Skip to content

Conversation

@elsh
Copy link
Contributor

@elsh elsh commented May 3, 2024

The exportability rules around package access level are updated in a way that an @_spi public can't be bound to a package var, similar to how @_spi public can't be bound to a public var. For reference, see swiftlang/swift#73161.
This PR corrects such use case.

@elsh
Copy link
Contributor Author

elsh commented May 3, 2024

@swift-ci smoke test

@elsh
Copy link
Contributor Author

elsh commented May 3, 2024

@swift-ci smoke test macOS

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) May 3, 2024 08:44
@MaxDesiatov MaxDesiatov merged commit 605e228 into main May 3, 2024
@MaxDesiatov MaxDesiatov deleted the elsh/fix-pkg-error branch May 3, 2024 11:28
ahoppen added a commit that referenced this pull request May 15, 2024
)

- **Explanation**: Cherry-pick swiftlang/swift#73161
to `releases/6.0` so that SwiftPM 6.0 can be built with a `main` Swift
development snapshot
- **Scope**: Changes access level of a member from `package` to SPI
- **Risk**: Very low, just changes the access lavel 
- **Testing**: n/a
- **Issue**: n/a
- **Reviewer**: @MaxDesiatov and @bnbarham on
#7525
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
…iftlang#7525)

The exportability rules around `package` access level are updated in a
way that an `@_spi public` can't be bound to a `package` var, similar to
how `@_spi public` can't be bound to a `public` var. For reference, see
swiftlang/swift#73161.
This PR corrects such use case.
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
…iftlang#7525)

The exportability rules around `package` access level are updated in a
way that an `@_spi public` can't be bound to a `package` var, similar to
how `@_spi public` can't be bound to a `public` var. For reference, see
swiftlang/swift#73161.
This PR corrects such use case.
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.

4 participants