ensure symlinks used when TBB_ROOT set

Author: kevinushey

NEWS.md

@@ -1,9 +1,12 @@
 
 ## RcppParallel 5.1.5 (UNRELEASED)
 
+* Fixed an issue where setting `TBB_ROOT` (or `TBB_INC` / `TBB_LIB`) would
+  copy rather than symlink the associated libraries. (#161)
+
 ## RcppParallel 5.1.4
 
-- Fixed an issue causing client packages of RcppParallel to fail to compile
+* Fixed an issue causing client packages of RcppParallel to fail to compile
   on Solaris.
 
 ## RcppParallel 5.1.3

R/RcppParallel-package.R

@@ -14,7 +14,7 @@
 #' 
 #' For additional documentation, see the package website at:
 #' 
-#' <https://rcppcore.github.io/RcppParallel>
+#' <https://rcppcore.github.io/RcppParallel/>
 #' 
 #' 
 #' @name RcppParallel-package

src/install.libs.R

@@ -17,9 +17,9 @@
    dir.create(libsDest, recursive = TRUE, showWarnings = FALSE)
    file.copy(files, libsDest, overwrite = TRUE)
 
-   # copy tbb
-   # TODO: use 'dest' library directory once rstan is updated
-   tbbDest <- paste0("../inst/lib", R_ARCH)
+   # copy tbb (NOTE: do not use inst/ folder as R will resolve symlinks,
+   # behavior which we do _not_ want here!)
+   tbbDest <- file.path(R_PACKAGE_DIR, paste0("lib", R_ARCH))
    dir.create(tbbDest, recursive = TRUE, showWarnings = FALSE)
 
    # check for bundled vs. system tbb