]> code.ossystems Code Review - openembedded-core.git/commitdiff
perl: fix Perl5 module builds
authorJens Rehsack <rehsack@gmail.com>
Thu, 8 Oct 2015 14:21:21 +0000 (16:21 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 29 Oct 2015 07:28:31 +0000 (07:28 +0000)
This patch fixes some issues in classes providing cpan module build support:

* add support even for xs modules with more than 3 levels as
  B::Hooks::End::Of::Scope or Math::Random::ISAAC::XS
* correct handling of Module::Build (as far as stolen from pkgsrc
  and my humble knowledge)
* configure to install to vendor_libs as default, even when
  inherited do_install remains unused (overwritten do_install)

Signed-off-by: Jens Rehsack <sno@netbsd.org>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/classes/cpan-base.bbclass
meta/classes/cpan.bbclass
meta/classes/cpan_build.bbclass

index d9817ba6b6b6d0b5db4fb6011dad22ae8a86349c..7810a4dc61c49b6adde28bce8bc02bca585b0f89 100644 (file)
@@ -49,7 +49,11 @@ PERLVERSION[vardepvalue] = ""
 FILES_${PN}-dbg += "${PERLLIBDIRS}/auto/*/.debug \
                     ${PERLLIBDIRS}/auto/*/*/.debug \
                     ${PERLLIBDIRS}/auto/*/*/*/.debug \
+                    ${PERLLIBDIRS}/auto/*/*/*/*/.debug \
+                    ${PERLLIBDIRS}/auto/*/*/*/*/*/.debug \
                     ${PERLLIBDIRS}/vendor_perl/${PERLVERSION}/auto/*/.debug \
                     ${PERLLIBDIRS}/vendor_perl/${PERLVERSION}/auto/*/*/.debug \
                     ${PERLLIBDIRS}/vendor_perl/${PERLVERSION}/auto/*/*/*/.debug \
+                    ${PERLLIBDIRS}/vendor_perl/${PERLVERSION}/auto/*/*/*/*/.debug \
+                    ${PERLLIBDIRS}/vendor_perl/${PERLVERSION}/auto/*/*/*/*/*/.debug \
                     "
index e2bbd2f63ac60434c29ff84fc884dc83c4fc5e80..8e079e0d55e403c670ae01e6836d1ffded2e32cf 100644 (file)
@@ -17,7 +17,7 @@ export PERLHOSTLIB = "${STAGING_LIBDIR_NATIVE}/perl-native/perl/${@get_perl_vers
 
 cpan_do_configure () {
        export PERL5LIB="${PERL_ARCHLIB}"
-       yes '' | perl ${EXTRA_PERLFLAGS} Makefile.PL ${EXTRA_CPANFLAGS}
+       yes '' | perl ${EXTRA_PERLFLAGS} Makefile.PL INSTALLDIRS=vendor ${EXTRA_CPANFLAGS}
 
        # Makefile.PLs can exit with success without generating a
        # Makefile, e.g. in cases of missing configure time
index 4f648a60cff4cc9d25191a934ec0781e6d9f2dc6..365374894fb3ae273673807d15eb6c25ad8bffd1 100644 (file)
@@ -8,6 +8,7 @@ EXTRA_CPAN_BUILD_FLAGS ?= ""
 # Env var which tells perl if it should use host (no) or target (yes) settings
 export PERLCONFIGTARGET = "${@is_target(d)}"
 export PERL_ARCHLIB = "${STAGING_LIBDIR}${PERL_OWN_DIR}/perl/${@get_perl_version(d)}"
+export PERLHOSTLIB = "${STAGING_LIBDIR_NATIVE}/perl-native/perl/${@get_perl_version(d)}/"
 export LD = "${CCLD}"
 
 cpan_build_do_configure () {
@@ -24,14 +25,22 @@ cpan_build_do_configure () {
                                --install_path bindoc=${mandir}/man1 \
                                --install_path libdoc=${mandir}/man3 \
                                 ${EXTRA_CPAN_BUILD_FLAGS}
+
+       # Build.PLs can exit with success without generating a
+       # Build, e.g. in cases of missing configure time
+       # dependencies. This is considered a best practice by
+       # cpantesters.org. See:
+       #  * http://wiki.cpantesters.org/wiki/CPANAuthorNotes
+       #  * http://www.nntp.perl.org/group/perl.qa/2008/08/msg11236.html
+       [ -e Build ] || bbfatal "No Build was generated by Build.PL"
 }
 
 cpan_build_do_compile () {
-        perl Build
+        perl Build verbose=1
 }
 
 cpan_build_do_install () {
-       perl Build install
+       perl Build install --destdir ${D}
 }
 
 EXPORT_FUNCTIONS do_configure do_compile do_install