[pacman-dev] [PATCH] makepkg: include all hash types in integlist
d at falconindy.com
Tue Sep 17 11:31:56 EDT 2013
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.
More information about the pacman-dev