Skip to content

build: repair the build with VS2019 16.10.0 #39595

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

Conversation

shahmishal
Copy link
Member

The newest VS2019 release updates CMake to 3.20, which picks up
llvm-mt as a preferred manifest tool. However, we do not build
llvm-mt with libxml2 support currently, which prevents the use of the
just built manifest tool for building libdispatch. Explicitly opt into
using the MSVC manifest-tool.

Cherry-pick 2990e8b

(cherry picked from commit d4d4d13)

The newest VS2019 release updates CMake to 3.20, which picks up
`llvm-mt` as a preferred manifest tool.  However, we do not build
`llvm-mt` with libxml2 support currently, which prevents the use of the
just built manifest tool for building libdispatch.  Explicitly opt into
using the MSVC manifest-tool.

Cherry-pick 2990e8b

(cherry picked from commit d4d4d13)
@shahmishal shahmishal requested a review from a team as a code owner October 5, 2021 21:23
@shahmishal
Copy link
Member Author

@swift-ci test

Explicitly opt into the Microsoft manifest tool to be resilient against
future CMake releases which prefer `llvm-mt` when building with clang
(including clang-cl).  This should ensure that we do not get broken with
the update to VS2019 16.10 and CMake 3.20.

(cherry picked from commit 3274984)
@shahmishal
Copy link
Member Author

@swift-ci test windows

@shahmishal
Copy link
Member Author

@swift-ci test macOS

@shahmishal
Copy link
Member Author

@swift-ci test Linux

@shahmishal shahmishal merged commit b5891c8 into release/5.5-08092021 Oct 7, 2021
@shahmishal shahmishal deleted the shahmishal/cherry-pick-d4d4d136a2c53cfe8828f85762c14def46ea5c99 branch October 7, 2021 08:10
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