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.

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

OS X-- build out of the box #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

OS X-- build out of the box #81

wants to merge 2 commits into from

Conversation

viega
Copy link

@viega viega commented Aug 20, 2023

Added a flag to link the default libclang that installs when you have already installed XCode.

I couldn't figure out how to get this into the nimble file without delving into nimble code, so it's living in a new config.nims.

Xcode.

I couldn't figure out how to get this into the nimble file, so it's
living in a new config.nims.
@viega
Copy link
Author

viega commented Aug 21, 2023

Note that this only helps the build; you still need to (as root) set up opir etc. to use the lib, eg:

install_name_tool -add_rpath /Library/Developer/CommandLineTools/usr/lib/ ~/.nimble/pkgs/futhark-0.9.3/opir
install_name_tool -change libclang.dylib @loader_path/Library/Developer/CommandLineTools/usr/lib/ ~/.nimble/pkgs/futhark-0.9.3/opir

…efaults to major version 14, unless you set MAC_SDK.
@PMunch
Copy link
Owner

PMunch commented Aug 21, 2023

Could you drop that last part into the README? And how does this differ from the logic already in the opir.nims file? I'm not familiar with how OSX handles their paths but I was under the impression that that would ask xcode where it had stored its libraries.

@viega
Copy link
Author

viega commented Aug 21, 2023

Sure, I didn't even take the time to look closely; didn't see opir.nims actually. I just dealt with the errors that came up when I moved to a Mac. I'll take a closer look and try to do it right, but might not be able to get to it until later in the week.

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.

2 participants