-
Notifications
You must be signed in to change notification settings - Fork 145
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
Assure that setuptools <45 is installed #245
Conversation
I don't think there was any deep reason, apart from "good hygiene to stay as up-to-date as possible". Now that we're stuck on a legacy python, of course, the opposite applies and we need to work to stay as out-of-date as we safely can :-( |
Usually upgrading pip or any other modules via pip to latest offered version is not an issue. There are several stages where compatibility with the local Python version is checked and all of them must fail somehow for an issue like this to happen 😉. Let me check if EDIT: |
In some circumstances pip pulls setuptools 45.0.0, which is not compatible with Python 2. To assure the correct version is installed, add the "--no-setuptools" flag to virtualenv and install the correct setuptools version manually afterwards. Workaround for: #239 Additionally this removes the obsolete pip upgrade. "make build" only runs once and is blocked afterwards by a flag file. Even when removing this flag file manually, "virtualenv" always reinstalls pip, which includes an upgrade to the latest version. A latest supported pip version is hence assured on every build. Signed-off-by: MichaIng <micha@dietpi.com>
Comment added, obsolete pip install/upgrade removed, commit text updated accordingly. That removed empty line indentation is not somehow required for Makefile syntax, is it? 😄 |
just tested it locally. Hilariously, I needed to pre-run |
Reads similar to the issues we face on Debian Bullseye and Ubuntu Focal where the
|
Yep, that works too. Fortunately, we're making fairly good progress on the rust syncstorage-rs server. There's still the tokenserver bits that need to be hammered out, but hoping to get those in order soon too. Then we can move things over to rust and send the python2 bits to a farm. |
Reasonable, however, from my side there is interest to keep things running until next summer when Debian Bullseye gets released 🙂. OOT, the next to take care of:
appears when running latest pip. |
…y-populate-collection-cache-in-read-transactions Fix mozilla-services#224 Only populate the CollectionCache when in a read transaction
In some circumstances pip pulls setuptools 45.0.0, which is not compatible with Python 2. To assure the correct version is installed, add the "--no-setuptools" flag to virtualenv and install the correct setuptools version manually afterwards.
Workaround for: #239
While this works, it could be added probably more cleanly, e.g. adding
--no-setuptools
toVIRTUALENV
variable and merging$(INSTALL) -U "setuptools<45"
with$(INSTALL) -i https://pypi.python.org/simple -U pip
to:What was actually the reason for adding the explicit pip install/upgrade? Since
virtualenv
does (or should) do this already. I can only imagine that it was a similar issue than the present one since pip is pulled from an explicit mirror?Also some comment in code would be a good idea. Please advice about how it is wanted.