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

Only filter sdk when PREINSTRUMENTED_SDK_VERSIONS is not empty #7787

Conversation

utzcoz
Copy link
Member

@utzcoz utzcoz commented Nov 5, 2022

This CL can fix building error when PREINSTRUMENTED_SDK_VERSIONS is empty/not set when running command
./gradlew :preinstrumented:instrumentAll.

I tested it locally with following two commands:

./gradlew :preinstrumented:instrumentAll
PREINSTRUMENTED_SDK_VERSIONS=33 ./gradlew :preinstrumented:instrumentAll

@utzcoz utzcoz requested a review from hoisie November 5, 2022 12:40
@utzcoz utzcoz force-pushed the fix-preinstrumented-instrumentAll-building-error branch 2 times, most recently from 51207a3 to ec88d79 Compare November 5, 2022 12:51
This CL can fix building error when PREINSTRUMENTED_SDK_VERSIONS is
empty/not set when running command
`./gradlew :preinstrumented:instrumentAll`.

I tested it locally with following two commands:

./gradlew :preinstrumented:instrumentAll
PREINSTRUMENTED_SDK_VERSIONS=33 ./gradlew :preinstrumented:instrumentAll

Signed-off-by: utzcoz <utzcoz@outlook.com>
Signed-off-by: utzcoz <utzcoz@outlook.com>
@utzcoz utzcoz force-pushed the fix-preinstrumented-instrumentAll-building-error branch from ec88d79 to d1bd0a8 Compare November 6, 2022 09:18
@utzcoz utzcoz merged commit bf95af2 into robolectric:master Nov 7, 2022
@utzcoz utzcoz deleted the fix-preinstrumented-instrumentAll-building-error branch November 7, 2022 07:35
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.

None yet

2 participants