[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#768340: marked as done (win32-loader: Please use "---" not "--" on installers kernel command line)

Your message dated Tue, 18 Nov 2014 07:33:45 +0000
with message-id <E1XqdIH-0005dj-W8@franck.debian.org>
and subject line Bug#768340: fixed in win32-loader 0.7.7
has caused the Debian Bug report #768340,
regarding win32-loader: Please use "---" not "--" on installers kernel command line
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org

768340: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=768340
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: debian-installer-utils
X-Debbugs-CC: debian-kernel@lists.debian.org

(CCing debian-kernel just FYI, since I don't think this can/should be
fixed with a kernel change, likewise filing against
debian-installer-utils and not the kernel even though a kernel change
introduced the breakage)

A recent change to the kernel[0] (from v3.15 onwards) has changed the
way the kernel handles its command line, such that it now ignores
anything passed after a "--" marker. This has broken d-i's own use of
that marker which was to separate options intended for the installer
only (before the marker) from those which are intended to be both
consumed by the d-i kernel and propagated to the final installation
(after the marker, returned by the user-params utility).

It used to be that you could do:
        vmlinuz some/preseed=value -- console=ttyFOO
which would have the dual affect of having the kernel console (and hence
installer UI) run on ttyFOO and also, via grub-installer's use of
user-params, propagate the console=ttyFOO into the final grub config
(similarly for other bootloaders).

With the kernel change this no longer works -- the kernel doesn't put
its console on ttyFOO since it stops parsing at the --. So you get
silence on boot.

To get the old behaviour you need
        vmlinuz some/preseed=value console=ttyFOO -- console=ttyFOO
which is pretty tedious.

Just using
        vmlinuz some/preseed=value console=ttyFOO
doesn't propagate the console=ttyFOO to the installed system.

Since the kernel change was related to the "systemd abusing kernel
cmdline" debacle I'm not overly keen on raising this upstream and I
don't think that changing the kernel in a way which diverges from
upstream would not be the right approach here.

I don't know how widely used/documented/Supported this ability was, but
I thought e.g. the pxe and isolinux cfgs made use of it.

Not sure what we can do about this. Perhaps choose another separator
("=="?) and make user-params support both?


[0] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=51e158c12aca3c9ac63988611a97c05109b14dc9

--- End Message ---
--- Begin Message ---
Source: win32-loader
Source-Version: 0.7.7

We believe that the bug you reported is fixed in the latest version of
win32-loader, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 768340@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
Didier Raboud <odyx@debian.org> (supplier of updated win32-loader package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)

Hash: SHA256

Format: 1.8
Date: Thu, 13 Nov 2014 14:50:11 +0100
Source: win32-loader
Binary: win32-loader
Architecture: source all
Version: 0.7.7
Distribution: unstable
Urgency: medium
Maintainer: Debian Install System Team <debian-boot@lists.debian.org>
Changed-By: Didier Raboud <odyx@debian.org>
 win32-loader - Debian-Installer loader for win32
Closes: 768340
 win32-loader (0.7.7) unstable; urgency=medium
   The « Lineus longissimus » release
   [ Cyril Brulebois ]
   * Fix typo in previous changelog entry.
   [ Didier Raboud ]
   * Use '---' instead of '--' for the kernel command-line (Closes: #768340)
 9512d88933dedebdf7088a4b045852a8e47ea9a7 1721 win32-loader_0.7.7.dsc
 3ea155b344afd5d026078f8deb6f507286a1fc65 243522 win32-loader_0.7.7.tar.bz2
 fc596c47987d7d784d03570df23a919bb9075bb6 525428 win32-loader_0.7.7_all.deb
 a24f9fad90d08c073dfa07f9528ecdd6b85e2047 1454518 win32-loader_0.7.7_all.exe
 64edf7ad243740feb3f7d9ff23cfcb33945801b7 4445 win32-loader_0.7.7_all.txt
 4042f6e48be3d73fc5736b2e70cca218055d38d830fb3f10c353df05c35dcbca 1721 win32-loader_0.7.7.dsc
 ce35ae8f039b72202ed9c4e2688e8c99b5ffde99d6f4a03d93ab06e87b945e5a 243522 win32-loader_0.7.7.tar.bz2
 be5df62bc734986baf9894b469970dee11231a65ab1cc16e4d3d3176152a46db 525428 win32-loader_0.7.7_all.deb
 6a54a11a523e79b3260fc1fafe5ebe94572331e3afc628e6cdd6fa8119e93fed 1454518 win32-loader_0.7.7_all.exe
 c04e00f59e8bc7bef9c36fb38c49aac0c1048cd732daf92576bf00fe85d18428 4445 win32-loader_0.7.7_all.txt
 7ff58ff2022a8f8906811f132d859e89 1721 utils extra win32-loader_0.7.7.dsc
 24fffa030d502e161a0c53e405f1dc0c 243522 utils extra win32-loader_0.7.7.tar.bz2
 a74862ae8866fe45fbe071d4031fd449 525428 utils extra win32-loader_0.7.7_all.deb
 2f859078c7f509654f7377e0205c1851 1454518 byhand - win32-loader_0.7.7_all.exe
 c1cc5c4aaaf8d36ebd8abac26a6164e6 4445 byhand - win32-loader_0.7.7_all.txt

Version: GnuPG v2


--- End Message ---

Reply to: