Set VSINSTALLDIR to where we install VS2013. Alternatively we can run VCVARSALL.bat in the script but the change seems to be equivalent and that'd mean we'd be running a batch script from a shell script which is being run from a batch script... maybe that's not that weird...
Created attachment 220256 [details] patch
Created attachment 220257 [details] vcvarsall version
Comment on attachment 220257 [details] vcvarsall version View in context: https://bugs.webkit.org/attachment.cgi?id=220257&action=review > Tools/EWSTools/start-queue-win.sh:25 > + cmd /c "$PROGRAMFILES\Microsoft Visual Studio 12.0\VC\vcvarsall.bat" This should be "CALL "$PROGRAMFILES...." We want the queue to run with the contents of the batch file added to the shell environment. The 'cmd' version will spawn a new shell, which will cease to exist once the command finishes running.
Comment on attachment 220256 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=220256&action=review > Tools/EWSTools/start-queue-win.sh:25 > + export VSINSTALLDIR="$PROGRAMFILES\Microsoft Visual Studio 12.0" I don't care for this version as much, because it doesn't set the other environment variables (such as LIBPATH, etc.) which might be useful for building.
Created attachment 220265 [details] vcvarsall version fixed
uh hold up, call command not found
Duh! Sorry -- I was thinking in DOS BATCH, not BASH. We want to introduce these variables to the Cygwin environment, so maybe we do need to go with your original version that just adds the environment variable.
(In reply to comment #7) > Duh! Sorry -- I was thinking in DOS BATCH, not BASH. > > We want to introduce these variables to the Cygwin environment, so maybe we do need to go with your original version that just adds the environment variable. Hmm, after some amount of research I don't think cygwin has anything that can save the environment settings set by a batch script. I could always convert the batch script to a bash script, check it in, and we can modify it as need be instead.
What do you think Brent? Think we should just go with setting VSINSTALLDIR ourselves?
Comment on attachment 220256 [details] patch Watch out for LIBPATH or INCLUDE_PATH problems in scripts since we aren't setting these, but this seems to work.
(In reply to comment #10) > (From update of attachment 220256 [details]) > Watch out for LIBPATH or INCLUDE_PATH problems in scripts since we aren't setting these, but this seems to work. Ok, I'll keep an eye out.
http://trac.webkit.org/changeset/161355 Keeping open until EWS bots are actually reporting working builds