Bug #69553 for XML-LibXML: install_sax_driver doesn't like custom INSTALLARCHLIB

Mon Jul 18 13:27:59 2011 https://www.google.com/accounts/o8/id?id=AItOawn1H1XofyQVeS7wUC8jmOudW_lS-OoTG_k - Ticket created

Subject:

install_sax_driver doesn't like custom INSTALLARCHLIB

install_sax_driver doesn't take into account that library files might be install somewhere other than INSTALLSITELIB and we end up with something like: Can't locate XML/LibXML/SAX/Parser.pm in @INC (@INC contains: /usr/local/lib/perl5/site_perl/5.12.4 /usr/local/lib/perl5/5.12.4/BSDPAN /usr/local/lib/perl5/site_perl/5.12.4/mach /usr/local/lib/perl5/site_perl/5.12.4 /usr/local/lib/perl5/5.12.4/mach /usr/local/lib/perl5/5.12.4 .) at /usr/local/lib/perl5/site_perl/5.12.4/XML/SAX.pm line 147. For example, FreeBSD might install the module with: perl Makefile.PL PREFIX=/usr/local INSTALLPRIVLIB=/usr/local/lib INSTALLARCHLIB=/usr/local/lib and XML::LibXML::SAX::Parser ends up in /usr/local/lib/perl5/site_perl/5.12.4/mach/XML/libXML/SAX, which is indeed not in @INC. I've attached the patch which simply adds the ARCH path to the commands in install_sax_driver.

Subject:

patch-Makefile.PL.in

Download patch-Makefile.PL.in
application/octet-stream 698b

Message body not shown because it is not plain text.

Wed Jul 20 16:07:35 2011 SHLOMIF [...] cpan.org - Correspondence added

Hi Milki, On Mon Jul 18 13:27:59 2011, https://www.google.com/accounts/o8/id?id=AItOawn1H1XofyQVeS7wUC8jmOudW_lS-OoTG_k wrote: Show quoted text

> install_sax_driver doesn't take into account that library files might be > install somewhere other than INSTALLSITELIB and we end up with something > like: > > Can't locate XML/LibXML/SAX/Parser.pm in @INC (@INC contains: > /usr/local/lib/perl5/site_perl/5.12.4 /usr/local/lib/perl5/5.12.4/BSDPAN > /usr/local/lib/perl5/site_perl/5.12.4/mach > /usr/local/lib/perl5/site_perl/5.12.4 /usr/local/lib/perl5/5.12.4/mach > /usr/local/lib/perl5/5.12.4 .) at > /usr/local/lib/perl5/site_perl/5.12.4/XML/SAX.pm line 147. > > > For example, FreeBSD might install the module with: > perl Makefile.PL PREFIX=/usr/local INSTALLPRIVLIB=/usr/local/lib > INSTALLARCHLIB=/usr/local/lib > > and XML::LibXML::SAX::Parser ends up in > /usr/local/lib/perl5/site_perl/5.12.4/mach/XML/libXML/SAX, which is > indeed not in @INC. > > I've attached the patch which simply adds the ARCH path to the commands > in install_sax_driver.

thanks! I applied your patch in the repository and will release it to CPAN soon. Sorry it took me so long. Regards, -- Shlomi Fish

Wed Jul 20 16:07:36 2011 The RT System itself - Status changed from 'new' to 'open'

Wed Jul 20 16:07:59 2011 SHLOMIF [...] cpan.org - Correspondence added

Resolving per the previous comment.

Wed Jul 20 16:08:00 2011 SHLOMIF [...] cpan.org - Status changed from 'open' to 'resolved'