Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Download Qhull source from Github, not Qhull servers, in meson build#27505
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
What is included / excluded in the download tarball vs what we are now getting from github? |
ContributorAuthor
Jacob-Stevens-Haas commentedDec 14, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I'm not familiar with exactly what command Qhull used to create their md5 from the tarball, but with + some python to sort the list, I was able to git diff the files. md5 sums from github tarballThis doesn't exactly match themd5 file listed on qhull's download page. Most of the |
ContributorAuthor
Jacob-Stevens-Haas commentedDec 14, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Oh, I see I took the qhull tarball "qhull-2020-src-8.0.2-tgz" to mean git tag 8.0.2, but I also see there's a git tag for 2020.2. I'll check that one. EDIT: Looks like both the github tarballs from the 2020.2 and 8.0.2 release match each other exactly, and neither matches the one on qhull's webpage. Of course, I'm using their stated md5s, since I can't download the tarball from their website, if you can, the simplest one to check is |
Member
ksunden commentedDec 14, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Beyond the superficial filepath (one had a subfolder of Most of the MD5 differences come from kyle@radon py@edge Downloads $ diff -r --strip-trailing-cr qhull_org githubOnly in github/qhull-8.0.2/build: qhulltestOnly in github/qhull-8.0.2/build: README-build.txtOnly in github/qhull-8.0.2: .gitignorediff -r --strip-trailing-cr qhull_org/qhull-8.0.2/html/qh-get.htm github/qhull-8.0.2/html/qh-get.htm96c96< <li>Sept. 3, 2020 22:00:41 EDT md5sum for Qhull 2020.2 downloads---> <li>Sept. 3, 2020 22:14 EDT md5sum for Qhull 2020.2 downloads98,101c98,101< 368f2aa9bf89677af04d57d281041359 *qhull-2020.2/qhull-2020.2-zip.md5sum< 27c44a7e2db5f127e1c6fc7807580566 *qhull-2020.2/qhull-2020-src-8.0.2-tgz.md5sum< a1a58969aa17d89088ebbd0b0eb44a82 *qhull-2020.2.zip< fa34047579937433e3a36c5d4033a988 *qhull-2020-src-8.0.2.tgz---> 4f68ebb4700849a548b332de500d42e9 *qhull-2020.2/qhull-2020.2-zip.md5sum> ed767244864488d5e9a22dfa788663a9 *qhull-2020.2/qhull-2020-src-8.0.2-tgz.md5sum> 7ed0bcce474bf12f6fbb5b4093760f3d *qhull-2020.2.zip> 295f7332269a38279478f555cc185296 *qhull-2020-src-8.0.2.tgzOnly in qhull_org/qhull-8.0.2: qhull-2020-src-8.0.2-tgz.md5sum So the only file which differs in actual content is the one listing the md5 sums of the tarballs/zips/md5sum files. The qhull.org version is the missing The github version is missing one md5sum file. None of these seem problematic to me |
QuLogic approved these changesDec 15, 2023
ksunden approved these changesDec 15, 2023
Romain-Geissler-1A commentedApr 22, 2024
Hi, I am currently failing to build matplotlib 3.8 using pip install, because again it seems the old qhull website has some issues and the connection is reset in the middle of the download. I see this was merged for meson only, which seems to be only in 3.9. How possible would it be possible to backport an equivalent of this, but in 3.8 with the previous build system ? |
Romain-Geissler-1A added a commit to Romain-Geissler-1A/matplotlib that referenced this pull requestApr 23, 2024
Change similar toomatplotlib#27505 but on the branch 3.8.x as right now theqhull website seems unable to properly deliver the tarball withoutdisconnecting in the middle.
1 task
Romain-Geissler-1A added a commit to Romain-Geissler-1A/matplotlib that referenced this pull requestApr 29, 2024
Change similar toomatplotlib#27505 but on the branch 3.8.x as right now theqhull website seems unable to properly deliver the tarball withoutdisconnecting in the middle.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading.Please reload this page.
PR summary
Closes#27159. As mentioned there, the Qhull server is HTTP (not HTTPS). While that's not a security problem in theory (we know the hash to expect), in practice corporate/academic may servers block the download. This includeseduroam, used by many large research universities. Furthermore, as@tacaswell mentioned, rather than hitting Qhull's server with every build, it's better to rely on github. (IIUC - lmk if I'm misrepresenting!).
The original issue got a lot of attention, but appeared resolved when I moved to a new network. When I met a new colleague who mentioned the same issue, I reopened the issue - understand if this needs more discussion beforehand.
PR checklist