Fix installation on Mac OS#432
Conversation
test for Mac OS 15
preliminary osx 26 support
preliminary Mac OS 26.2 support
add preliminary macOS 26 support
fix Mac OS paraview
add paraview bin to path for Macos
universal script for macOS and linux
fix Xcode version finding
did not work form script
bypass checks to completely support Mac OS 26 for now
|
Dear @sportokalidis, For supporting macOS 26 a version of root in Xcode 26 is required can you please provide one? |
fix for centos 7
fix m1
|
hey @sportokalidis, can you take a look on this PR please? |
|
please accept the pull request after check completion |
sportokalidis
left a comment
There was a problem hiding this comment.
Hi @johnpapad24, and thanks for the PR! Since this PR adds support for macos-26, please also add a CI job that runs on macos-26 to validate the changes.
There was a problem hiding this comment.
Could you please revert the change that hardcodes the tarball name in paraview.cmake? We should keep generating the tar file name dynamically.
There was a problem hiding this comment.
hey @johnpapad24, please take care the points @sportokalidis mentioned to finalize this please, thanks
There was a problem hiding this comment.
changes resolved yesterday please also accept the new pull requests for mac os 26
I have done an initial work to fix the downloaded paraview detection on Mac os
however I think it also needs root