Skip to content

Extend the availability annotations for the Netrc support to cover all the Darwin platforms #218

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 1 commit into from
Jun 9, 2021

Conversation

abertelrud
Copy link
Contributor

@abertelrud abertelrud commented Jun 6, 2021

This PR extends the availability annotation for the Netrc support to cover all the version numbers for the various Darwin platforms in the annotation for NSTextCheckingResult.range(withName name: String) -> NSRange in the SDK. Otherwise, ToolsSupportCore can no longer build on non-macOS Darwin platforms such as iOS, because the default minimum deployment targets for the various platforms are really old (e.g. iOS 9).

…l the Darwin platforms

This extends the availability based on the version numbers in the annotation for `NSTextCheckingResult.range(withName name: String) -> NSRange`.  Otherwise, ToolsSupportCore can no longer build on for example iOS, because the default minimum deployment targets for the various platforms are really old (e.g. iOS 9).
@abertelrud
Copy link
Contributor Author

@swift-ci please smoke test

@abertelrud abertelrud self-assigned this Jun 6, 2021
@abertelrud
Copy link
Contributor Author

@swift-ci please test

@abertelrud
Copy link
Contributor Author

The Linux failure seem odd:

/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/latest_toolchain/swift-5.3-RELEASE-ubuntu16.04/usr/lib/swift/dispatch/module.modulemap:1:8: error: redefinition of module 'Dispatch'
module Dispatch {
       ^
/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/swift-corelibs-libdispatch/dispatch/module.modulemap:1:8: note: previously defined here
module Dispatch {
       ^
/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/latest_toolchain/swift-5.3-RELEASE-ubuntu16.04/usr/lib/swift/dispatch/module.modulemap:8:8: error: redefinition of module 'DispatchIntrospection'
module DispatchIntrospection [system] [extern_c] {
       ^
/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/swift-corelibs-libdispatch/dispatch/module.modulemap:8:8: note: previously defined here
module DispatchIntrospection [system] [extern_c] {
       ^
/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/latest_toolchain/swift-5.3-RELEASE-ubuntu16.04/usr/lib/swift/dispatch/module.modulemap:13:8: error: redefinition of module 'CDispatch'
module CDispatch [system] [extern_c] {
       ^
/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/swift-corelibs-libdispatch/dispatch/module.modulemap:13:8: note: previously defined here
module CDispatch [system] [extern_c] {
       ^

@abertelrud
Copy link
Contributor Author

It's while building libTestPureSwiftSharedLib and doesn't seem related. Trying again in case it's transient.

@abertelrud
Copy link
Contributor Author

There shouldn't really be a need to build the Swift compiler, but it doesn't seem that TSC supports smoke tests.

@abertelrud
Copy link
Contributor Author

@swift-ci please test linux

@abertelrud
Copy link
Contributor Author

abertelrud commented Jun 6, 2021

Failed again. Looks like we're building with swift-5.3-RELEASE-ubuntu16.04 which seems rather old for building TSC main (since 5.4 has been released, 5.5 has been branched, and main is even beyond that). Not sure if that has anything to do with the failure, though, but it's interesting.

@abertelrud
Copy link
Contributor Author

@swift-ci please test linux

@tomerd
Copy link
Contributor

tomerd commented Jun 7, 2021

22:29:37 error: could not decode frontend target info; compiler driver and frontend executables may be incompatible
22:29:37 details: frontend: /home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/latest_toolchain/swift-5.3-RELEASE-ubuntu16.04/usr/bin/swift
22:29:37          arguments: ["-frontend", "-print-target-info", "-target", "x86_64-unknown-linux-gnu", "-sdk", "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/stdlib/public/core", "-resource-dir", "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/lib/swift", "-runtime-compatibility-version", "none"]
22:29:37          error: key missing: CodingKeys(stringValue: "compilerVersion", intValue: nil), path: []
22:29:37 output
22:29:37 : "{
22:29:37   "target": {
22:29:37     "triple": "x86_64-unknown-linux-gnu",
22:29:37     "unversionedTriple": "x86_64-unknown-linux-gnu",
22:29:37     "moduleTriple": "x86_64-unknown-linux-gnu",
22:29:37     "librariesRequireRPath": false
22:29:37   },
22:29:37   "paths": {
22:29:37     "sdkPath": "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/stdlib/public/core",
22:29:37     "runtimeLibraryPaths": [
22:29:37       "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/lib/swift/linux"
22:29:37     ],
22:29:37     "runtimeLibraryImportPaths": [
22:29:37       "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/lib/swift/linux/x86_64",
22:29:37       "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/stdlib/public/core/usr/lib/swift/linux/x86_64"
22:29:37     ],
22:29:37     "runtimeResourcePath": "/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/build/buildbot_incremental/swift-linux-x86_64/lib/swift"
22:29:37   }
22:29:37 }
22:29:37 "

@shahmishal CI infra issue?

@shahmishal
Copy link
Member

swiftlang/swift#37837
@swift-ci please test linux

@abertelrud abertelrud merged commit d6c996e into main Jun 9, 2021
@abertelrud abertelrud deleted the eng/anders/fix-netrc-availability branch June 12, 2021 23:03
abertelrud added a commit that referenced this pull request Jun 12, 2021
…ver all the Darwin platforms (#218)

This extends the availability based on the version numbers in the annotation for `NSTextCheckingResult.range(withName name: String) -> NSRange`.  Otherwise, ToolsSupportCore can no longer build on for example iOS, because the default minimum deployment targets for the various platforms are really old (e.g. iOS 9).

(cherry picked from commit d6c996e)
abertelrud added a commit that referenced this pull request Jun 14, 2021
…ver all the Darwin platforms (#218) (#223)

This extends the availability based on the version numbers in the annotation for `NSTextCheckingResult.range(withName name: String) -> NSRange`.  Otherwise, ToolsSupportCore can no longer build on for example iOS, because the default minimum deployment targets for the various platforms are really old (e.g. iOS 9).

(cherry picked from commit d6c996e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants