Summary: | PATH needs to include cygwin for WebKitVersion | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alex Christensen <alex.christensen> | ||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Trivial | CC: | bfulgham, commit-queue | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Windows 8 | ||||||
Attachments: |
|
Description
Alex Christensen
2013-12-09 14:31:32 PST
Created attachment 218801 [details]
Patch
Comment on attachment 218801 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218801&action=review r=me > Source/WebKit/WebKit.vcxproj/WebKit/WebKitVersion.cmd:5 > +set PATH=%SystemDrive%\cygwin\bin;%PATH% This assumes that Cygwin is always installed in "C:\cygwin", which may not be the case on all user's systems. Still, it's a reasonable assumption for most people. We might want to check "HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Cygwin\setup\rootdir" to get the true location. Comment on attachment 218801 [details] Patch Clearing flags on attachment: 218801 Committed r160465: <http://trac.webkit.org/changeset/160465> All reviewed patches have been landed. Closing bug. |