Loïc Minier <l...@dooz.org> wrote:
> On Wed, Jan 20, 2010, Måns Rullgård wrote:
>> If you make that IFS=${local_ifs:-$(printf ' \t\n')} it should be safe.
>> Likewise if you set the value first.
>
> Ok; see attached patches
I still think that path_of is a complication that we don't really need.
"type" command exist in posix, and althought its output is not defined,
if the output of a real command don't show the path that we need (in
this case /usr/bin/install), then type is pretty wrong in my humble
opinion.
With respect of the rest of the patch. Only real difference is that
mine uses "prog_exists" vs yours use "has" name.
I obviosly think that prog_exists is a more descriptive name, but I
really don't care one way or another.
Can we agree that we don't need path_of, and then just stop playing with
IFS and friends?
Later, Juan.
