-
-
Notifications
You must be signed in to change notification settings - Fork 565
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
More fixes to support tox 4 #34849
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. New commits:
|
Commit: |
Author: Matthias Koeppe |
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
comment:17
with tox 4.2.8, I get
|
comment:18
and with this branch, I get
|
comment:20
oops, sorry, I just got logged out of docker, that's the reason for the error |
comment:21
seems to work |
Reviewer: Dima Pasechnik |
comment:22
Thanks! |
Changed branch from u/mkoeppe/more_fixes_to_support_tox_4 to |
The tox.ini files in pkgs/*/ are broken with tox 4.
Early versions of tox 4.x have additional defects.
Environment variables containing strings with spaces are not accepted any longer for passenv/pass_env. tox-dev/tox#2699 (comment) - factors no longer work in
passenv
- fixed in tox 4.0.13negated factors in
deps
, like!pipenv-!norequirements: -rrequirements.txt
do not work - tox 4 does not support factor all conditionals as 3 tox-dev/tox#2747 - fixed in tox 4.0.15And autoprovisioning for downgrade to tox 3 is not working - tox auto-provisioning not working tox-dev/tox#2634
In SAGE_ROOT/tox.ini, in tox 4 earlier than 4.2.7, numerical factors are broken because they are misinterpreted as Python versions (https://tox.wiki/en/latest/changelog.html#bugfixes-4-2-7); e.g., 4.0.14 is broken:
Here we update affected
tox.ini
files so that:spkg-configure.m4
to check for such versions.Follow up:
Depends on #34839
CC: @fchapoton @dimpase
Component: scripts
Author: Matthias Koeppe
Branch/Commit:
a2eb816
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/34849
The text was updated successfully, but these errors were encountered: