python3.pkgs.pythonRuntimeDepsCheckHook: init #270457
Closed
mweinelt wants to merge 76 commits intoNixOS:masterfrom
Closed
python3.pkgs.pythonRuntimeDepsCheckHook: init #270457mweinelt wants to merge 76 commits intoNixOS:masterfrom
mweinelt wants to merge 76 commits intoNixOS:masterfrom
Conversation
a6ff14e to
30e73a2
Compare
Member
Author
Why is the version we pull from the tag 2.0.21.dev0? And why is that not greater than or equal 2.0.16? if package.version not in requirement.specifier:
error(
f"{package_name}{requirement.specifier} not satisifeid by version {package.version}"
)
return FalseNot the whole truth. I delegate that to Solved in bdbeb01. |
11c1327 to
562ad5c
Compare
105d5d5 to
14e811e
Compare
13 tasks
fb3650f to
146297f
Compare
13 tasks
13 tasks
caarlos0
approved these changes
Nov 30, 2023
13 tasks
9e131c4 to
c6ac6e8
Compare
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
The badly maintained tests require nose3, which requires the imp module, which has been ripped out in 3.12.
Checking runtime dependencies for homeassistant-2023.11.3-py3-none-any.whl - httpx==0.25.0 not satisfied by version 0.25.2 - yarl==1.9.2 not satisifeid by version 1.9.3
They use nose, which relies on the imp module, which was been removed from Python 3.12.
during check phase.
Upstream has no released version with Python 3.12 supported yet, but a patch was already merged, which works fine.
The `async_generator` module is a backport of async generators to Python 3.5 and it started failing on 3.12, but pulling it is safe.
The `async_generator` module is a backport of async generators to Python 3.5 and it started failing on 3.12, but pulling it is safe.
c6ac6e8 to
0cbd9ca
Compare
This was referenced Dec 2, 2023
Member
Author
|
Continues over in #271586. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of changes
Implements a hook, that checks whether all dependencies, as specified by
the wheel manifest, are present in the current environment.
Complains about missing packages, as well as version specifier
mismatches.
Requires at least
packaging>=23.2and creates a bootstrap version of it.Not sure if this is necessary, or if we should just not check on bootstrap packages.
Breaks the build for some packages, e.g., when we prevent infinite
recursion, like on sphinxcontrib packages, which have a dependency on
sphinx.What do we want to do here? Will set
dontCheckRuntimeDepsfor now.Fixes: #250865
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Priorities
Add a 👍 reaction to pull requests you find important.