Skip to content
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

wireguard: 20161209 -> 20161218 #21282

Closed
wants to merge 1 commit into from
Closed

wireguard: 20161209 -> 20161218 #21282

wants to merge 1 commit into from

Conversation

zx2c4
Copy link
Contributor

@zx2c4 zx2c4 commented Dec 19, 2016

Hi,

Upstream here. This is a rather important version bump. I made it because it's important, but I'm not a NixOS user. I computed the checksum like this:

zx2c4@thinkpad ~ $ ./nix-1.11.4/src/nix-hash/nix-hash --type sha256 --base32 WireGuard-0.0.20161218.tar.xz
0d5grn3jvc1byammwlq0hbpdbcs7d28fjarw8kam6fpcnalw2y4y

Hopefully that's correct.

@mention-bot
Copy link

@zx2c4, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fpletz, @ericsagnes and @womfoo to be potential reviewers.

@Mic92 Mic92 added 9.needs: port to stable A PR needs a backport to the stable release. 1.severity: security and removed 1.severity: security labels Dec 19, 2016
@joachifm
Copy link
Contributor

The hash is incorrect; probably you needed to use the --flat option. Anyway that is easily rectified. I'm curious to know what makes this release "important"?

@Mic92
Copy link
Member

Mic92 commented Dec 19, 2016

@joachifm it breaks the protocol (header +3 bytes) for 4-byte aligning.


src = fetchurl {
url = "https://git.zx2c4.com/WireGuard/snapshot/WireGuard-${version}.tar.xz";
sha256 = "11n8dq8a8w0qj8xg5np9w02kmk14hn5hphv2h4bjw9hs8yxvkaya";
sha256 = "0d5grn3jvc1byammwlq0hbpdbcs7d28fjarw8kam6fpcnalw2y4y";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are looking for this one: 9522e59#diff-cdcb747370d48836cde962abc9adbba1R5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.needs: port to stable A PR needs a backport to the stable release. 12. first-time contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants