SZEDER Gábor <szeder....@gmail.com> writes:

>> +    git commit ${3+"$3"} -m "$2" >act &&
>> +    head -1 <act >tmp &&
>> +    mv tmp act &&
>
> Here you could swap the order of when you write to 'act' and 'tmp',
> and thus make the 'mv' unnecessary, like this:
>
>   git commit [...] >tmp &&
>   head -1 act >tmp &&
>   [...rest of the test...]
>
> Note also that 'head' (or 'sed' in later tests) can open its input
> file on its own, there's no need to redirect its standard input.

Yup, that is better with a slight fix

        git testing-output-from-this-command ... >tmp &&
        head -n 1 tmp >actual &&

to actually read from the output kept in the temporary file.

Especially I think bare -<num> is a BSDism that has been
removed from POSIX some time ago.

Reply via email to