Skip to content

Support Build Tools#9

Merged
dra27 merged 2 commits intometastack:masterfrom
diskuv:feature-msbuild
Sep 3, 2021
Merged

Support Build Tools#9
dra27 merged 2 commits intometastack:masterfrom
diskuv:feature-msbuild

Conversation

@jonahbeckford
Copy link
Copy Markdown
Contributor

@jonahbeckford jonahbeckford commented Aug 30, 2021

Changes:

  1. Visual Studio can install Build Tools which is not detected by default by vswhere.exe. This PR adds -products * to the vswhere command line options.
    So & "C:\Program Files (x86)\Microsoft Visual Studio\Installer\vswhere.exe" -all which is the existing code does not detect Build Tools installations.
    While & "C:\Program Files (x86)\Microsoft Visual Studio\Installer\vswhere.exe" -all -products * does detect Build Tools installations.
  2. There is a small bug with the strong/weak detection logic. Before it compared a guaranteed semicolon terminated path with a possibly semicolon terminated path; this PR makes sure both side of the comparison are done with semicolon termination.

(Can split PR if needed)

@jonahbeckford jonahbeckford marked this pull request as draft August 31, 2021 20:08
@jonahbeckford jonahbeckford marked this pull request as ready for review August 31, 2021 21:34
@jonahbeckford
Copy link
Copy Markdown
Contributor Author

@dra27 Not urgent; just name-dropping since you had mentioned this repo last week, so this and #8 were opened.

FYI: I won't be able to use msvs-tools during my distribution's installation because msvs-tools requires MSYS2/Cygwin which won't be available at the start of the installation, but I will be using it at runtime when switching from Windows into the MSYS2 environment.

@dra27 dra27 merged commit a1c7bd3 into metastack:master Sep 3, 2021
@dra27
Copy link
Copy Markdown
Contributor

dra27 commented Sep 3, 2021

Thanks for these!

@dra27
Copy link
Copy Markdown
Contributor

dra27 commented Sep 6, 2021

I just cut a release with your two fixes (and another more minor one I discovered working on OCaml Labs' docker images).

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