Skip to content

Fix installation on Mac OS#432

Open
johnpapad24 wants to merge 18 commits into
masterfrom
fix-macos-paraview
Open

Fix installation on Mac OS#432
johnpapad24 wants to merge 18 commits into
masterfrom
fix-macos-paraview

Conversation

@johnpapad24

Copy link
Copy Markdown
Collaborator

I have done an initial work to fix the downloaded paraview detection on Mac os

however I think it also needs root

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
@johnpapad24 johnpapad24 changed the title Update CMakeLists.txt Fix installation on Mac OS Dec 11, 2025
@johnpapad24

Copy link
Copy Markdown
Collaborator Author

Dear @sportokalidis,

For supporting macOS 26 a version of root in Xcode 26 is required can you please provide one?

@johnpapad24 johnpapad24 self-assigned this Dec 11, 2025
@johnpapad24 johnpapad24 requested a review from vasvav December 11, 2025 05:55
@vasvav

vasvav commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

hey @sportokalidis, can you take a look on this PR please?

@johnpapad24

johnpapad24 commented Jan 27, 2026

Copy link
Copy Markdown
Collaborator Author

please accept the pull request after check completion
@sportokalidis @vasvav

@johnpapad24

Copy link
Copy Markdown
Collaborator Author

@sportokalidis @vasvav

@sportokalidis sportokalidis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please revert the change that hardcodes the tarball name in paraview.cmake? We should keep generating the tar file name dynamically.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @johnpapad24, please take care the points @sportokalidis mentioned to finalize this please, thanks

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes resolved yesterday please also accept the new pull requests for mac os 26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants