Merge pull request #364 from epiforecasts/update-to-c++17

Set C++17 as a system requirement

DESCRIPTION

@@ -139,4 +139,5 @@ Roxygen: list(markdown = TRUE)
 RoxygenNote: 7.2.3
 NeedsCompilation: yes
 SystemRequirements: GNU make
+       C++17
 VignetteBuilder: knitr

configure

@@ -1,5 +1,4 @@
-#! /bin/sh
-
 # Generated by rstantools.  Do not edit by hand.
 
-"${R_HOME}/bin/Rscript" -e "rstantools::rstan_config()"
+#! /bin/sh
+"${R_HOME}/bin/Rscript" -e "rstantools::rstan_config()" -e 'writeLines(sub("CXX14$", "CXX17", readLines("src/Makevars")), "src/Makevars")'

configure.win

@@ -1,5 +1,4 @@
-#! /bin/sh
-
 # Generated by rstantools.  Do not edit by hand.
 
-"${R_HOME}/bin${R_ARCH_BIN}/Rscript.exe" -e "rstantools::rstan_config()"
+#! /bin/sh
+"${R_HOME}/bin${R_ARCH_BIN}/Rscript.exe" -e "rstantools::rstan_config()" -e 'writeLines(sub("CXX14$", "CXX17", readLines("src/Makevars.win")), "src/Makevars.win")'

cran-comments.md

@@ -10,4 +10,6 @@
 
 All issues flagged by CRAN have been resolved. In particular the roxygen2 documentation issue that was flagged as the reason for archiving has been resolved by rebuilding the documentation. The change in policy to no longer allow a non-CRAN package as a suggest has also been resolved by removing this functionality.
 
-Additional CRAN comments flagged post archiving (i.e temporary directory usage, no examples for internal functions, and no return values) have been updated as requested where action was required to comply with CRAN policies.
+Additional CRAN comments flagged post archiving (i.e temporary directory usage, no examples for internal functions, and no return values) have been updated as requested where action was required to comply with CRAN policies.
+
+C++17 has been made a system requirement as requested by CRAN. Whilst rstan (on which we depend) currently only supports C++14 the package compiles as expected and passes all tests on C++17.