* [PATCH 0/1] shared-mime-info: add libxml-parser-perl to DEPENDS
@ 2011-09-22 18:49 Darren Hart
2011-09-22 18:49 ` [PATCH 1/1] " Darren Hart
0 siblings, 1 reply; 10+ messages in thread
From: Darren Hart @ 2011-09-22 18:49 UTC (permalink / raw)
To: openembedded-core, sgw; +Cc: Darren Hart
The following changes since commit ff8f0ea563ba3e9d6f8b8e770cfbf4dca8ad5288:
libxml2: reinclude a -staticdev package (2011-09-22 05:12:31 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib dvhart/qemuppc
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/qemuppc
Darren Hart (1):
shared-mime-info: add libxml-parser-perl to DEPENDS
.../shared-mime-info/shared-mime-info.inc | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
--
1.7.6.2
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 18:49 [PATCH 0/1] shared-mime-info: add libxml-parser-perl to DEPENDS Darren Hart
@ 2011-09-22 18:49 ` Darren Hart
2011-09-22 19:58 ` Richard Purdie
0 siblings, 1 reply; 10+ messages in thread
From: Darren Hart @ 2011-09-22 18:49 UTC (permalink / raw)
To: openembedded-core, sgw; +Cc: Darren Hart
Fixes [YOCTO #1514]
Building core-image-minimal on qemuppc (and likely others) can fail in
the shared-mime-info configure step with the following error:
checking for XML::Parser... configure: error: XML::Parser perl module is
required for intltool
Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
successfully.
Testing: build tested the native version on qemuppc. I run into tool chain
build failures before I can get to building the target version.
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
CC: Joshua Lock <josh@linux.intel.com>
---
.../shared-mime-info/shared-mime-info.inc | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
index 64eef9d..e206305 100644
--- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
+++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
@@ -5,8 +5,8 @@ SECTION = "base"
LICENSE = "GPLv2+"
LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
-DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
-DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
+DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
+DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
--
1.7.6.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 18:49 ` [PATCH 1/1] " Darren Hart
@ 2011-09-22 19:58 ` Richard Purdie
2011-09-22 20:03 ` Saul Wold
2011-09-22 20:27 ` Darren Hart
0 siblings, 2 replies; 10+ messages in thread
From: Richard Purdie @ 2011-09-22 19:58 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Darren Hart
On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
> Fixes [YOCTO #1514]
>
> Building core-image-minimal on qemuppc (and likely others) can fail in
> the shared-mime-info configure step with the following error:
>
> checking for XML::Parser... configure: error: XML::Parser perl module is
> required for intltool
>
> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
> successfully.
>
> Testing: build tested the native version on qemuppc. I run into tool chain
> build failures before I can get to building the target version.
>
> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> CC: Joshua Lock <josh@linux.intel.com>
> ---
> .../shared-mime-info/shared-mime-info.inc | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> index 64eef9d..e206305 100644
> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> @@ -5,8 +5,8 @@ SECTION = "base"
> LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>
> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>
> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
There must be some underlying problem here. intltool-native should
depend on libxml-parser-perl-native so it should already be built. We
need to fix the underlying issue, not the symptom :/
Cheers,
Richard
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 19:58 ` Richard Purdie
@ 2011-09-22 20:03 ` Saul Wold
2011-09-22 20:19 ` Darren Hart
2011-09-22 20:32 ` Darren Hart
2011-09-22 20:27 ` Darren Hart
1 sibling, 2 replies; 10+ messages in thread
From: Saul Wold @ 2011-09-22 20:03 UTC (permalink / raw)
To: Richard Purdie
Cc: Darren Hart, Patches and discussions about the oe-core layer
On 09/22/2011 12:58 PM, Richard Purdie wrote:
> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
>> Fixes [YOCTO #1514]
>>
>> Building core-image-minimal on qemuppc (and likely others) can fail in
>> the shared-mime-info configure step with the following error:
>>
I was thinking about this and went "why is shared-mime-info even in
core-image-minimal", are you sure that was the image you were building.
This might point to some other problem also.
Sau!
>> checking for XML::Parser... configure: error: XML::Parser perl module is
>> required for intltool
>>
>> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
>> successfully.
>>
>> Testing: build tested the native version on qemuppc. I run into tool chain
>> build failures before I can get to building the target version.
>>
>> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
>> CC: Joshua Lock<josh@linux.intel.com>
>> ---
>> .../shared-mime-info/shared-mime-info.inc | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> index 64eef9d..e206305 100644
>> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> @@ -5,8 +5,8 @@ SECTION = "base"
>> LICENSE = "GPLv2+"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>>
>> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
>> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
>> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
>> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>>
>> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
>
>
> There must be some underlying problem here. intltool-native should
> depend on libxml-parser-perl-native so it should already be built. We
> need to fix the underlying issue, not the symptom :/
>
> Cheers,
>
> Richard
>
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 20:03 ` Saul Wold
@ 2011-09-22 20:19 ` Darren Hart
2011-09-22 20:53 ` Richard Purdie
2011-09-22 20:32 ` Darren Hart
1 sibling, 1 reply; 10+ messages in thread
From: Darren Hart @ 2011-09-22 20:19 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On 09/22/2011 01:03 PM, Saul Wold wrote:
> On 09/22/2011 12:58 PM, Richard Purdie wrote:
>> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
>>> Fixes [YOCTO #1514]
>>>
>>> Building core-image-minimal on qemuppc (and likely others) can fail in
>>> the shared-mime-info configure step with the following error:
>>>
> I was thinking about this and went "why is shared-mime-info even in
> core-image-minimal", are you sure that was the image you were building.
> This might point to some other problem also.
Yes, core-image-minimal. It also would fail if I just tried to build
shared-mime-info-native.
--
Darren
>
> Sau!
>
>>> checking for XML::Parser... configure: error: XML::Parser perl module is
>>> required for intltool
>>>
>>> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
>>> successfully.
>>>
>>> Testing: build tested the native version on qemuppc. I run into tool chain
>>> build failures before I can get to building the target version.
>>>
>>> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
>>> CC: Joshua Lock<josh@linux.intel.com>
>>> ---
>>> .../shared-mime-info/shared-mime-info.inc | 4 ++--
>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> index 64eef9d..e206305 100644
>>> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> @@ -5,8 +5,8 @@ SECTION = "base"
>>> LICENSE = "GPLv2+"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>>>
>>> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
>>> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
>>> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
>>> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>>>
>>> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
>>
>>
>> There must be some underlying problem here. intltool-native should
>> depend on libxml-parser-perl-native so it should already be built. We
>> need to fix the underlying issue, not the symptom :/
>>
>> Cheers,
>>
>> Richard
>>
>>
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 20:19 ` Darren Hart
@ 2011-09-22 20:53 ` Richard Purdie
0 siblings, 0 replies; 10+ messages in thread
From: Richard Purdie @ 2011-09-22 20:53 UTC (permalink / raw)
To: Darren Hart; +Cc: oe-core layer, Patches
On Thu, 2011-09-22 at 13:19 -0700, Darren Hart wrote:
> On 09/22/2011 01:03 PM, Saul Wold wrote:
> > On 09/22/2011 12:58 PM, Richard Purdie wrote:
> >> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
> >>> Fixes [YOCTO #1514]
> >>>
> >>> Building core-image-minimal on qemuppc (and likely others) can fail in
> >>> the shared-mime-info configure step with the following error:
> >>>
> > I was thinking about this and went "why is shared-mime-info even in
> > core-image-minimal", are you sure that was the image you were building.
> > This might point to some other problem also.
>
> Yes, core-image-minimal. It also would fail if I just tried to build
> shared-mime-info-native.
Right, its a native dependency being pulled in by one of the tools, not
the image itself...
Cheers,
Richard
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 20:03 ` Saul Wold
2011-09-22 20:19 ` Darren Hart
@ 2011-09-22 20:32 ` Darren Hart
1 sibling, 0 replies; 10+ messages in thread
From: Darren Hart @ 2011-09-22 20:32 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
On 09/22/2011 01:03 PM, Saul Wold wrote:
> On 09/22/2011 12:58 PM, Richard Purdie wrote:
>> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
>>> Fixes [YOCTO #1514]
>>>
>>> Building core-image-minimal on qemuppc (and likely others) can fail in
>>> the shared-mime-info configure step with the following error:
>>>
> I was thinking about this and went "why is shared-mime-info even in
> core-image-minimal", are you sure that was the image you were building.
> This might point to some other problem also.
Perhaps these are pulled in from:
EXTRA_IMAGE_FEATURES = "tools-debug tools-profile tools-testapps
debug-tweaks"
Removing this does remove shared-mime-info from the package list (per
depexp). They do not get built during a subsequent core-image-minimal
build either.
--
Darren
>
> Sau!
>
>>> checking for XML::Parser... configure: error: XML::Parser perl module is
>>> required for intltool
>>>
>>> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
>>> successfully.
>>>
>>> Testing: build tested the native version on qemuppc. I run into tool chain
>>> build failures before I can get to building the target version.
>>>
>>> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
>>> CC: Joshua Lock<josh@linux.intel.com>
>>> ---
>>> .../shared-mime-info/shared-mime-info.inc | 4 ++--
>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> index 64eef9d..e206305 100644
>>> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> @@ -5,8 +5,8 @@ SECTION = "base"
>>> LICENSE = "GPLv2+"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>>>
>>> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
>>> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
>>> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
>>> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>>>
>>> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
>>
>>
>> There must be some underlying problem here. intltool-native should
>> depend on libxml-parser-perl-native so it should already be built. We
>> need to fix the underlying issue, not the symptom :/
>>
>> Cheers,
>>
>> Richard
>>
>>
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 19:58 ` Richard Purdie
2011-09-22 20:03 ` Saul Wold
@ 2011-09-22 20:27 ` Darren Hart
2011-09-22 20:34 ` Koen Kooi
2011-09-22 20:52 ` Richard Purdie
1 sibling, 2 replies; 10+ messages in thread
From: Darren Hart @ 2011-09-22 20:27 UTC (permalink / raw)
To: Richard Purdie; +Cc: Josh Lock, Patches and discussions about the oe-core layer
On 09/22/2011 12:58 PM, Richard Purdie wrote:
> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
>> Fixes [YOCTO #1514]
>>
>> Building core-image-minimal on qemuppc (and likely others) can fail in
>> the shared-mime-info configure step with the following error:
>>
>> checking for XML::Parser... configure: error: XML::Parser perl module is
>> required for intltool
>>
>> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
>> successfully.
>>
>> Testing: build tested the native version on qemuppc. I run into tool chain
>> build failures before I can get to building the target version.
>>
>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>> CC: Joshua Lock <josh@linux.intel.com>
>> ---
>> .../shared-mime-info/shared-mime-info.inc | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> index 64eef9d..e206305 100644
>> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>> @@ -5,8 +5,8 @@ SECTION = "base"
>> LICENSE = "GPLv2+"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>>
>> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
>> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
>> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
>> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>>
>> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
>
>
> There must be some underlying problem here. intltool-native should
> depend on libxml-parser-perl-native so it should already be built. We
> need to fix the underlying issue, not the symptom :/
So intltool.inc:
DEPENDS = "libxml-parser-perl-native"
#RDEPENDS_${PN} = "libxml-parser-perl"
RRECOMMENDS_${PN} = "perl-modules"
RRECOMMENDS_${PN}_virtclass-native = ""
Shouldn't this instead look like:
DEPENDS = "libxml-parser-perl"
DEPENDS_virtclass-native = "libxml-parser-perl-native"
Dropping my change above and using this fix instead allows the build to
complete. Does look like the right fix to you? Shall I send this as a
patch?
Thanks,
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 20:27 ` Darren Hart
@ 2011-09-22 20:34 ` Koen Kooi
2011-09-22 20:52 ` Richard Purdie
1 sibling, 0 replies; 10+ messages in thread
From: Koen Kooi @ 2011-09-22 20:34 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Josh Lock
Op 22 sep. 2011, om 22:27 heeft Darren Hart het volgende geschreven:
>
>
> On 09/22/2011 12:58 PM, Richard Purdie wrote:
>> On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
>>> Fixes [YOCTO #1514]
>>>
>>> Building core-image-minimal on qemuppc (and likely others) can fail in
>>> the shared-mime-info configure step with the following error:
>>>
>>> checking for XML::Parser... configure: error: XML::Parser perl module is
>>> required for intltool
>>>
>>> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
>>> successfully.
>>>
>>> Testing: build tested the native version on qemuppc. I run into tool chain
>>> build failures before I can get to building the target version.
>>>
>>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>>> CC: Joshua Lock <josh@linux.intel.com>
>>> ---
>>> .../shared-mime-info/shared-mime-info.inc | 4 ++--
>>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> index 64eef9d..e206305 100644
>>> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
>>> @@ -5,8 +5,8 @@ SECTION = "base"
>>> LICENSE = "GPLv2+"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
>>>
>>> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
>>> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
>>> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
>>> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
>>>
>>> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
>>
>>
>> There must be some underlying problem here. intltool-native should
>> depend on libxml-parser-perl-native so it should already be built. We
>> need to fix the underlying issue, not the symptom :/
>
> So intltool.inc:
>
> DEPENDS = "libxml-parser-perl-native"
> #RDEPENDS_${PN} = "libxml-parser-perl"
> RRECOMMENDS_${PN} = "perl-modules"
> RRECOMMENDS_${PN}_virtclass-native = ""
>
> Shouldn't this instead look like:
>
> DEPENDS = "libxml-parser-perl"
> DEPENDS_virtclass-native = "libxml-parser-perl-native"
It needs -native for both, sadly
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/1] shared-mime-info: add libxml-parser-perl to DEPENDS
2011-09-22 20:27 ` Darren Hart
2011-09-22 20:34 ` Koen Kooi
@ 2011-09-22 20:52 ` Richard Purdie
1 sibling, 0 replies; 10+ messages in thread
From: Richard Purdie @ 2011-09-22 20:52 UTC (permalink / raw)
To: Darren Hart; +Cc: Josh Lock, Patches and discussions about the oe-core layer
On Thu, 2011-09-22 at 13:27 -0700, Darren Hart wrote:
>
> On 09/22/2011 12:58 PM, Richard Purdie wrote:
> > On Thu, 2011-09-22 at 11:49 -0700, Darren Hart wrote:
> >> Fixes [YOCTO #1514]
> >>
> >> Building core-image-minimal on qemuppc (and likely others) can fail in
> >> the shared-mime-info configure step with the following error:
> >>
> >> checking for XML::Parser... configure: error: XML::Parser perl module is
> >> required for intltool
> >>
> >> Adding libxml-parser-perl to DEPENDS allows share-mime-info-native to build
> >> successfully.
> >>
> >> Testing: build tested the native version on qemuppc. I run into tool chain
> >> build failures before I can get to building the target version.
> >>
> >> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> >> CC: Joshua Lock <josh@linux.intel.com>
> >> ---
> >> .../shared-mime-info/shared-mime-info.inc | 4 ++--
> >> 1 files changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> >> index 64eef9d..e206305 100644
> >> --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> >> +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
> >> @@ -5,8 +5,8 @@ SECTION = "base"
> >> LICENSE = "GPLv2+"
> >> LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3"
> >>
> >> -DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native"
> >> -DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native"
> >> +DEPENDS = "libxml2 intltool-native glib-2.0 shared-mime-info-native libxml-parser-perl"
> >> +DEPENDS_virtclass-native = "libxml2-native intltool-native glib-2.0-native libxml-parser-perl-native"
> >>
> >> SRC_URI = "http://freedesktop.org/~hadess/shared-mime-info-${PV}.tar.bz2"
> >
> >
> > There must be some underlying problem here. intltool-native should
> > depend on libxml-parser-perl-native so it should already be built. We
> > need to fix the underlying issue, not the symptom :/
>
> So intltool.inc:
>
> DEPENDS = "libxml-parser-perl-native"
> #RDEPENDS_${PN} = "libxml-parser-perl"
> RRECOMMENDS_${PN} = "perl-modules"
> RRECOMMENDS_${PN}_virtclass-native = ""
>
> Shouldn't this instead look like:
>
> DEPENDS = "libxml-parser-perl"
> DEPENDS_virtclass-native = "libxml-parser-perl-native"
>
> Dropping my change above and using this fix instead allows the build to
> complete. Does look like the right fix to you? Shall I send this as a
> patch?
That certainly sounds like the more correct solution, yes.
Cheers,
Richard
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2011-09-22 20:59 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-09-22 18:49 [PATCH 0/1] shared-mime-info: add libxml-parser-perl to DEPENDS Darren Hart
2011-09-22 18:49 ` [PATCH 1/1] " Darren Hart
2011-09-22 19:58 ` Richard Purdie
2011-09-22 20:03 ` Saul Wold
2011-09-22 20:19 ` Darren Hart
2011-09-22 20:53 ` Richard Purdie
2011-09-22 20:32 ` Darren Hart
2011-09-22 20:27 ` Darren Hart
2011-09-22 20:34 ` Koen Kooi
2011-09-22 20:52 ` Richard Purdie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox