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: Robert Millan
Subject: Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments
Date: Mon, 3 Mar 2008 21:58:47 +0100
User-agent: Mutt/1.5.13 (2006-08-11)

On Fri, Feb 29, 2008 at 02:46:17PM +0100, Fabian Greffrath wrote:
> 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... ;)

Maybe Okuji knows... ;-)

> 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>.

Ok no big deal.

-- 
Robert Millan

<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call… if you are unable to speak?
(as seen on /.)




reply via email to

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