Add option to opencascade to apply patch for C2-continuous Coons patches #22853
Conversation
This file contains 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
ghost
mentioned this pull request
joergbrech
changed the title
conan-center-bot
added
Failed
Missing dependencies
Conan v1 pipeline ✔️All green in build 2 (
Conan v2 pipeline ❌
The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping See details:Failure in build 2 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Hooks produced the following warnings for commit 33f7418opencascade/7.6.0@#e5ad5156ea46d5c6c0e28167af52e5e6opencascade/7.6.2@#26320450d78b569f6c91ecdd1f65b6abopencascade/7.5.0@#910198c6be73eac8a8bbd94d7cd6a067 |
|
Note that my PR adds a new option to the recipe, which is disabled by default. So in CI we are missing the build with this option enabled, i.e. Below are some infos about the pipeline failures. If I understand them correctly, they are unrelated to my changes and result from problems in the dependencies, rather than the recipe I am trying to update in this PR. How can I contribute to get the pipeline to work? Missing prebuilt binary for tk/8.6.10The conan v2 pipeline fails. This seems to be unrelated to my changes, but due to a missing prebuilt binary for tk/8.6.10: Click to expand the log of `conan graph explain`conan@1b264254d276:~/conan-center-index/recipes/opencascade$ conan graph explain all/conanfile.py --version=7.6.2 -o opencascade/*:shared=False======== Computing dependency graph ======== ======== Computing necessary packages ======== ======== Retrieving and computing closest binaries ======== Finding binaries in the cache ======== Closest binaries ======== If I read the log correctly, then I can produce a similar output with So this seems to be an issue of Hook warnings about missing system librariesI couldn not reproduce the warnings |
ghost
mentioned this pull request
Sign up for free
to 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.
opencascade/7.6.2
I am contributing this as developer of TiGL (https://github.com/DLR-SC/tigl). TiGL depends on a patched version of opencascade for "C2 continuous Coons-Patches" (A low-level surface modeling algorithm). We want to migrate our build system to conan so I added an option to the existing opencascade recipe to include this patch.
Note that I needed to bump the freetype version to avoid a version conflict.