Matthias Neeracher <[EMAIL PROTECTED]> said:
> Update of /cvsroot/fink/fink/perlmod/Fink
> Modified Files:
> PkgVersion.pm
> Log Message:
> Integrated support for Ruby modules from ruby-branch into trunk
I kept meaning to look at this earlier, but got side-tracked. Ah well..
>===================================================================
> RCS file: /cvsroot/fink/fink/perlmod/Fink/PkgVersion.pm,v
> diff -u -d -r1.211 -r1.212
> @@ -1595,10 +1600,15 @@
> } elsif (!exists $self->{parent} and $self->param("_type") eq "perl") {
> # grab perl version, if present
> my ($perldirectory, $perlarchdir) = $self->get_perl_dir_arch();
>
> $install_script .=
> "make install PREFIX=\%i
> INSTALLPRIVLIB=\%i/lib/perl5$perldirectory
> INSTALLARCHLIB=\%i/lib/perl5$perldirectory/$perlarchdir
> INSTALLSITELIB=\%i/lib/perl5$perldirectory
> INSTALLSITEARCH=\%i/lib/perl5$perldirectory/$perlarchdir
> INSTALLMAN1DIR=\%i/share/man/man1 INSTALLMAN3DIR=\%i/share/man/man3
> INSTALLSITEMAN1DIR=\%i/share/man/man1 INSTALLSITEMAN3DIR=\%i/share/man/man3
> INSTALLBIN=\%i/bin INSTALLSITEBIN=\%i/bin INSTALLSCRIPT=\%i/bin\n";
> + } elsif ($self->param("_type") eq "ruby") {
> + # grab ruby version, if present
> + my ($rubydirectory, $rubyarchdir) = $self->get_ruby_dir_arch();
> +
> + $install_script .= "make install prefix=\%i\n";
> } elsif (not $do_splitoff) {
> $install_script .= "make install prefix=\%i\n";
> }
What is the effect of get_ruby_dir_arch() here (i.e., why are you
handling type:ruby separately)? But more importantly, it appears you
are having the default action to be "make install" even in a SplitOff
of a type:ruby package. That seems bad.
dan
--
Daniel Macks
[EMAIL PROTECTED]
http://www.netspace.org/~dmacks
-------------------------------------------------------
This SF.net email is sponsored by: Perforce Software.
Perforce is the Fast Software Configuration Management System offering
advanced branching capabilities and atomic changes on 50+ platforms.
Free Eval! http://www.perforce.com/perforce/loadprog.html
_______________________________________________
Fink-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/fink-devel