All of lore.kernel.org
 help / color / mirror / Atom feed
* [STABLE][PATCH] gdbm: use LDFLAGS (fixes QA error)
@ 2009-04-14  7:35 Koen Kooi
  2009-04-14 21:42 ` Denys Dmytriyenko
  0 siblings, 1 reply; 2+ messages in thread
From: Koen Kooi @ 2009-04-14  7:35 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Koen Kooi

From: Koen Kooi <koen@openembedded.org>


Signed-off-by: Koen Kooi <koen@openembedded.org>
---
 recipes/gdbm/gdbm_1.8.3.bb |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/recipes/gdbm/gdbm_1.8.3.bb b/recipes/gdbm/gdbm_1.8.3.bb
index f3beb47..883022e 100644
--- a/recipes/gdbm/gdbm_1.8.3.bb
+++ b/recipes/gdbm/gdbm_1.8.3.bb
@@ -4,7 +4,7 @@ SECTION = "libs"
 PRIORITY = "optional"
 LICENSE = "GPL"
 
-PR = "r2"
+PR = "r3"
 
 SRC_URI = "${GNU_MIRROR}/gdbm/gdbm-${PV}.tar.gz \
 	   file://makefile.patch;patch=1 \
@@ -12,6 +12,8 @@ SRC_URI = "${GNU_MIRROR}/gdbm/gdbm-${PV}.tar.gz \
 
 inherit autotools
 
+TARGET_CC_ARCH += "${LDFLAGS}"
+
 do_stage () {
 	oe_libinstall -so -a libgdbm ${STAGING_LIBDIR}
 	install -m 0644 ${S}/gdbm.h ${STAGING_INCDIR}/
-- 
1.5.6.5




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

* Re: [STABLE][PATCH] gdbm: use LDFLAGS (fixes QA error)
  2009-04-14  7:35 [STABLE][PATCH] gdbm: use LDFLAGS (fixes QA error) Koen Kooi
@ 2009-04-14 21:42 ` Denys Dmytriyenko
  0 siblings, 0 replies; 2+ messages in thread
From: Denys Dmytriyenko @ 2009-04-14 21:42 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Koen Kooi

On Tue, Apr 14, 2009 at 09:35:34AM +0200, Koen Kooi wrote:
> From: Koen Kooi <koen@openembedded.org>

Is it a proper use of TARGET_CC_ARCH? As name suggests, it wasn't meant to 
deliver LDFLAGS to CC, only to specify the architecture. But as I can see, 
many recipes reuse this variable to stick additional flags for CC... So, I 
guess it's a common practice.

Acked-by: Denys Dmytriyenko <denis@denix.org>


> Signed-off-by: Koen Kooi <koen@openembedded.org>
> ---
>  recipes/gdbm/gdbm_1.8.3.bb |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/gdbm/gdbm_1.8.3.bb b/recipes/gdbm/gdbm_1.8.3.bb
> index f3beb47..883022e 100644
> --- a/recipes/gdbm/gdbm_1.8.3.bb
> +++ b/recipes/gdbm/gdbm_1.8.3.bb
> @@ -4,7 +4,7 @@ SECTION = "libs"
>  PRIORITY = "optional"
>  LICENSE = "GPL"
>  
> -PR = "r2"
> +PR = "r3"
>  
>  SRC_URI = "${GNU_MIRROR}/gdbm/gdbm-${PV}.tar.gz \
>  	   file://makefile.patch;patch=1 \
> @@ -12,6 +12,8 @@ SRC_URI = "${GNU_MIRROR}/gdbm/gdbm-${PV}.tar.gz \
>  
>  inherit autotools
>  
> +TARGET_CC_ARCH += "${LDFLAGS}"
> +
>  do_stage () {
>  	oe_libinstall -so -a libgdbm ${STAGING_LIBDIR}
>  	install -m 0644 ${S}/gdbm.h ${STAGING_INCDIR}/
> -- 
> 1.5.6.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel



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

end of thread, other threads:[~2009-04-14 21:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-04-14  7:35 [STABLE][PATCH] gdbm: use LDFLAGS (fixes QA error) Koen Kooi
2009-04-14 21:42 ` Denys Dmytriyenko

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.