Skip to content

Work around a crash importing FoundationXML. #786

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
Oct 28, 2024

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Oct 28, 2024

This PR works around a new crash importing FoundationXML See:

Works around rdar://138761752.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@grynspan grynspan added bug 🪲 Something isn't working windows 🪟 Windows support linux 🐧 Linux support (all distros) workaround Workaround for an issue in another component (may need to revert later) labels Oct 28, 2024
@grynspan grynspan self-assigned this Oct 28, 2024
@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan grynspan requested a review from parkera October 28, 2024 15:40
@grynspan grynspan added this to the Swift 6.1 milestone Oct 28, 2024
@grynspan grynspan merged commit 405d8c9 into main Oct 28, 2024
3 checks passed
@grynspan grynspan deleted the jgrynspan/workaround-foundationxml-crash branch October 28, 2024 18:09
@stmontgomery
Copy link
Contributor

@grynspan Looks like we may be able to revert this now that swiftlang/swift#77309 has been merged. Care to do so?

grynspan added a commit that referenced this pull request Nov 4, 2024
grynspan added a commit that referenced this pull request Jan 3, 2025
grynspan added a commit that referenced this pull request Jan 6, 2025
This reverts commit 405d8c9.

The issue should no longer be occurring. See #786.

### Checklist:

- [x] Code and documentation should follow the style of the [Style
Guide](https://github.com/apple/swift-testing/blob/main/Documentation/StyleGuide.md).
- [x] If public symbols are renamed or modified, DocC references should
be updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working linux 🐧 Linux support (all distros) windows 🪟 Windows support workaround Workaround for an issue in another component (may need to revert later)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants