fix issue with registration of 'stale' symbols (#939)

Author: kevinushey

ChangeLog

@@ -1,3 +1,8 @@
+2019-02-11  Kevin Ushey  <[email protected]>
+
+	* R/Attributes.R: better handle pre-existing RcppExports.cpp,
+	RcppExports.R when calling compileAttributes()
+
 2018-12-26  Zhuoer Dong  <[email protected]>
 
 	* vignettes/Rcpp-quickref.Rmd: Fix three bugs: use `Named()`, define

R/Attributes.R

@@ -421,9 +421,15 @@ compileAttributes <- function(pkgdir = ".", verbose = getOption("verbose")) {
     if (!file.exists(rDir))
         dir.create(rDir)
 
+    # remove the old RcppExports.R file
+    unlink(file.path(rDir, "RcppExports.R"))
+
     # get a list of all source files
     cppFiles <- list.files(srcDir, pattern = "\\.((c(c|pp)?)|(h(pp)?))$", ignore.case = TRUE)
 
+    # don't include RcppExports.cpp
+    cppFiles <- setdiff(cppFiles, "RcppExports.cpp")
+
     # locale independent sort for stable output
     locale <- Sys.getlocale(category = "LC_COLLATE")
     Sys.setlocale(category = "LC_COLLATE", locale = "C")

inst/NEWS.Rd

@@ -15,6 +15,9 @@
     \item Changes in Rcpp Attributes:
     \itemize{
       \item A new plugin was added for C++20 (Dirk in \ghpr{927})
+      \item Fixed an issue where 'stale' symbols could become registered in
+      RcppExports.cpp, leading to linker errors and other related issues.
+      (\ghit{733}; \ghit{934})
     }
     \item Changes in Rcpp Documentation:
     \itemize{