Bug#486039: closed by Robert Millan <rmh at aybabtu.com> (Bug#486039: fixed in grub2 1.96+20080617-1)

[Raphael Geissert]

On Tuesday 17 June 2008, Robert Millan wrote:
> On Mon, Jun 16, 2008 at 10:06:00PM -0500, Raphael Geissert wrote:
> > Source: grub2
> > Version: 1.96+20080617-1
> >
> > On Monday 16 June 2008, Debian Bug Tracking System wrote:
> > > This is an automatic notification regarding your Bug report
> > > which was filed against the grub-efi package:
> > >
> > > #486039: grub-efi: bashism in /bin/sh script
> > >
> > > It has been closed by Robert Millan <rmh at aybabtu.com>.
> > >
> > >      - Uses EUID instead of UID in update-grub.  (Closes: #486043,
> > > #486039, #486040, #486041).
> >
> > At first I though they had just s/UID/EUID/g, but seems not.
> >
> > But checking the code...
> >
> > > if [ "x$EUID" = "x" ] ; then
> > >   EUID=`id -u`
> > > fi
> >
> > is wrong, id -u is the equivalent of $UID, id -ru of $EUID.
>
> $ sudo bash -c "id -ru"
> 0
> $ sudo bash -c "echo $EUID"
> 1000

It should be sudo bash -c 'echo $EUID', otherwise $EUID is replaced before 
sudo is called.

>
> I think this is all bogus, and I don't have time to trace down these
> things.
>
> Fortunately, this has such a low priority that it doesn't matter at all.


Cheers,
-- 
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://lists.alioth.debian.org/pipermail/pkg-grub-devel/attachments/20080619/763b8810/attachment.pgp