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.importlib-metadata: 1.3.0 -> 1.5.0 #80893
pythonPackages.importlib-metadata: 1.3.0 -> 1.5.0 #80893
Conversation
I hit cancel mid-request, but didn't catch it, sorry -- this should be a draft, this is a WIP... It currently does not work, this introduces infinite recursion when building this package.
|
Note: |
I couldn't quite figure out how to skip the |
this rebuilds the majority of the python package set, please target staging |
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.
they just added pyfakefs as a checkInput, this is very do-able
Hi @jonringer. Thanks for looking. As for your suggestion - that's what I did at first, and it yielded the infinite recursion error that I posted. If you look at my two commits, you'll see one added the pyfakefs checkDep and then my workaround removed it again since it caused the inf recur error. Obviously deleting the test is not ideal, but I don't know how to proceed with your recommendation without hitting the recursion error. |
8ae3611
to
f4b8092
Compare
I think deleting the test, and just noting that it's done to avoid infinite recursion is fine |
f4b8092
to
61ec463
Compare
# removing test_main.py - it requires 'pyflakefs' | ||
# and adding `pyflakefs` to `checkInputs` causes infinite recursion. |
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 would move the comments outside of the string, if we want to edit the comments later, the string comments will cause a rebuild, while nix comments will not
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.
makes sense, thx. addressed and pushed already.
61ec463
to
a2d8f10
Compare
I was able to build (on nixos-unstable) for: |
a2d8f10
to
424fe9a
Compare
@GrahamcOfBorg build pythonPackages.importlib-metadata |
darwin builds are timing out due to trying to rebuild llvm |
Motivation for this change
Update
importlib-metadata
to latest (1.5.0
).Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)