-
Notifications
You must be signed in to change notification settings - Fork 10
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
Update snap edge/beta to OCC 7.7.2 #69
Comments
link to forum discussion: https://forum.freecadweb.org/viewtopic.php?p=639505#p639505 |
you'll need to change this to V7_7_1_BF it's possible that you will need to move to core 22 also: occt 7.7 requires tbb >=2021.5 which I doubt would be in ubuntu 20.04 |
You're right, @adrianinsaval. Do you know if I can keep things as they are for stable and change OCC and core 22 just for snap edge and snap beta builds? |
I would expect yes, but I'm not all that familiar with how making snaps work |
There's some bugs with occt 7.7.1 that I don't think we're going to fix before release, I'll move this out of the 0.21 project. Better to do this next cycle. |
Are we not building Blobfish for the snaps? I should fix if not. |
has blobfish rebased his patches on 7.7.2 yet ? |
I don't think so: the latest I see is 7.7.1BF |
the blobfish patch set keeps growing but none of it is going upstream, should we worry about that? Can we do something to get those merged upstream? |
Maybe we can forge a relationship with upstream (there is a gentleman that has represented OCC in previous FLOSS conferences) and have a (paid?) liason work to merge patches ? |
We cannot "merge" his code upstream, he has not signed the OpenCASCADE developer agreement, and is not interested in doing so. His patches would have to be rewritten. |
Ugh, I wish they would drop that silly requirement. |
Per wwmayer OCC 7.7.0 compiles fine with latest FreeCAD HEAD. We should add it to snap edge in that case
The text was updated successfully, but these errors were encountered: