Re: [PATCH 1/1] libxml2: fix python path and enable libxml2-python

[Robert Yang <liezhi.yang@windriver.com>]

On 09/15/2014 06:43 AM, Burton, Ross wrote:
> On 9 September 2014 08:34, Robert Yang <liezhi.yang@windriver.com> wrote:
>> We have libxml2-python for native and nativesdk, but don't have it for
>> target, and can't find the reason from the git log, libxml2-python is
>> widely used, after looked into it's configure.in, we can enable it
>> easily.
>
> It seems like adding a PACKAGECONFIG for Python would be a good idea,
> we don't want to force target Python to be built.

Thanks, updated in the repo:

   git://git.openembedded.org/openembedded-core-contrib rbt/libxml2

commit ff4a9f5118f39d82fe1bb7fcc7870ef01a9cdb0b
Author: Robert Yang <liezhi.yang@windriver.com>
Date:   Mon Sep 8 20:24:38 2014 -0700

     libxml2: fix python path and add libxml2-python

     We have libxml2-python for native and nativesdk, but don't have it for
     target, and can't find the reason from the git log, libxml2-python is
     widely used, after looked into it's configure.in, we can add it
     by PACKAGECONFIG.

     The previous --with-python=${STAGING_BINDIR}/python is incorrect, it
     acted as work becase it's conigure can check automatically, python is in
     ${STAGING_BINDIR_NATIVE}/${PYTHON_PN}-native/${PYTHON_PN}, as known as
     ${PYTHON}.

     Signed-off-by: Robert Yang <liezhi.yang@windriver.com>

diff --git a/meta/recipes-core/libxml/libxml2.inc 
b/meta/recipes-core/libxml/libxml2.inc
index bcf9a62..4d11c2b 100644
--- a/meta/recipes-core/libxml/libxml2.inc
+++ b/meta/recipes-core/libxml/libxml2.inc
@@ -36,15 +36,14 @@ do_configure_prepend () {
         sed -i -e '/.*ansidecl.h.*/d' ${S}/configure.in
  }

-do_configure_prepend_class-nativesdk () {
-       # Ensure we get the correct site-packages path
-       export PYTHON_SITE_PACKAGES="${PYTHON_SITEPACKAGES_DIR}"
-}
+export PYTHON_SITE_PACKAGES="${PYTHON_SITEPACKAGES_DIR}"

+PACKAGECONFIG ??= ""
+PACKAGECONFIG[python] = 
"--with-python=${PYTHON},--without-python,python,python-core"
  # WARNING: zlib is require for RPM use
-EXTRA_OECONF = "--without-python --without-debug --without-legacy 
--with-catalog --without-docbook --with-c14n --without-lzma --with-fexceptions"
-EXTRA_OECONF_class-native = "--with-python=${STAGING_BINDIR}/python 
--without-legacy --without-docbook --with-c14n --without-lzma --with-zlib"
-EXTRA_OECONF_class-nativesdk = "--with-python=${STAGING_BINDIR}/python 
--without-legacy --without-docbook --with-c14n --without-lzma --with-zlib"
+EXTRA_OECONF = "--without-debug --without-legacy --with-catalog 
--without-docbook --with-c14n --without-lzma --with-fexceptions"
+EXTRA_OECONF_class-native = "--with-python=${PYTHON} --without-legacy 
--without-docbook --with-c14n --without-lzma --with-zlib"
+EXTRA_OECONF_class-nativesdk = "--with-python=${PYTHON} --without-legacy 
--without-docbook --with-c14n --without-lzma --with-zlib"
  EXTRA_OECONF_linuxstdbase = "--without-python --with-debug --with-legacy 
--with-docbook --with-c14n --without-lzma --with-zlib"

  # required for pythong binding

// Robert

>
> Ross
>
>