All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-oe][PATCH] libtinyxml: Add native support and use relative paths
@ 2013-03-14 16:45 Stefan Herbrechtsmeier
  2013-03-22  9:22 ` Martin Jansa
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Herbrechtsmeier @ 2013-03-14 16:45 UTC (permalink / raw)
  To: openembedded-devel

Replace CXXFLAGS += "-fPIC" with EXTRA_CXXFLAGS to allow native
build and use relative paths to simplify recipe.

Signed-off-by: Stefan Herbrechtsmeier <stefan@herbrechtsmeier.net>
---
 .../recipes-support/libtinyxml/libtinyxml_2.6.2.bb |   18 ++++++++++--------
 1 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb b/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
index fdadebd..8670fac 100644
--- a/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
+++ b/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
@@ -16,20 +16,21 @@ SRC_URI[sha256sum] = "15bdfdcec58a7da30adc87ac2b078e4417dbe5392f3afb719f9ba6d062
 
 S = "${WORKDIR}/tinyxml"
 
-CXXFLAGS += "-fPIC"
+EXTRA_CXXFLAGS = "-I. -fPIC"
 
 do_compile() {
-    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxml.o ${S}/tinyxml.cpp
-    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxmlerror.o ${S}/tinyxmlerror.cpp
-    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxmlparser.o ${S}/tinyxmlparser.cpp
+    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxml.o tinyxml.cpp
+    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxmlerror.o tinyxmlerror.cpp
+    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxmlparser.o tinyxmlparser.cpp
     ${CXX} ${CXXFLAGS} \
             -shared \
             -Wl,-soname,libtinyxml.so.${PV} \
-            -o ${S}/libtinyxml.so.${PV} \
+            -o libtinyxml.so.${PV} \
             ${LDFLAGS} \
-            ${S}/tinyxml.o \
-            ${S}/tinyxmlparser.o \
-            ${S}/tinyxmlerror.o
+            tinyxml.o \
+            tinyxmlparser.o \
+            tinyxmlerror.o
+
 }
 
 do_install() {
@@ -41,3 +42,4 @@ do_install() {
     install -m 0644 ${S}/tinyxml.h ${D}${includedir}
 }
 
+BBCLASSEXTEND += "native"
-- 
1.7.0.4




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

* Re: [meta-oe][PATCH] libtinyxml: Add native support and use relative paths
  2013-03-14 16:45 [meta-oe][PATCH] libtinyxml: Add native support and use relative paths Stefan Herbrechtsmeier
@ 2013-03-22  9:22 ` Martin Jansa
  0 siblings, 0 replies; 2+ messages in thread
From: Martin Jansa @ 2013-03-22  9:22 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 2311 bytes --]

On Thu, Mar 14, 2013 at 05:45:57PM +0100, Stefan Herbrechtsmeier wrote:
> Replace CXXFLAGS += "-fPIC" with EXTRA_CXXFLAGS to allow native
> build and use relative paths to simplify recipe.

Applied, thanks!

> 
> Signed-off-by: Stefan Herbrechtsmeier <stefan@herbrechtsmeier.net>
> ---
>  .../recipes-support/libtinyxml/libtinyxml_2.6.2.bb |   18 ++++++++++--------
>  1 files changed, 10 insertions(+), 8 deletions(-)
> 
> diff --git a/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb b/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
> index fdadebd..8670fac 100644
> --- a/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
> +++ b/meta-oe/recipes-support/libtinyxml/libtinyxml_2.6.2.bb
> @@ -16,20 +16,21 @@ SRC_URI[sha256sum] = "15bdfdcec58a7da30adc87ac2b078e4417dbe5392f3afb719f9ba6d062
>  
>  S = "${WORKDIR}/tinyxml"
>  
> -CXXFLAGS += "-fPIC"
> +EXTRA_CXXFLAGS = "-I. -fPIC"
>  
>  do_compile() {
> -    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxml.o ${S}/tinyxml.cpp
> -    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxmlerror.o ${S}/tinyxmlerror.cpp
> -    ${CXX} ${CXXFLAGS} -I${S} -c -o ${S}/tinyxmlparser.o ${S}/tinyxmlparser.cpp
> +    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxml.o tinyxml.cpp
> +    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxmlerror.o tinyxmlerror.cpp
> +    ${CXX} ${CXXFLAGS} ${EXTRA_CXXFLAGS} -c -o tinyxmlparser.o tinyxmlparser.cpp
>      ${CXX} ${CXXFLAGS} \
>              -shared \
>              -Wl,-soname,libtinyxml.so.${PV} \
> -            -o ${S}/libtinyxml.so.${PV} \
> +            -o libtinyxml.so.${PV} \
>              ${LDFLAGS} \
> -            ${S}/tinyxml.o \
> -            ${S}/tinyxmlparser.o \
> -            ${S}/tinyxmlerror.o
> +            tinyxml.o \
> +            tinyxmlparser.o \
> +            tinyxmlerror.o
> +
>  }
>  
>  do_install() {
> @@ -41,3 +42,4 @@ do_install() {
>      install -m 0644 ${S}/tinyxml.h ${D}${includedir}
>  }
>  
> +BBCLASSEXTEND += "native"
> -- 
> 1.7.0.4
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

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

end of thread, other threads:[~2013-03-22  9:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-14 16:45 [meta-oe][PATCH] libtinyxml: Add native support and use relative paths Stefan Herbrechtsmeier
2013-03-22  9:22 ` Martin Jansa

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.