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
NetworkManager fixes #62956
NetworkManager fixes #62956
Conversation
Logic expression was incorrect before.
…vers Before only one of them could work at the same time.
Avoid using `with` which isn't particularly needed.
@worldofpeace Did you manage to try this perchance? |
I'm afraid it flew away from me 🐦 I should be able to give it a test today, I think I've looked over the changes prior and didn't see any glaring issues. |
Co-Authored-By: worldofpeace <worldofpeace@protonmail.ch>
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.
Seems fine to me, and tested.
OK, let's merge then - I used it quite a bit too and tested with nameservers. |
Motivation for this change
Assortment of NetworkManager fixes that I did while working on #62955. In particular
insertNameservers
andappendNameservers
now work simultaneously andstrongswan
configuration isn't installed when not enabled.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)