RESOLVED FIXED 217068
[tvOS] [watchOS] Add support in libTestWTFAlwaysMissing-iOS-v2.tbd
https://bugs.webkit.org/show_bug.cgi?id=217068
Summary [tvOS] [watchOS] Add support in libTestWTFAlwaysMissing-iOS-v2.tbd
Ryan Haddad
Reported 2020-09-28 17:15:02 PDT
Since the tvOS simulator bot was updated to the tvOS 14 SDK, the build has been failing with the following: ld: warning: directory not found for option '-F/Applications/Xcode.app/Contents/Developer/Platforms/AppleTVSimulator.platform/Developer/SDKs/AppleTVSimulator14.0.sdk/System/Library/Frameworks/Quartz.framework/Frameworks' ld: warning: building for tvOS Simulator, but linking in .tbd built for iOS Simulator, 'TestWTFAlwaysMissing-iOS-v2' ld: warning: Could not find or use auto-linked library 'TestWTFAlwaysMissing-iOS-v2' Undefined symbols for architecture x86_64: "_TestWTFAlwaysMissing", referenced from: TestWebKitAPI::WeakLinking_WeakImport_Test::TestBody() in libTestWTF.a(WeakLinking.o) "_TestWTFAlwaysMissingWithoutAttributeWeakImport", referenced from: TestWebKitAPI::WeakLinking_WeakImport_Test::TestBody() in libTestWTF.a(WeakLinking.o) ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation) https://build.webkit.org/builders/Apple-tvOS-Simulator-14-Release-Build/builds/1/steps/compile-webkit/logs/stdio The tvOS device build seems OK.
Attachments
Patch (1.51 KB, patch)
2020-09-29 09:42 PDT, Jonathan Bedard
no flags
Patch (1.51 KB, patch)
2020-09-29 12:31 PDT, Jonathan Bedard
no flags
Radar WebKit Bug Importer
Comment 1 2020-09-28 17:15:43 PDT
Jonathan Bedard
Comment 2 2020-09-28 17:21:05 PDT
My bet is that we're somehow missing Tools/TestWebKitAPI/Tests/WTF/darwin/libTestWTFAlwaysMissing-iOS.tbd in the tvOS Simulator build.
Jonathan Bedard
Comment 3 2020-09-29 09:42:45 PDT
Jonathan Bedard
Comment 4 2020-09-29 09:45:16 PDT
Looks like the rules for platform specifiers in tbd has gotten a bit more strict, I was able to reproduce this locally and this change fixes the problem. I'm pretty much certain that watchOS needs it as well.
Aakash Jain
Comment 5 2020-09-29 09:49:13 PDT
rs=me if EWS passes.
Jonathan Bedard
Comment 6 2020-09-29 12:31:00 PDT
EWS
Comment 7 2020-09-29 13:13:32 PDT
Committed r267757: <https://trac.webkit.org/changeset/267757> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410031 [details].
Note You need to log in before you can comment on or make changes to this bug.