-
Notifications
You must be signed in to change notification settings - Fork 56
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
Build Error #163
Comments
Which package? RcppArmadillo? Can bisect to a working commit? Do you have access to another machine besides macOS? |
What is the entire error message that you're seeing? Are you seeing, for example, something like:
I think this is because RcppArmadillo has an entry that confuses the new version of fastLmPure <- function(X, y) {
stopifnot(is.matrix(X), is.numeric(y), nrow(y)==nrow(X))
.Call( "RcppArmadillo_fastLm", X, y, PACKAGE = "RcppArmadillo" )
} The R native routine skeleton sees that, and through that
However, the function is now exported with the symbol name tl;dr: this line:
should become
for compatibility with the newer version of Rcpp. As an aside, it looks like
but it looks like the version defined in |
Yes, the same error message. I test it on Windows and the problem is the same. And every time I press
which is consistent with what you mention. |
Uh-oh. I see now. Unbelievable that did not hit us earlier. What Kevin is of course correct, but there is an additional riddle. I had something similar moons ago in RProtoBuf. We have (had) symbols |
With that the call becomes a newer-school .Call(`_RcppArmadillo_fastLm_cpp`, X, y) without the |
With a little delay due to the Saturday morning neighbourhood run, try this branch. I renamed one of the two |
It works. @eddelbuettel @kevinushey Thanks for your time and quick remedy. Still not quite familiar with how the whole package magically runs, so I should spend time doing some research on it. Anyway, thanks for all your effort helping me. |
That was a weird one pushed down on part by R 3.4.* (wanting registration of C-level routines, optionally with Really surprised it did not bite any one of us til now. Oh well. Two pending PRs, plus change to |
It'll get closed automatically once the PR is merged. |
Looked like I was wrong w.r.t. squashed merged. Learn something new. PR #164 merged. |
Hi Dirk, I use RcppArmadillo for one my packages. Travis builds used to work before, but now RcppArmadillo fails with a similar error as this one. Here is the log: I see the current github version is Is this indeed the problem that is causing my travis to fail? Any suggestion how to overcome it for now? |
@dorianps: Please edit the question to make it useable. Random links to random sites, AWS or not, don't count. Note that we run reverse depends against all CRAN packages depending on RcppArmadillo and found no issues. |
Hi @eddelbuettel , Sorry, I thought the link was open for all. Here is the error:
Here is the full travis log: |
So, it looks like the compile is failing because you are missing Try: sudo apt-get install libiomp5 However, this might also be because your c.f. https://stackoverflow.com/questions/33357029/using-openmp-with-clang |
Thank's @coatless Here is the most recent travis log: In searching google for the error Yet, I am not so knowledgeable to understand what is wrong with openmp. |
Well, I have obviously been using Rcpp since day one, and I have always been on Debian or Ubuntu. This never ever failed me for me. There is something "unusual" or hosed about your system. Not an RcppArmadillo. Also, note that every Travis build is a full log of what works on a minimal Ubuntu 14.04 system. See the build logs here. |
There is nothing wrong with libomp. It should always be installed as |
I managed to fix the error by installing RcppArmadillo the good old way with
Since my problem is resolved, the issue can be closed. Thank you all. |
There is nothing to be closed as it was alreday close but I am glad you took care of that self-inflicted wound. |
@chitrak-banerjee Why are you adding this to a complete unrelated question? This makes no sense. Please delete and open a fresh one with something reproducible. |
@eddelbuettel , I've met the same problem. I solved this by updating libstdc++.so.6 to libstdc++.so.6.0.28. It requires GLIBCXX_3.4.20 or later. |
I am locking this now. We are not here to support outdated build systems. The rule is: if you OS versions and / or compilers are years behind what CRAN uses, we cordially invite you to stick to package versions corresponding to when your tools were current and allow us to do our volunteer work on the current versions. You are welcome to update too. |
I try to build and check this package locally in RStudio and an error comes out:
Error: package or namespace load failed for ‘RcppArmadillo’ in dyn.load(file, DLLpath = DLLpath, ...):
. Another warning message is"WARNING: support for OpenMP requires C++11/C++14; add -std=c++11 or -std=c++14 to compiler flags"
. I google it and there is someone meeting the same problem during installation, but I don't know where to fix yet.When I check files, there are two files
R/RcppExports.R
andsrc/RcppExports.cpp
possibly modified by RStudioBuild
and a new fileinst/include/RcppArmadilloLapack.h
. I have no idea whether they are to do with the error. Could anyone do me a favor?The text was updated successfully, but these errors were encountered: