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
openvpn plugins: use absolute paths to configuration gui #47212
Conversation
@@ -28,6 +28,7 @@ in stdenv.mkDerivation rec { | |||
"--without-libnm-glib" | |||
"--with-gnome=${if withGnome then "yes" else "no"}" | |||
"--localstatedir=/tmp" | |||
"--enable-absolute-paths=yes" |
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.
No need to specify yes, it is the default for --enable-foo
options
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.
fixed
4aabe90
to
e091396
Compare
|
Was this cherry-picked to 18.09? |
It seems it was not. The basic NetworkManager functionality is broken and it should be backported IMHO. I created a backport to 18.09 PR here: #48220 |
#48220 is merged into release-18.09 now |
Motivation for this change
This is based on the suggestion of @jtojnar here #46901 (comment)
This should fix all network manager plugins; I have tested openvpn iodine and vpnc.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)