After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 694768 - NetworkManager might use different libexecdir than gnome-shell
NetworkManager might use different libexecdir than gnome-shell
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: VPN: pptp
git master
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
: 679212 679225 (view as bug list)
Depends on: 693590
Blocks:
 
 
Reported: 2013-02-26 22:00 UTC by Dominique Leuenberger
Modified: 2013-04-20 05:55 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
nm-pptp.service: use absolute paths for all helpers (833 bytes, patch)
2013-02-26 22:01 UTC, Dominique Leuenberger 		none Details | Review

Description Dominique Leuenberger 2013-02-26 22:00:53 UTC 
+++ This bug was initially created as a clone of Bug #693590 +++

In openSUSE, for example (debian seems to do similar things), gnome-shell is condifured with
configure --libexecdir=/usr/lib/gnome-shell

and NetworkManager accordingly with
configure --libexecdir=/usr/lib/NetworkManager

=> gnome-shell fails to understand this and looks for the NM vpn agents in /usr/lib/gnome-shell

A patch against gnome-shell was rejected and a fix on NM-pptp proposed (similar patches would be needed for all NM specific VPN modules).
Comment 1 Dominique Leuenberger 2013-02-26 22:01:28 UTC 
Created attachment 237474 [details] [review]
nm-pptp.service: use absolute paths for all helpers
Comment 2 Dan Winship 2013-02-27 09:12:33 UTC 
Yeah, this patch makes sense (of course we'd want it for all the vpn types), but I think we should also add a NetworkManager.pc variable as discussed in bug 693590
Comment 3 Dominique Leuenberger 2013-02-27 18:03:59 UTC 
I agree that the combination of the .pc variable and the absolute paths would be the best indeed...

I can create patches for the NM-vpn modules, no problem there... I have on my radar:
- pptp (attached)
- openconnect
- openvpn
- strongswan
- vpnc

Would you prefer one bug each or can I attach them all here?
Comment 4 Dan Williams 2013-02-28 09:41:33 UTC 
(In reply to comment #2)
> Yeah, this patch makes sense (of course we'd want it for all the vpn types),
> but I think we should also add a NetworkManager.pc variable as discussed in bug
> 693590

+1 from me on this and 693590
Comment 5 Dan Winship 2013-03-12 19:14:58 UTC 
fixed in all VPN modules in master
Comment 6 Dan Winship 2013-03-12 19:18:10 UTC 
*** Bug 679225 has been marked as a duplicate of this bug. ***
Comment 7 Clemens Buchacher 2013-04-20 05:55:17 UTC 
*** Bug 679212 has been marked as a duplicate of this bug. ***