LGTM, maybe @kfunk or @brauch know more?

Why did you replace the registry query by a fixed path?

echo Define which compiler for VS2017 to use. Possible architectures are:

VS2017 should change too. Any ideas if this should print the VS exact version or just a generic question?

In D21589#477604, @brauch wrote:

Why did you replace the registry query by a fixed path?

Which part of the patch are you referring to? I didn't intentionally do anything differently than the VS2017 hack, I tried to just duplicated that same method for VS2019.
For VS2015 and earlier, it appears the installer defined an environment variable pointing to the tools which the previous versions of this script used. For VS2017+, that variable doesn't seem to exist
(I have VS2015 and VS2019 on my computer, which is my basis for making the above claims)

In D21589#477661, @Petross404 wrote:

echo Define which compiler for VS2017 to use. Possible architectures are:

VS2017 should change too. Any ideas if this should print the VS exact version or just a generic question?

Oops. Good catch. Would it work to just say Define which toolchain to use. Possible architectures are: rather than me having to go any further into Windows Batch scripting than I have to? :)

brauch is referring to this

for /f "usebackq tokens=3*" %%a in (`reg query "HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\VisualStudio\SxS\VS7" /s`) do (
     set vs15_path=%%a %%b
     if exist "!vs15_path!Common7\Tools\VsDevCmd.bat" (
	set "VS150COMNTOOLS=!vs15_path!Common7\Tools\"
	goto :end
     )
   )
)
:end

It's best to leave windows decide where VS is installed rather than guessing (even if it's successful for 99% of the cases). I used to install software which required many GB to another partition since C: was tiny.

About the message of the toolchain I think it's correct. Keep it simple ;) You might want to keep your changes as a backup and start over, but consult the devs too. Good luck!

EDIT: Wrong code quote

In D21589#477664, @Petross404 wrote:

brauch is referring to this

for /f "usebackq tokens=3*" %%a in (`reg query "HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\VisualStudio\SxS\VS7" /s`) do (
     set vs15_path=%%a %%b
     if exist "!vs15_path!Common7\Tools\VsDevCmd.bat" (
	set "VS150COMNTOOLS=!vs15_path!Common7\Tools\"
	goto :end
     )
   )
)
:end

Oops. What happened here is I was hacking on the release version which looks more like the patch I posted. I'll see if I can make sense of the master branch version and make it more like that!

It's best to leave windows decide where VS is installed rather than guessing (even if it's successful for 99% of the cases). I used to install software which required many GB to another partition since C: was tiny.

About the message of the toolchain I think it's correct. Keep it simple ;) You might want to keep your changes as a backup and start over, but consult the devs too. Good luck!

Ping? Please keep hammering the iron while hot :)

In D21589#492284, @kossebau wrote:

Ping? Please keep hammering the iron while hot :)

I'm sorry. I haven't forgotten. I have just covered with hot irons for quite awhile :). I am currently on travel and I will be for several more weeks but when I am able to get my VM working again I will make sure to update this patch

Actually, there is a bit of a problem... The version of the file currently on master is bugged :(

See: https://phabricator.kde.org/D17908#492404

In D21589#492408, @sredman wrote:

Actually, there is a bit of a problem... The version of the file currently on master is bugged :(

See: https://phabricator.kde.org/D17908#492404

Any progress with the MS tool?

In D21589#494068, @Petross404 wrote:

In D21589#492408, @sredman wrote:

Actually, there is a bit of a problem... The version of the file currently on master is bugged :(

See: https://phabricator.kde.org/D17908#492404

Any progress with the MS tool?

Not really. Here is a blog post which provides suggestions for how to use it: https://renenyffenegger.ch/notes/Windows/development/Visual-Studio/environment-variables/index

Things are going to be crazy for me until probably September, so if you want to have a look in to this I would appreciate it. I can probably test whatever you come up with if you ping me :)

Hi,

I have installed Build Tools for VS2019 and the installation is at

C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools

I found this string inside HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\1eb19443 which honestly doesn't seem the proper key to extract this value but anyway.

A small preview of what I wrote is this:

@echo off

setlocal enabledelayedexpansion
setlocal enableextensions

REM Visual Studio 2019 is tricky to be found.
set VS2019_KEY_NAME="HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\1eb19443"
set VS2019_VAL_NAME="InstallLocation"
for /f "usebackq skip=2 tokens=3*" %%a in (`^(reg query %VS2019_KEY_NAME% /v %VS2019_VAL_NAME% ^| find "%VS2019_VAL_NAME%"^) 2^>nul `) do (
	set vs16_path=%%b
	if exist "!vs16_path!Common7\Tools\LaunchDevCmd.bat" (
		set "VS160COMNTOOLS=!vs15_path!Common7\Tools\"
		goto :end
	)
)

No matter what I try I get the error "ERROR: The system was unable to find the specified registry key or value." Any ideas?

EDIT My script accounts only for BuildTools, thats wrong.