Subject: Re: [PATCH 1/2] perf: compare diff algorithms

From: Jakub Narebski <[email-masked]>
Date: Wed, 7 Mar 2012 19:03:01 +0100

Thomas Rast wrote:
> Jakub Narebski <[email-masked]> writes:
> > Junio C Hamano <[email-masked]> writes:
> >> 
> >> But of course, this from a command line would not work:
> >> 
> >> 	$ perl -MGit
> >> 
> >> I do not expect it to, and for the ease of testing new versions, I
> >> prefer it not to work.
> >> 
> >> In any case, you should be able to do anything under t/ _before_
> >> installing, so relying on having Git.pm in normal @INC is a double
> >> no-no.
> >
> > Thomas, take a look at how it is solved in 't/t9700/test.pl', used by
> > 't/t9700-perl-git.sh':
> >
> >   use lib (split(/:/, $ENV{GITPERLLIB}));
> 
> Hum.  The problem is that the user may invoke aggregate.perl manually,
> and GITPERLLIB won't be set in that case.
> 
> Is there a better solution than duplicating the logic that sets
> GITPERLLIB in test-lib.sh within aggregate.perl?

Beside extracting logic that sets GITPERLLIB into separate file like
in Junio proposal?  You can always assume that it is fixed relative
to perl/Git.pm, and use __DIR__ or $FindBin to make "use lib", e.g.

  use FindBin;
  use lib "$FindBin::Bin/../../perl";

-- 
Jakub Narebski
Poland