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

Install platforms;android-32 #1745

Merged
merged 1 commit into from
Sep 29, 2023
Merged

Install platforms;android-32 #1745

merged 1 commit into from
Sep 29, 2023

Conversation

meteorcloudy
Copy link
Member

Try to fix #1588

@meteorcloudy
Copy link
Member Author

meteorcloudy commented Sep 26, 2023

We'll have to rebuild docker images + Windows VM image and install on macOS machines manually.

@meteorcloudy meteorcloudy merged commit c6bfff5 into master Sep 29, 2023
7 checks passed
@meteorcloudy meteorcloudy deleted the install_new_android_sdk branch September 29, 2023 08:05
@sgowroji
Copy link
Member

sgowroji commented Oct 3, 2023

Hi @meteorcloudy, We are still seeing the error in flogger https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/3350#018af351-1325-4b64-b7b7-dbb5e79bc2a4 and main pipeline

ERROR: /var/lib/buildkite-agent/builds/bk-docker-7g38/bazel-downstream-projects/flogger/google/BUILD:36:16: in javadoc_library rule //google:flogger_javadoc:
Traceback (most recent call last):
	File "/var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/c3ea6ac52571282244a9490118bd3aa5/external/google_bazel_common/tools/javadoc/javadoc.bzl", line 29, column 49, in _javadoc_library
		transitive_deps.append(dep[JavaInfo].transitive_deps)
Error: 'JavaInfo' value has no field or method 'transitive_deps'

@meteorcloudy
Copy link
Member Author

Sorry, I haven't deployed the change yet.

@meteorcloudy
Copy link
Member Author

The new android SDK has been installed. But Flogger now fails with a different error: https://buildkite.com/bazel/flogger/builds/2701#018d64f0-b10a-4a8c-92e6-beb4d40f5085

@sgowroji Can you please file an issue against Flogger?

@sgowroji
Copy link
Member

sgowroji commented Feb 2, 2024

@meteorcloudy  raised here google/flogger#373

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.

Flogger needs Android SDK upgrade for Bazel CI VMs
2 participants