Re: [PATCH] Parse commandline in grub-xen

[Vladimir 'phcoder' Serbinenko]

On May 11, 2015 3:34 PM, "Olaf Hering" <address@hidden> wrote:
>
> On Mon, May 11, Vladimir 'phcoder' Serbinenko wrote:
>
> > On May 11, 2015 2:16 PM, "Olaf Hering" <address@hidden> wrote:
> > > The use of uninitialized vars has to be caught by the script author no?
> > It never works this way
>
> So? How does it work then? Because thats what I will propose to my
> package maintainer later this week...
>
Never rely on people checking their scripts for anything. If you can make introducing bugs into scripts by adding prefix then do so. I fail to understand your stubbornness for a bad design
> Olaf
>
> > > Code like that is valid IMO:
> > >
> > > set localvar=""
> > > if [ -n "${whatever}" ];then
> > >   echo "do whatever implies"
> > >   set localvar="val"
> > > fi
> > > if [ -n "${localvar}" ];then
> > >   echo "do whatever local things"
> > > fi
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/grub-devel