-
Notifications
You must be signed in to change notification settings - Fork 311
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
Sync to modern FindOpenVDB.cmake #5102
base: master
Are you sure you want to change the base?
Sync to modern FindOpenVDB.cmake #5102
Conversation
Oh, it looks like perhaps you want to always use the "8.2 patched" specified in Is there any reason you're sticking with "8.2 patched"? Was the reasoning perhaps from a previous time, and now 10.x is suitable? I'd be happy to update the |
2601f44
to
1041dac
Compare
You can refer to this discussion about OpenVDB version issues; it might be helpful. |
@MackBambu got it, will do! i'm eagerly looking forward to being able to build from source on Debian Unstable. It looks like any fixes for this would be best put through Prusaslicer, which Bambu Studio would then pull in? What's the preferred workflow there? I only have Bambu machines, but would prefer any generic fixes indeed go to the open source root project. |
Steps for Updating Dependencies: 1.Select a stable official version as the base. If any modifications to the dependency code are needed, apply them as patches to keep changes isolated and manageable. |
thanks, but do you think i maybe ought do this work with PrusaSlicer, and then count on it coming into BambuStudio when you next pull from there (i understand BS to be derived from PS, but perhaps i am wrong)? |
You can work directly on Bambu Studio, but if you prefer working from PrusaSlicer, that's also fine. We will cherry-pick relevant code from PrusaSlicer when appropriate. |
I don't personally understand why we're shipping our own copies of these files; we ought IMHO be getting them from the installed system (i.e.
/usr/{local}/lib/cmake/OpenVDB
), where they're guaranteed to match the installed libopenvdb. Regardless, we probably want a more modern version ofFindOpenVDB
and its new dependency,FindBlosc
. With this change, I can build on Debian Unstable.If you really want to support OpenVCB 5 (quite old), we'll want to test this against such a vintage installation. I unfortunately do not have a setup on which to do so.
With the existing files, attempting to build on e.g. Debian Unstable gets a failure about libilmbase, which hasn't existed in some time (replaced by OpenEXR):