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.dulwich: 0.20.14 -> 0.20.15 #107452
Conversation
Result of 5 packages failed to build:
16 packages built:
|
Result of 3 packages failed to build:
20 packages built:
check-manifest
|
Thanks. Updates are included. |
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.
Thanks! The changes to the code look good to me. I haven’t tried to build these packages or dependencies.
@GrahamcOfBorg build python3Packages.build python3Packages.dulwich python3Packages.pep517 python3Packages.proto-plus |
Some of the updates causes a massive rebuild. Can you please split the PR into 4 different PRs so we can review/accept them individually? The one (or ones) causing this massive rebuild will be targeted against the |
Break-outs: |
Can you please remove pep517 from the changeset to see whether the rebuild is smaller now? |
Removed |
@GrahamcOfBorg build python3Packages.dulwich python3Packages.build python3Packages.proto-plus |
This PR is blocked until master is rebuilt with #108255 merged in from 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.
diff LGTM
Updated commit messages to match the comment in another PR. |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 23 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.
LGTM
Result of nixpkgs-review pr 107452 run on x86_64-linux 1
23 packages built:
- breezy (python38Packages.breezy)
- cabal2nix
- dep2nix
- devpi-client
- errbot
- flatpak-builder
- hydra-unstable
- klaus (python38Packages.klaus)
- nix-prefetch-bzr
- nix-prefetch-scripts
- nix-update-source
- python37Packages.breezy
- python37Packages.check-manifest
- python37Packages.dulwich
- python37Packages.klaus
- python38Packages.check-manifest
- python38Packages.dulwich
- python39Packages.dulwich
- python39Packages.klaus
- rabbitvcs
- reno
- vcstool
- xandikos
Motivation for this change
Update to latest upstream release 0.20.15
Additionally:
homepage
entry to metadatachangelog
entry to metadataCOPYING
: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)