-
Notifications
You must be signed in to change notification settings - Fork 33
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
Switch to official vtk 7 available in hombrew/science #29
Comments
It looks like there is progress by the vtk team that may make this possible. See: Homebrew #3811 has been merged that includes VTK: fix linking to python |
The upstream fix still doesn't resolve the "segmentation fault" we get almost immediately upon launch when using hombrew/science/vtk that defaults to --with-python but purportedly fixed so there are no longer references to the system python...
This will have to stay on the list a bit longer... |
The root cause for the crash was due to incorrectly configured dynamic loader paths that resulted in loading binaries external to the bundle. PR 535 resolves the path issues that prevented us from using the production VTK with python bindings. |
PR FreeCAD/FreeCAD#554 is also required and was just merged by Yorik. All green for moving to standard Homebrew VTK install. |
Re-generating the ports-cache to cache the Homebrew vtk port to fully resolve this issue. |
vtk is available in homebrew-science but packaged with python bindings that use hard coded paths to the macOS python interpreter resulting in segfault exceptions. We host the formula and companion bottle built with —without-python. Once the vtk team eliminates their hard-coded path problem, we should be able to eliminate this custom formula to reduce maintenance.
The homebrew team made some progress here - https://github.com/Homebrew/homebrew-science/pull/4342 but including the python bindings still causes references to the system python interpreter to be included in the Salome dylibs. Additional context from the homebrew community https://github.com/Homebrew/homebrew-science/issues/3401
The text was updated successfully, but these errors were encountered: