remove the hardcoded default args for bootloader
authorJF Ding <jian-feng.ding@intel.com>
Fri, 3 Feb 2012 11:27:16 +0000 (19:27 +0800)
committerJF Ding <jian-feng.ding@intel.com>
Fri, 3 Feb 2012 11:27:16 +0000 (19:27 +0800)
removed the following settings:
  default "liveinst":"Installation Only" bootmenu item
  default "quiet" kernel args if not specified in ks

mic/imager/livecd.py
mic/kickstart/__init__.py

index 9dc33cf..92d7e99 100644 (file)
@@ -118,17 +118,17 @@ class LiveImageCreatorBase(LoopImageCreator):
 
             A sensible default implementation is provided.
         """
+
         if self.ks is None:
-            r = "ro liveimg quiet"
-            if os.path.exists(self._instroot + "/usr/bin/rhgb"):
-                r += " rhgb"
-            if os.path.exists(self._instroot + "/usr/bin/plymouth"):
-                r += " rhgb"
-            return r
-        r = kickstart.get_kernel_args(self.ks)
+            r = "ro liveimg"
+        else:
+            r = kickstart.get_kernel_args(self.ks)
+
         if os.path.exists(self._instroot + "/usr/bin/rhgb") or \
-           os.path.exists(self._instroot + "/usr/bin/plymouth"):
-            r += " rhgb"
+           os.path.exists(self._instroot + "/usr/bin/plymouth") and \
+           ' rhgb' not in r:
+            r += ' rhgb'
+
         return r
 
     def _get_mkisofs_options(self, isodir):
index d8c264e..d08121d 100644 (file)
@@ -45,8 +45,8 @@ def read_kickstart(path):
     be then passed to an ImageCreator constructor.
 
     If an error occurs, a CreatorError exception is thrown.
-
     """
+
     #version = ksversion.makeVersion()
     #ks = ksparser.KickstartParser(version)
 
@@ -649,7 +649,7 @@ def get_kernel_args(ks, default = "ro liveimg"):
         return default
     return "%s %s" %(default, ks.handler.bootloader.appendLine)
 
-def get_menu_args(ks, default = "liveinst"):
+def get_menu_args(ks, default = ""):
     if not hasattr(ks.handler.bootloader, "menus"):
         return default
     if ks.handler.bootloader.menus in (None, ""):