-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
libtatsu: init, libimobiledevice-drvs: update all with minor rebuilds #349567
base: master
Are you sure you want to change the base?
libtatsu: init, libimobiledevice-drvs: update all with minor rebuilds #349567
Conversation
Drafting to perform a minor version bump on all of these derivations, including rebasing to the current master |
205c16e
to
78011da
Compare
|
]; | ||
|
||
# the package uses zip_get_num_entries, which is deprecated | ||
env.NIX_CFLAGS_COMPILE = toString [ | ||
NIX_CFLAGS_COMPILE = toString [ |
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_CFLAGS_COMPILE = toString [ | |
env.NIX_CFLAGS_COMPILE = toString [ |
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.
Whoops, seems I've made a change that isn't the best. Could I ask why this is preferred? Afaict both will propagate as an env var for the derivation. Is this preferred due to its more explicit use of "env" to imply "as an env var"?
EDIT: fixed!
78011da
to
4639e34
Compare
This is a separation commit from #342961 targetting all derivations that result in minor rebuilds. The maximum rebuilds I have observed from these are 33.
These changes were accompanied by cleanup that is shared between both PRs. Cleanup includes:
unstableGitUpdater
to all derivations (see explanation by @restoration578 here)meta.license
more consistently to upstreammeta.homepage
platforms.unix
for all derivations inmeta.platforms
pkgs/by-name
cc @restoration578 @clebs @RossComputerGuy @winterqt as commentors from the previous pr.
Relevant Issues
Closes #342738
Closes #342736
Closes #342731
Closes #342728
Closes #342729
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.