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

x/dashboard: request for updated C compilers on windows builders #43195

Closed
thanm opened this issue Dec 15, 2020 · 5 comments
Closed

x/dashboard: request for updated C compilers on windows builders #43195

thanm opened this issue Dec 15, 2020 · 5 comments
Labels
Builders x/build issues (builders, bots, dashboards) FeatureRequest FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Windows
Milestone

Comments

@thanm
Copy link
Contributor

thanm commented Dec 15, 2020

While working on issue #39326, I noticed that the C compilers we have installed on our windows builders (at least the ones I have used via gomote) are pretty ancient -- GCC 5.

It would be great if we could update the version to something a bit more recent (V10 maybe). Also worth noting that many folks are now using clang instead of gcc on windows -- it would be very helpful if our windows builders had clang installed as well.

@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Dec 15, 2020
@gopherbot gopherbot added this to the Unreleased milestone Dec 15, 2020
@thanm
Copy link
Contributor Author

thanm commented Dec 15, 2020

@golang/release

@dmitshur dmitshur changed the title x/build/cmd/release: request for updated C compilers on windows builders x/dashboard: request for updated C compilers on windows builders Dec 15, 2020
@dmitshur dmitshur added FeatureRequest NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Dec 15, 2020
@ALTree
Copy link
Member

ALTree commented Dec 15, 2020

This is #35006.

I've been using the toolchain recommended in #35006 (comment) and it works much better than mingw. It's also completely static and requires no installation.

@thanm
Copy link
Contributor Author

thanm commented Dec 15, 2020

@ALTree thanks, that's helpful. I'll maybe try the musl.cc copy and see if I can make progress with that.

@thanm
Copy link
Contributor Author

thanm commented Dec 17, 2020

The toolchain from musl.cc helped a lot -- I was able to develop a fix for my lld bug using that.

I tried running all.bash on a windows gomote with the musl.cc toolchain (overriding the default gomote toolchain). There were a couple of failures of the form

    nm_test.go:121: building test executable failed: exit status 2 # command-line-arguments
        c:/workdir/x86_64-w64-mingw32-native/bin/../lib/gcc/x86_64-w64-mingw32/9.3.1/../../../libmsvcrt.a(/2203): duplicate symbol reference: _unlock_file in both libgcc(.text) and libgcc(.data)
        libgcc(.text): relocation target ___chkstk_ms not defined
        libgcc(.text): relocation target atexit not defined

This is with internal linking. Might be worth poking at those to see if they can be worked around, since other than that it looked good to me.

@bcmills
Copy link
Contributor

bcmills commented Jan 26, 2022

Duplicate of #35006

@bcmills bcmills marked this as a duplicate of #35006 Jan 26, 2022
@bcmills bcmills closed this as completed Jan 26, 2022
@golang golang locked and limited conversation to collaborators Jan 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Builders x/build issues (builders, bots, dashboards) FeatureRequest FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Windows
Projects
None yet
Development

No branches or pull requests

5 participants