-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Sun, 11 Oct 2009, Steffen Maier wrote:
David, I could not find this in master nor in f12-branch.
Am I missing something?
Looks like I forgot to git am this patch, even though I had it queued up.
Committed now. Sorry about that.
On 10/06/2009 09:09 PM, David Cantrell wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Ack.
On Mon, 5 Oct 2009, Steffen Maier wrote:
On s390x, linuxrc.s390 brings up the network completely and
writes an NM compliant /etc/sysconfig/network-scripts/ifcfg-<DEVICE>.
Apparently, loader never hits STEP_IP or writeEnabledNetworkInfo().
The latter would convert LAYER2= and PORTNO= into OPTIONS="".
Since it does not seem to be called, linuxrc.s390 now writes OPTIONS="".
Fixed readIfcfgContents() to not swallow OPTIONS which contains '='.
With this in place, network.py writes a correct ifcfg to /mnt/sysimage.
This patch completes ifcfg production on s390x of the following commits:
25f58fe7c701c453d39d2a9c5c0850eefce07f76
9249e40f42ffbbdcf42cd1caad72e3d622c7a75b
5f0fcf6688d08f83826c2892bb9fc97d6b4d7dd0
81163960a6137d39a5f2082e9f8d9f0a7b2ada57
This patch is intended for both devel and F-12.
---
loader/linuxrc.s390 | 15 ++++++++++-----
network.py | 2 +-
2 files changed, 11 insertions(+), 6 deletions(-)
mode change 100644 => 100755 loader/linuxrc.s390
diff --git a/loader/linuxrc.s390 b/loader/linuxrc.s390
old mode 100644
new mode 100755
index baec1e3..3b2a8d4
--- a/loader/linuxrc.s390
+++ b/loader/linuxrc.s390
@@ -3110,12 +3110,17 @@ fi
[ "$PEERID" != "" ] && echo "PEERID=$PEERID" >> $IFCFGFILE
[ "$PORTNAME" != "" ] && echo "PORTNAME=$PORTNAME" >> $IFCFGFILE
[ "$CTCPROT" != "" ] && echo "CTCPROT=$CTCPROT" >> $IFCFGFILE
-[ "$LAYER2" != "" ] && echo "LAYER2=$LAYER2" >> $IFCFGFILE
[ "$MACADDR" != "" ] && echo "MACADDR=$MACADDR" >> $IFCFGFILE
-[ "$PORTNO" != "" ] && echo "PORTNO=$PORTNO" >> $IFCFGFILE
-# also needs support in stage2 (loader.c,(net.c)) to make PORTNO
persistent,
-# i.e. OPTION="portno=1" in
/etc/sysconfig/network-scripts/ifcfg-<netdevname>
-# (see loader of RHEL 5.3)
+optstr=""
+for option in LAYER2 PORTNO; do
+ [ -z "${!option}" ] && continue
+ [ -n "$optstr" ] && optstr=${optstr}" "
+ optstr=${optstr}$(echo ${option} | tr [[:upper:]]
[[:lower:]])"="${!option}
+done
+# write single quotes since network.py removes double quotes but we
need quotes
+echo "OPTIONS='$optstr'" >> $IFCFGFILE
+unset option
+unset optstr
if [ -z "$testing" ]; then
diff --git a/network.py b/network.py
index e9f6ad8..3a0e721 100644
--- a/network.py
+++ b/network.py
@@ -366,7 +366,7 @@ class Network:
if line.startswith('#') or line == '':
continue
- var = string.splitfields(line, '=')
+ var = string.splitfields(line, '=', 1)
if len(var) == 2:
var[1] = var[1].replace('"', '')
contents[var[0]] = string.strip(var[1])
- -- David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
iEYEARECAAYFAkrLlfQACgkQ5hsjjIy1VkmfnwCgvOy7Bgey2Cq9qoyJOWZwaZgZ
hbkAmwbMcWiYhD2k0X7eaiQKBXKv8ZW9
=KZc8
-----END PGP SIGNATURE-----
Steffen
Linux on System z Development
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Erich Baier
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
- --
David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
iEYEARECAAYFAkrXjvMACgkQ5hsjjIy1VkkCFwCgsczggp34YLpLFFkGXgifXYv8
hFoAmgLPfkDCi0SeFrNuLM/YG864uo30
=fYSZ
-----END PGP SIGNATURE-----
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list