Skip to content
Permalink
Browse files

Fix shellcheck warning

$@ is only supposed to be used as "$@", for all uses within strings, $*
is the correct variable.
  • Loading branch information
neverpanic committed Mar 10, 2018
1 parent f64c0da commit a995cfe9cbb30a234ee79b277596844fdd8a8386
Showing with 1 addition and 1 deletion.
  1. +1 −1 mpbb-install-port
@@ -38,7 +38,7 @@ install-port() {
# $option_prefix and $thisdir are set in mpbb
# shellcheck disable=SC2154
if [[ -f $("${option_prefix}/bin/port-tclsh" "${thisdir}/tools/archive-path.tcl" "$@") ]]; then
echo "$@ already installed, nothing to do"
echo "$* already installed, nothing to do"
# log: summary for the portwatcher
echo "Building '$port' ... [OK]" >> "$log_subports_progress"
echo "$@" >> "${option_work_dir}/all_ports"

0 comments on commit a995cfe

Please sign in to comment.
You can’t perform that action at this time.