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.urwid: 1.3.1 -> 2.0.1 #67096
Conversation
@GrahamcOfBorg build pythonPackages.urwid python35Packages.urwid python36Packages.urwid python37Packages.urwid |
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.
Builds fine on ofborg 👍
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.
nix-review
passes on NixOS (1 package broken on master)
diff LGTM
[26 built (1 failed)]
error: build of '/nix/store/i2dnj3513448k8ibh3nixzy519szcpsv-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/67096
1 package failed to build:
python37Packages.python_openzwave
27 package were build:
alot khal mitmproxy python27Packages.bpython python27Packages.configshell python27Packages.pudb python27Packages.pycarddav python27Packages.upass python27Packages.urwid python27Packages.urwidtrees python37Packages.bpython python37Packages.configshell python37Packages.homeassistant-pyozw python37Packages.pudb python37Packages.upass python37Packages.urwid python37Packages.urwidtrees s-tui salut_a_toi speedometer targetcli todoman topydo turses urlscan wicd wikicurses
@doronbehar Can you squash commits? |
f6d5ec8
to
45d3623
Compare
Squashed.
|
Thanks 🎆️ |
Motivation for this change
A new package I'd like to contribute depends on urwid>=2.0.0, from upstream's changelog, it seems no breaking changes were done. I hope this is reliable enough and that this update won't break existing packages.
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)