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

Add support of Zephyr OS #4692

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

anobli
Copy link

@anobli anobli commented Jan 8, 2025

This updates googletest in order to use it with Zephyr.
The first patch update gtest itself to support Zephyr OS it self.
The second adds files used by Zephyr build system. If this is not acceptable, then I will drop it and find another way to integrate it to zephyr build system.

Copy link

google-cla bot commented Jan 8, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@anobli anobli force-pushed the abailon/zephyr/dev branch 3 times, most recently from c767d2d to 53e0458 Compare February 10, 2025 15:22
@anobli anobli requested a review from dkaszews February 10, 2025 15:26
@anobli anobli force-pushed the abailon/zephyr/dev branch from 53e0458 to ef017e6 Compare February 11, 2025 08:48
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 <[email protected]>
This update googletest to use it as Zephyr OS external module.
This allows downloading, configure and building googltest using
Zephyr west build system, without any user manipulation.

Signed-off-by: Alexandre Bailon <[email protected]>
@anobli anobli force-pushed the abailon/zephyr/dev branch from ef017e6 to 8e3886a Compare February 11, 2025 08:50
@anobli anobli requested a review from dkaszews February 11, 2025 08:50
@dkaszews
Copy link

What little I can review and understand looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants