Skip to content

Fix clang-win.jam so that it works#361

Merged
eldiener merged 1 commit intodevelopfrom
pr/fix-clang-win
Oct 29, 2018
Merged

Fix clang-win.jam so that it works#361
eldiener merged 1 commit intodevelopfrom
pr/fix-clang-win

Conversation

@pdimov
Copy link
Member

@pdimov pdimov commented Oct 27, 2018

No description provided.

@pdimov
Copy link
Member Author

pdimov commented Oct 27, 2018

Fixes #330.

@eldiener eldiener merged commit 6910438 into develop Oct 29, 2018
@pdimov
Copy link
Member Author

pdimov commented Oct 29, 2018

With this in place, clang-cl should now be usable on Appveyor (which has it installed and in PATH), by just using toolset=clang-win.

@eldiener
Copy link
Contributor

The new clang-win.jam worked well. Is there a way to choose a VC++ version as the backend headers and libraries, or is the latest VC++ version found always chosen ? The old clang-win had a property but the new one does not have anything like that which I can see.

@pdimov
Copy link
Member Author

pdimov commented Oct 30, 2018

I'm not sure how clang-cl picks the version to use. Will need to check the sources. I think that if you invoke it from a developer command prompt, it uses whatever is set by it, but if not, it scans and picks the most recent MSVC toolset (and Windows SDK - these two are separate.)

@grafikrobot grafikrobot deleted the pr/fix-clang-win branch December 26, 2020 16:56
@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2021

Thank you for your contributions. Main development of B2 has moved to https://github.com/bfgroup/b2 Please consider following up at https://github.com/bfgroup/b2/pulls

@github-actions github-actions bot added the transition Transition to bfgroup/b2 label Oct 2, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

transition Transition to bfgroup/b2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants