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

Working OS/X build #39

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

Working OS/X build #39

wants to merge 2 commits into from

Conversation

jancona
Copy link

@jancona jancona commented Jul 4, 2018

This builds and runs on OS/X Sierra. It also still works on a Linux box. This should fix #24.

@darksidelemm
Copy link

Under OSX 10.11.6 I'm getting a few errors of the kind:
csdr.c:2080:31: error: use of undeclared identifier 'CLOCK_MONOTONIC'

I'm using macports to provide the build system, and I had to modify line 47 in the Makefile to add the /opt/local/include and /opt/local/lib directories, else libfftw wasn't being found.

Not sure if this is a compile issue (gcc isn't really gcc... it's LLVM), or something else.

@jancona
Copy link
Author

jancona commented Jul 6, 2018

It looks like clock_gettime and CLOCK_MONOTONIC were only added in OS/X 10.12 Sierra. I pushed this solution, which works on Sierra. Let me know if it works for you @darksidelemm.

@darksidelemm
Copy link

It builds fine now! Not able to do any serious testing at the moment, but that's a good start!
(Still have to manually specify the include and lib directories mind you, but i'm not sure what the right solution to that is)

@Sevyls
Copy link

Sevyls commented Oct 18, 2018

Why is this change not being merged?

@Sevyls
Copy link

Sevyls commented Oct 18, 2018

Building works under macOS Mojave, but installing fails:

Michaels-MacBook-Pro:csdr user$ sudo make install
install -m 0755 libcsdr.so.0.15 /usr/lib
install: /usr/lib/libcsdr.so.0.15: Operation not permitted
make: *** [install] Error 71

Changing the PREFIX in Makefile to /usr/local worked for me.

@ha7ilm
Copy link
Owner

ha7ilm commented Oct 18, 2018 via email

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.

can't make for mac osx
4 participants