149818 2015-10-05 14:58:03 -0700 Disable Bitcode when building for iOS device 2015-10-05 15:36:25 -0700 1 1 1 Unclassified WebKit Tools / Tests WebKit Local Build iPhone / iPad iOS 9.0 RESOLVED FIXED P2 Normal --- 140828 1 dbates dbates ap ddkilzer lforschler oldest_to_newest 1130747 0 dbates 2015-10-05 14:58:03 -0700 Do not generate bitcode when building for iOS device. Xcode 7 generates bitcode for iOS device apps by default. 1130748 1 dbates 2015-10-05 14:59:36 -0700 With the patch for bug #140828, running build-webkit --debug --device eventually fails with the following message: ld: -unexported_symbol and -bitcode_bundle (Xcode setting ENABLE_BITCODE=YES) cannot be used together 1130750 2 262465 dbates 2015-10-05 15:00:26 -0700 Created attachment 262465 Patch 1130756 3 262465 ap 2015-10-05 15:14:10 -0700 Comment on attachment 262465 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=262465&action=review > Tools/ChangeLog:9 > + Xcode 7 generates bitcode for iOS device apps by default. For now, do not generate bitcode > + when building for iOS device. Why is it "for now"? Not sure why we'd need that for a system framework. 1130759 4 dbates 2015-10-05 15:26:59 -0700 (In reply to comment #3) > Comment on attachment 262465 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=262465&action=review > > > Tools/ChangeLog:9 > > + Xcode 7 generates bitcode for iOS device apps by default. For now, do not generate bitcode > > + when building for iOS device. > > Why is it "for now"? Not sure why we'd need that for a system framework. We do not need it. Will remove the phrase "For now". 1130763 5 dbates 2015-10-05 15:36:25 -0700 Committed r190584: <http://trac.webkit.org/changeset/190584> 262465 2015-10-05 15:00:26 -0700 2015-10-05 15:14:10 -0700 Patch bug-149818-20151005145956.patch text/plain 1581 dbates U3VidmVyc2lvbiBSZXZpc2lvbjogMTkwNTc3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggODJkOGFhNmViMjdkZjdiYTU5NzI0NThjNjU4YjAxY2Q3 ODBiZjc3Ny4uYjYxNjg4NTdiMWY1MWE2MmQxYzk0NDcwYjQ5ZjQyOGIzYWRiZjA5YSAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE4 IEBACiAyMDE1LTEwLTA1ICBEYW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KIAorICAg ICAgICBEaXNhYmxlIEJpdGNvZGUgd2hlbiBidWlsZGluZyBmb3IgaU9TIGRldmljZQorICAgICAg ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQ5ODE4CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgWGNvZGUgNyBnZW5lcmF0 ZXMgYml0Y29kZSBmb3IgaU9TIGRldmljZSBhcHBzIGJ5IGRlZmF1bHQuIEZvciBub3csIGRvIG5v dCBnZW5lcmF0ZSBiaXRjb2RlCisgICAgICAgIHdoZW4gYnVpbGRpbmcgZm9yIGlPUyBkZXZpY2Uu CisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdGRpcnMucG06CisgICAgICAgIChYY29kZU9wdGlv bnMpOgorCisyMDE1LTEwLTA1ICBEYW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KKwog ICAgICAgICBbaU9TXSBNYWtlIGl0IHBvc3NpYmxlIHRvIGJ1aWxkIFdlYktpdCB1c2luZyBpcGhv bmVvcyBTREsgd2l0aG91dCBhIGRldmVsb3BlciBjZXJ0aWZpY2F0ZSBpbnN0YWxsZWQKICAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0MDgyOAogICAgICAg ICA8cmRhcjovL3Byb2JsZW0vMTk1MjA1OTk+CmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3dl YmtpdGRpcnMucG0gYi9Ub29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0KaW5kZXggODk0NDFjMTZh NWU0ZmZiYjA4ZGFkMzU1MTVhYzg0MWZkZGI3MmY0Zi4uNjJkMGQ5MzllMTA2ZjYwNDM1YWIyN2Rh Mzk0YWU5MDNmYjc0YzllMyAxMDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBt CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0ZGlycy5wbQpAQCAtNjg1LDYgKzY4NSw3IEBAIHN1 YiBYY29kZU9wdGlvbnMKICAgICBwdXNoIEBvcHRpb25zLCAiQVJDSFM9JGFyY2hpdGVjdHVyZSIg aWYgJGFyY2hpdGVjdHVyZTsKICAgICBwdXNoIEBvcHRpb25zLCAiU0RLUk9PVD0keGNvZGVTREsi IGlmICR4Y29kZVNESzsKICAgICBpZiAod2lsbFVzZUlPU0RldmljZVNES1doZW5CdWlsZGluZygp KSB7CisgICAgICAgIHB1c2ggQG9wdGlvbnMsICJFTkFCTEVfQklUQ09ERT1OTyI7CiAgICAgICAg IGlmIChoYXNJT1NEZXZlbG9wbWVudENlcnRpZmljYXRlKCkpIHsKICAgICAgICAgICAgICMgRklY TUU6IE1heSBtYXRjaCBtb3JlIHRoYW4gb25lIGluc3RhbGxlZCBkZXZlbG9wbWVudCBjZXJ0aWZp Y2F0ZS4KICAgICAgICAgICAgIHB1c2ggQG9wdGlvbnMsICJDT0RFX1NJR05fSURFTlRJVFk9IiAu IElPU19ERVZFTE9QTUVOVF9DRVJUSUZJQ0FURV9OQU1FX1BSRUZJWDsK