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
python3Packages.localzone: Downgrade dnspython dependency #99496
Conversation
e3983fd
to
7eddb66
Compare
7eddb66
to
336d1be
Compare
localzone is currently incompatible with dnspython 2
336d1be
to
7a85f34
Compare
@mweinelt I adopted your recommended changes. Is this what you meant? |
Yes, I feel this makes this change alot cleaner and better maintainable. |
LGTM. What is broken, was broken before as well. Result of 2 packages failed to build:
2 packages built:
|
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.
Libraries cannot use a different version. If it is not compatible with the current dnspython
it needs to be marked as broken.
So revert? |
Yes |
I already made an issue upstream ags-slc/localzone#1 |
The current localezone release is not compatible with dnspython 2.x that
was introduced in nixpkgs with commit a7b5b18
Motivation for this change
Fix broken build
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)