Re: [PATCH 08/12] t9150-svk-mergetickets.sh: use the $( ... ) construct for command substitution

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Elia Pinto <gitter.spiros@xxxxxxxxx> writes:

>  test_expect_success 'svk merges were represented coming in' "
> -	[ `git cat-file commit HEAD | grep parent | wc -l` -eq 2 ]
> +	[ $(git cat-file commit HEAD | grep parent | wc -l) -eq 2 ]
>  	"

Entirely outside of the theme of this topic, but this should be
corrected to use "test".

More importantly, the outer quote should become single quote.  As it
currently is written, "git cat-file" runs outside the protection of
test_expect_success when the command line of the test_expect_success
is formulated, which defeats the whole purpose of test helper macros
to run everything "git" inside them.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]