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

Race detector *.syso files missing for cross-compilation #181

Closed
tamird opened this issue Jan 10, 2022 · 7 comments
Closed

Race detector *.syso files missing for cross-compilation #181

tamird opened this issue Jan 10, 2022 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@tamird
Copy link

tamird commented Jan 10, 2022

This is roughly golang/go#42382.

My understanding is that binaries downloaded from https://github.com/actions/go-versions are built from source, rather than being downloaded from https://go.dev/dl/. But apparently not:

tamird@tamird:~/src/goid/go$ curl -sfSL https://github.com/actions/go-versions/releases/download/1.17.5-1559554870/go-1.17.5-linux-x64.tar.gz | tar zx
tamird@tamird:~/src/goid/go$ find src/runtime -name '*.syso'
src/runtime/race/race_linux_amd64.syso

The official Go binaries have long omitted these files for non-host platforms (https://go-review.googlesource.com/c/build/+/144281/) but that shouldn't have affected go-versions if those packages are indeed built from source. But they appear not to be. It would be nice to fix that so that cross-compilation of race binaries works.

@tamird tamird added bug Something isn't working needs triage labels Jan 10, 2022
@vsafonkin
Copy link

Hi @tamird, thank you for your report, we will take a look it.

@vsafonkin
Copy link

@tamird, sorry for the delay, is this issue still actual for you?

@tamird
Copy link
Author

tamird commented Apr 4, 2022

I included reproduction instructions in the description. Can you try them?

@vsafonkin
Copy link

I included reproduction instructions in the description. Can you try them?

Sure, thank you!

@vsafonkin vsafonkin self-assigned this Apr 18, 2022
@vsafonkin
Copy link

vsafonkin commented Apr 18, 2022

@tamird, I'm afraid that building Go from source code will cause a lot of problems and pretty complicated for maintenance. We already have experience with setup-python action where we use Python builds from source code for each OS platform and it cause many issues with self-hosted runners.

@tamird
Copy link
Author

tamird commented Apr 18, 2022

That's fine - it should perhaps be documented how these packages are built.

@vsafonkin
Copy link

@tamird, I've added a note about it to the readme file, I believe we can close this issue, thank you for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants