Skip to content

Conversation

@proyan
Copy link
Contributor

@proyan proyan commented Dec 19, 2025

Summary

Changes to recipe: qhull/8.1-alpha6

Motivation

This PR is in a series of PR meant to add Collision support to Pinocchio (https://conan.io/center/recipes/pinocchio)

Details

Current version of qhull is not supported by coal (https://conan.io/center/recipes/coal?version=3.0.1)
This adds the latest version 8.1-alpha6 to conan

The subsequent PR #29186 includes this dependency into coal


  • [ ✓] Read the contributing guidelines
  • [ ✓] Checked that this PR is not a duplicate: list of PRs by recipe
  • [ ✓] If this is a bug fix, please link related issue or provide bug details
  • [ ✓] Tested locally with at least one configuration using a recent version of Conan

Add a 👍 reaction to pull requests you find important to help the team prioritize, thanks!

Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

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

You will need to also add an entry in config.yml file for this to be mergeable.

Seeing the release cadence of upstream, and that the alpha version have not been marked as a rc, I think it's ok to add the tag as a prerelease

@proyan
Copy link
Contributor Author

proyan commented Dec 19, 2025

Hi @AbrilRBS
Thanks for the quick reply. I've updated the config.yml. Could you please point me to where I should add the pre-release tag?

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