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

BasicNativeAndroidTest: Move gtest to a single flavor #489

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

utzcoz
Copy link
Contributor

@utzcoz utzcoz commented Oct 28, 2023

The junit-gtest library will bring native test library and its c++ shared library into the final package. It's not easy to keep these necessary shared libraries into androidTest flavor only. So this CL adds a new flavor called nativeTest to include native test related libraries and run it with androidTest. For normal usage, we can use core flavor, the new default flavor without any native test related libraries.

@utzcoz utzcoz force-pushed the move-gtest-to-a-single-flavor branch from bcd2d99 to db23ecd Compare October 28, 2023 09:09
@utzcoz
Copy link
Contributor Author

utzcoz commented Oct 28, 2023

cc @fsladkey and @brettchabot for reviewing and discussion. I am experimenting junit-gtest, and I want to find a better approach to resolve junit-gtest packaged so files, and if Google has more experience, really hope to listen your response.

The junit-gtest library will bring native test library and its c++
shared library into the final package. It's not easy to keep these
necessary shared libraries into androidTest flavor only. So this CL adds
a new flavor called nativeTest to include native test related libraries
and run it with androidTest. For normal usage, we can use core flavor,
the new default flavor without any native test related libraries.

Signed-off-by: utzcoz <[email protected]>
@utzcoz utzcoz force-pushed the move-gtest-to-a-single-flavor branch from db23ecd to d092540 Compare October 28, 2023 09:17
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.

1 participant