[pykickstart] Set bootloader location constructor default value to "none" (#916529)

mulhern amulhern at redhat.com
Thu Oct 31 14:31:46 UTC 2013


Related: rhbz#916529

Bootloader location default set in constructor was previously "", which
is not one of the allowed choices; client code
is not likely to expect or check for this value.

There is a possiblity that some client code somewhere uses
bootloader.location=="" as a proxy for not bootloader.seen, in which case
it will break after this change.

Signed-off-by: mulhern <amulhern at redhat.com>
---
 pykickstart/commands/bootloader.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pykickstart/commands/bootloader.py b/pykickstart/commands/bootloader.py
index 32035cf..ae0c0af 100644
--- a/pykickstart/commands/bootloader.py
+++ b/pykickstart/commands/bootloader.py
@@ -32,7 +32,7 @@ class FC3_Bootloader(KickstartCommand):
         self.appendLine = kwargs.get("appendLine", "")
         self.forceLBA = kwargs.get("forceLBA", False)
         self.linear = kwargs.get("linear", True)
-        self.location = kwargs.get("location", "")
+        self.location = kwargs.get("location", "none")
         self.md5pass = kwargs.get("md5pass", "")
         self.password = kwargs.get("password", "")
         self.upgrade = kwargs.get("upgrade", False)
-- 
1.8.3.1



More information about the anaconda-patches mailing list