search

jashubbard / edfR

an R package for importing EDF files from Eyelink eyetrackers
23 stars 4 forks source link

edfR doesn't compile on Linux #1

Closed tmalsburg closed 8 years ago

tmalsburg commented 8 years ago

I tried installing via install_github('jashubbard/edfR') on Ubuntu 15.04. For some reason this seems to be using the Makevars for OSX instead of those for Linux:

g++ -shared -L/usr/local/lib -o edfR.so RcppExports.o edf_api.o process_eyedata.o -framework edfapi -F/Library/Frameworks/
g++: error: edfapi: No such file or directory
g++: error: unrecognized command line option ‘-framework’

The configure file also adds /usr/local/lib to PKG_LIBS although edfapi actually gets installed into /usr/lib (when following the instructions given in the SR Research forum).

(I'm planning to use the package over the next weeks. So hopefully I will be able to make some contributions.)

jashubbard commented 8 years ago

Ok I updated configure so that it creates a new Makevars no matter what, not just when it doesn't exist. I also changed the path to /usr/lib . Give it another shot.

On Jan 20, 2016, at 12:30 PM, Titus von der Malsburg notifications@github.com wrote:

I tried installing via install_github('jashubbard/edfR') on Ubuntu 15.04. For some reason this seems to be using the Makevars for OSX instead of those for Linux:

g++ -shared -L/usr/local/lib -o edfR.so RcppExports.o edf_api.o process_eyedata.o -framework edfapi -F/Library/Frameworks/ g++: error: edfapi: No such file or directory g++: error: unrecognized command line option ‘-framework’ The configure file also adds /usr/local/lib to PKG_LIBS although edfapi actually gets installed into /usr/lib (when following the instructions given in the SR Research forum).

(I'm planning to use the package over the next weeks. So hopefully I will be able to make some contributions.)

— Reply to this email directly or view it on GitHub https://github.com/jashubbard/edfR/issues/1.

tmalsburg commented 8 years ago

Hm, same behavior as before. Unfortunately, I don't know much about R's build process. I'll look into it ...

tmalsburg commented 8 years ago

Fixed in 0052e1331bbc4d0abc11db953534bcb8b95e7fdd. There was an error in the conditional for detecting the OS.