Skip to content

[Qhull] Install with apt#1950

Merged
maxfirmbach merged 3 commits into
mainfrom
docker-update-get-qhull-from-apt
Apr 9, 2026
Merged

[Qhull] Install with apt#1950
maxfirmbach merged 3 commits into
mainfrom
docker-update-get-qhull-from-apt

Conversation

@maxfirmbach

@maxfirmbach maxfirmbach commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Description and Context

Install Qhull with apt instead of our manual procedure.

I talked with @eulovi about the failing tests, which were already fixed in #1649.

@maxfirmbach maxfirmbach self-assigned this Apr 8, 2026
@maxfirmbach maxfirmbach added the dependencies Pull requests that update a dependency file label Apr 8, 2026

@amgebauer amgebauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you!

@c-p-schmidt c-p-schmidt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Do we have some information on the version in our documentation that needs to be updated as well?

@maxfirmbach

Copy link
Copy Markdown
Contributor Author

@c-p-schmidt Not sure about the version. I did not check which version libqhull-dev uses.

@c-p-schmidt

Copy link
Copy Markdown
Contributor

@c-p-schmidt Not sure about the version. I did not check which version libqhull-dev uses.

I just checked our documentation and did not find a version that we specify. Since we now do NOT install a specific version via an installation script, but use apt to install it, we should not mention a version IMO (at least as long as we are not aware of incompatibility of specific versions), because the version obtained by apt install is most likely varying across different OS versions.

FYI: Ubuntu 24.04, the Qhull version is 2020.2-6.

@maxfirmbach maxfirmbach merged commit 4568163 into main Apr 9, 2026
44 checks passed
@maxfirmbach maxfirmbach deleted the docker-update-get-qhull-from-apt branch April 9, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants