Skip to content

Commit

Permalink
Merge pull request #49 from asmacdo/no-seteu-in-make
Browse files Browse the repository at this point in the history
No -eu needed in Make target
  • Loading branch information
TheChymera authored Oct 5, 2023
2 parents 2f69935 + ae182c6 commit a4cc508
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -57,16 +57,7 @@ analysis-reproman:
--jp walltime=120:00:00 \
make analysis-singularity

# These don't work:
#SCRATCH_PATH?=/dartfs-hpc/scratch/$(USER)
#$(eval if [[ -z "$SCRATCH_PATH" ]]; then SCRATCH_PATH="/home/user/$(USER)/.scratch"; fi)
#$(eval SCRATCH_PATH?="/home/user/$(USER)/.scratch")
#$(eval SCRATCH_PATH ?= "/home/user/$(USER)/.scratch")
#SCRATCH_PATH ?= "/home/user/$(USER)/.scratch"
#SCRATCH_PATH = $(if $(SCRATCH_PATH),$(SCRATCH_PATH),"/home/user/$(USER)/.scratch")

analysis-singularity:
set -eu
$(if $(USER),,$(error USER is not defined and currently required for the SCRATCH_PATH variable))
@echo "Selected \`SCRATCH_PATH\` $(SCRATCH_PATH). You may want to customize this by setting it on the command line, after the make command, i.e. \`make SCRATCH_PATH=... target\`."
mkdir -p $(SCRATCH_PATH)
Expand Down

0 comments on commit a4cc508

Please sign in to comment.