155078 2016-03-05 16:58:35 -0800 build-webkit prints an error on iOS simulator bots 2016-03-06 00:33:25 -0800 1 1 1 Unclassified WebKit Tools / Tests WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 ap ap commit-queue dbates ddkilzer dean_johnson lforschler mitz ossy oldest_to_newest 1171316 0 ap 2016-03-05 16:58:35 -0800 https://build.webkit.org/builders/Apple%20iOS%209%20Simulator%20Debug%20%28Build%29/builds/0/steps/compile-webkit/logs/stdio perl Tools/Scripts/copy-webkitlibraries-to-product-directory --sdk iphonesimulator --wksi /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator Updating /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator/libWebKitSystemInterfaceIOSDevice9.0.a Updating /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator/libWebKitSystemInterfaceIOSSimulator9.0.a Updating /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator/libWebKitSystemInterfaceIOSDevice9.2.a Updating /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator/libWebKitSystemInterfaceIOSSimulator9.2.a Updating /Volumes/Data/slave/ios-simulator-9-debug/build/WebKitBuild/Debug-iphonesimulator/usr/local/include/WebKitSystemInterface.h Use of uninitialized value $extraOptions[8] in system at /Volumes/Data/slave/ios-simulator-9-debug/build/Tools/Scripts/webkitdirs.pm line 1754. 1171330 1 ap 2016-03-05 17:49:32 -0800 xcodeSDK() returns an uninitialized value. 1171339 2 ap 2016-03-05 17:57:49 -0800 main::setXcodeSDK(undef) called at Tools/Scripts/build-layouttestrelay line 63 1171353 3 ap 2016-03-05 18:58:08 -0800 This is easy to fix by checking xcodeSDK() for undefined in buildXCodeProject(): push(@extraOptions, ("-sdk", xcodeSDK())) if isIOSWebKit(); But I'm trying to understand why -sdk is added in buildXCodeProject() in the first place, not in XcodeOptions(). 1171365 4 273120 ap 2016-03-05 20:07:42 -0800 Created attachment 273120 naive fix I don't quite understand the relationship between -sdk and SDKROOT, but looks like we don't need -sdk. 1171390 5 273120 commit-queue 2016-03-06 00:33:20 -0800 Comment on attachment 273120 naive fix Clearing flags on attachment: 273120 Committed r197636: <http://trac.webkit.org/changeset/197636> 1171391 6 commit-queue 2016-03-06 00:33:25 -0800 All reviewed patches have been landed. Closing bug. 273120 2016-03-05 20:07:42 -0800 2016-03-06 00:33:20 -0800 naive fix LayoutTestRelaySDK.txt text/plain 2348 ap SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDE5NzYyOCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDE4IEBACisyMDE2LTAzLTA1ICBBbGV4ZXkgUHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29t PgorCisgICAgICAgIGJ1aWxkLXdlYmtpdCBwcmludHMgYW4gZXJyb3Igb24gaU9TIHNpbXVsYXRv ciBib3RzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0x NTUwNzgKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBU aGlzIGhhcHBlbnMgd2hlbiBidWlsZC1sYXlvdXR0ZXN0cmVsYXkgc2V0cyAkeGNvZGVTREsgdG8g dW5kZWZpbmVkLCB0byB1c2UgaG9zdCBTREsuCisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdGRp cnMucG06IChidWlsZFhDb2RlUHJvamVjdCk6IFRoZXJlIGlzIG5vIG5lZWQgdG8gcGFzcyAtc2Rr LCBiZWNhdXNlCisgICAgICAgIFhjb2RlT3B0aW9ucygpIGFscmVhZHkgcGFzc2VzIFNES1JPT1Qg YXMgYXBwcm9wcmlhdGUuCisKKyAgICAgICAgKiBTY3JpcHRzL2J1aWxkLWxheW91dHRlc3RyZWxh eTogQWRkZWQgYSBGSVhNRS4gUmVzZXR0aW5nICR4Y29kZVNESyBkb2Vzbid0IHJlc2V0CisgICAg ICAgIGFueSBvdGhlciBjb25maWd1cmF0aW9uIHZhcmlhYmxlcyB0aGF0IHdlcmUgZGVyaXZlZCBm cm9tIGl0cyBpbml0aWFsIHZhbHVlLCBsaWtlICRwb3J0TmFtZS4KKwogMjAxNi0wMy0wNSAgQ2Fy bG9zIEFsYmVydG8gTG9wZXogUGVyZXogIDxjbG9wZXpAaWdhbGlhLmNvbT4KIAogICAgICAgICBb R1RLXSBGaXggbG9nZ2luZyBkZXRhaWxzIGFmdGVyIHIxOTc1MDUgKFJ1biB0aGUgcnVuLWJlbmNo bWFyayBzY3JpcHQgb24gdGhlIHBlcmZvcm1hbmNlIGJvdC4pCkluZGV4OiBUb29scy9TY3JpcHRz L2J1aWxkLWxheW91dHRlc3RyZWxheQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9TY3JpcHRzL2J1aWxk LWxheW91dHRlc3RyZWxheQkocmV2aXNpb24gMTk3NjIyKQorKysgVG9vbHMvU2NyaXB0cy9idWls ZC1sYXlvdXR0ZXN0cmVsYXkJKHdvcmtpbmcgY29weSkKQEAgLTYwLDYgKzYwLDggQEAgY2hkaXIg IlRvb2xzL0xheW91dFRlc3RSZWxheSIgb3IgZGllOwogIyAgICAgICAgdG8gYXZvaWQgYnJlYWtp bmcgc29tZSBBcHBsZSBJbnRlcm5hbCB0b29scyB3ZSBzdXBwb3J0IGNhbGxpbmcgdGhpcyBzY3Jp cHQKICMgICAgICAgIHdpdGhvdXQgLS1pb3Mtc2ltdWxhdG9yIHNvIGxvbmcgYXMgd2UgYXJlIGJ1 aWxkaW5nIG9uIGEgTWFjLgogaWYgKGlzQXBwbGVNYWNXZWJLaXQoKSAmJiAhd2lsbFVzZUlPU0Rl dmljZVNESygpKSB7CisgICAgIyBGSVhNRTogQ2hhbmdpbmcgU0RLIGF0IHRoaXMgcG9pbnQgcmVz dWx0cyBpbiBhbiBpbmNvbnNpc3RlbnQgd2Via2l0ZGlycyBzdGF0ZS4KKyAgICAjIFdlIHNob3Vs ZCBtYWtlIGl0IHNvIHRoYXQgd2Via2l0ZGlycyBmdW5jdGlvbnMgbmV2ZXIgc2VlIHRoZSB3cm9u ZyBTREsuCiAgICAgc2V0WGNvZGVTREsodW5kZWYpOyAjIEZvcmNlIHVzZSBvZiBNYWMgU0RLIGlm IC0taW9zLXNpbXVsYXRvciB3YXMgcGFzc2VkCiAgICAgZXhpdCBleGl0U3RhdHVzKGJ1aWxkWENv ZGVQcm9qZWN0KCJMYXlvdXRUZXN0UmVsYXkiLCAkY2xlYW4sIFhjb2RlT3B0aW9ucygpLCBAQVJH VikpOwogfQpJbmRleDogVG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBtCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IFRvb2xzL1NjcmlwdHMvd2Via2l0ZGlycy5wbQkocmV2aXNpb24gMTk3NjIyKQorKysgVG9vbHMv U2NyaXB0cy93ZWJraXRkaXJzLnBtCSh3b3JraW5nIGNvcHkpCkBAIC0xNzQ4LDggKzE3NDgsNiBA QCBzdWIgYnVpbGRYQ29kZVByb2plY3QoJCRAKQogICAgICAgICBwdXNoKEBleHRyYU9wdGlvbnMs ICJjbGVhbiIpOwogICAgIH0KIAotICAgIHB1c2goQGV4dHJhT3B0aW9ucywgKCItc2RrIiwgeGNv ZGVTREsoKSkpIGlmIGlzSU9TV2ViS2l0KCk7Ci0KICAgICBjaG9tcCgkRU5We0RTWU1VVElMX05V TV9USFJFQURTfSA9IGBzeXNjdGwgLW4gaHcuYWN0aXZlY3B1YCk7CiAgICAgcmV0dXJuIHN5c3Rl bSAieGNvZGVidWlsZCIsICItcHJvamVjdCIsICIkcHJvamVjdC54Y29kZXByb2oiLCBAZXh0cmFP cHRpb25zOwogfQo=