diff options
| author | Colin Guthrie <colin@mageia.org> | 2014-01-24 09:57:13 +0000 |
|---|---|---|
| committer | Olivier Blin <dev@blino.org> | 2014-01-27 01:27:34 +0100 |
| commit | 360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491 (patch) | |
| tree | d3b07ba39400d4bff52e37dea362ab2fdcefa1f3 | |
| parent | 8e18c8b9e3ae6db0687b240ccf7ecb242cded322 (diff) | |
| download | drakx-net-360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491.tar drakx-net-360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491.tar.gz drakx-net-360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491.tar.bz2 drakx-net-360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491.tar.xz drakx-net-360f3b5937e1fb1ff173240ec5c3ee9f8e9f9491.zip | |
ifcfg: Do not write the NM_CONTROLLED flag unless we know it's value.
If it's not read from the config, it will be read as undef, but written
back as 'no'. This changes the behaviour where no setting means
'automatic' - i.e. if NM is installed, use it, otherwise don't.
If the user wants to be specific, then they make the consious choice
to tick the box in drakx-net.
mga#6675 mga#9261
| -rw-r--r-- | NEWS | 4 | ||||
| -rw-r--r-- | lib/network/connection.pm | 3 |
2 files changed, 6 insertions, 1 deletions
@@ -1,3 +1,7 @@ +- do not write NM_CONTROLLED flag in configs if it's not been explicitly set. + This allows, e.g. NM to run on gnome live CD/DVDs. Drak tools will still + work fine when NM is not installed. mga#6675 mga#9261 + 2.11 - add minimal (i.e. installer) support for shorewall-ipv6 (mga#11928) - drop shorewall 2->3 upgrade support diff --git a/lib/network/connection.pm b/lib/network/connection.pm index 5cab3ac..443c0e4 100644 --- a/lib/network/connection.pm +++ b/lib/network/connection.pm @@ -269,7 +269,8 @@ sub build_ifcfg_settings { DEVICE => $self->get_interface, ONBOOT => bool2yesno($self->{control}{onboot}), ACCOUNTING => bool2yesno($self->{control}{accounting}), - NM_CONTROLLED => bool2yesno($self->{control}{nm_controlled}), + # Only write NM_CONTROLLED if we absolutely know it's value + if_(defined $self->{control}{nm_controlled}, NM_CONTROLLED => bool2yesno($self->{control}{nm_controlled})), USERCTL => bool2yesno($self->{control}{userctl}), METRIC => $self->{control}{metric}, MTU => $self->{control}{mtu}, |