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

Add an "Upgrade clang" step #2201

Closed

Conversation

wantehchang
Copy link
Collaborator

In the current windows-latest (windows-2022) image, the installed Clang is version 16, but the Visual Studio STL requires Clang version 17 or later. The "Upgrade clang" workaround is recommended in actions/runner-images#10001 (comment).

In the current windows-latest (windows-2022) image, the installed Clang
is version 16, but the Visual Studio STL requires Clang version 17 or
later. The "Upgrade clang" workaround is recommended in
actions/runner-images#10001 (comment).
@vrabaud
Copy link
Collaborator

vrabaud commented Jun 11, 2024

No need for that if you remove it from the path in #2202 right ?

@wantehchang
Copy link
Collaborator Author

Yes, we only need either this pull request or #2202. This pull request is simpler, but the upgrade takes a long time. So I also prefer #2202.

@wantehchang
Copy link
Collaborator Author

I abandoned this pull request.

@wantehchang wantehchang deleted the ci-windows-use-vs-clang-cl branch June 12, 2024 00:07
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.

2 participants