[Buildroot] [PATCH 1/2] pkgconf: pkg-config.in: double quote $@

Thomas Petazzoni thomas.petazzoni at bootlin.com
Thu Feb 22 15:49:49 UTC 2018


Hello,

On Wed, 21 Feb 2018 18:48:54 -0500, Gaël PORTAY wrote:

> > Could you give more details about what this is fixing, i.e a specific
> > scenario that is fixed by this patch ?  
> 
> In the case pkgconf, I can not see a real situation where this patch
> fixes an issue.
> 
> There is an important difference between $@ and "$@". The shell expands
> "$@" as "$1" "$2" "$3"... while it expands $@ as $1 $2 $3.
> 
> With the second form, we losts spaces in positional parameters.
> 
> As example, the following call
> 
> 	pkg-config --cflags "one two" three
> 
> is wrapped as
> 
> 	pkgconf --cflags one two three
> 
> while we are expecting
> 
> 	pkgconf --cflags "one two" three
> 
> "$@" is really useful when writing wrappers. It passes the positional
> arguments *as* they are given.

OK, thanks for the explanation, makes sense. A better commit log would
definitely help :)

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com



More information about the buildroot mailing list