-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Sat, 20 Feb 2010, Steffen Maier wrote:
Comments below.
On 02/19/2010 09:38 PM, David Cantrell wrote:
---
iw/zipl_gui.py | 8 ++++----
textw/zipl_text.py | 9 +++++----
2 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/iw/zipl_gui.py b/iw/zipl_gui.py
index a4c36aa..9f2bdc2 100644
--- a/iw/zipl_gui.py
+++ b/iw/zipl_gui.py
@@ -19,11 +19,11 @@
#
# must replace with explcit form so update disks will work
-import isys
import gtk
import string
from iw_gui import *
from constants import *
+from storage.dasd import getDasdPorts
import gettext
_ = lambda x: gettext.ldgettext("anaconda", x)
@@ -101,11 +101,11 @@ class ZiplWindow (InstallWindow):
kernelparms = self.bl.args.get()
else:
kernelparms = ""
- if isys.getDasdPorts() and (kernelparms.find("dasd=") == -1):
+ if getDasdPorts() and (kernelparms.find("dasd=") == -1):
if len(kernelparms) > 0:
- kernelparms = "%s dasd=%s" %(kernelparms, isys.getDasdPorts())
+ kernelparms = "%s dasd=%s" %(kernelparms, getDasdPorts())
This looks like the old style dasd_mod option to activate DASDs on
device driver probing. We have deprecated its use more or less in
anaconda. Using this old style option would be incompatible to rd_DASD
dracutSetupString for root-fs DASDs and s390util's /etc/dasd.conf for
non-root-fs DASDs.
The surrounding code seems to be related to some UI for the zipl bootloader.
I have never seen such UI so far, but anaconda did zipl stuff fully
automatically without choice.
Is this reachable by the user in some way?
Have no idea right now. Will look at this later, hopefully it can go. I'm
mostly interested in just removing unused code at this point. Unnecessary
code is for separate patches.
else:
- kernelparms = "dasd=%s" %(isys.getDasdPorts(),)
+ kernelparms = "dasd=%s" %(getDasdPorts(),)
dito
self.kernelEntry.set_text(kernelparms)
if self.bl.args and self.bl.args.chandevget():
diff --git a/textw/zipl_text.py b/textw/zipl_text.py
index 23789ff..9a1b202 100644
--- a/textw/zipl_text.py
+++ b/textw/zipl_text.py
@@ -18,12 +18,13 @@
# along with this program. If not, see <http://www.gnu.org/licenses/>.
#
-import isys
import string
from snack import *
from constants_text import *
from constants import *
+from storage.dasd import getDasdPorts
+
import gettext
_ = lambda x: gettext.ldgettext("anaconda", x)
@@ -45,11 +46,11 @@ class ZiplWindow:
if self.bl.args and self.bl.args.get():
kernelparms = self.bl.args.get()
- if isys.getDasdPorts() and (kernelparms.find("dasd=") == -1):
+ if getDasdPorts() and (kernelparms.find("dasd=") == -1):
if len(kernelparms) > 0:
- kernelparms = "%s dasd=%s" %(kernelparms, isys.getDasdPorts())
+ kernelparms = "%s dasd=%s" %(kernelparms, getDasdPorts())
dito
else:
- kernelparms = "dasd=%s" %(isys.getDasdPorts(),)
+ kernelparms = "dasd=%s" %(getDasdPorts(),)
dito
kernelentry.set(kernelparms)
if self.bl.args and self.bl.args.chandevget():
Steffen
Linux on System z Development
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Dirk Wittkopp
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.10 (GNU/Linux)
iEYEARECAAYFAkuDIX0ACgkQ5hsjjIy1VknvMgCg9bPnFD9ETxbIZUAwG3m9StfS
HhUAoO9EJeynaNo1ULsDf13mkTTrAx9A
=Dwyg
-----END PGP SIGNATURE-----
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
