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
nix-prefetch: 0.3.1 -> 0.4.0 #109450
nix-prefetch: 0.3.1 -> 0.4.0 #109450
Conversation
There is no changelog. |
@SuperSandro2000 I have added the changes under motivation for this change. I assume that is what you meant? |
Result of 2 packages built:
Seems to work well, tested both |
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.
Code-wise LGTM, but please squash the commits.
I meant the downstream repo. It is missing the changelog for 4.0. |
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.
can be merged when ofborg is green
@thiagokokada Pity you cannot do that from Github, but managed to do it manually. @SuperSandro2000 Where are you looking for the changelogs? I added them here: |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
@SuperSandro2000 Could you elaborate on why this needs a backport? |
I thought it was also broken on stable. |
Motivation for this change
Created a new release making the manual patching no longer necessary.
Changes in 0.4.0:
nix-prefetch
overlay when the ad-hoc approach fails.XDG_RUNTIME_DIR
.fetchFromGithub
dummy in--list
.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)