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.

Sign up for GitHub

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

Jump to bottom

[draft] Add a mindeps CI and update minimum versions #1943

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

[draft] Add a mindeps CI and update minimum versions #1943

wants to merge 15 commits into from
+547 −60

Conversation

mwestphal
Copy link
Contributor

@mwestphal mwestphal commented Jan 22, 2025 •
edited
Loading

Add a mindeps CI testing minium version for:

  • alembic
  • assimp
  • draco
  • occt
  • openexr
  • openvdb

Update minimum version for our direct dependencies:

  • alembic: 1.8.5
  • assimp: v5.4.0
  • araco: 1.5.6
  • occt: V7_6_3
  • openexr: 3.0.1
  • openvdb: 10.0.0

Dependencies that were not added to mindeps:

  • blosc : No find_package in F3D, not listed as dependencies in our doc, no specific versions listed by VFX
  • imath: No find_package in F3D, not listed as dependencies in our doc, no specific versions listed by VFX
  • ospray: No find_package in F3D, not listed as dependencies in our doc, no specific versions listed by VFX, problem with recent releases of F3D, we are "already" testing mindeps and cannot update.

Dependencies that should be added to mindeps but more work needed:

  • Python
  • Java

@mwestphal mwestphal force-pushed the mindeps branch 3 times, most recently from d5c23e3 to 028cb89 Compare January 25, 2025 10:11
@mwestphal mwestphal changed the title [draft] Add a mindeps CI [draft] Add a mindeps CI and update minimum versions Jan 25, 2025
@mwestphal mwestphal force-pushed the mindeps branch 3 times, most recently from 12813df to 6f45d8c Compare January 25, 2025 11:22
@mwestphal mwestphal force-pushed the mindeps branch 6 times, most recently from 5170096 to e511ae8 Compare January 25, 2025 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Meakk Meakk Meakk left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mwestphal @Meakk