The fix I was proposing actually has implications much deeper than I thought. I 
would need to investigate further and will take no action at this time.

On Jun 6, 2014, at 10:47 PM, Dirk Eddelbuettel <e...@debian.org> wrote:

> 
> On 6 June 2014 at 22:16, Dmitry Nesterov wrote:
> | It was even simpler, I just ran install.package(“Rcpp") in R and it pulled 
> in all the headers…
> 
> Which version?  Was that recently?
> 
> | I have found the repo, will do the change and then the pull request,
> | I assume the regression tests are done on your side?
> 
> The tests are part of the sources, which is common for R package. 
> 
> In general, you want to run 'R CMD check nameOfThePackage/'. In this
> particular case you also want to set a variable 'RunAllRcppTests' to 'yes' eg
> 
>     RunAllRcppTests='yes' R CMD check rcpp
> 
> should do. 
> 
> Dirk
> 
> -- 
> http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org

_______________________________________________
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