Skip to content

Commit

Permalink
commenting out CXX ... = 11 since this was giving a CRAN comment.
Browse files Browse the repository at this point in the history
  • Loading branch information
jamesmurray7 committed Mar 13, 2023
1 parent b6b4c55 commit f3ea2e1
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,5 @@ i.e. gmvjoint.so has size 8.3Mb.
R-hub build OK.
Note upon installation one may be warned about inconsistencies across TMB/Matrix versions, but this
is not within my control.
Some warning about C++11 flags also, but this comes from RcppArmadillo, and not sure I can relax this.
Commented warnings about CXX_STD=CXX11 in Makevars(/.win) which resulted in a NOTE. Hopefully this doesn't result
in deterioration of performance since I thought RcppArmadillo required this!
2 changes: 1 addition & 1 deletion src/Makevars
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
##
## And with R 3.4.0, and RcppArmadillo 0.7.960.*, we turn C++11 on as OpenMP
## support within Armadillo prefers / requires it
CXX_STD = CXX11
##CXX_STD = CXX11

PKG_CXXFLAGS = $(SHLIB_OPENMP_CXXFLAGS)
PKG_LIBS = $(SHLIB_OPENMP_CXXFLAGS) $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)
2 changes: 1 addition & 1 deletion src/Makevars.win
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
##
## And with R 3.4.0, and RcppArmadillo 0.7.960.*, we turn C++11 on as OpenMP
## support within Armadillo prefers / requires it
CXX_STD = CXX11
##CXX_STD = CXX11

PKG_CXXFLAGS = $(SHLIB_OPENMP_CXXFLAGS)
PKG_LIBS = $(SHLIB_OPENMP_CXXFLAGS) $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)

0 comments on commit f3ea2e1

Please sign in to comment.