ARMmbed / mbed-tools

⚠️ Beta Status: New command line tooling for Mbed OS
Apache License 2.0
45 stars 30 forks source link

Add include guard to `mbed_config.cmake` #321

Closed rwalton-arm closed 2 years ago

rwalton-arm commented 2 years ago

Description

Some users may want to include mbed_config.cmake in their application CMakeLists.txt and modify some of the preset variables. This is currently impossible: the variables defined in mbed_config.cmake will be overwritten when the file is included again in the mbed-os root CMakeLists.txt.

We can use CMake's include_guard feature to force CMake to skip processing of the file if it has already been processed previously in the build.

Azure Pipelines have removed Python 3.6 from the macOS-latest runners. Python3.6 is nearing end of life so we can remove testing for it.

https://www.python.org/dev/peps/pep-0494/#lifespan

Test Coverage

codecov[bot] commented 2 years ago

Codecov Report

Merging #321 (fd9d186) into master (2a421e7) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #321   +/-   ##
=======================================
  Coverage   97.08%   97.08%           
=======================================
  Files          92       92           
  Lines        2779     2779           
=======================================
  Hits         2698     2698           
  Misses         81       81