David, I could not find this in master nor in f12-branch. Am I missing something? 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