[Xen-devel] [PATCH OSSTEST 5/7] Debian: ARM: only apply no bootloader workaround if xopts{PvMenuLst}

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

This workaround is only necessary because of how pv-menu-list works,
so we should only apply both or neither of them.

This results in a long line and I'm about to add a second workaround
to this block, so switch to a regular if block instead of postfixing
on the one command. Move the comment inside that block in preparation
for other workarounds as well.

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 Osstest/Debian.pm | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
index 7ce5d67..ff7975e 100644
--- a/Osstest/Debian.pm
+++ b/Osstest/Debian.pm
@@ -913,15 +913,19 @@ d-i     grub-installer/bootdev          string /dev/xvda
 
 END
 
-    # Debian doesn't currently know what bootloader to install in a
-    # Xen guest on ARM. We install pv-grub-menu above which actually
-    # does what we need, but the installer doesn't treat that as a
-    # "bootloader".
     logm("\$arch is $arch, \$suite is $suite");
-    $preseed_file.= (<<END) if $arch =~ /^arm/ && $suite =~ 
/wheezy|jessie|sid/;
+    if ($xopts{PvMenuLst} && $arch =~ /^arm/ &&
+       $suite =~ /wheezy|jessie|sid/ ) {
+
+       # Debian doesn't currently know what bootloader to install in
+       # a Xen guest on ARM. We install pv-grub-menu above which
+       # actually does what we need, but the installer doesn't treat
+       # that as a "bootloader".
+       $preseed_file.= (<<END);
 d-i     nobootloader/confirmation_common boolean true
 
 END
+    }
 
     $preseed_file .= preseed_hook_cmds();
 
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel