Bug#461367: grub: please use ucf for managing user changes to menu.lst

[Steve Langasek]

Hi Robert,

On Sat, Jan 19, 2008 at 12:47:46PM +0100, Robert Millan wrote:
> On Thu, Jan 17, 2008 at 05:30:29PM -0800, Steve Langasek wrote:
> > --- grub-0.97.orig/debian/grub.postinst
> > +++ grub-0.97/debian/grub.postinst
> > @@ -0,0 +1,9 @@
> > +#!/bin/sh
> > +
> > +if [ "$1" = "configure" ]; then
> > +	if dpkg --compare-versions "$2" lt-nl "0.97-29.1"; then
> > +		update-grub
> > +	fi
> > +fi

> You can't autorun update-grub in Legacy GRUB.  People will yell at you.

Why would that be, aside from the very bug that this is trying to fix?

Feel free to discard that part of the patch if it's inappropriate.  Ubuntu
is already calling update-grub in the postinst for handling updates from
dapper (Ubuntu 6.06), and of course update-grub is already called by default
for each kernel package install, so I think this part of the patch will
remain in the Ubuntu package regardless.

On Sat, Jan 19, 2008 at 01:07:14PM +0100, Robert Millan wrote:

> Besides ...

> On Thu, Jan 17, 2008 at 05:30:29PM -0800, Steve Langasek wrote:
> > At the last Ubuntu Developer Summit, it was decided that we should
> > do something about this for Ubuntu 8.04.[1]

> ... notwithstanding what Ubuntu Developer Summit decides about their own
> bussiness ...

> > Since grub2 is not yet ready to
> > be a default bootloader,

> ... what makes you think it won't be for lenny?

I have no opinion about whether it will be ready for lenny.  It won't be
ready for hardy, which is due out in three months, so for Ubuntu this is
necessary.  If you think grub1 will be obsolete and not ship with lenny,
then feel free to ignore this patch.  If grub1 will ship with lenny even as
a non-default bootloader, though, I think this is an enhancement that's
worth considering, and I would welcome any other feedback you have on the
patch itself.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slangasek at ubuntu.com                                     vorlon at debian.org