csdr icon indicating copy to clipboard operation
csdr copied to clipboard

Working OS/X build

Open jancona opened this issue 6 years ago • 6 comments

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

jancona avatar Jul 04 '18 12:07 jancona

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.

darksidelemm avatar Jul 06 '18 12:07 darksidelemm

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.

jancona avatar Jul 06 '18 13:07 jancona

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)

darksidelemm avatar Jul 06 '18 13:07 darksidelemm

Why is this change not being merged?

Sevyls avatar Oct 18 '18 09:10 Sevyls

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.

Sevyls avatar Oct 18 '18 09:10 Sevyls

Because of me struggling to find time on reviewing this and other pull requests. I don't have a Mac either, so what I'll be able to test is that it still works in Linux.

Michael Auß [email protected] ezt írta (időpont: 2018. okt. 18., Cs, 11:45):

Why is this change not being merged?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/simonyiszk/csdr/pull/39#issuecomment-430945518, or mute the thread https://github.com/notifications/unsubscribe-auth/AFoUMHRjS30gb4ERlaYMx4ZRI-jTQUbhks5umE2ogaJpZM4VCfWM .

ha7ilm avatar Oct 18 '18 09:10 ha7ilm