Skip to content

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