[pacman-dev] [PATCH] makepkg: include all hash types in integlist

[Dave Reisner]

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.