Bug 155838

Summary: REGRESSION (r197358): WebKitSystemInterface.h copied into directory named "--llvm"
Product: WebKit Reporter: Hofi <hofione>
Component: Tools / TestsAssignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Blocker CC: dbates, dean_johnson, hofione, lforschler, ossy
Priority: P2 Keywords: Regression
Version: WebKit Local Build   
Hardware: Mac   
OS: OS X 10.11   
Bug Depends on: 154821    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Hofi 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
Comment 1 Alexey Proskuryakov 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?
Comment 2 Dean Johnson 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.
Comment 3 Daniel Bates 2016-03-25 10:11:35 PDT
Created attachment 274914 [details]
Patch
Comment 4 Daniel Bates 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>
Comment 5 Daniel Bates 2016-03-25 11:42:54 PDT
All reviewed patches have been landed.  Closing bug.