Skip to content

Work around a crash importing FoundationXML.#786

Merged
grynspan merged 2 commits into
mainfrom
jgrynspan/workaround-foundationxml-crash
Oct 28, 2024
Merged

Work around a crash importing FoundationXML.#786
grynspan merged 2 commits into
mainfrom
jgrynspan/workaround-foundationxml-crash

Conversation

@grynspan

@grynspan grynspan commented Oct 28, 2024

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

@swift-ci test

@grynspan

Copy link
Copy Markdown
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
@grynspan grynspan deleted the jgrynspan/workaround-foundationxml-crash branch October 28, 2024 18:09
@stmontgomery

Copy link
Copy Markdown
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