evanpurkhiser / keyfinder-cli

A CLI wrapper for libkeyfinder. Making DJs lives easier.
GNU General Public License v3.0
111 stars 23 forks source link

Cannot find keyfinder library on OSX using make #7

Closed andersfischernielsen closed 8 years ago

andersfischernielsen commented 8 years ago

I get a ld: library not found for -lkeyfinder when trying to make.

I'm running El Capitan, have run qmake, make and make install in libKeyFinder, even with the fixes mentioned in the other commit.

I have a DJ gig tomorrow, but I can't seem to build keyfinder-cli.

evanpurkhiser commented 8 years ago

Have you seen this https://github.com/ibsh/libKeyFinder/pull/9?

andersfischernielsen commented 8 years ago

I just pulled your fork, did everything again. I still get ld: library not found for -lkeyfinder clang: error: linker command failed with exit code 1 (use -v to see invocation) make: *** [keyfinder-cli] Error 1 when trying to make keyfinder-cli.

Thanks for making this, btw! I hope I can use it soon. :smile:

evanpurkhiser commented 8 years ago

And you built from my forked branch fixup/install-target right?

Can you verify that after doing make install for libKeyFinder is installing the shared object into /usr/local/lib/libkeyfinder.*

andersfischernielsen commented 8 years ago

Yeah, I built from that branch. :)

And no, there's no libkeyfinder.* lib in /usr/local/lib/, unfortunately.

evanpurkhiser commented 8 years ago

I discovered an issue with my changes to the Project file in that pull request causing the headers and library files to install into the incorrect place on OSX. See: https://github.com/ibsh/libKeyFinder/pull/9#issuecomment-193675543

Go ahead and rebuild libkeyfinder and see if that solves the issue!

For posterity the build steps I took

brew install fftw qt5 ffmpeg
git clone git@github.com:ibsh/libKeyFinder
cd libKeyFinder
git fetch origin pull/9/head:target-fix
git checkout target-fix
/usr/local/opt/qt5/bin/qmake
make
make install

git clone git@github.com:EvanPurkhiser/keyfinder-cli
cd keyfinder-cli
make
andersfischernielsen commented 8 years ago

Awesome! It works now. :) Thanks.