From: André Draszik <andre.dras...@jci.com>

cmake_do_compile() and cmake_do_install() basically do the
same, except they use a different --target, and at the
moment this is copy/pasted code with a minor modification.

Other recipes which e.g. might want to support compilation
as part of ptest have to do the same. This is a bit
inconvenient.

By factoring out all of this into a common helper,
cmake_runcmake_build(), this is easily re-used. An
(imaginary) recipe can compile ptest support simply by
using
  cmake_runcmake_build --target buildtest-TESTS
(assuming such a build target exists).

Also, this now is very similar to oe_runmake().

Signed-off-by: André Draszik <andre.dras...@jci.com>
---
 meta/classes/cmake.bbclass | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/classes/cmake.bbclass b/meta/classes/cmake.bbclass
index d60dad8d66..fcfd5dda4f 100644
--- a/meta/classes/cmake.bbclass
+++ b/meta/classes/cmake.bbclass
@@ -159,14 +159,17 @@ cmake_do_configure() {
          -Wno-dev
 }
 
+cmake_runcmake_build() {
+       bbnote ${DESTDIR:+DESTDIR=${DESTDIR} }VERBOSE=1 cmake --build '${B}' 
"$@" -- ${EXTRA_OECMAKE_BUILD}
+       eval ${DESTDIR:+DESTDIR=${DESTDIR} }VERBOSE=1 cmake --build '${B}' "$@" 
-- ${EXTRA_OECMAKE_BUILD}
+}
+
 cmake_do_compile()  {
-       bbnote VERBOSE=1 cmake --build '${B}' --target 
${OECMAKE_TARGET_COMPILE} -- ${EXTRA_OECMAKE_BUILD}
-       VERBOSE=1 cmake --build '${B}' --target ${OECMAKE_TARGET_COMPILE} -- 
${EXTRA_OECMAKE_BUILD}
+       cmake_runcmake_build --target ${OECMAKE_TARGET_COMPILE}
 }
 
 cmake_do_install() {
-       bbnote DESTDIR='${D}' cmake --build '${B}' --target 
${OECMAKE_TARGET_INSTALL} -- ${EXTRA_OECMAKE_BUILD}
-       DESTDIR='${D}' cmake --build '${B}' --target ${OECMAKE_TARGET_INSTALL} 
-- ${EXTRA_OECMAKE_BUILD}
+       DESTDIR='${D}' cmake_runcmake_build --target ${OECMAKE_TARGET_INSTALL}
 }
 
 EXPORT_FUNCTIONS do_configure do_compile do_install do_generate_toolchain_file
-- 
2.16.1

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to