Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple sketch support #2283

Merged
merged 3 commits into from
Mar 4, 2024
Merged

Conversation

ddowling
Copy link
Contributor

@ddowling ddowling commented Feb 21, 2024

Summary

Allow CMakeLists.txt files to contain multiple build_sketch() targets.

This PR fixes #2282

A lot of Arduino library projects will contain multiple example applications that demonstrate the library functionality. It is also common practice on larger embedded applications to have a number of small test programs that test specific parts of the systems so these can be tested in isolation from the overall application code.
This change means all of the examples and tests can be built together from the top level CMakeList.txt file.

There is a dependency error when .ino files are not in the parent directory of the project that is fixed with this pull request.

…file

build multiple targets.
* Add an include_guard() on build_sketch.cmake to prevent errors if this
file is included multiple time.
@fpistm fpistm added enhancement New feature or request New feature labels Feb 21, 2024
@fpistm fpistm added this to the 2.8.0 milestone Feb 21, 2024
@fpistm fpistm requested review from massonal and fpistm February 21, 2024 13:40
@fpistm fpistm merged commit 7f5d45e into stm32duino:main Mar 4, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

cmake build fails if more than one build_sketch definition is used
3 participants