diff mbox

[Xen-devel,RFC,OSSTEST,07/19] ts-host-install: include console before *and* after the -- marker.

Message ID 1412942554-752-7-git-send-email-ian.campbell@citrix.com
State New
Headers show

Commit Message

Ian Campbell Oct. 10, 2014, 12:02 p.m. UTC
As of v3.15 Linux now stop parsing its command line at the -- mark and passes
the remainder to init. This has broken Debian Installer's feature where
anything after the -- is propagated to the installed system.

See also https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=762007

To work around this we need to pass console= twice, once before the -- for the
current kernel's use and then again after the -- which is propagated to the
installed system.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
 ts-host-install | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Ian Jackson Oct. 10, 2014, 2:05 p.m. UTC | #1
Ian Campbell writes ("[PATCH RFC OSSTEST 07/19] ts-host-install: include console before *and* after the -- marker."):
> As of v3.15 Linux now stop parsing its command line at the -- mark and passes
> the remainder to init. This has broken Debian Installer's feature where
> anything after the -- is propagated to the installed system.
> 
> See also https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=762007

Urgh.  Thanks, systemd.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
diff mbox

Patch

diff --git a/ts-host-install b/ts-host-install
index cf9506d..cdbd774 100755
--- a/ts-host-install
+++ b/ts-host-install
@@ -208,12 +208,16 @@  END
     push @installcmdline,
         get_host_property($ho, "install-append $ho->{Suite}", '');
 
-    push @installcmdline, qw(--);
-
     my $console = get_host_native_linux_console($ho);
 
     push @installcmdline, "console=$console" unless $console eq "NONE";
 
+    push @installcmdline, qw(--);
+
+    # See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=762007 for
+    # why this is repeated.
+    push @installcmdline, "console=$console" unless $console eq "NONE";
+
     my $installcmdline= join ' ', @installcmdline;
 
     setup_pxeboot($ho, <<END);