seveas / python-networkmanager

Easy communication with NetworkManager
http://packages.python.org/python-networkmanager/
Other
163 stars 90 forks source link

Bug fixes #54

Closed afreof closed 6 years ago

afreof commented 6 years ago

Hi Dennis, With NM 1.4.4 and python 2.7 I was facing some issues. Hope this helps.

One more detail: commit 7453a4b6e0c3a71d7062e4971a56cac1857368e3 "Add dependencies to setup.py" leads to a python egg installation, at least on Yocto. Without this commit NetworkManager.py(c) is installed as expected. Is this related to my setup or is an egg the preferred installation for python-networkmanager?

seveas commented 6 years ago

Hi @afreof,

Thanks for your PR! I agree that you found some bugs, but didn't agree with the fixes (except the indentation, but that had a conflict with another change I made). So I fixed those bugs slightly differently instead in the now released 2.1 version.