-
Notifications
You must be signed in to change notification settings - Fork 81
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
Horizon does not support OpenCascade 7.8.0 #764
Comments
None of the distros we have CI for already has opencascade 7.8 Any ideas how to reproduce this? |
From release notes (https://dev.opencascade.org/doc/overview/html/occt__upgrade.html):
I think this is a similar issue to this one: |
I guess that most distros won't upgrade OpenCascade as long as it breaks its consumers - at least this is what we are doing in FreeBSD: see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277294 . Thus, ATM, you have to compile it yourself. |
I'm using a locally built horizon with locally built OCCT 7.8.0 and it works fine here (kde neon)... didn't have to change anything.
Edit: Are you sure meson / cmake is finding the correct OCCT? I had to uninstall the liboce and libocct packages. |
If you patch out https://github.com/horizon-eda/horizon/blob/master/meson.build#L31-L35 does it work then? |
I have just built it successfully with the attached patch But this is for the 2.5.0 release, and it does not contain a meson.build file. I'd like to provide a patch for meson.build according to the OpenCascade version, but I don't know how to test it with meson. Could you please explain? |
Do you mean test the opencascade version or test building with meson? I haven't used meson much but this seems to work: assert(opencascade.found(), 'OpenCASCADE or OCE is required')
if opencascade.version() >= '7.8.0'
message('opencascade is >=7.8.0')
else
message('opencascade <7.8.0')
endif |
Yes, that works, thanks! |
The build fails with the message:
There is no more libTKSTEP in OpenCascade 7.8.0.
The text was updated successfully, but these errors were encountered: