Openembedded Core Discussions
 help / color / mirror / Atom feed
* [dylan][PATCH 0/2] Backports from master
@ 2014-01-08 17:22 Paul Eggleton
  2014-01-08 17:22 ` [dylan][PATCH 1/2] cmake: set system name correctly Paul Eggleton
  2014-01-08 17:23 ` [dylan][PATCH 2/2] cmake.bbclass: ensure CMAKE_SYSTEM_NAME is correct Paul Eggleton
  0 siblings, 2 replies; 3+ messages in thread
From: Paul Eggleton @ 2014-01-08 17:22 UTC (permalink / raw)
  To: openembedded-core

A couple of patches to fix the CMAKE_SYSTEM_NAME value.



The following changes since commit 394fa61d00b97c19610bfb38f02174c4e8aeeb15:

  libsoup-2.4: add intltool-native to DEPENDS (2013-12-19 11:44:49 +0000)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib paule/dylan-next
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/dylan-next

Richard Purdie (1):
  cmake: set system name correctly

Saul Wold (1):
  cmake.bbclass: ensure CMAKE_SYSTEM_NAME is correct

 meta/classes/cmake.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.8.1.2



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

* [dylan][PATCH 1/2] cmake: set system name correctly
  2014-01-08 17:22 [dylan][PATCH 0/2] Backports from master Paul Eggleton
@ 2014-01-08 17:22 ` Paul Eggleton
  2014-01-08 17:23 ` [dylan][PATCH 2/2] cmake.bbclass: ensure CMAKE_SYSTEM_NAME is correct Paul Eggleton
  1 sibling, 0 replies; 3+ messages in thread
From: Paul Eggleton @ 2014-01-08 17:22 UTC (permalink / raw)
  To: openembedded-core

From: Richard Purdie <richard.purdie@linuxfoundation.org>

For unknown reasons, the cmake class is using SDK_OS as the
target system OS. This makes no sense but only shows up as a problem
when you try a different SDK OS. Fix it to use TARGET_OS which is
the correct thing to do. For the vast majority of users this will
make no difference.

(From OE-Core master rev: 57be84259f0885865c85d7bac350979430b956b5)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/classes/cmake.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/cmake.bbclass b/meta/classes/cmake.bbclass
index e64c30c..4ef9fca 100644
--- a/meta/classes/cmake.bbclass
+++ b/meta/classes/cmake.bbclass
@@ -35,7 +35,7 @@ cmake_do_generate_toolchain_file() {
 	cat > ${WORKDIR}/toolchain.cmake <<EOF
 # CMake system name must be something like "Linux".
 # This is important for cross-compiling.
-set( CMAKE_SYSTEM_NAME `echo ${SDK_OS} | sed 's/^./\u&/'` )
+set( CMAKE_SYSTEM_NAME `echo ${TARGET_OS} | sed 's/^./\u&/'` )
 set( CMAKE_SYSTEM_PROCESSOR ${TARGET_ARCH} )
 set( CMAKE_C_COMPILER ${OECMAKE_C_COMPILER} )
 set( CMAKE_CXX_COMPILER ${OECMAKE_CXX_COMPILER} )
-- 
1.8.1.2



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

* [dylan][PATCH 2/2] cmake.bbclass: ensure CMAKE_SYSTEM_NAME is correct
  2014-01-08 17:22 [dylan][PATCH 0/2] Backports from master Paul Eggleton
  2014-01-08 17:22 ` [dylan][PATCH 1/2] cmake: set system name correctly Paul Eggleton
@ 2014-01-08 17:23 ` Paul Eggleton
  1 sibling, 0 replies; 3+ messages in thread
From: Paul Eggleton @ 2014-01-08 17:23 UTC (permalink / raw)
  To: openembedded-core

From: Saul Wold <sgw@linux.intel.com>

Using TARGET_OS can add the ABIEXTENSION so ensure that is is removed for the Linux
TARGET_OS, we might have other TARGET_OSes so don't hard code CMAKE_SYSTEM_NAME

[YOCTO #5145]

(From OE-Core master rev: 7d8b700242b1b32c6b6d0735b497701800f54fc4)

Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/classes/cmake.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/cmake.bbclass b/meta/classes/cmake.bbclass
index 4ef9fca..5144cdf 100644
--- a/meta/classes/cmake.bbclass
+++ b/meta/classes/cmake.bbclass
@@ -35,7 +35,7 @@ cmake_do_generate_toolchain_file() {
 	cat > ${WORKDIR}/toolchain.cmake <<EOF
 # CMake system name must be something like "Linux".
 # This is important for cross-compiling.
-set( CMAKE_SYSTEM_NAME `echo ${TARGET_OS} | sed 's/^./\u&/'` )
+set( CMAKE_SYSTEM_NAME `echo ${TARGET_OS} | sed -e 's/^./\u&/' -e 's/^\(Linux\).*/\1/'` )
 set( CMAKE_SYSTEM_PROCESSOR ${TARGET_ARCH} )
 set( CMAKE_C_COMPILER ${OECMAKE_C_COMPILER} )
 set( CMAKE_CXX_COMPILER ${OECMAKE_CXX_COMPILER} )
-- 
1.8.1.2



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

end of thread, other threads:[~2014-01-08 17:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-08 17:22 [dylan][PATCH 0/2] Backports from master Paul Eggleton
2014-01-08 17:22 ` [dylan][PATCH 1/2] cmake: set system name correctly Paul Eggleton
2014-01-08 17:23 ` [dylan][PATCH 2/2] cmake.bbclass: ensure CMAKE_SYSTEM_NAME is correct Paul Eggleton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox