Skip to content

Add Action for MacOS installation #138

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 30 commits into
base: develop
Choose a base branch
from
Open

Add Action for MacOS installation #138

wants to merge 30 commits into from

Conversation

@ajaust
Copy link
Collaborator

@ajaust ajaust commented Mar 29, 2022 •
edited
Loading

This PR adds an action which installs the Python bindings on MacOS X.

@ajaust ajaust force-pushed the add-macos-action branch 3 times, most recently from f4fc64e to df85b98 Compare March 29, 2022 09:07
@ajaust ajaust force-pushed the add-macos-action branch from df85b98 to 488ee9f Compare March 29, 2022 09:46
@ajaust ajaust mentioned this pull request Mar 29, 2022
Open
@ajaust
Copy link
Collaborator Author

ajaust commented Mar 29, 2022

The action currently fails as the bindings do not find the preCICE headers. I am not sure whether I should fix it by setting the CPATH as this action currently installs preCICE in the /usr directory. We should find out, if this should be in the standard search path of MacOS X first.

@kursatyurt
Copy link
Contributor

kursatyurt commented Apr 24, 2022

On my computer using miniconda python installation without any problem.

I just changed CFLAGS in setup.py to include -L\usr\local\include

But again I have linker problem

@BenjaminRodenberg BenjaminRodenberg added the good first issue Good for newcomers label Mar 12, 2025
@BenjaminRodenberg
Copy link
Member

BenjaminRodenberg commented Mar 12, 2025

Just merged develop into this PR to keep it synchronized. Seems like there are some problems. I'm not planning to work on this PR since I do not have particular interest in MacOS but would be great if any MacOS user could pick this up.

@IshaanDesai
Copy link
Member

IshaanDesai commented Aug 13, 2025

Keep this PR updated in the hope that we would merge the MacOS Action at some point.

@IshaanDesai IshaanDesai added the help wanted Extra attention is needed label Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

good first issue Good for newcomers help wanted Extra attention is needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ajaust @kursatyurt @BenjaminRodenberg @IshaanDesai