Bug 197288 – [AppleWin] Upgrade bots to Visual Studio 2019

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Don Olmstead]

Visual Studio 2019 was released this month, https://docs.microsoft.com/en-us/visualstudio/releases/2019/release-notes . WinCairo currently requires either an old version of 2017 or 2019 to build. Currently our bots are using 2019 with ninja but we'd like to switch general development to 2019. AppleWin should bump the version as well.

I would suggest installing Visual Studio 2019 Build Tools on the bots, direct link is https://aka.ms/vs/16/release/vs_buildtools.exe or expand the Tools section at https://visualstudio.microsoft.com/downloads/ , which has just the build tools and not the IDE. Its meant more for bots.

Our bots have the following workloads installed see https://docs.microsoft.com/en-us/visualstudio/install/workload-component-id-vs-build-tools?view=vs-2019 for a list of all workloads

    Microsoft.VisualStudio.Component.Roslyn.Compiler
    Microsoft.Component.MSBuild
    Microsoft.VisualStudio.Component.CoreBuildTools
    Microsoft.VisualStudio.Workload.MSBuildTools
    Microsoft.VisualStudio.Component.Windows10SDK
    Microsoft.VisualStudio.Component.VC.CoreBuildTools
    Microsoft.VisualStudio.Component.VC.Tools.x86.x64
    Microsoft.VisualStudio.Component.VC.Redist.14.Latest
    Microsoft.VisualStudio.Component.Windows10SDK.17763
    Microsoft.VisualStudio.Workload.VCTools

At the same time if you're intending to use MSBuild through build-webkit you'll need to update CMake to a 3.14.x version which has support for 2019. Currently 3.14.3 is the latest.

[Don Olmstead]

Alex I think this will make the bots green again.

[Alex Christensen]

I think we need to try to keep 2017 for another little while.

[Don Olmstead]

(In reply to Alex Christensen from comment #2)
> I think we need to try to keep 2017 for another little while.

You may have to roll out that C++17 patch then. If you're not able to reproduce the build breakage in the bots locally and you're using 2017 my bet is that the bots need to be updated to a newer version of 2017. MS is testing AppleWin and WinCairo internally to check for compiler breakages.

2019 is probably a good idea for you all in the long term. WinCairo flat out doesn't work without it.

[Alex Christensen]

We successfully covered up all the bugs for now, but we should still push to upgrade.

[Don Olmstead]

Here's the command line we use for installing on WinCairo. Adding Brent since he was asking for information on the Windows SDK.

C:\Users\ContainerAdministrator\AppData\Local\Temp\VSBuildTools2019.exe --quiet --norestart --nocache --wait --add Microsoft.VisualStudio.Component.Roslyn.Compiler --add Microsoft.Component.MSBuild --add Microsoft.VisualStudio.Component.CoreBuildTools --add Microsoft.VisualStudio.Workload.MSBuildTools --add Microsoft.VisualStudio.Component.Windows10SDK --add Microsoft.VisualStudio.Component.VC.CoreBuildTools --add Microsoft.VisualStudio.Component.VC.Tools.x86.x64 --add Microsoft.VisualStudio.Component.VC.Redist.14.Latest --add Microsoft.VisualStudio.Component.Windows10SDK.17763 --add Microsoft.VisualStudio.Workload.VCTools --installPath C:\MSVS

Again we use Visual Studio 2019 Build Tools which is meant for build servers. You won't get an IDE but you'll get all the things to build with.

[Don Olmstead]

Any updates on this?

[Brent Fulgham]

Pulling into radar to track updates on bots and EWS.

[Radar WebKit Bug Importer]

<rdar://problem/54520227>

[Fujii Hironori]

Some bots seem to start using Visual Studio 2019. Will it be available soon?

https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/8331

> -- Building for: Visual Studio 16 2019
> -- The C compiler identification is MSVC 19.23.28107.0
> -- The CXX compiler identification is MSVC 19.23.28107.0

[Alexey Proskuryakov]

The upgrade is supposed to be all done.

[Fujii Hironori]

Win7 bot still seems using VS2017?
> Using Visual Studio: /cygdrive/c/Program Files (x86)/Microsoft Visual Studio/2017/Community
https://build.webkit.org/builders/Apple%20Win%207%20Release%20%28Build%29/builds/8567/steps/compile-webkit/logs/stdio

[Alexey Proskuryakov]

I thought that Win7 was no longer supported. But of course we need to remove the bots if that's the case. Brent and Per Arne, what do you say?