Bug 155078 – build-webkit prints an error on iOS simulator bots

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Alexey Proskuryakov]

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.

[Alexey Proskuryakov]

xcodeSDK() returns an uninitialized value.

[Alexey Proskuryakov]

main::setXcodeSDK(undef) called at Tools/Scripts/build-layouttestrelay line 63

[Alexey Proskuryakov]

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().

[Alexey Proskuryakov]

Created attachment 273120 [details]
naive fix

I don't quite understand the relationship between -sdk and SDKROOT, but looks like we don't need -sdk.

[WebKit Commit Bot]

Comment on attachment 273120 [details]
naive fix

Clearing flags on attachment: 273120

Committed r197636: <http://trac.webkit.org/changeset/197636>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.