Re: [PATCH 2/2] libuser: enable python support

[Kang Kai <Kai.Kang@windriver.com>]

On 2012年12月07日 08:46, Saul Wold wrote:
> On 12/06/2012 12:16 AM, Kang Kai wrote:
>> Compile libuser with python support.
>>
>> Signed-off-by: Kang Kai <kai.kang@windriver.com>
>> ---
>> meta/recipes-extended/libuser/libuser_0.57.1.bb | 12 +++++++++---
>> 1 files changed, 9 insertions(+), 3 deletions(-)
>>
>> diff --git a/meta/recipes-extended/libuser/libuser_0.57.1.bb 
>> b/meta/recipes-extended/libuser/libuser_0.57.1.bb
>> index 99b661b..3a29695 100644
>> --- a/meta/recipes-extended/libuser/libuser_0.57.1.bb
>> +++ b/meta/recipes-extended/libuser/libuser_0.57.1.bb
>> @@ -15,10 +15,16 @@ SRC_URI = 
>> "https://fedorahosted.org/releases/l/i/libuser/libuser-${PV}.tar.xz"
>>
>> SRC_URI[md5sum] = "be82c6941264d0b4bd04f95fb342ec7d"
>> SRC_URI[sha256sum] = 
>> "a61289867581fa715354a3fafe09c3e481173ce0a2dcb33b04588b6ac13cead5"
>> -PR = "r2"
>> +PR = "r3"
>>
>> DEPENDS = "popt libpam glib-2.0 xz-native docbook-utils-native 
>> linuxdoc-tools-native"
>>
>> -EXTRA_OECONF += "--without-python"
>> +inherit autotools gettext pythonnative python-dir
>> +
>> +EXTRA_OEMAKE = "PYTHON_CPPFLAGS=-I${STAGING_INCDIR}/${PYTHON_DIR}"
>> +
> This change seems to have a problem with the non-gplv3, possibly some 
> library or newer api is needed by the python code.
>
> Please review, you might need a wrapper with INCOMPATIBLE_LICENSE 
> check for enabling or disabling the python.

Sorry for missing test on non-gplv3, I will update and send V2 after 
test on non-gplv3 image.

Regards,
Kai

>
> Thanks
> Sau!
>
>> +PACKAGES += "${PN}-python "
>> +
>> +FILES_${PN}-dbg += "${PYTHON_SITEPACKAGES_DIR}/.debug"
>> +FILES_${PN}-python = "${PYTHON_SITEPACKAGES_DIR}"
>>
>> -inherit autotools gettext
>>
>