[pacman-dev] [PATCH] makepkg: include all hash types in integlist
xyne at archlinux.ca
Tue Sep 17 11:35:57 EDT 2013
On 2013-09-17 11:31 -0400
Dave Reisner wrote:
>On Tue, Sep 17, 2013 at 03:28:31PM +0000, Xyne wrote:
>> Dave Reisner wrote:
>> >This isn't the right fix. It should be:
>> > IFS=$'\n' read -rd '' -a integlist < <(get_integlist)
>> >> else
>> >> integlist=("$@")
>> >> fi
>> Out of curiosity, what difference does it make? Is subshell invocation more
>> expensive than file substitution?
>It's a reliance on word expansion (and the side effects that potentially
>come with it) versus just reading the data as is. Using read is about
>being correct, not about cost.
With IFS set to newline, there will be no unintentional word expansion in
either case. The results are the same. I do not see how one approach is more
correct than the other.
Note that I have submitted a patch with your recommended fix (it's your code
More information about the pacman-dev