Skip to content

Require macOS 10.15 #369

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
Feb 4, 2021
Merged

Conversation

yim-lee
Copy link
Contributor

@yim-lee yim-lee commented Feb 3, 2021

Motivation:
The same requirement is being added to SwiftPM as part of swiftlang/swift-package-manager#3202. Without this change SourceKit-LSP will fail to build with error:

the library 'SKSwiftPMWorkspace' requires macos 10.10, but depends on the product 'SwiftPM-auto' which requires macos 10.15; consider changing the library 'SKSwiftPMWorkspace' to require macos 10.15 or later, or the product 'SwiftPM-auto' to require macos 10.10 or earlier.

Modification:
Require macOS 10.15.

Result:
SourceKit-LSP builds successfully with changeset
swiftlang/swift-package-manager#3202.

Motivation:
The same requirement is being added to SwiftPM as part of swiftlang/swift-package-manager#3202. Without this change SourceKit-LSP will fail to build with error:

```
the library 'SKSwiftPMWorkspace' requires macos 10.10, but depends on the product 'SwiftPM-auto' which requires macos 10.15; consider changing the library 'SKSwiftPMWorkspace' to require macos 10.15 or later, or the product 'SwiftPM-auto' to require macos 10.10 or earlier.
```

Modification:
Require macOS 10.15.

Result:
SourceKit-LSP builds successfully with changeset
swiftlang/swift-package-manager#3202.
@yim-lee yim-lee requested a review from benlangmuir as a code owner February 3, 2021 22:57
@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 3, 2021

cc @neonichu @abertelrud @tomerd

@benlangmuir
Copy link
Contributor

@swift-ci please test

@yim-lee
Copy link
Contributor Author

yim-lee commented Feb 4, 2021

@benlangmuir Can you please merge this when ready since I don't have permissions to? Thank you.

@benlangmuir benlangmuir merged commit fb707d1 into swiftlang:main Feb 4, 2021
@yim-lee yim-lee deleted the require-mac10.15 branch February 4, 2021 23:00
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.

2 participants