-
Notifications
You must be signed in to change notification settings - Fork 10.6k
Add support of Zephyr OS #4692
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
base: main
Are you sure you want to change the base?
Add support of Zephyr OS #4692
Conversation
|
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. |
c767d2d to
53e0458
Compare
53e0458 to
ef017e6
Compare
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>
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 <abailon@baylibre.com>
ef017e6 to
8e3886a
Compare
|
What little I can review and understand looks good to me |
|
+1 This is fantastic, and works great out of the box. |
|
+1 @dkaszews can you merge it? |
|
@awojasinski I'm not a maintainer, I just reviewed it because I'd like to see it merged myself |
|
We are also really interested in the change. But we have the problem that the main zephyr thread is still running when the tests are executed. Does anybody else have this issue? |
|
Any updates on this? When will this be merged? |
Could you elaborate more about the issue you are getting ? |
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.