Hi,
On Mon, May 18, 2015 at 11:20 PM, Johannes Schindelin
wrote:
> Hi Paul,
>
> On 2015-05-18 15:32, Paul Tan wrote:
>
>> @@ -95,7 +94,11 @@ test_expect_success 'test . as a remote' '
>> git checkout copy &&
>> test "$(cat file)" = file &&
>> git pull &&
>> - test "$(cat file)" = updated
>> + test "$(cat file)" = updated &&
>> + git reflog -1 >reflog.actual &&
>> + sed "s/$_x05[0-9a-f]*/OBJID/g" reflog.actual >reflog.fuzzy &&
>
> Actually, let's use "s/^[0-9a-f]*/OBJID/" instead: you only want to replace
> the first few characters.
Did you mean "s/^$_x05[0-9a-f]*/OBJID/"? (with "$_x05" expanding to
'[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f]' from test-lib.sh). If not,
then it would match even if there was no SHA1 hash.
But yes, without the "^" there will very likely be false positives.
Thanks for catching.
>> @@ -106,7 +109,11 @@ test_expect_success 'the default remote . should
>> not break explicit pull' '
>> git reset --hard HEAD^ &&
>> test "$(cat file)" = file &&
>> git pull . second &&
>> - test "$(cat file)" = modified
>> + test "$(cat file)" = modified &&
>> + git reflog -1 >reflog.actual &&
>> + sed "s/$_x05[0-9a-f]*/OBJID/g" reflog.actual >reflog.fuzzy &&
>
> Same here.
Regards,
Paul
--
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