Jiang Xin <worldhello.net@xxxxxxxxx> writes:
>> This couldn't have possibly passed with the trailing &&, or am I
>> missing something? There is already "add another remote" before
>> this test that adds "second" remote. Is this test about "add
>> yet another remote", or is it checking the result of adding "second"
>> that was done in the previous step?
>
> The trailing "&&“ is a copy & paste error. I only run my fixup in
> GIT_GETTEXT_POISON mode in a harry, not noticed the bypassed
> testcase has this serious bug.
>
> I split the original "add another remote" into two blocks. One is a
> normal testcase, and another has a C_LOCALE_OUTPUT prereq
> flag. This is because other testcases depend on the operations in
> "add another remote" testcase ('git remote add -f second ../two'),
> and these testcases would fail if add C_LOCALE_OUTPUT
> prereq to the whole "add another remote" testcase.
OK, so in short, I think the fix-up I made while queuing (and the
typofix in the commit log message for the other one) this round to
'pu' should match what you wanted to do. If you want to make
further changes, I wouldn't mind re-queuing an update, but otherwise
there is no need to resend this series (and it will be less work for
me ;-).
Thanks.
--
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