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
p7zip: fix src URL for debian gitlab move #41769
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Another broken URL related to: NixOS#39927 Note that the patch file has legitimately changed, because ~4 months ago Debian replaced their CVE security fix with a newer version that fixes some additional bugs: https://salsa.debian.org/debian/p7zip/commit/d6fd3b37345489ec2907fcf70aabf0c754f5371f
@GrahamcOfBorg build p7zip |
Success on x86_64-linux (full log) Attempted: p7zip Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: p7zip Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: p7zip Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: p7zip Partial log (click to expand)
|
vcunat
pushed a commit
that referenced
this pull request
Jun 10, 2018
Another broken URL related to: #39927 Note that the patch file has legitimately changed, because ~4 months ago Debian replaced their CVE security fix with a newer version that fixes some additional bugs: https://salsa.debian.org/debian/p7zip/commit/d6fd3b37345489ec2907fcf70aabf0c754f5371f (cherry picked from commit e20abf8)
bhipple
added a commit
to bhipple/nixpkgs
that referenced
this pull request
Aug 16, 2018
A few months ago I moved these patches to the new debian alsa instance [1], but it looks like their `sha256`s on the tag at the remote have changed again. It doesn't appear that debian's source remote is stable in the way we need it to be; let's just vendor the patches to avoid future issues. [1] NixOS#41769
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Another broken URL related to: #39927
Note that the patch file has legitimately changed, because ~4 months ago Debian
replaced their CVE security fix with a newer version that fixes some additional
bugs: https://salsa.debian.org/debian/p7zip/commit/d6fd3b37345489ec2907fcf70aabf0c754f5371f
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)