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
cocotb: 1.2.0 -> 1.3.0 #81417
cocotb: 1.2.0 -> 1.3.0 #81417
Conversation
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.
do you mind quoting the homepage link?
otherwise LGTM
@jonringer sorry I'm not sure I understand what you mean |
Also I've never contributed to nixpkgs before, so if I've missed something please let me know! |
change
to
:) otherwise LGTM |
This looks ok to me. Thanks for the contribution @Nitori-! |
The fix for the CC/CXX variables was merged some time before 1.3.0. The `-Werror` seems to be new. It is removed in upstream's master so it should be removable in the next release. Something in glibc is causing a warning that is killing the build if `-Werror` isn't removed.
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.
diff LGTM
commit LGTM
shows usage
[5 built, 1 copied (7.0 MiB), 1.9 MiB DL]
https://github.com/NixOS/nixpkgs/pull/81417
3 package built:
python27Packages.cocotb python37Packages.cocotb python38Packages.cocotb
@GrahamcOfBorg build python27Packages.cocotb python37Packages.cocotb python38Packages.cocotb |
Motivation for this change
New release of cocotb. Changelog: https://cocotb.readthedocs.io/en/latest/release_notes.html#cocotb-1-3-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)