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

R Build Failing #2929

Closed
sadatnfs opened this issue Nov 10, 2018 · 4 comments
Closed

R Build Failing #2929

sadatnfs opened this issue Nov 10, 2018 · 4 comments

Comments

@sadatnfs
Copy link

I'm trying to install the R bindings for Apache Arrow and it's been erroring out for me for the last 3-5 days with the following trace (I'm building in a Debian Stretch Docker, with Arrow already built and make installd:

g++  -std=gnu++11 -I"/opt/R/lib/R/include" -DNDEBUG -DNDEBUG -I/usr/local/include -I"/opt/R/lib/R/library/Rcpp/include" -w -g -O3   -m64 -fno-gnu-unique -I/opt/intel/compilers_and_libraries/linux/mkl/include -I/usr/include -mavx -msse3 -msse4.1 -msse4.2 -msse2 -mfpmath=sse    -fvisibility=hidden -fPIC   -w -g -O3   -m64 -fno-gnu-unique -I/opt/intel/compilers_and_libraries/linux/mkl/include -I/usr/include -mavx -msse3 -msse4.1 -msse4.2 -msse2 -mfpmath=sse   -c message.cpp -o message.o
message.cpp: In functionstd::shared_ptr<arrow::RecordBatch> ipc___ReadRecordBatch__Message__Schema(const std::unique_ptr<arrow::ipc::Message>&, const std::shared_ptr<arrow::Schema>&)’:
message.cpp:61:3: error:R_ERROR_NOT_OKwas not declared in this scope
   R_ERROR_NOT_OK(arrow::ipc::ReadRecordBatch(*message, schema, &batch));
   ^~~~~~~~~~~~~~
message.cpp:61:3: note: suggested alternative:RETURN_NOT_OK’
   R_ERROR_NOT_OK(arrow::ipc::ReadRecordBatch(*message, schema, &batch));
   ^~~~~~~~~~~~~~
   RETURN_NOT_OK
message.cpp: In functionstd::shared_ptr<arrow::Schema> ipc___ReadSchema_InputStream(const std::shared_ptr<arrow::io::InputStream>&)’:
message.cpp:69:3: error:R_ERROR_NOT_OKwas not declared in this scope
   R_ERROR_NOT_OK(arrow::ipc::ReadSchema(stream.get(), &schema));
   ^~~~~~~~~~~~~~
message.cpp:69:3: note: suggested alternative:RETURN_NOT_OK’
   R_ERROR_NOT_OK(arrow::ipc::ReadSchema(stream.get(), &schema));
   ^~~~~~~~~~~~~~
   RETURN_NOT_OK
message.cpp: In functionstd::unique_ptr<arrow::ipc::Message> ipc___MessageReader__ReadNextMessage(const std::unique_ptr<arrow::ipc::MessageReader>&)’:
message.cpp:85:3: error:R_ERROR_NOT_OKwas not declared in this scope
   R_ERROR_NOT_OK(reader->ReadNextMessage(&message));
   ^~~~~~~~~~~~~~
message.cpp:85:3: note: suggested alternative:RETURN_NOT_OK’
   R_ERROR_NOT_OK(reader->ReadNextMessage(&message));
   ^~~~~~~~~~~~~~
   RETURN_NOT_OK
message.cpp: In functionstd::unique_ptr<arrow::ipc::Message> ipc___ReadMessage(const std::shared_ptr<arrow::io::InputStream>&)’:
message.cpp:93:3: error:R_ERROR_NOT_OKwas not declared in this scope
   R_ERROR_NOT_OK(arrow::ipc::ReadMessage(stream.get(), &message));
   ^~~~~~~~~~~~~~
message.cpp:93:3: note: suggested alternative:RETURN_NOT_OK’
   R_ERROR_NOT_OK(arrow::ipc::ReadMessage(stream.get(), &message));
   ^~~~~~~~~~~~~~
   RETURN_NOT_OK
/opt/R/lib/R/etc/Makeconf:168: recipe for target 'message.o' failed
make: *** [message.o] Error 1
ERROR: compilation failed for packagearrow* removing/opt/R/lib/R/library/arrow
@wesm
Copy link
Member

wesm commented Nov 10, 2018

Could you provide complete details about how you are building the project? You might want to have a look at #2770 also

@sadatnfs
Copy link
Author

@wesm for the time being, I'm installing the package directly pointing to the Github repo, something like pacman::p_load_gh("apache/arrow/r") would that do.

One thing to note though is that commit d5f7b02fdf3c42663db8db6f385d94c17a865a10 is the last commit where I have been able to bulid the R package successfully, and so I am pinning my install to that commit for now. Looks like the commit right after (happens to be an R commit) causes my installation to fail and cannot find that R_ERROR_NOT_OK pointer (referring to #2915).

@wesm
Copy link
Member

wesm commented Nov 10, 2018

Try the latest master commit. Please post reproducible failure so @romainfrancois or others can take a look

@sadatnfs
Copy link
Author

Can confirm that its working now! Thanks @wesm !

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

No branches or pull requests

2 participants