Skip to content

Windows: override msvc-cl config if BAZEL_BUILD_EXTRA_OPTIONS set#12945

Merged
lizan merged 1 commit intoenvoyproxy:masterfrom
greenhouse-org:windows-allow-override-msvc-cl
Sep 7, 2020
Merged

Windows: override msvc-cl config if BAZEL_BUILD_EXTRA_OPTIONS set#12945
lizan merged 1 commit intoenvoyproxy:masterfrom
greenhouse-org:windows-allow-override-msvc-cl

Conversation

@sunjayBhatia
Copy link
Member

Commit Message: Windows: override msvc-cl config if BAZEL_BUILD_EXTRA_OPTIONS set
Additional Description: Will allow us to cleanly add a clang-cl build in CI
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Will allow us to cleanly add a clang-cl build in CI

Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
@sunjayBhatia
Copy link
Member Author

@envoyproxy/windows-dev

@sunjayBhatia
Copy link
Member Author

sunjayBhatia commented Sep 2, 2020

@snowp Bill is out this week on vacation

@sunjayBhatia
Copy link
Member Author

Windows RBE pool might be struggling a little

@wrowe wrowe self-requested a review September 2, 2020 18:45
Copy link
Contributor

@wrowe wrowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, permits passing BAZEL_BUILD_EXTRA_OPTIONS to include the toolchain desired. @lizan can you give this an glance and senior-maintainer approval?

@lizan lizan merged commit 69cf872 into envoyproxy:master Sep 7, 2020
@sunjayBhatia sunjayBhatia deleted the windows-allow-override-msvc-cl branch September 14, 2020 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants