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
libexif: 0.6.21 -> 0.6.22 #88100
libexif: 0.6.21 -> 0.6.22 #88100
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.
since this change causes over 500 rebuilds, do you mind targeting the staging branch
git rebase --onto=staging HEAD~1
git push .. ... --force
then change the base branch in the github PR from master
-> staging
See https://nixos.org/nixpkgs/manual/#submitting-changes-staging-branch for more details on staging branch
also, duplicate of #88096 |
In #88096 I didn't look into cross compilation so it's probably better to merge this one. Another difference is that the other PR uses the release archive which doesn't need autoreconf, not sure what is preferable. |
@mmilata Oh sorry, this time I didn't look for duplicates because I felt I'd be early enough. Will check next time definetely. I'd always prefer building from source, as this leverages the hurdle for upstream contributions. @jonringer Thanks, I know about the staging branch but first wanted to ask ofBorg how much is going to be rebuilt. |
Also: - build from git - enable cross compilation
@prusnak Thanks for the review. I added your suggestions and rebased to latest |
Motivation for this change
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)