[Qemu-devel] Re: Stop using "which" in ./configure

[Paolo Bonzini]

On 01/21/2010 05:53 PM, Jamie Lokier wrote:
> > >  If you make that IFS=${local_ifs:-$(printf ' \t\n')} it should be safe.
> > >  Likewise if you set the value first.
> Remove the colon.  The above will wrongly change empty IFS, which
> is not the same as unset IFS.

local_ifs would never be unset anyway, so it would never trigger.

After all, Loic's code can be considered okay as it was in the first 
place (sorry).  Instead, we should just make sure that no code ever 
unsets IFS.  I committed this recommendation to the Autoconf manual's 
shell portability section, so it's not necessary to add any comment here.

Paolo