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

net/wireguard: version bump #3664

Closed
wants to merge 1 commit into from
Closed

net/wireguard: version bump #3664

wants to merge 1 commit into from

Conversation

zx2c4
Copy link
Contributor

@zx2c4 zx2c4 commented Dec 16, 2016

Maintainer: @zorun
Upstream: me

Description:

Update to new snapshot version. We also make IPV6 optional, and
conditionally selecting the udptunnel6 module, using the same trick that
the strongswan package also uses for this kind of dependency expression.
This solves a request from users.

Update to new snapshot version. We also make IPV6 optional, and
conditionally selecting the udptunnel6 module, using the same trick that
the strongswan package also uses for this kind of dependency expression.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
@danrl
Copy link
Contributor

danrl commented Dec 16, 2016

Makes sense. I hate to see IPv6 "go". As if it is optional nowadays. It isn't. Too bad some people still insist on compiling without IPv6. Nevertheless, I support this change for the sake of the legacy-only people out there.

I wish we could make IPv4 optional as well. It's 2016. I rarely use legacy IP on some of my routers.

@zx2c4
Copy link
Contributor Author

zx2c4 commented Dec 16, 2016

@danrl the kernel module supports no-v6 mode by popular request from small system builders.

Maintainers: please do not yet merge this pull request. I'll send an update in a few days when it's ready.

@zorun
Copy link

zorun commented Dec 18, 2016

@zx2c4 The change looks good to me, feel free to submit it again with the new snapshot.

@zx2c4
Copy link
Contributor Author

zx2c4 commented Dec 18, 2016

Reopened in #3674. Please merge that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants