Comment 10 for bug 15374

Revision history for this message
In , Steve Langasek (vorlon) wrote : Re: lilo ignores failures on upgrade, leaving the system silently unbootable

tags 304260 -fixed
thanks

Hi Bill,

On Wed, May 25, 2005 at 10:18:49AM +0200, <email address hidden> wrote:
> tags 304260 - patch
> thanks
> On Sun, May 15, 2005 at 06:20:45AM -0700, Steve Langasek wrote:
> >
> > Hi Andrés,
> >
> > I've prepared a 0-day NMU for this bug, which will be uploaded shortly. The
> > patch is attached.
> >
> > * debian/patches/15_space-bug.dpatch:
> > diff -u lilo-22.6.1/debian/lilo.postinst lilo-22.6.1/debian/lilo.postinst
> > --- lilo-22.6.1/debian/lilo.postinst
> > +++ lilo-22.6.1/debian/lilo.postinst
> > @@ -68,7 +68,7 @@
> > db_get lilo/runme || true;
> > if [ "$RET" = "true" ]; then
> > echo "Running lilo..."
> > - /sbin/lilo || true
> > + /sbin/lilo
> > fi
> >
> > ;;

> This patch is a noop, because lilo.postinst does not 'set -x', (which is
> yet another policy violation), so the error status of lilo is ignored
> anyway. This caused problems in upgrade-report bug #310653.

Thanks for catching this. Looks like another upload is needed. :/

--
Steve Langasek
postmodern programmer