Skip to content
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

macOS does not have ldconfig #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

macOS does not have ldconfig #68

wants to merge 1 commit into from

Conversation

tachoknight
Copy link

Since macOS does not have the ldconfig command, an error was generated when installing. This PR does not run the ldconfig command no macOS.

@noxdafox noxdafox force-pushed the master branch 2 times, most recently from 26d1ee4 to 0fccf70 Compare January 26, 2025 21:08
@noxdafox
Copy link
Owner

noxdafox commented Jan 26, 2025 •
edited
Loading

Hello,

I am not sure this is actually an issue. Here's github actions runner for MAC OS 13 and 14 (Intel and M1) where we successfully run make install-clips with the ldconfig command. EDIT: just noticed now as the ldconfig is prefixed with a - effectively ignoring the error. Are you getting any issue with it?

clipspy is now shipped as a binary distribution for both Intel and ARM architecture. Why are you trying to install it from source?

@tachoknight
Copy link
Author

When I did a pip3 install clipspy within a venv environment it wanted to build from source, thus me finding the error.

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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tachoknight @noxdafox