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

Reapply the multi-ip code #3908

Merged
merged 3 commits into from
Sep 1, 2014
Merged

Reapply the multi-ip code #3908

merged 3 commits into from
Sep 1, 2014

Conversation

wkennington
Copy link
Contributor

This patchset reapplies the multi-ip changes with a fix that corrects a bug in translating old syntax to new syntax.

@robberer
Copy link
Contributor

robberer commented Sep 1, 2014

Nice. Ipv4 part tested and proven to work with my network setup. No errors, neither with the old syntax [1] nor with the new syntax [2].

[1] https://gist.github.com/robberer/5cf11e13b798cdf6c318
[2] https://gist.github.com/robberer/3922578e831d4a4ee720

Can't say anything about ipv6.

@wkennington
Copy link
Contributor Author

It's been working fine in my multi-ipv6 and single ipv6 deployments.

The only reason it was reverted is because I quickly hacked together compatibility support and made a typo.

7c6f434c added a commit that referenced this pull request Sep 1, 2014
@7c6f434c 7c6f434c merged commit 9e3d1b1 into NixOS:master Sep 1, 2014
@@ -20,10 +40,36 @@ let
description = "Name of the interface.";
};

ip4 = mkOption {
Copy link
Member

Choose a reason for hiding this comment

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

This should be ipv4 and ipv6, respectively, for consistency with other NixOS options.

@wkennington wkennington deleted the master.ip branch September 7, 2014 00:46
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.

4 participants