Skip to content
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

xalan-c: Add patch for Xerces-C 3.2.0 #180

Merged
merged 1 commit into from
Sep 10, 2017
Merged

xalan-c: Add patch for Xerces-C 3.2.0 #180

merged 1 commit into from
Sep 10, 2017

Conversation

rleigh-codelibre
Copy link
Contributor

@rleigh-codelibre rleigh-codelibre commented Sep 6, 2017

Xalan-C 3.2.0 changes the type of XMLCh from a 16-bit integer type to char16_t. Xerces made some broken assumptions about casting which require fixing.

See https://issues.apache.org/jira/browse/XALANC-773

Note I also renamed the existing patch to be more descriptive. This is safe because it's not actually used by the xalan-c formula (I'll make the formula use both patches once this is merged).

Xalan-C 3.2.0 changes the type of XMLCh from a 16-bit integer type
to char16_t.  Xerces made some broken assumptions about casting
which require fixing.

See https://issues.apache.org/jira/browse/XALANC-773
@JCount JCount merged commit 71f4b09 into Homebrew:master Sep 10, 2017
@JCount
Copy link
Contributor

JCount commented Sep 10, 2017

@rleigh-codelibre Merged, you should be able to update your PR with the patch now.

@rleigh-codelibre rleigh-codelibre deleted the xalan-c-patch branch September 16, 2017 20:13
@lock lock bot added the outdated label Jan 18, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants