Skip to content

Commit 78654f1

Browse files
authored
Merge pull request #3329 from BsAtHome/fix_halcommand-wordsplit
Undo quoting of HALCOMMAND
2 parents cf4976f + 7da4f08 commit 78654f1

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

scripts/linuxcnc.in

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -954,7 +954,9 @@ HALCOMMAND=$($INIVAR -ini "$INIFILE" -var HALCMD -sec HAL -num $NUM 2> /dev/null
954954
while [ -n "$HALCOMMAND" ] ; do
955955
if [ -n "$HALCOMMAND" ] ; then
956956
echo "Running HAL command: $HALCOMMAND" >> "$PRINT_FILE"
957-
if ! $HALCMD "$HALCOMMAND" && [ -z "$DASHK" ]; then
957+
# The HALCOMMAND must be word-split
958+
# shellcheck disable=SC2086
959+
if ! $HALCMD $HALCOMMAND && [ -z "$DASHK" ]; then
958960
echo "INI file HAL command $HALCOMMAND failed."
959961
Cleanup
960962
exit 1

0 commit comments

Comments
 (0)