[Webkit-unassigned] [Bug 126317] webkitdirs.pm does not use environment variable VSINSTALLDIR

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=126317





--- Comment #3 from David Delaune <david.delaune at hotmail.com>  2013-12-31 12:47:11 PST ---
(In reply to comment #1) 
> This fix was needed to support our build farm. I'm surprised it caused any issues for you; do you not have VS2013 installed?

So it is exactly as I suspected. This quick hack was added to fix a single-user scenario while potentially breaking compilations for the rest of the world.

Someone should invest 5 minutes looking into this. It only takes a few minutes to peruse vcvarsall.bat where you will find that depending on the architecture it will call \VC\bin\vcvars[*].bat which calls "%VS120COMNTOOLS%VCVarsQueryRegistry.bat" to set %VSINSTALLDIR%.

or perhaps if you only have 10 seconds to invest you could open a Visual Studio command prompt and type 'echo %VSINSTALLDIR%'

If you really want to check the version then you should consider taking advantage of the %VISUALSTUDIOVERSION% environment variable.

Best Wishes,
-David Delaune

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.