-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[deps] Update PCRE2 URL #43884
[deps] Update PCRE2 URL #43884
Conversation
Similar to JuliaPackaging/Yggdrasil#4293. The old release tarballs are gone, so we have to use GitHub-generated snapshots and also to run `./autogen.sh` before `./configure`.
Do we not need to update the checksums? |
Likely, the tarballs changed. Where are the checksums? |
https://github.com/JuliaLang/julia/blob/master/deps/checksums/pcre
I often |
Autogen requires programs that we avoid having in our dependency list. |
Is there a place we can get tarballs that don't require us to run |
Alternatively, we could download the tarballs, run autogen, tar up the results, and host the resultant tarballs ourselves. Then users don't need autogen. |
Valentin had already asked in spack the PCRE2 dev to reupload the release tarballs somewhere. We're waiting for a reply |
The original tarball for 10.36 is now available on GitHub (same checksums): https://github.com/PhilipHazel/pcre2/releases/tag/pcre2-10.36. I updated this PR to just change the URL |
(cherry picked from commit 9769024)
(cherry picked from commit 9769024)
(cherry picked from commit 9769024)
Similar to JuliaPackaging/Yggdrasil#4293. The old
release tarballs are gone, so we have to use GitHub-generated snapshots and also
to run
./autogen.sh
before./configure
.Should fix #43878 but I didn't test it locally, I mainly followed the changes I had to in Yggdrasil as well.