Felipe Contreras <felipe.contre...@gmail.com> writes:
> On Fri, May 10, 2013 at 3:28 PM, Junio C Hamano <gits...@pobox.com> wrote:
>> Felipe Contreras <felipe.contre...@gmail.com> writes:
>>
>>> +test_expect_success 'push update refs failure' '
>>> + (cd local &&
>>> + git checkout update &&
>>> + echo "update fail" >>file &&
>>> + git commit -a -m "update fail" &&
>>> + git rev-parse --verify testgit/origin/heads/update >expect &&
>>> + GIT_REMOTE_TESTGIT_PUSH_ERROR="non-fast forward" \
>>> + test_expect_code 1 git push origin update &&
>>
>> This is not portable
>
> Why not? Other parts of this script run commands with environment
> variables like this.
There is a difference between a proper command and a shell function
with respect to the single-shot environment assignment.
VAR=VAL command args
is fine, but
VAR=VAL shell_function args
is not.
People often make this mistake and we had to fix it number of times
in t/ directory.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
