Re: [PATCHv2 6/6] t5543-atomic-push.sh: add basic tests for atomic pushes
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Dec 16, 2014, 20:46 UTC
- Message-ID
- <xmqq7fxr9vc7.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <CAGZ79kZnGKNTpRZndgG_CjAC9gqafGXoq8rFOWuHZkaqY_LK7w@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
Show 11 quoted lines
> On Tue, Dec 16, 2014 at 11:46 AM, Junio C Hamano <gitster@pobox.com> wrote:
>
>>
>> Seeing that all callers of test_ref_upstream computes $2 as
>>
>> git -C workbench rev-parse --verify <something>
>>
>
> Only in the first tests, where this should be the case after push.
> In the failure tests, we go with HEAD@{N} which needs to be computed
> inside the workbench repo.Aren't we saying the same thing? The suggested alternative is to do the "git -C workbench rev-parse" bit inside the helper.