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
