-
Notifications
You must be signed in to change notification settings - Fork 116
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
Enable on-device tests for evergreen AOSP #3247
base: main
Are you sure you want to change the base?
Conversation
44aee2d
to
8d8de60
Compare
67b2a6e
to
476c94b
Compare
target_name != "nplb_evergreen_compat_tests" && | ||
target_name != "player_filter_benchmarks" && | ||
# TODO: This one looks fishy. | ||
target_name != "player_filter_tests" && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is expected. Per @andrewsavage1 the test suite was disabled because it depends too much on the starboard implementation.
fdc6ec0
to
b0fec46
Compare
b/291130824
Change-Id: I715f754fc40573584523009eb538cf43432f4b3b
Change-Id: Ibece0510d5e1ad9dd4f63a2f3ff10913739b0bf1
Change-Id: Ica6b7a17a1d3c7ad77025c6a954070c7acf139d9
Change-Id: I3c5b13a3d8fad52175c91f8923ff6dd83f94c946
Change-Id: Ia34dda84ff18817ecd019fb06c324af246d1d3a1
b0fec46
to
d9d046b
Compare
gradle_files_dir = "$root_out_dir/gradle/$installable_target_name" | ||
# The gradle build expects libcrashpad_handler.so to be present in the out directory. | ||
if (sb_is_evergreen_compatible) { | ||
deps += [ "//starboard/loader_app:copy_crashpad_handler_named_as_so" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The crashpad_handler target is not needed for unit test targets. Removing the dependency will save us some space.
b/268244534
b/291130824