Skip to content

Commit 934e346

Browse files
Apply suggestions from code review
Co-authored-by: Hugues Kamba <[email protected]>
1 parent 95d3716 commit 934e346

File tree

2 files changed

+14
-10
lines changed

2 files changed

+14
-10
lines changed

tools/cmake/README.md

+10-6
Original file line numberDiff line numberDiff line change
@@ -79,16 +79,20 @@ cmake -S <source-dir> -B <build-dir> -DCMAKE_BUILD_TYPE=debug
7979
## How to build a greentea test
8080
8181
Install prerequisites suggested in the previous section and follow the below steps to build:
82-
* Generate .mbed_build configuration for DISCO_L475VG_IOT01A target from blinky example and copied into the test_suite directory.
83-
* Change directory into the test suite directory
84-
* run below command to build full profile green tea
82+
* Generate the `.mbedbuild/` configuration directory for the Mbed target you want to run the test on using [mbed-os-example-blinky](https://github.com/ARMmbed/mbed-os-example-blinky)
83+
``'
84+
$ mbedtools configure -t <TOOLCHAIN> -m <MBED_TARGET>
85+
``'
86+
* Copy `.mbedbuild/` into the test suite directory.
87+
* Set your current directory to the test suite directory
88+
* Run the following command to build the test binary with the full profile
8589
8690
```
87-
touch mbed-os.lib;mkdir build;cd build;cmake .. -GNinja;cmake --build .
91+
touch mbed-os.lib && mkdir build && cd build && cmake .. -G Ninja && cmake --build .
8892
```
89-
* run below command to build baremetal profile green tea
93+
* Run the following command to build the test binary with the baremetal profile
9094
```
91-
touch mbed-os.lib;mkdir build;cd build;cmake .. -GNinja -DMBED_BAREMETAL_GREENTEA_TEST=ON;cmake --build .
95+
touch mbed-os.lib && mkdir build && cd build && cmake .. -G Ninja -DMBED_BAREMETAL_GREENTEA_TEST=ON && cmake --build .
9296
```
9397
9498
Note: This steps will get evolve once mbedtools have a proper way of invoking greentea test using mbedtools command

tools/cmake/mbed_greentea.cmake

+4-4
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,10 @@ macro(mbed_greentea_cmake_macro)
2929
TEST_REQUIRED_LIBS
3030
)
3131
cmake_parse_arguments(MBED_GREENTEA
32-
"${options}"
33-
"${singleValueArgs}"
34-
"${multipleValueArgs}"
35-
${ARGN}
32+
"${options}"
33+
"${singleValueArgs}"
34+
"${multipleValueArgs}"
35+
${ARGN}
3636
)
3737

3838
set(TEST_NAME ${MBED_GREENTEA_TEST_NAME})

0 commit comments

Comments
 (0)