All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] cmake.bbclass: set the modules directory correctly
@ 2016-06-17 15:01 Jose Pardeiro
  2016-06-18 12:56 ` Jussi Kukkonen
  0 siblings, 1 reply; 2+ messages in thread
From: Jose Pardeiro @ 2016-06-17 15:01 UTC (permalink / raw)
  To: openembedded-core; +Cc: Jose Pardeiro

The CMake recipes contain a mismatch between the environmental variable
which defines where the Modules are installed and the location where they
actually are. This patch fixes the environmental variable to point to the
proper folder defined according to the cmake version.

Signed-off-by: Jose Pardeiro <jpardeiro@rapyuta-robotics.com>
---
 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 e788310..6dbac2b 100644
--- a/meta/classes/cmake.bbclass
+++ b/meta/classes/cmake.bbclass
@@ -74,7 +74,7 @@ set( ENV{QT_CONF_PATH} ${WORKDIR}/qt.conf )
 set( CMAKE_INSTALL_RPATH ${OECMAKE_RPATH} )
 
 # Use native cmake modules
-list(APPEND CMAKE_MODULE_PATH "${STAGING_DATADIR}/cmake/Modules/")
+list(APPEND CMAKE_MODULE_PATH "${STAGING_DATADIR}/cmake-\${CMAKE_MAJOR_VERSION}.\${CMAKE_MINOR_VERSION}/Modules/")
 
 # add for non /usr/lib libdir, e.g. /usr/lib64
 set( CMAKE_LIBRARY_PATH ${libdir} ${base_libdir})
-- 
1.9.1



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

* Re: [PATCH v2] cmake.bbclass: set the modules directory correctly
  2016-06-17 15:01 [PATCH v2] cmake.bbclass: set the modules directory correctly Jose Pardeiro
@ 2016-06-18 12:56 ` Jussi Kukkonen
  0 siblings, 0 replies; 2+ messages in thread
From: Jussi Kukkonen @ 2016-06-18 12:56 UTC (permalink / raw)
  To: Jose Pardeiro; +Cc: Patches and discussions about the oe-core layer

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

On 17 June 2016 at 18:01, Jose Pardeiro <jpardeiro@rapyuta-robotics.com>
wrote:
>
> The CMake recipes contain a mismatch between the environmental variable
> which defines where the Modules are installed and the location where they
> actually are. This patch fixes the environmental variable to point to the
> proper folder defined according to the cmake version.
>
> Signed-off-by: Jose Pardeiro <jpardeiro@rapyuta-robotics.com>
> ---
>  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 e788310..6dbac2b 100644
> --- a/meta/classes/cmake.bbclass
> +++ b/meta/classes/cmake.bbclass
> @@ -74,7 +74,7 @@ set( ENV{QT_CONF_PATH} ${WORKDIR}/qt.conf )
>  set( CMAKE_INSTALL_RPATH ${OECMAKE_RPATH} )
>
>  # Use native cmake modules
> -list(APPEND CMAKE_MODULE_PATH "${STAGING_DATADIR}/cmake/Modules/")
> +list(APPEND CMAKE_MODULE_PATH
"${STAGING_DATADIR}/cmake-\${CMAKE_MAJOR_VERSION}.\${CMAKE_MINOR_VERSION}/Modules/"


This broke "bitbake cmake":

| CMake Error at
/home/jku/src/poky/build/tmp/work/core2-64-poky-linux/cmake/3.5.2-r0/toolchain.cmake:34
(list):
|   Syntax error in cmake code at
|
|
/home/jku/src/poky/build/tmp/work/core2-64-poky-linux/cmake/3.5.2-r0/toolchain.cmake:34
|
|   when parsing string
|
|
/home/jku/src/poky/build/tmp/sysroots/qemux86-64/usr/share/cmake-\3.5.${CMAKE_MINOR_VERSION}/Modules/
|
|   Invalid character escape '\3'.
| Call Stack (most recent call first):
|
/home/jku/src/poky/build/tmp/sysroots/x86_64-linux/usr/share/cmake-3.5/Modules/CMakeDetermineSystem.cmake:98
(include)
|   CMakeLists.txt:19 (project)


>
>
>
>  # add for non /usr/lib libdir, e.g. /usr/lib64
>  set( CMAKE_LIBRARY_PATH ${libdir} ${base_libdir})
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

[-- Attachment #2: Type: text/html, Size: 2605 bytes --]

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

end of thread, other threads:[~2016-06-18 12:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-17 15:01 [PATCH v2] cmake.bbclass: set the modules directory correctly Jose Pardeiro
2016-06-18 12:56 ` Jussi Kukkonen

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.