[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webkit-unassigned
Subject:    [Webkit-unassigned] [Bug 126317] webkitdirs.pm does not use environment variable VSINSTALLDIR
From:       bugzilla-daemon () webkit ! org
Date:       2013-12-31 20:49:19
Message-ID: 20131231204714.3CF1B159509 () lists ! macosforge ! org
[Download RAW message or body]

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





--- Comment #3 from David Delaune <david.delaune@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.
_______________________________________________
webkit-unassigned mailing list
webkit-unassigned@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-unassigned


[prev in list] [next in list] [prev in thread] [next in thread]