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
depends: build expat with CMake #29878
base: master
Are you sure you want to change the base?
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
Looks like there could be issues with expat, CMake and 32-bit builds: |
Guix builds (on x86_64)
|
760bf23
to
6cec148
Compare
(Testing a script I'm developing on this PR)
|
6cec148
to
e92be65
Compare
I've pushed a change that switches to downloading the unbootstraped source tarball. Want to re-run your script? |
Those generated scripts are short and look harmless, btw.
GitHub generated source archives may not have stable hashes: https://github.blog/2023-02-21-update-on-the-future-stability-of-source-code-archives-and-hashes/ I'm not sure if |
FWIW #29923 removes the need to deal with expat in any way (as it's a dependency of a Qt dependency). |
9b54b40
to
c23d45d
Compare
c23d45d
to
ef85ac0
Compare
Bumps expat from
2.4.8
->2.6.2
Switch to building Expat with CMake, instead of Autotools.