Skip to content

Make PackageModel resources optional #7348

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 2 commits into from
Feb 19, 2024

Conversation

MaxDesiatov
Copy link
Contributor

@MaxDesiatov MaxDesiatov commented Feb 19, 2024

#7337 is breaking toolchain builds, so we should pass SKIP_RESOURCE_SUPPORT by default to avoid that.

@MaxDesiatov MaxDesiatov added the WIP Work in progress label Feb 19, 2024
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test windows

@MaxDesiatov MaxDesiatov added bug resources Package resources support and removed WIP Work in progress labels Feb 19, 2024
@MaxDesiatov MaxDesiatov marked this pull request as ready for review February 19, 2024 14:37
@MaxDesiatov MaxDesiatov merged commit 9c6493f into main Feb 19, 2024
@MaxDesiatov MaxDesiatov deleted the maxd/optional-libraries-metadata branch February 19, 2024 15:25
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
swiftlang#7337 is breaking
toolchain builds, so we should pass `SKIP_RESOURCE_SUPPORT` by default
to avoid that.
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
swiftlang#7337 is breaking
toolchain builds, so we should pass `SKIP_RESOURCE_SUPPORT` by default
to avoid that.
xedin added a commit to xedin/swift-package-manager that referenced this pull request Jul 19, 2024
xedin added a commit to xedin/swift-package-manager that referenced this pull request Jul 19, 2024
xedin added a commit to xedin/swift-package-manager that referenced this pull request Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug resources Package resources support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants