-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Docker train failure: ModuleNotFoundError: No module named 'setuptools._distutils' #3295
Comments
FWIW I'm getting this error as well when trying to rebuild DeepSpeech from my docker scripts for 0.7.4 https://github.com/techiaith/docker-deepspeech-cy/blob/deepspeech-0.7.4/Makefile#L19 it used to build! If I change to 0.8.2 or latest on master, the same bug appears. |
It's an issue at pypa setuptools... Hopefully this doesn't require any changes or workarounds on DeepSpeech's side. |
Unfortunately, we can't upgrade this release. Also, forcing |
It builds successfully for me if you instead specify the last version of setuptools before today's 50.0.0 release:
|
Yeah there are a lot of reports of bad breakages because of 50.0.0. Unfortunately we wont be able to issue that kind of fix for older release so i prefer to wait and see for pypa/setuptools feedback on all those issues. |
People report that |
in the case of Docker, that would still need a change in the Dockerfile (you can't set and pass an environment variable in a |
There's so much mess, I really don't get. Some of what seems to be one of the maintainer's of the project seems to imply there is no bug here, but seeing how this explodes everywhere it looks suspicious. As to the setuptools version, in my testing it was not enough. |
This really comes at a very bad time, we have so much to do that is equally important, I can't spare the time to try and debug pip/setuptools as well ... |
@DewiBrynJones Seems like 50.0.3 works |
Except the |
Great! I got the 0.8.2 release to build and train models with the following workaround to use 49.6.3:
I could try and debug pip/setuptools the latest on master (?) if it's any help (although I'm no expert) |
It looks like we are facing https://github.com/pypa/setuptools/issues/2350:
|
I can't find this one. |
It seems to work here as well. I'd welcome a PR against
And we could make a 0.8.3 to release the fix as well. |
@DewiBrynJones I took the liberty to assign you the bug and I'm waiting on your PR since you have nailed that down quite well :) |
Ha ha! me and my big mouth. i'll get on it. |
No, it's really welcome, but if you can't do it I'll take care of it, no pressure here. |
Fix for setuptools._distutils issue (#3295)
Thanks @DewiBrynJones it's now fixed on both branches |
@DewiBrynJones Do you mind also sending a PR for |
Thanks! |
The text was updated successfully, but these errors were encountered: