No longer set CXX_STD to C++14

eddelbuettel · Nov 29, 2023 · e852651 · e852651
1 parent 6246503
commit e852651
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -1,9 +1,11 @@
-2023-11-28  Dirk Eddelbuettel  <[email protected]>
+2023-11-29  Dirk Eddelbuettel  <[email protected]>
 
 	* src/RcppExports.cpp: Regenerated under updated Rcpp to address
 	format string issue reported by R-devel
 	* R/RcppExports.R: Idem
 
+	* src/Makevars: No longer set CXX_STD to C++14
+
 	* .github/workflows/ci.yaml (jobs): Update to actions/checkout@v4
 
 2022-11-18  Dirk Eddelbuettel  <[email protected]>

diff --git a/src/Makevars b/src/Makevars
@@ -1,6 +1,7 @@
 
-## This is a C++14 package
-CXX_STD = CXX14
+## This is a C++14 package -- but R 4.2.0 defaults to C++14 so only uncomment this if needed
+## on an R version older that 4.2.0 in which case the explicit setting will help
+#CXX_STD = CXX14
 
 ## We need the headers in inst/include
 ## This one helps with Boost noise but is not allowed by CRAN as 'not portable': -Wno-parentheses