Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Control: tags -1 + pending On Sun, 2015-12-13 at 20:42 -0500, Milan Kupcevic wrote: > On 12/13/2015 05:21 PM, Adam D. Barratt wrote: > > Control: tags -1 -moreinfo +confirmed > > > > On Sun, 2015-12-13 at 13:01 -0500, Milan Kupcevic wrote: > >> Hi Adam, > >> > >> We've got third party confirmations and requests[0][1] for this fix. > >> Should I upload mkvmlinuz/37+deb8u1 to s-p-u? > > > > I was rather hoping that your poking of debian-kernel might elicit some > > response, but apparently not. Please go ahead. > > > > Thanks. Uploaded. Flagged for acceptance. Regards, Adam
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
On 12/13/2015 05:21 PM, Adam D. Barratt wrote: > Control: tags -1 -moreinfo +confirmed > > On Sun, 2015-12-13 at 13:01 -0500, Milan Kupcevic wrote: >> Hi Adam, >> >> We've got third party confirmations and requests[0][1] for this fix. >> Should I upload mkvmlinuz/37+deb8u1 to s-p-u? > > I was rather hoping that your poking of debian-kernel might elicit some > response, but apparently not. Please go ahead. > Thanks. Uploaded. Milan signature.asc Description: OpenPGP digital signature
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Hi Adam, We've got third party confirmations and requests[0][1] for this fix. Should I upload mkvmlinuz/37+deb8u1 to s-p-u? Milan [0] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=793556#28 [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=741642#56 signature.asc Description: OpenPGP digital signature
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Control: tags -1 -moreinfo +confirmed On Sun, 2015-12-13 at 13:01 -0500, Milan Kupcevic wrote: > Hi Adam, > > We've got third party confirmations and requests[0][1] for this fix. > Should I upload mkvmlinuz/37+deb8u1 to s-p-u? I was rather hoping that your poking of debian-kernel might elicit some response, but apparently not. Please go ahead. Regards, Adam
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
On 12/08/2015 01:10 PM, Gerhard Pircher wrote: [ ... ] > It has been verified that the bugfix in version mkvmlinuz/37+deb8u1 > fixes this problem. Thus I would to ask, if this new mkvmlinuz > version will be available anytime soon in Debian stable? > I'm waiting for release managers decision at this point. Milan signature.asc Description: OpenPGP digital signature
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Hello, Currently fresh installations of Debian Jessie are left in a non-bootable state on the Pegasos2, if the boot image is loaded directly from an ext3 or AFFS partition (for Linux/MorphOS/ AmigaOS4 multi boot setups) by OpenFirmware resp. mkvmlinuz is installed in place of a real bootloader. It has been verified that the bugfix in version mkvmlinuz/37+deb8u1 fixes this problem. Thus I would to ask, if this new mkvmlinuz version will be available anytime soon in Debian stable? Best regards, Gerhard
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
On 07/25/2015 06:38 AM, Adam D. Barratt wrote: Control: tags -1 + moreinfo On Fri, 2015-07-24 at 22:32 -0400, Milan Kupcevic wrote: mkvmlinuz/37+deb8u1 is fixing bug #741642 already fixed in sid/testing to allow for smooth upgrade from wheezy to jessie. See attached diff. changelog |6 ++ kernel-image/postinst |2 ++ kernel-image/postrm |2 ++ --- mkvmlinuz-37/debian/kernel-image/postinst 2012-06-28 21:01:13.0 -0400 +++ mkvmlinuz-37+deb8u1/debian/kernel-image/postinst2015-07-23 22:45:48.0 -0400 @@ -1,5 +1,7 @@ #!/bin/sh +echo 2 Will the mkvmlinuz invocation later in the postinst ever possibly produce output on stdout? The db_get debconf call does communicate with debconf through stdin/stdout by design. That is why nothing else should use stdin nor stdout at stake for not to interfere with such communication. The violation actually comes from 'run-parts --report' itself, not from the hook. Wheezy kernel package has been using 'run-parts --verbose' to run the hooks. This worked fine as run-parts printed out to stderr only. Jessie kernel package is using 'run-parts --report' instead where output depends on the behavior of the hook in question. See run-parts manual for the exact --report run-parts option description. I'm dealing with this problem by printing out a newline character to stderr from the hook itself, therefore pushing run-parts to output the hook's name to stderr, thus causing run-parts to leave the debconf communication alone. This fix is as minimalist as you possibly could get for a bug fix in a stable release. CC: debian-kernel to get possible input from there While I can see the intent behind the above (forcing run-parts --report to output the script name to stderr rather than stdout), it wasn't immediately obvious. The kernel handbook (in https://kernel-handbook.alioth.debian.org/ch-update-hooks.html ), suggests exec /dev/null 2, which seems much more idiomatic. The kernel-handbook suggestion won't fix this issue because run-parts interprets debconf communication as hook output and decides that it has to print out the hook script name to the same place, thus interfering with debconf communication and causing the error described in bug reports #791868 and #741642. Milan signature.asc Description: OpenPGP digital signature
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Control: tags -1 + moreinfo On Fri, 2015-07-24 at 22:32 -0400, Milan Kupcevic wrote: mkvmlinuz/37+deb8u1 is fixing bug #741642 already fixed in sid/testing to allow for smooth upgrade from wheezy to jessie. See attached diff. changelog |6 ++ kernel-image/postinst |2 ++ kernel-image/postrm |2 ++ --- mkvmlinuz-37/debian/kernel-image/postinst 2012-06-28 21:01:13.0 -0400 +++ mkvmlinuz-37+deb8u1/debian/kernel-image/postinst2015-07-23 22:45:48.0 -0400 @@ -1,5 +1,7 @@ #!/bin/sh +echo 2 Will the mkvmlinuz invocation later in the postinst ever possibly produce output on stdout? While I can see the intent behind the above (forcing run-parts --report to output the script name to stderr rather than stdout), it wasn't immediately obvious. The kernel handbook (in https://kernel-handbook.alioth.debian.org/ch-update-hooks.html ), suggests exec /dev/null 2, which seems much more idiomatic. Regards, Adam -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#793556: jessie-pu: package mkvmlinuz/37+deb8u1
Package: release.debian.org Severity: normal Tags: jessie User: release.debian@packages.debian.org Usertags: pu mkvmlinuz/37+deb8u1 is fixing bug #741642 already fixed in sid/testing to allow for smooth upgrade from wheezy to jessie. See attached diff. changelog |6 ++ kernel-image/postinst |2 ++ kernel-image/postrm |2 ++ 3 files changed, 10 insertions(+) -- System Information: Debian Release: 8.1 APT prefers stable-updates APT policy: (500, 'stable-updates'), (500, 'stable') Architecture: powerpc (ppc64) Kernel: Linux 3.16.0-4-powerpc64 (SMP w/4 CPU core) Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system) diff -Nru mkvmlinuz-37/debian/changelog mkvmlinuz-37+deb8u1/debian/changelog --- mkvmlinuz-37/debian/changelog 2015-04-10 07:14:08.0 -0400 +++ mkvmlinuz-37+deb8u1/debian/changelog 2015-07-23 23:40:49.0 -0400 @@ -1,3 +1,9 @@ +mkvmlinuz (37+deb8u1) stable; urgency=medium + + * Push run-parts output to stderr. (Closes: #741642) + + -- Milan Kupcevic mi...@physics.harvard.edu Thu, 23 Jul 2015 23:00:46 -0400 + mkvmlinuz (37) unstable; urgency=medium * Include only necessary modules to further reduce vmlinuz size on Pegasos. diff -Nru mkvmlinuz-37/debian/kernel-image/postinst mkvmlinuz-37+deb8u1/debian/kernel-image/postinst --- mkvmlinuz-37/debian/kernel-image/postinst 2012-06-28 21:01:13.0 -0400 +++ mkvmlinuz-37+deb8u1/debian/kernel-image/postinst 2015-07-23 22:45:48.0 -0400 @@ -1,5 +1,7 @@ #!/bin/sh +echo 2 + set -e . /usr/share/debconf/confmodule diff -Nru mkvmlinuz-37/debian/kernel-image/postrm mkvmlinuz-37+deb8u1/debian/kernel-image/postrm --- mkvmlinuz-37/debian/kernel-image/postrm 2012-06-28 21:01:13.0 -0400 +++ mkvmlinuz-37+deb8u1/debian/kernel-image/postrm 2015-07-23 22:45:48.0 -0400 @@ -1,5 +1,7 @@ #!/bin/sh +echo 2 + set -e . /usr/share/debconf/confmodule