All of lore.kernel.org
 help / color / mirror / Atom feed
* A few fixes (resend, hopefully not munged this time)
@ 2010-08-12 15:35 kernel-hacker
  2010-08-12 15:35 ` [PATCH 1/2] freetype: include perl-native dependancy kernel-hacker
  2010-08-12 15:35 ` [PATCH 2/2] recode: Fix configure step kernel-hacker
  0 siblings, 2 replies; 7+ messages in thread
From: kernel-hacker @ 2010-08-12 15:35 UTC (permalink / raw)
  To: openembedded-devel

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.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/2] freetype: include perl-native dependancy
  2010-08-12 15:35 A few fixes (resend, hopefully not munged this time) kernel-hacker
@ 2010-08-12 15:35 ` kernel-hacker
  2010-08-12 17:04   ` Tom Rini
  2010-08-12 15:35 ` [PATCH 2/2] recode: Fix configure step kernel-hacker
  1 sibling, 1 reply; 7+ messages in thread
From: kernel-hacker @ 2010-08-12 15:35 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Alex Bennee

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




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/2] recode: Fix configure step.
  2010-08-12 15:35 A few fixes (resend, hopefully not munged this time) kernel-hacker
  2010-08-12 15:35 ` [PATCH 1/2] freetype: include perl-native dependancy kernel-hacker
@ 2010-08-12 15:35 ` kernel-hacker
  2010-08-12 17:05   ` Tom Rini
  1 sibling, 1 reply; 7+ messages in thread
From: kernel-hacker @ 2010-08-12 15:35 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Alex Bennee

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




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] freetype: include perl-native dependancy
  2010-08-12 15:35 ` [PATCH 1/2] freetype: include perl-native dependancy kernel-hacker
@ 2010-08-12 17:04   ` Tom Rini
  2010-08-12 17:23     ` Alex Bennee
  0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2010-08-12 17:04 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Alex Bennee

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] recode: Fix configure step.
  2010-08-12 15:35 ` [PATCH 2/2] recode: Fix configure step kernel-hacker
@ 2010-08-12 17:05   ` Tom Rini
  0 siblings, 0 replies; 7+ messages in thread
From: Tom Rini @ 2010-08-12 17:05 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Alex Bennee

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] freetype: include perl-native dependancy
  2010-08-12 17:04   ` Tom Rini
@ 2010-08-12 17:23     ` Alex Bennee
  2010-08-12 18:57       ` Tom Rini
  0 siblings, 1 reply; 7+ messages in thread
From: Alex Bennee @ 2010-08-12 17:23 UTC (permalink / raw)
  To: openembedded-devel; +Cc: 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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] freetype: include perl-native dependancy
  2010-08-12 17:23     ` Alex Bennee
@ 2010-08-12 18:57       ` Tom Rini
  0 siblings, 0 replies; 7+ messages in thread
From: Tom Rini @ 2010-08-12 18:57 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Alex Bennee

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2010-08-12 18:57 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-12 15:35 A few fixes (resend, hopefully not munged this time) kernel-hacker
2010-08-12 15:35 ` [PATCH 1/2] freetype: include perl-native dependancy kernel-hacker
2010-08-12 17:04   ` Tom Rini
2010-08-12 17:23     ` Alex Bennee
2010-08-12 18:57       ` Tom Rini
2010-08-12 15:35 ` [PATCH 2/2] recode: Fix configure step kernel-hacker
2010-08-12 17:05   ` Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.