[OE-core] [PATCH] libxpm-native: add dependency on gettext-native
Saul Wold
sgw at linux.intel.com
Tue Aug 21 15:40:08 UTC 2012
On 08/20/2012 06:35 PM, Jesse Zhang wrote:
> Hi Saul,
>
> On 08/21/2012 05:16 AM, Saul Wold wrote:
>> On 08/02/2012 06:57 PM, Jesse Zhang wrote:
>>> libxpm needs xgettext to build po files. Add explicit dependency on
>>> gettext-native.
>>>
>>> Signed-off-by: Jesse Zhang <sen.zhang at windriver.com>
>>> ---
>>> meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb | 3 ++-
>>> 1 files changed, 2 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb b/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
>>> index 9eaa212..6830216 100644
>>> --- a/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
>>> +++ b/meta/recipes-graphics/xorg-lib/libxpm_3.5.10.bb
>>> @@ -10,7 +10,8 @@ protocol."
>>> LICENSE = "BSD"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=3e07763d16963c3af12db271a31abaa5"
>>> DEPENDS += "libxext libsm libxt"
>>> -PR = "r0"
>>> +DEPENDS_virtclass-native = "gettext-native"
>>
>> Is there a reason you don't do an inherit gettext here to ensure other bits are set?
>
> Actually I don't know what's the preferred way to do this. What is
> required by libxpm is the xgettext command; using DEPENDS will be enough
> to make it available, but I didn't know the other solution.
>
Ok, well I am telling you there is another, more correct, way to do this
using the inherit gettext, so please try using inherit gettext and see
if that works correctly. The meta/classes/gettext.bbclass in
conjunction with native.bbclass correctly set the DEPENDS to use
gettext-native.
Also, on another note, please modify your patch subject to just say
libxpm, this initial confused a couple of us thinking that libxpm was a
native specific recipe.
Sau!
>> Also for the native, you maybe able to use the gettext-minimal-native if there is reason you can't use the inherit.
>
> gettext-minimal-native is already pulled in through the implicit
> dependency chain. But clearly it doesn't provide xgettext?
>
> jesse
>
>>
>>> +PR = "r1"
>>> PE = "1"
>>>
>>> XORG_PN = "libXpm"
>>>
>>
>>
>
>
More information about the Openembedded-core
mailing list