Skip to content

[5.3][Foundation] Adjust Foundation shims for compatibility #31382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 29, 2020

Conversation

lorentey
Copy link
Member

@lorentey lorentey commented Apr 28, 2020

Cherry-picked from #31380.

  • Adjust stray public import of _SwiftFoundationOverlayShims
  • Re-add Foundation shims that were removed in [Foundation] Import shims with @implementationOnly #28918. The current version of the Foundation overlay doesn’t use these, but we should still keep them in case a toolchain snapshot build picks up an overlay module from one of the SDKs in Xcode 11.

rdar://62339802

The current version of the Foundation overlay doesn’t use these, but we should still keep them in case a toolchain snapshot build picks up on overlay module from one of the SDKs in Xcode 11.

rdar://62339802
(cherry picked from commit 057b27b)
@lorentey
Copy link
Member Author

@swift-ci test

@lorentey lorentey requested review from kylemacomber and edymtt April 28, 2020 22:59
@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9a842f0

@lorentey
Copy link
Member Author

Hm, I'm pretty sure this is unrelated.

19:00:06 FAILED: bin/LLDB.framework/Versions/A/LLDB 
...
clang: error: unable to execute command: Bus error: 10

@lorentey
Copy link
Member Author

@swift-ci test macOS platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9a842f0

@lorentey
Copy link
Member Author

lorentey commented Apr 29, 2020

¯\(°_o)/¯

20:13:20 ******************** TEST 'lldb-shell :: SwiftREPL/Basic.test' FAILED ********************
20:13:20 Script:
20:13:20 --
20:13:20 : 'RUN: at line 3';   /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/bin/lldb --no-lldbinit -S /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/test/Shell/lit-lldb-init --repl --repl-language swift | /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/llvm-macosx-x86_64/bin/FileCheck /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/llvm-project/lldb/test/Shell/SwiftREPL/Basic.test --check-prefix=SWIFT
20:13:20 : 'RUN: at line 6';   /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/bin/lldb --no-lldbinit -S /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/test/Shell/lit-lldb-init --repl --repl-language c++ 2>&1 | /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/llvm-macosx-x86_64/bin/FileCheck /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/llvm-project/lldb/test/Shell/SwiftREPL/Basic.test --check-prefix=CPP
20:13:20 : 'RUN: at line 9';   not /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/bin/lldb --no-lldbinit -S /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/test/Shell/lit-lldb-init --repl --repl-language patatino 2>&1 | /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/llvm-macosx-x86_64/bin/FileCheck /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/llvm-project/lldb/test/Shell/SwiftREPL/Basic.test   --check-prefix=INVALID
20:13:20 : 'RUN: at line 13';   echo '2 + 3' | /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/bin/lldb --no-lldbinit -S /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/test/Shell/lit-lldb-init --repl | /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/llvm-macosx-x86_64/bin/FileCheck /Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/llvm-project/lldb/test/Shell/SwiftREPL/Basic.test --check-prefix=INT
20:13:20 --
20:13:20 Exit Code: 1
20:13:20 
20:13:20 Command Output (stderr):
20:13:20 --
20:13:20 0  lldb                     0x0000000108f17a08 llvm::sys::PrintStackTrace(llvm::raw_ostream&) + 40
20:13:20 1  lldb                     0x0000000108f16cc8 llvm::sys::RunSignalHandlers() + 248
20:13:20 2  lldb                     0x0000000108f182bd SignalHandler(int) + 285
20:13:20 3  libsystem_platform.dylib 0x00007fff735705fd _sigtramp + 29
20:13:20 4  libsystem_platform.dylib 000000000000000000 _sigtramp + 18446603338581080608
20:13:20 5  LLDB                     0x000000010b9dc1b2 lldb_private::SwiftASTContext::CreateInstance(lldb::LanguageType, lldb_private::Module&, lldb_private::Target*, bool) + 7538
20:13:20 6  LLDB                     0x000000010b9e257b CreateTypeSystemInstance(lldb::LanguageType, lldb_private::Module*, lldb_private::Target*, char const*) + 43
20:13:20 7  LLDB                     0x000000010ba1e682 lldb_private::TypeSystemMap::GetTypeSystemForLanguage(lldb::LanguageType, lldb_private::Module*, bool) + 498
20:13:20 8  LLDB                     0x000000010b8d97fe lldb_private::Module::GetTypeSystemForLanguage(lldb::LanguageType) + 30
20:13:20 9  LLDB                     0x000000010ba0013b std::__1::__function::__func<lldb_private::SwiftASTContext::CreateInstance(lldb::LanguageType, lldb_private::Target&, char const*)::$_1, std::__1::allocator<lldb_private::SwiftASTContext::CreateInstance(lldb::LanguageType, lldb_private::Target&, char const*)::$_1>, void ()>::operator()() + 27
20:13:20 10 LLDB                     0x000000010bf2476f void* std::__1::__thread_proxy<std::__1::tuple<std::__1::unique_ptr<std::__1::__thread_struct, std::__1::default_delete<std::__1::__thread_struct> >, llvm::ThreadPool::ThreadPool(unsigned int)::$_0> >(void*) + 735
20:13:20 11 libsystem_pthread.dylib  0x00007fff7357c109 _pthread_start + 148
20:13:20 12 libsystem_pthread.dylib  0x00007fff73577b8b thread_start + 15
20:13:20 
/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/llvm-project/lldb/test/Shell/SwiftREPL/Basic.test:14:9: error: INT: expected string not found in input
20:13:20 // INT: Welcome to {{.*}}Swift
20:13:20         ^
20:13:20 <stdin>:1:1: note: scanning from here
20:13:20 (lldb) command source -s 0 '/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/lldb-macosx-x86_64/test/Shell/lit-lldb-init'
20:13:20 ^
20:13:20 <stdin>:4:104: note: possible intended match here
20:13:20 (lldb) env DYLD_LIBRARY_PATH='/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/swift-macosx-x86_64/lib/swift/macosx' LD_LIBRARY_PATH='/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/swift-macosx-x86_64/lib/swift/x86_64' SIMCTL_CHILD_DYLD_LIBRARY_PATH='/Users/buildnode/jenkins/workspace/swift-PR-osx/branch-release/5.3/buildbot_incremental/swift-macosx-x86_64/lib/swift/macosx'
20:13:20                                                                                                        ^
20:13:20 
20:13:20 --
20:13:20 
20:13:20 ********************

@lorentey
Copy link
Member Author

@swift-ci test macOS platform

@lorentey lorentey merged commit eca272f into swiftlang:release/5.3 Apr 29, 2020
@lorentey lorentey deleted the shims-shmims-5.3 branch April 29, 2020 17:53
@AnthonyLatsis AnthonyLatsis added swift 5.3 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants