Fix the sandbox scripts in absence of the OPAMROOT variable

ocaml · Sep 11, 2023 · fbff854 · fbff854
1 parent ef3e65f
commit fbff854
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/state/shellscripts/bwrap.sh b/src/state/shellscripts/bwrap.sh
@@ -101,7 +101,7 @@ add_dune_cache_mount() {
 
 # In case OPAMROOT happens to be in one of the writeable directories we
 # need to make sure it is read-only
-if [ -n ${OPAMROOT+x} ]; then
+if [ -n "${OPAMROOT:-}" ]; then
   add_mounts ro "$OPAMROOT"
 fi
 

diff --git a/src/state/shellscripts/sandbox_exec.sh b/src/state/shellscripts/sandbox_exec.sh
@@ -58,7 +58,7 @@ add_dune_cache_mount() {
 
 # In case OPAMROOT happens to be in one of the writeable directories we
 # need to make sure it is read-only
-if [ -n ${OPAMROOT+x} ]; then
+if [ -n "${OPAMROOT:-}" ]; then
   add_mounts ro "$OPAMROOT"
 fi