A few fixes (resend, hopefully not munged this time)

A few fixes (resend, hopefully not munged this time)

[kernel-hacker]

Hi,

Sorry for the delay resubmitting these. Please find attached the
following patches we added to OE during bringing up our tree. Our
actual patch series runs to around 14 patches but I assume your not
interested in more non-consumer boards or patches that "work for us".
However I'm happy to send them to the mailing list or privately if
someone wants to review them. A number of the patches were from Phil
Blundel and I believe have already worked their way upstream from his
trees.

[PATCH 1/2] freetype: include perl-native dependancy
[PATCH 2/2] recode: Fix configure step.

--
Alex.

[PATCH 1/2] freetype: include perl-native dependancy

[kernel-hacker]

From: Alex Bennee <ajb@cbnl.com>

Freetype requires a native perl to build during it's configure step.
I'm unsure how it built before unless other uses had brought in
perl-native through another dependancy (our GTK setup is fairly
minimal).

Signed-off-by: Alex Bennee <ajb@cbnl.com>
---
 recipes/freetype/freetype_2.3.11.bb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/recipes/freetype/freetype_2.3.11.bb b/recipes/freetype/freetype_2.3.11.bb
index 1474bb9..8d3ca63 100644
--- a/recipes/freetype/freetype_2.3.11.bb
+++ b/recipes/freetype/freetype_2.3.11.bb
@@ -11,12 +11,15 @@ SRC_URI = "\
 "
 S = "${WORKDIR}/freetype-${PV}"
 
+DEPENDS	     = "perl-native"
+
 inherit autotools pkgconfig binconfig
 
 LIBTOOL = "${HOST_SYS}-libtool"
 EXTRA_OEMAKE = "'LIBTOOL=${LIBTOOL}'"
 EXTRA_OECONF = "--without-zlib"
 
+
 do_configure() {
 	cd builds/unix
 	libtoolize --force --copy
-- 
1.7.2.1

Re: [PATCH 1/2] freetype: include perl-native dependancy

[Tom Rini]

kernel-hacker@bennee.com wrote:
> From: Alex Bennee <ajb@cbnl.com>
> 
> Freetype requires a native perl to build during it's configure step.
> I'm unsure how it built before unless other uses had brought in
> perl-native through another dependancy (our GTK setup is fairly
> minimal).
> 
> Signed-off-by: Alex Bennee <ajb@cbnl.com>

OE assumes that there is a host perl already.  perl-native is mainly 
provided for some very special cases (such as building perl for the target).

-- 
Tom Rini
Mentor Graphics Corporation

Re: [PATCH 1/2] freetype: include perl-native dependancy

[Alex Bennee]

On 12 August 2010 18:04, Tom Rini <tom_rini@mentor.com> wrote:
> kernel-hacker@bennee.com wrote:
>>
>> From: Alex Bennee <ajb@cbnl.com>
>>
>> Freetype requires a native perl to build during it's configure step.
>> I'm unsure how it built before unless other uses had brought in
>> perl-native through another dependancy (our GTK setup is fairly
>> minimal).
>>
>> Signed-off-by: Alex Bennee <ajb@cbnl.com>
>
> OE assumes that there is a host perl already.  perl-native is mainly
> provided for some very special cases (such as building perl for the target).

How odd, I couldn't get configure to run without a perl-native. Maybe
because I was building for i686 on an x86_64. I guess it relegates
that patch to a "works for us" status.

-- 
Alex, homepage: http://www.bennee.com/~alex/
http://www.half-llama.co.uk

Re: [PATCH 1/2] freetype: include perl-native dependancy

[Tom Rini]

Alex Bennee wrote:
> On 12 August 2010 18:04, Tom Rini <tom_rini@mentor.com> wrote:
>> kernel-hacker@bennee.com wrote:
>>> From: Alex Bennee <ajb@cbnl.com>
>>>
>>> Freetype requires a native perl to build during it's configure step.
>>> I'm unsure how it built before unless other uses had brought in
>>> perl-native through another dependancy (our GTK setup is fairly
>>> minimal).
>>>
>>> Signed-off-by: Alex Bennee <ajb@cbnl.com>
>> OE assumes that there is a host perl already.  perl-native is mainly
>> provided for some very special cases (such as building perl for the target).
> 
> How odd, I couldn't get configure to run without a perl-native. Maybe
> because I was building for i686 on an x86_64. I guess it relegates
> that patch to a "works for us" status.

So, were you building for say MACHINE=qemux86

-- 
Tom Rini
Mentor Graphics Corporation

[PATCH 2/2] recode: Fix configure step.

[kernel-hacker]

From: Alex Bennee <ajb@cbnl.com>

Add space to appended CC/CXX otherwise configure fails

Signed-off-by: Alex Bennee <ajb@cbnl.com>
---
 recipes/recode/recode_3.6.bb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/recode/recode_3.6.bb b/recipes/recode/recode_3.6.bb
index 3e7c6cd..0cdd43d 100644
--- a/recipes/recode/recode_3.6.bb
+++ b/recipes/recode/recode_3.6.bb
@@ -8,7 +8,7 @@ SRC_URI = "${GNU_MIRROR}/recode/recode-${PV}.tar.gz \
 
 inherit autotools
 
-TARGET_CC_ARCH_append = "${LDFLAGS}"
+TARGET_CC_ARCH_append = " ${LDFLAGS}"
 
 do_configure() {
 	gnu-configize
-- 
1.7.2.1

Re: [PATCH 2/2] recode: Fix configure step.

[Tom Rini]

kernel-hacker@bennee.com wrote:
> From: Alex Bennee <ajb@cbnl.com>
> 
> Add space to appended CC/CXX otherwise configure fails
> 
> Signed-off-by: Alex Bennee <ajb@cbnl.com>
> ---
>  recipes/recode/recode_3.6.bb |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/recode/recode_3.6.bb b/recipes/recode/recode_3.6.bb
> index 3e7c6cd..0cdd43d 100644
> --- a/recipes/recode/recode_3.6.bb
> +++ b/recipes/recode/recode_3.6.bb
> @@ -8,7 +8,7 @@ SRC_URI = "${GNU_MIRROR}/recode/recode-${PV}.tar.gz \
>  
>  inherit autotools
>  
> -TARGET_CC_ARCH_append = "${LDFLAGS}"
> +TARGET_CC_ARCH_append = " ${LDFLAGS}"
>  
>  do_configure() {
>  	gnu-configize

iirc this should become TARGET_CC_ARCH += "${LDFLAGS}" as then the space 
will be added and it looks cleaner.

-- 
Tom Rini
Mentor Graphics Corporation