Skip to content

Commit

Permalink
Now quote arguments with double qoutes by default, i.e. SCRIPT_ARG_FM…
Browse files Browse the repository at this point in the history
…T='"%s"' is the new default
  • Loading branch information
HenrikBengtsson committed Apr 30, 2024
1 parent 09b9ad5 commit 2e9971b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,6 @@ check-api/seguid-r:
$(MAKE) check-api SCRIPT_CALL="Rscript --no-init-file" SCRIPT_PREAMBLE="library(seguid)" SCRIPT_PRINT_FMT="cat(%s)" SCRIPT_ARGS_SEP=", "

check-api/seguid-tcl:
$(MAKE) check-api SCRIPT_CALL="tclsh" SCRIPT_PREAMBLE="source ../src/base64.tcl; source ../src/sha1.tcl; source ../src/seguid.tcl;" SCRIPT_PRINT_FMT="puts stdout [%s]" SCRIPT_ARG_FMT='"%s"' SCRIPT_ARGS_SEP=" " SCRIPT_CALL_FMT="%s %s"
$(MAKE) check-api SCRIPT_CALL="tclsh" SCRIPT_PREAMBLE="source ../src/base64.tcl; source ../src/sha1.tcl; source ../src/seguid.tcl;" SCRIPT_PRINT_FMT="puts stdout [%s]" SCRIPT_ARGS_SEP=" " SCRIPT_CALL_FMT="%s %s"

check-api/ALL: check-api/seguid-python check-api/seguid-r
2 changes: 1 addition & 1 deletion tests-api/utils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ setup_api_test() {
fi
echo "script_args_sep: ${script_args_sep}"

script_arg_fmt="${SCRIPT_ARG_FMT:-'%s'}"
script_arg_fmt="${SCRIPT_ARG_FMT:-\"%s\"}"
echo "script_arg_fmt: ${script_arg_fmt}"
[[ -n ${script_arg_fmt} ]] || fail "SCRIPT_ARG_FMT is empty"

Expand Down

0 comments on commit 2e9971b

Please sign in to comment.