| Summary: | devenv.com not available in VC++ Express installations | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Björn Graf (boki) <bjoern.graf> | ||||||||||||||||
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||||||||||
| Status: | RESOLVED FIXED | ||||||||||||||||||
| Severity: | Normal | CC: | bugs-webkit, cryo, ddkilzer, gavin.sharp, hyatt, ian | ||||||||||||||||
| Priority: | P2 | ||||||||||||||||||
| Version: | 420+ | ||||||||||||||||||
| Hardware: | PC | ||||||||||||||||||
| OS: | Windows XP | ||||||||||||||||||
| Attachments: |
|
||||||||||||||||||
|
Description
Björn Graf (boki)
2006-03-16 03:27:06 PST
Created attachment 7111 [details]
Utilize vcbuild.exe in webkit.pm
Created attachment 7124 [details]
Utilize vcbuild.exe in webkit.pm with VCINSTALLDIR check
Comment on attachment 7124 [details]
Utilize vcbuild.exe in webkit.pm with VCINSTALLDIR check
If this works, we should definitely do it! But I don't have a Windows machine to test on.
Comment on attachment 7124 [details]
Utilize vcbuild.exe in webkit.pm with VCINSTALLDIR check
r=me but whoever lands this should test on windows to make sure it works ok with full VC
*** Bug 7871 has been marked as a duplicate of this bug. *** Committed revision 15015. Watching buildbot for Windows build errors. http://build.webkit.org/ This patch did not work on the buildbot. http://build.webkit.org/post-commit-win32/builds/1746/step-compile-release/0 Looks like vcvars32.bat needs to be run on the buildbot. Hyatt, can you do this, or is this something that needs to be done every time the user logs in? Run "<Program Files>\Microsoft Visual Studio 8\VC\bin\vcvars32.bat" to set VC variables at /cygdrive/c/WebKit-BuildSlave/post-commit-win32/build/WebKitTools/Scripts/webkitdirs.pm line 350. program finished with exit code 2 I am backing out the patch for now. Created attachment 9007 [details]
Updated patch with ChangeLog
If the vcvars32.bat script gets run on the Win32 buildbot, this patch may work again.
(In reply to comment #8) > Created an attachment (id=9007) [edit] > Updated patch with ChangeLog I did not change anything in the patch itself, but the original patch applied with fuzz 1. Created attachment 9008 [details]
Patch with no reviewer set
Same as previous patch, but no reviewer set. *sigh*
(In reply to comment #10) > Same as previous patch, but no reviewer set. *sigh* The reason this had Maciej as a reviewer is that he long since said "r=me" (see comment #4 above). But we need someone to test that this works on a system with Visual Studio before we land it. Sorry it's been sitting around so long. (In reply to comment #11) > (In reply to comment #10) > > Same as previous patch, but no reviewer set. *sigh* > > The reason this had Maciej as a reviewer is that he long since said "r=me" (see > comment #4 above). Actually, I was sighing at myself because I forgot to clear Maciej from previous patch as the reviewer before uploading it to this bug. :) As soon as this lands, I'll install VC++ Express on my Windows laptop so I can play with WebKit there, too. This forces people with full visual studio to set VCINSTALLDIR, which hasn't been a requirement before. Probably it could just default to using "$programFilesPath/Microsoft Visual Studio 8/VC/" instead of VCINSTALLDIR if its not set, but r- for the technicality. Created attachment 9189 [details]
Updated patch to default to $ProgramFiles if VCINSTALLDIR is not set
Committed revision 15252. Hopefully, the buildbot won't break this time. :) (In reply to comment #15) > Committed revision 15252. > > Hopefully, the buildbot won't break this time. :) No such luck. Attempted one build-fix in r15254. (In reply to comment #16) > (In reply to comment #15) > > Committed revision 15252. > > Hopefully, the buildbot won't break this time. :) > No such luck. Attempted one build-fix in r15254. Okay, that worked. If VC++ Express still isn't working, please open a new bug and file a new patch. Thanks! :) Created attachment 9298 [details]
Final build fix patch
Reopening once again. This broke the Windows buildbot yet again. Will mark the patches r-. Created attachment 11333 [details] another attempt Updates the earlier patches to use VCExpress.exe, which takes the same parameters as devenv.com. Unfortunately, VCExpress.exe doesn't print anything to the console, saving its results to per-project HTML files. Still, this is hopefully slightly better than the current situation (and I suppose that everyone builds from VC Express IDE anyway). Cmake apparently has a workaround, but I couldn't make it work: <http://www.cmake.org/pipermail/cmake/2006-February/008292.html>. Comment on attachment 11333 [details]
another attempt
r=me
Committed revision 17551. |