Minimal changes in order to build for Zephyr#4883
Open
thorsten-klein wants to merge 2 commits intogoogle:mainfrom
Open
Minimal changes in order to build for Zephyr#4883thorsten-klein wants to merge 2 commits intogoogle:mainfrom
thorsten-klein wants to merge 2 commits intogoogle:mainfrom
Conversation
Although Zephyr has it own test suite, it doesn't work well with C++. gtest and gmock seem more adapted. This adds support of Zephyr OS in order to test C++ libraries and applications. Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
88171dc to
55da831
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR introduces minimal changes required to successfully build and run GoogleTest on Zephyr.
It is based on PR #4692 and it includes a cherry-picked commit from @anobli (Thank you!).
I open this PR to gather feedback from googletest maintainers. Maybe you prefer this more minimal solution for the start?
Otherwise please feel free to close this PR if #4692 will be merged.
Notes
These changes only contain the changes that are really necessary to build successful for Zephyr.
All further additional Zephyr-specific integration files can be maintained in downstream Zephyr projects, so there is no hard need to add them to googletest.
Usage Instructions (for Zephyr projects)
Add googletest to your
west.ymlmanifest, for example underlib/googletest:Create the module integration files within your project:
modules/googletest/CMakeLists.txtmodules/googletest/KconfigRefer to PR Add support of Zephyr OS #4692 how they can look like. In CMake, you can locate the googletest sources with:
Include the module’s Kconfig in your project’s Zephyr module
Kconfig:Add the module directory in your project’s Zephyr module
CMakeLists.txt: