patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 7 Packages Projects Releases Wiki Activity

cmake: openamp: Fixed a bug where the wrong BOARD was set

Browse source 
Fixed a bug in the OpenAMP sample's recursive build scripts, where it
would pass on the wrong board to it's second CMake invocation.

This fixes #10345

Fixed the same bug in ipm_mcux.

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
This commit is contained in:
Sebastian Bøe 2018-10-03 10:23:46 +02:00 committed by Kumar Gala
parent 58ec6fd30f
commit 8c791a999e
2 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
samples/subsys/ipc/ipm_mcux/CMakeLists.txt
View file

@ -14,6 +14,8 @@ ExternalProject_Add(
SOURCE_DIR ${APPLICATION_SOURCE_DIR}/remote
INSTALL_COMMAND "" # This particular build system has no install command
BUILD_BYPRODUCTS "${CMAKE_CURRENT_BINARY_DIR}/ipm_mcux_remote-prefix/src/ipm_mcux_remote-build/zephyr/zephyr.bin"
CMAKE_CACHE_ARGS -DBOARD:STRING=${BOARD}
# NB: Do we need to pass on more CMake variables?
)
project(NONE)

2
samples/subsys/ipc/openamp/CMakeLists.txt
View file

@ -25,6 +25,8 @@ ExternalProject_Add(
SOURCE_DIR ${APPLICATION_SOURCE_DIR}/remote
INSTALL_COMMAND "" # This particular build system has no install command
BUILD_BYPRODUCTS "${CMAKE_CURRENT_BINARY_DIR}/openamp_remote-prefix/src/openamp_remote-build/zephyr/zephyr.bin"
CMAKE_CACHE_ARGS -DBOARD:STRING=${BOARD}
# NB: Do we need to pass on more CMake variables?
)
add_dependencies(core_m0_inc_target openamp_remote)