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

libs(qt6): vendor xerces-c for windows and mac builds #1011

Merged
merged 3 commits into from
Aug 17, 2023
Merged

Conversation

csett86
Copy link
Collaborator

@csett86 csett86 commented Aug 8, 2023

Source of the xerces-c headers and binaries is https://github.com/FashionFreedom/xerces-c/releases/tag/v3.2.4

@changelogg
Copy link

changelogg bot commented Aug 8, 2023

Hey! Changelogs info seems to be missing or might be in incorrect format.
Please use the below template in PR description to ensure Changelogg can detect your changes:
- (tag) changelog_text
or
- tag: changelog_text
OR
You can add tag in PR header or while doing a commit too
(tag) PR header
or
tag: PR header
Valid tags: added / feat, changed, deprecated, fixed / fix, removed, security, build, ci, chore, docs, perf, refactor, revert, style, test
Thanks!
For more info, check out changelogg docs

@ghost
Copy link

ghost commented Aug 8, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@csett86 csett86 mentioned this pull request Aug 9, 2023
Signed-off-by: Christoph Settgast <csett86_git@quicksands.de>
@csett86
Copy link
Collaborator Author

csett86 commented Aug 10, 2023

This is now ready to be reviewed :)

@slspencer slspencer self-requested a review August 17, 2023 17:11
Signed-off-by: slspencer <susan.spencer@gmail.com>
@slspencer slspencer merged commit 0a4e29b into qt6 Aug 17, 2023
3 of 5 checks passed
@slspencer slspencer deleted the vendor-xerces branch August 17, 2023 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants