Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: > Elia Pinto <gitter.spiros@xxxxxxxxx> writes: > >> --- a/t/test-lib-functions.sh >> +++ b/t/test-lib-functions.sh >> @@ -478,7 +478,7 @@ test_external_without_stderr () { >> test_path_is_file () { >> if ! test -f "$1" >> then >> - echo "File $1 doesn't exist. $*" >> + echo "File $1 doesn't exist. $2" >> false >> fi >> } >> @@ -486,7 +486,7 @@ test_path_is_file () { >> test_path_is_dir () { >> if ! test -d "$1" >> then >> - echo "Directory $1 doesn't exist. $*" >> + echo "Directory $1 doesn't exist. $2" >> false >> fi >> } > > Sounds straightforwardly correct to me. Thanks. This however makes me wonder why you were nominated for reviewing this patch, though... -- 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