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
pythonPackages.pytest_3: fix build #64445
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't override modules within the Python package set because it can lead to conflicting versions. Instead, we should likely set the package set wide version to 0.8.1 in case of Python 2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the old build as a versioned attribute, i.e. pluggy_0_8_1
. Then we can override the version for the python-2.x package set on the top-level so that all builds depending on pluggy
get a consistent version.
Sorry for the delay, I've added |
@GrahamcOfBorg build pythonPackages.pytest_3 python3Packages.pytest_3 |
@@ -5293,6 +5293,8 @@ in { | |||
|
|||
pluggy = callPackage ../development/python-modules/pluggy {}; | |||
|
|||
pluggy_0_8 = callPackage ../development/python-modules/pluggy/0_8.nix {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one will have to always be used with Python 2. Note though, if we notice breakage due to this change, then that means we won't be able to support pytest 3 anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried, but it breaks
python3Packages.pytest_3
it would use pluggy=0.12.0, which was the original bug reportpython{2,3}Packages.pytest
because it requirespluggy>=0.12
then that means we won't be able to support pytest 3 anymore.
I'm inclined to drop pytest_3, only 9 packages depend on it
@@ -18,7 +18,7 @@ buildPythonPackage rec { | |||
|
|||
checkInputs = [ hypothesis mock ]; | |||
buildInputs = [ setuptools_scm ]; | |||
propagatedBuildInputs = [ attrs py setuptools six pluggy more-itertools atomicwrites] | |||
propagatedBuildInputs = [ attrs py setuptools six pluggy_0_8 more-itertools atomicwrites] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can't have multiple versions of the same library in use at the same time.
We indeed need to drop |
Dropping |
pluggy>=0.9 introduced a breaking change to pytest_3, and is not feasible to keep pytest_3 around in nixpkgs. See: NixOS#64445
Motivation for this change
Fixes #64440
cc: @peti
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)