You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So far, I've come across two separate points in this regard, both relating to the removal of distutils/setuptools from the stdlib:
You now need to install setuptools explicitly into the venv, because it does not happen by default anymore. This has the added complication that pip freeze on previous Python versions does not include setuptools, so appenv-update-lockfile does not put it in the requirements.lock -- which probably made sense previously, since setuptools was "managed by venv", but now it isn't anymore. (My workaround is adding it to the lockfile manually.)
The "are we on a lobotomized debian python without pip" sanity check now always fails, because it uses import distutils as part of its detection. I don't know what the updated version of this should look like.
The text was updated successfully, but these errors were encountered:
wosc
added a commit
to wosc/pharos-deployment
that referenced
this issue
Nov 15, 2023
Our bootstrap instruction already include installing python-venv which
takes care of the situation that this workaround was guarding against.
See also flyingcircusio/batou#402
So far, I've come across two separate points in this regard, both relating to the removal of distutils/setuptools from the stdlib:
You now need to install
setuptools
explicitly into the venv, because it does not happen by default anymore. This has the added complication thatpip freeze
on previous Python versions does not includesetuptools
, soappenv-update-lockfile
does not put it in therequirements.lock
-- which probably made sense previously, since setuptools was "managed by venv", but now it isn't anymore. (My workaround is adding it to the lockfile manually.)The "are we on a lobotomized debian python without pip" sanity check now always fails, because it uses
import distutils
as part of its detection. I don't know what the updated version of this should look like.The text was updated successfully, but these errors were encountered: