Summary: | [Win] Use a bundled vswhere.exe instead of downloading by update-vswhere.py | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Fujii Hironori <Hironori.Fujii> | ||||||||||||
Component: | Tools / Tests | Assignee: | Fujii Hironori <Hironori.Fujii> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | commit-queue, dbates, don.olmstead, ews-watchlist, lforschler, pvollan, ross.kirsling, stephan.szabo, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Bug Depends on: | |||||||||||||||
Bug Blocks: | 196622 | ||||||||||||||
Attachments: |
|
Description
Fujii Hironori
2019-04-09 20:33:18 PDT
Created attachment 367100 [details]
Patch
AppleWin EWS failed.
> -- Building for: Visual Studio 15 2017
> -- The C compiler identification is MSVC 19.11.25547.0
> -- The CXX compiler identification is MSVC 19.11.25547.0
> -- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx64/x86/cl.exe
> -- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/Hostx64/x86/cl.exe -- works
> (...)
> Died at /home/buildbot/WebKit/Tools/Scripts/webkitdirs.pm line 1825.
> Using MSBuild: Visual Studio Locator version 2.1.3 [query version 1.12.134.36697]
> *************************************************************
> Cannot find 'Visual Studio Locator version 2.1.3 [query version 1.12.134.36697]
> '
> Please make sure execute that the Microsoft .NET Framework SDK
> is installed on this machine.
> *************************************************************
It seems using Visual Studio 2017 version 15.3. Umm, why?
Created attachment 367104 [details]
Patch
* Added -no-logo
Comment on attachment 367104 [details] Patch Oh, no. AppleWin EWS failed again. > Using MSBuild: Error 0x57: Unknown parameter: find > ************************************************************* > Cannot find 'Error 0x57: Unknown parameter: find > ' > Please make sure execute that the Microsoft .NET Framework SDK > is installed on this machine. > ************************************************************* Created attachment 367109 [details]
Patch
Created attachment 367112 [details]
Patch
* Remove CR LF of the output of vswhere
Comment on attachment 367112 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367112&action=review > Tools/ChangeLog:9 > + vswhere.exe has been bundled since Visual Studio 2017 version > + 15.2. We shouldn't download vswhere.exe. I'm not sure that we shouldn't have a download as a fallback. Actually, if we're doing this, maybe it'd be easier to switch to just finding the msbuild location at the same time since we're not using any of the other information from vswhere. (In reply to Don Olmstead from comment #7) > I'm not sure that we shouldn't have a download as a fallback. We shouldn’t keep update-vswhere without knowing the reason. I’ll restore it if we’ll have the reason we need to download. Created attachment 367167 [details]
Have vswhere find msbuild itself
Per earlier comment and separate comments on the 2019 patch, maybe something like this to just have vswhere do the lifting for finding msbuild in the non-VSINSTALLDIR case.
I missed that was what was tried in one of the patches that was obsolete. Since it was asked elsewhere, from my standpoint the steps after this would be to get some version of the 2019 patch done so that we can find the msbuild and to let cmake pick the generator for wincairo. And then try to work out how to handle playstation port's version requirements (whether that is to force vcvars + update the script to set the platform, force vcvars and users to update PLATFORM or to find 2017). Both of which can be followups. Comment on attachment 367112 [details] Patch Clearing flags on attachment: 367112 Committed r244171: <https://trac.webkit.org/changeset/244171> All reviewed patches have been landed. Closing bug. |