-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Draft: Windows: Add clang-cl CI job #12946
Draft: Windows: Add clang-cl CI job #12946
Conversation
@envoyproxy/windows-dev |
current
|
Just to document the failure, which is well-understood, the command;
is an invocation of AR (static library creation), dispatched to the Once we figure out why these arguments are added for static AR invocations in the clang-cl step, we'll be one step closer to a functional clang-cl build. |
Note this issue may be local to libcurl's CMakeLists.txt configuration and have nothing at all to do with bazel's rules_foreign_cc or the envoy build itself, it would not be the first issue we have had to hack around on that schema. |
I can confirm the curl CMakeLists.txt logic is doing some inappropriate things. Presently correlating this to my use of cmake for httpd builds, and jumping forwards to curl 7.72.0 so we don't spend cycles on an already archaic release. |
This PR development is also blocked on merging #13066 due to gcc compilation issues to be resolved. |
Signed-off-by: Sunjay Bhatia <[email protected]>
Signed-off-by: William A Rowe Jr <[email protected]>
This is presently blocked on local effort and changes to flow through from #13133 /wait |
/retest |
Retrying Azure Pipelines, to retry CircleCI checks, use |
superceded by #13133, closing |
Draft PR for now to demonstrate and iterate on Windows clang-cl build in CI
Commit Message: Windows: Add clang-cl CI job
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A