Re: [PATCH 1/1] vpnc: add support to change interface name



Dan Winship <danw gnome org> on Tue, 2014/09/23 08:46:
On 09/23/2014 04:58 AM, Christian Hesse wrote:
Dan Williams <dcbw redhat com> on Mon, 2014/09/22 10:09:
So this is great, and ties in with some other stuff we're doing.
However, one change I'd like to request.  Instead of putting the
interface name into the VPN data (like the other vpnc-specific options),
lets put it into the NMSettingConnection setting's 'interface-name'
property.

Sounds reasonable.

But (NMConnection *connection) is specific to a VPN connection and does
not have an interface-name property, no? If I got it right I can not pass
it to nm_connection_get_setting_connection() and therefore
(NMSettingConnection *s_con) is missing. So do we have to create
(NMConnection *connection) and (NMConnection *vpn) and pass around both?

No, the NMConnection contains multiple NMSettings (basically, one for
each page of the connection editor). So you can call both
get_setting_connection() and get_setting_vpn() on it.

Ok, I think I got it... Will reply with a patch. Thanks for your help!
-- 
Schoene Gruesse
Chris
                         O< ascii ribbon campaign
                   stop html mail - www.asciiribbon.org

Attachment: signature.asc
Description: PGP signature



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]