On 7 May 2014 at 10:08, Martyn Plummer wrote:
| On Tue, 2014-05-06 at 22:50 -0700, Kevin Ushey wrote:
| > Looks like it was a bug on our end -- R-exts specifies that USE_CXX1X
| > should be set to any value; we try to set it to nothing (ie, define it
| > but leave it empty) but apparently that is not accepted.
| 
| Yes it is. We use Sys.getenv("USE_CXX1X", NA) to distinguish the case
| where USE_CXX1X is set to an empty string (as you did) from the case
| where it is unset. So I don't know why this isn't working for you.
| 
| The relevant code is src/library/tools/R/install.R.

Indeed. 

I figured this out tpp after after JJ's better patch replaced my first patch
which also set a value. The construct used here is elegant (but I still find
the entire issue bizarre as one could as easily have required that USE_CXX1X
be set to yes, or 1, or ...   This paragraph is still hard to parse IMHO.

Dirk

-- 
Dirk Eddelbuettel | e...@debian.org | http://dirk.eddelbuettel.com
_______________________________________________
Rcpp-devel mailing list
Rcpp-devel@lists.r-forge.r-project.org
https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel

Reply via email to