RESOLVED FIXED 155838
REGRESSION (r197358): WebKitSystemInterface.h copied into directory named "--llvm" 
https://bugs.webkit.org/show_bug.cgi?id=155838
Summary REGRESSION (r197358): WebKitSystemInterface.h copied into directory named "--...
Hofi
Reported 2016-03-24 09:29:56 PDT
the problem similar to this http://stackoverflow.com/questions/22019412/build-webkit-error-on-mac-os-x-10-9-webkitsysteminterface-h-file-not-found building WebKitLegacy target for iOS simulator first got a compile time error <webkit_root>/Source/WebKit/ios/WebCoreSupport/WebChromeClientIOS.mm:38:9: fatal error: 'WebKitSystemInterface.h' file not found than after that resolved got a linker error during link time of WebKitLegacy ld: library not found for -lWebKitSystemInterfaceIOSSimulator9.2 My case could be resolved by 2 steps 1. copied "<webkit_root>/WebKitLibraries/--llvm/usr" over to "<webkit_root>/WebKitBuild/<Configuration>/" 2. copied "<webkit_root>/WebKitLibraries/libWebKit"* over to "<webkit_root>/WebKitBuild/<Configuration>/" the build made from the 'snapshot of the WebKit source tree' http://nightly.webkit.org/files/WebKit-SVN-source.tar.bz2 was available today (marc. 24 2016) using Xcode 7.2.1 final on osx 10.11.4 final I've been building from Xcode using the 'All Source' scheme for 9.2 iOS simulator
Attachments
Patch (1.88 KB, patch)
2016-03-25 10:11 PDT, Daniel Bates 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2016-03-24 18:09:54 PDT
> 1. copied "<webkit_root>/WebKitLibraries/--llvm/usr" over to "<webkit_root>/WebKitBuild/<Configuration>/" Sounds similar to bug 154880, must be another problem with command line parsing in the script. I'm not sure how/if WKSI archive is downloaded when building from Xcode, perhaps you ran some terminal commands prior to that?
Dean Johnson
Comment 2 2016-03-24 18:11:43 PDT
This sounds like the --llvm argument isn't supported as an option anymore, and it's still being passed. --llvm is then interpreted as the output directory, instead of as an option.
Daniel Bates
Comment 3 2016-03-25 10:11:35 PDT
Created attachment 274914 [details] Patch
Daniel Bates
Comment 4 2016-03-25 11:42:51 PDT
Comment on attachment 274914 [details] Patch Clearing flags on attachment: 274914 Committed r198681: <http://trac.webkit.org/changeset/198681>
Daniel Bates
Comment 5 2016-03-25 11:42:54 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.