Bug 155838 - REGRESSION (r197358): WebKitSystemInterface.h copied into directory named "--llvm"
Summary: REGRESSION (r197358): WebKitSystemInterface.h copied into directory named "--...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Local Build
Hardware: Mac OS X 10.11
: P2 Blocker
Assignee: Daniel Bates
Keywords: Regression
Depends on: 154821
  Show dependency treegraph
Reported: 2016-03-24 09:29 PDT by Hofi
Modified: 2016-03-25 11:42 PDT (History)
5 users (show)

See Also:

Patch (1.88 KB, patch)
2016-03-25 10:11 PDT, Daniel Bates
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hofi 2016-03-24 09:29:56 PDT
the problem similar to this


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]
Comment 4 Daniel Bates 2016-03-25 11:42:51 PDT
Comment on attachment 274914 [details]

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.