Author: JF <jf@codingfield.com>
Fix builds that do not specify CMAKE_BUILD_TYPE during CMake generation. Apply Release by default.
README.md | 8 ++++++++ src/CMakeLists.txt | 4 ++++
diff --git a/README.md b/README.md index e728969d715ae3c904794fd8640d5b0e12de4556..42efda205d419ee637b9da882d4f52d5fa7500a0 100644 --- a/README.md +++ b/README.md @@ -102,6 +102,14 @@ $ cd build $ cmake -DARM_NONE_EABI_TOOLCHAIN_PATH=... -DNRF5_SDK_PATH=... -DUSE_OPENOCD=1 -DGDB_CLIENT_BIN_PATH=[optional] ../ ``` + +**Note** : By default, the build configuration *Release* is applied. It means that the code is built with optimisations enabled (-O3). If you wanto to compile in *Debug* mode to ease debugging and step-by-step debugging, specify the CMake variable `CMAKE_BUILD_TYPE` in the CMake command line: + +``` +-DCMAKE_BUILD_TYPE=Debug +``` + + * Make ``` $ make -j pinetime-app diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 286a792a78c285bf50bc795e2af214ef6109453e..1609196a05d1cd1bfc8bad6a08c8a8a4e0132dc7 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -546,6 +546,10 @@ add_definitions(-DNRF52 -DNRF52832 -DNRF52832_XXAA -DNRF52_PAN_74 -DNRF52_PAN_64 -DNRF52_PAN_12 -DNRF52_PAN_58 -DNRF52_PAN_54 -DNRF52_PAN_31 -DNRF52_PAN_51 -DNRF52_PAN_36 -DNRF52_PAN_15 -DNRF52_PAN_20 -DNRF52_PAN_55 -DBOARD_PCA10040) add_definitions(-DFREERTOS) add_definitions(-DDEBUG_NRF_USER) +if(NOT CMAKE_BUILD_TYPE) + set(CMAKE_BUILD_TYPE "Release") +endif() + # Build autonomous binary (without support for bootloader) set(EXECUTABLE_NAME "pinetime-app") set(NRF5_LINKER_SCRIPT "${CMAKE_SOURCE_DIR}/gcc_nrf52.ld")