On Wed, Oct 01, 2003 at 11:22:01AM -0700, [EMAIL PROTECTED] wrote:
> The only one worth noting is:
> 
> @@ -2092,6 +2092,11 @@
>          $self->{PERLPREFIX}   ||= $iprefix;
>          $self->{SITEPREFIX}   ||= $sprefix;
>          $self->{VENDORPREFIX} ||= $vprefix;
> +     my $p = $self->{PREFIX} = $self->{PERLPREFIX};
> +     for my $t (qw/PERL SITE VENDOR/)
> +     {
> +         $self->{"${t}PREFIX"} =~ s!^\Q$p\E(?=/|$)!\$(PREFIX)!;
> +     }
>      }
>  
>      my $arch    = $Config{archname};
> 
> Which is required to allow PREFIX to be overridden meaningfully on the
> make command line (vs. on the Makefile.PL command line).
> 
> The change was made principally because the Debian perl policy currently
> recommends using "make install PREFIX=..." to install into a temporary
> directory as part of the package building process.

There's a very good reason why I can't allow the above and I don't
remember what it is.  I very specificly couldn't make:

    perl Makefile.PL
    make test
    make install PREFIX=...

work because it broke something...somewhere.  I don't have access to my
mail archives at the moment to figure it out.  It was probably a VMS thing
in which case I might be able to throw an if !VMS around your code above.


> We should probably change policy to suggest using the new DESTDIR
> support, or at least VENDORPREFIX but for the moment I didn't want to
> break existing practice.

Yes, DESTDIR is the way forward.


> I appreciate all your work on MakeMaker BTW, this has been the most
> painless upgrade that I recall.  Thanks.

Thank you. :)


> Various debian-specific ExtUtils changes:
> 
>  * Respect umask during installation, and set as appropriate for each of
>    perl, vendor and site (policy requires group writable site dirs).

Upstream bug for umask() issue...
http://rt.cpan.org/NoAuth/Bug.html?id=3927


>  * Don't install .packlist or perllocal.pod for perl or vendor.

Once the below happens, doing the above should be easier.
http://rt.cpan.org/NoAuth/Bug.html?id=3003

Reply via email to