-
Notifications
You must be signed in to change notification settings - Fork 34
Disable this library for Swift 5.9 and above. #68
Conversation
@swift-server-bot test this please |
For now, I've left this enabled on Windows, as Windows support isn't landed in the Swift runtime yet. |
@swift-server-bot add to allowlist |
@al45tair the CI test run with warnings as errors, so you may want to also conditionalize any test and sample code. note the CI jobs can be reproduced locally by invoking the same docker compose commands the CI job does |
5bed8f3
to
e75348f
Compare
@swift-server-bot test this please |
e75348f
to
4fef46c
Compare
@swift-server-bot test this please |
Swift 5.9 has a new built-in backtrace-on-crash facility, which supersedes this library. rdar://111109405
4fef46c
to
1d503b4
Compare
We'll want to hold off on merging this until swiftlang/swift#67483 and its 5.9 counterpart have gone through and the results have landed in a nightly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thumbs up for merging this!
cc @tomerd @weissi @fabianfett :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice one, thanks!
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [vapor/vapor](https://togithub.com/vapor/vapor) | minor | `4.80.0` -> `4.81.0` | --- ### Release Notes <details> <summary>vapor/vapor (vapor/vapor)</summary> ### [`v4.81.0`](https://togithub.com/vapor/vapor/releases/tag/4.81.0): - The swift-backtrace library is no longer needed in 5.9 [Compare Source](https://togithub.com/vapor/vapor/compare/4.80.0...4.81.0) #### What's Changed The swift-backtrace library is no longer needed in 5.9 by [@​gwynne](https://togithub.com/gwynne) in [#​3064](https://togithub.com/vapor/vapor/issues/3064) > Per [https://github.com/swift-server/swift-backtrace/pull/68](https://togithub.com/swift-server/swift-backtrace/pull/68). ###### *This patch was released by [@​gwynne](https://togithub.com/gwynne)* **Full Changelog**: vapor/vapor@4.80.0...4.81.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44My4wIiwidXBkYXRlZEluVmVyIjoiMzYuODMuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==--> Co-authored-by: Self-hosted Renovate Bot <361546+cgrindel-self-hosted-renovate[bot]@users.noreply.github.enterprise.com>
Pull request swift-server#68 prevents Backtrace from being built on Swift 5.9 and above, but Demangle is still built and fails when the importing project uses musl.
Pull request swift-server#68 prevents Backtrace from being built on Swift 5.9 and above, but Demangle is still built and fails when the importing project uses musl.
Pull request swift-server#68 prevents Backtrace from being built on Swift 5.9 and above, but Demangle is still built and fails when the importing project uses musl.
Swift 5.9 has a new built-in backtrace-on-crash facility, which supersedes this library.
rdar://111109405