Debian specific requirements for your docs

[jkeithostertag]

Just a reminder for your docs: with Debian I needed to
sudo apt install python3-xmltodict and sudo apt install python3-aiohttp in order to run your example at this date.

[null8626]

Thank you for the suggestion. Will make changes to the documentation later!

[MalikMlitat]

@null8626 I could takeover this issue :)

[null8626]

thanks!

[MalikMlitat]

@null8626 I didn't found a proper place where such documentations are belonging too, so I created a debian folder and manifested the dependencies in the control file, would it be an idea, to include a .deb package in the python-weather release, what do you think?

main...MalikMlitat:python-weather:main

the generated package includes:

`
dpkg -c ../python3-weather_0.3.8_all.deb
drwxr-xr-x root/root 0 2022-08-01 21:24 ./
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/lib/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/lib/python3/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/lib/python3/dist-packages/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/
-rw-r--r-- root/root 1399 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/init.py
-rw-r--r-- root/root 1603 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/client.py
-rw-r--r-- root/root 494 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/exceptions.py
-rw-r--r-- root/root 6555 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/response.py
-rw-r--r-- root/root 1427 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather/rest.py
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather-0.3.8.egg-info/
-rw-r--r-- root/root 2046 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather-0.3.8.egg-info/PKG-INFO
-rw-r--r-- root/root 1 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather-0.3.8.egg-info/dependency_links.txt
-rw-r--r-- root/root 0 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather-0.3.8.egg-info/requires.txt
-rw-r--r-- root/root 15 2022-08-01 21:24 ./usr/lib/python3/dist-packages/python_weather-0.3.8.egg-info/top_level.txt
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/share/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/share/doc/
drwxr-xr-x root/root 0 2022-08-01 21:24 ./usr/share/doc/python3-weather/
-rw-r--r-- root/root 155 2022-08-01 21:24 ./usr/share/doc/python3-weather/changelog.gz
-rw-r--r-- root/root 1299 2022-08-01 21:24 ./usr/share/doc/python3-weather/copyright

`

[jkeithostertag]

No, i do not think it would be advisable to include a .deb file- they are easily available through the repositories where they are always kept updated. It is enough to alert people for the need of them.

…

On Mon, Aug 1, 2022, at 4:07 PM, MalikMlitat wrote: @null8626 <https://github.com/null8626> I didn't found a proper place where such documentations are belonging too, so I created a debian folder and manifested the dependencies in the control file, would it be an idea, to include a .deb package in the python-weather release, what do you think? main...MalikMlitat:python-weather:main — Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A2HILTNZZUM4YJ6R2QIOSELVXAVALANCNFSM54UIYDAQ>. You are receiving this because you authored the thread.Message ID: ***@***.***>

-- Keith Ostertag ***@***.***

[MalikMlitat]

But from where did you got the .deb file?
This is something which needs to be fixed in the downstream not the upstream

[jkeithostertag]

The packages I needed to download in order to run your program are in the Debian repository, here are the url's where you can find the debs:

https://packages.debian.org/bullseye/python3-xmltodict
https://packages.debian.org/bullseye/python3-aiohttp

This was for Debian Bullseye on amd64 for python3, other architectures may need other debs, that's why I don't think providing debs is a good idea... but you know best! I am not a developer!

Hope this helps, just ignore it if it is wrong or does not make sense for your program.

Keith

[MalikMlitat]

Hey @jkeithostertag ,

now I got the issue, please do the following:

1. remove the manual installed packages: sudo apt remove python3-xmltodict python3-aiohttp -y
2. install the deps from the source code: sudo python3 setup.py install
3. run the example: python3 example.py

[null8626]

please note that the xmltodict dependency won't be used anymore in a future update, since #25

[null8626]

Changes landed in this commit