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
python2Packages.cfn-lint: fix importlib-resources version bounds #97762
Conversation
Why is this package being disabled? What's failing? From looking at upstream, it supports |
importlib-resource is required to be 1.4, and we have 3.0.0 in the repository |
Applying this patch seemed to work (tested against GitHub source, not PyPi). Don't need to disable python2.7 then.
|
90ca438
to
f0805f1
Compare
Thank you for the patch |
f0805f1
to
08f3e69
Compare
Commit should be renamed to something like
|
Seeing as we're now patching the source, would you be able to add a |
Also, added import checks to have a minimal test for the package
08f3e69
to
25d2cc2
Compare
As it is supposed to be an app more than a library, I only took the packages in the root folder's Built for python2 and 3 locally |
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.
does in fact fix the build, other failures can be dealt with in other PRs
Motivation for this change
package dependencies don't support python 2
ZHF: #97479
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)