Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:
> No, the only thing that changed was the introduction of Git::Packet (and
> t0021/*.perl uses that). And that Perl module is not yet installed.
Ahh, that is the difference among other users of split(/:/,
$ENV{GITPERLLIB}). Scripts other than 0021 may be using installed
ones and using the wrong separator does not affect them.
> Granted, we tested incorrect versions of those modules, but this late in
> the -rc phase, I would prefer to be cautious. People might be relying on
> the current bug.
People meaning those who run "make test"?
In any case, the patch to 0021 as posted sounds like the most
conservative thing to do at this point, even though we would
definitely want to revisit it and clean it up after the release. As
long as the reason why only 0021 wants the special case while others
are OK is understood, I am OK with the patch ;-)
Thanks.
