-
-
Notifications
You must be signed in to change notification settings - Fork 704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v0.10.1 broke Ubuntu 16.04 install via pip #635
Comments
Hello, Could you tell us the |
Seems to happen with the system-default |
Could you post the output of |
Using Python 3, yes. I'll have to get back to you on the exact setuptools version; I don't have the VM up and running at the moment. But, again, this is whatever the default one that comes on Ubuntu 16.04. Since Watchdog does run great when you install the dependency and Ubuntu 16.04 is going to be supported (therefore common in cloud providers) for another 14 months or so, I would urge you to change the way you specify the dependency (back to how it was) to make it "just work" rather than pinning a minimum version. |
The thing is that if we move back to how it was done before the commit, it will break elsewhere (again, because of outdated |
I found the right minimum version: 36.2.7. But I do not know yet how to deal with that :) |
I manage a package (Dactyl) that uses watchdog as a dependency. Starting with the release of Watchdog v0.10.1, our Dockerized builds (Ubuntu 16.04) are failing on an
ImportError
forpathtools
that ultimately comes from watchdog.After some investigation, it appears to be that
pip3 install watchdog
no longer installs the necessarypathtools
dependency because pip doesn't look at thesetup.cfg
file. This issue may be masked on some OSes because it's common for pathtools to be installed by default, but I was able to reproduce as follows:Creating a fresh Ubuntu 16.04 install (I did it on a VM)
sudo apt-get install python3-pip
pip3 install watchdog
python3
At the Python prompt:
The result is similar to this error:
Manually installing
pathtools
with pip fixes the error.I suspect the changes in commit 6baa210 to remove
pathtools
frominstall_requires
are the cause.The text was updated successfully, but these errors were encountered: