Skip to content

package -> @_spi public due to package exportability rule updates #7525

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
May 3, 2024

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
5 checks passed
@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.

3 participants