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

Bump qhull and CppAD to latest versions #1011

Merged
merged 3 commits into from
Feb 3, 2022
Merged

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Feb 1, 2022

This is part of the work on branch https://github.com/traversaro/robotology-superbuild/tree/fix727 that was done to solve #727 . However, this part can be contributed on its own PR.

This PR bumps the qhull and CppAD version to track the latest version. In particular, this ensures that the versions used in the superbuild are compatible with the latest versions used in conda-forge:

Note that qhull has a bit of a complex versioning scheme, as it has releases like 8.0.2 and 2020.2 . We switch to the 2020.2-style tags for compatibility with most linux distros (see https://repology.org/project/qhull/versions) and with conda-forge.

As this are external packages, they are fixed to releases in LatestReleases and also for Stable and Unstable branches.

@GiulioRomualdi
Copy link
Member

By the way, is this going to affect blf? There we require qhull v8

@traversaro
Copy link
Member Author

The 2020.2 tag was done later then the v8.0.* tags, so I think this should be ok: https://github.com/qhull/qhull/tags .

@traversaro
Copy link
Member Author

Anyhow, blf is compiling fine in the CI, so I think we can merge.

@traversaro traversaro merged commit db524f2 into master Feb 3, 2022
@traversaro traversaro deleted the bumpexternaldeps branch February 3, 2022 08:14
Nicogene pushed a commit to Nicogene/robotology-superbuild that referenced this pull request Feb 14, 2022
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.

2 participants