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.

Sign up for GitHub

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

Jump to bottom

x/build: misc-compile coverage for GOOS=android ports in LUCI #61762

Open
dmitshur opened this issue Aug 4, 2023 · 2 comments
Open

x/build: misc-compile coverage for GOOS=android ports in LUCI #61762

dmitshur opened this issue Aug 4, 2023 · 2 comments
Labels
Builders x/build issues (builders, bots, dashboards) NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. OS-Android
Milestone
Unreleased

Comments

@dmitshur
Copy link
Contributor

dmitshur commented Aug 4, 2023 •
edited

This is like #25963, but for LUCI.

The current failure mode (for android/arm specifically):

# compress/bzip2.test
loadinternal: cannot find runtime/cgo
/.../goroot/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
gcc: error: unrecognized command-line option ‘-marm’; did you mean ‘-mabm’?

See https://logs.chromium.org/logs/golang/buildbucket/cr-buildbucket/8773728632941952977/+/u/step/10/log/2.

misc-compile coverage for android/386, android/amd64, and android/arm64 needs similar attention.

CC @golang/android.
@dmitshur dmitshur added Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Aug 4, 2023
@dmitshur dmitshur added this to the Unreleased milestone Aug 4, 2023
@dmitshur dmitshur added OS-Android arch-arm Issues solely affecting the 32-bit arm architecture. labels Aug 4, 2023
@ianlancetaylor
Copy link
Contributor

In case it's not clear, this is trying to pass an ARM option to an x86 C compiler. Somehow the tool is using the wrong C compiler.

@dmitshur
Copy link
Contributor Author

dmitshur commented Aug 4, 2023 •
edited

Thanks. I’ll move this to NeedsDecision, since the current problem is understood, and next up we need to decide what we’d like the LUCI misc-compiler trybot to test for these ports.

@dmitshur dmitshur added NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Aug 4, 2023
@dmitshur dmitshur changed the title x/build: misc-compile coverage for android/arm port in LUCI x/build: misc-compile coverage for GOOS=android ports in LUCI Aug 10, 2023
@dmitshur dmitshur removed the arch-arm Issues solely affecting the 32-bit arm architecture. label Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Builders x/build issues (builders, bots, dashboards) NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. OS-Android
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
2 participants
@dmitshur @ianlancetaylor