Do not set different cache directories per job (#4460)

* Do not set different cache directories per job

This should not be necessary as the disk caches are now safer in how
they handle race conditions. Using a unified cache directory should
improve performance.

Author: connorjward

scripts/firedrake-run-split-tests

@@ -31,10 +31,6 @@ num_procs=$1
 num_jobs=$2
 extra_args=${@:3}
 
-# Set per-job cache directories to avoid race conditions when multiple jobs attempt to compile the same form.
-cache_cmd="PYOP2_CACHE_DIR=\$VIRTUAL_ENV/.cache/pyop2/job{#} \
-           FIREDRAKE_TSFC_KERNEL_CACHE_DIR=\$VIRTUAL_ENV/.cache/tsfc/job{#}"
-
 if [ $num_procs = 1 ]; then
     # Cannot use mpiexec -n 1 because this can sometimes hang with
     # OpenMPI at MPI_Finalize
@@ -57,7 +53,7 @@ set -x
 # * Uses tee to pipe stdout+stderr to both stdout and a log file
 # * Writes pytest's exit code to a file called jobN.errcode (for later inspection)
 parallel --line-buffer --tag \
-    "${cache_cmd}; ${pytest_cmd} |& tee ${log_file_prefix}{#}.log; \
+    "${pytest_cmd} |& tee ${log_file_prefix}{#}.log; \
     echo \${PIPESTATUS[0]} > job{#}.errcode" \
     ::: $(seq ${num_jobs})