-
Notifications
You must be signed in to change notification settings - Fork 754
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
Some cleanups #39
Some cleanups #39
Conversation
Looks good, up to the last commit. The problem there is that there is still a call to |
That is: the .name file and the D-Bus policy. It's needed for useful use of charon-nm, unlike the GUI.
The auth dialog is built separately from the actual plugin -- we need to guess where the auth dialog will end up. If the distros override it, they set it to the same path NetworkManager uses.
It's the preferred location for system-provided plugins. A compatible file in /etc is still kept. Also, the compatibility /etc file needs to use a full path due to a bug in GNOME Shell. The full path to a arch-dependent file in a supposedly arch-independent file is a sin and a multilib violation in some distributions. Hovewer. some pre-release versions of NetworkManager-1.2 as shipped by distributions require a full path. Let's keep a configure-time option for that.
PKG_CHECK_MODULES does the substitutions.
The former is deprecated and the newer API is nicer anyway.
Split off from pull reques #15 for ease of review.