From c2412542060226e4fffcdc0e8e2ed12a13923478 Mon Sep 17 00:00:00 2001 From: Joris Goosen Date: Wed, 29 Nov 2023 11:56:54 +0100 Subject: [PATCH] always install matrix when installing jaspBase (#5326) (#5332) * always install matrix when installing jaspBase (#5326) * install Matix from source on macos, requires gfortran to be installed on the local system --------- Co-authored-by: RensDofferhoff <20978635+RensDofferhoff@users.noreply.github.com> --- Modules/install-jaspBase.R.in | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/Modules/install-jaspBase.R.in b/Modules/install-jaspBase.R.in index 79f5780c4a..6eeee30924 100644 --- a/Modules/install-jaspBase.R.in +++ b/Modules/install-jaspBase.R.in @@ -47,15 +47,20 @@ if (md5SumsChanged(modulePkg, moduleLibrary)) { } +#This is necessary because of conflicts with Matrix dep of base R lib. +#Can be removed when a new version of R updates its Matrix +if (!@IS_FLATPAK_USED@) { + + if (Sys.info()["sysname"] == "Darwin") { + options(pkgType = "source") + } + + renv::install("Matrix", library = moduleLibrary, prompt = prompt) +} + # Converting the absolute symlinks to relative symlinks on macOS # Todo, I can do this using CMake like I do on Windows if (Sys.info()["sysname"] == "Darwin") { source('@MODULES_BINARY_PATH@/symlinkTools.R') convertAbsoluteSymlinksToRelative('@R_LIBRARY_PATH@', '@MODULES_RENV_CACHE_PATH@') } - -#This is necessary because of conflicts with Matrix dep of base R lib. -#Can be removed when a new version of R updates its Matrix -if (@IS_LINUX_LOCAL_BUILD@) { - renv::install("Matrix", type="source") -}