grub-devel
[Top][All Lists]
Advanced

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

Re: [NEW PATCH] Add option to grub-probe to accept system devices as arg


From: Fabian Greffrath
Subject: Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments
Date: Fri, 29 Feb 2008 14:46:17 +0100
User-agent: Thunderbird 2.0.0.12 (Windows/20080213)

Robert Millan schrieb:
Better do this like it's done in other scripts (check grub-install).  Not sure
why, I guess it's more portable that way...

Ah, now I know what you ment. However, I cannot really see the advantage in

 grub_probe=${sbindir}/`echo grub-probe | sed ${transform}`

with ${transform}="s,x,x,".

I rather reads like WTF to me... ;)


BTW, in another mail from 4 Feb you asked my if the
"echo ... | cut ..." mess in my script could be simplified with the
"echo ... | read ..." feature. I am sorry, but the answer is "no".

This is because read will run in a separate process and cannot set the variables in the parent shell, see <http://www.unixguide.net/unix/bash/E4.shtml>.

Cheers,
Fabian
--
Dipl.-Phys. Fabian Greffrath

Ruhr-Universität Bochum
Lehrstuhl für Energieanlagen und Energieprozesstechnik (LEAT)
Universitätsstr. 150, IB 3/134
D-44780 Bochum

Telefon: +49 (0)234 / 32-26334
Fax:     +49 (0)234 / 32-14227
E-Mail:  address@hidden




reply via email to

[Prev in Thread] Current Thread [Next in Thread]