v4l2-settings-restore.sh: don't use "echo -n" master
authorAntonio Ospite <ao2@ao2.it>
Sat, 24 Nov 2018 18:22:04 +0000 (19:22 +0100)
committerAntonio Ospite <ao2@ao2.it>
Sat, 24 Nov 2018 21:48:53 +0000 (22:48 +0100)
Dont' use "echo -n" as it's not POSIX compliant, this fixes two
shellcheck warnings:

In v4l2-settings-restore.sh line 27:
    CTRL=$(echo -n "$setting" | cut -d ' ' -f 1)
                ^-- SC2039: In POSIX sh, echo flags are undefined.

In v4l2-settings-restore.sh line 28:
    VAL=$(echo -n "$setting" | cut -d ' ' -f 2)
               ^-- SC2039: In POSIX sh, echo flags are undefined.

v4l2-settings-restore.sh

index b255da8..cb6b572 100755 (executable)
@@ -24,8 +24,8 @@ if [ "$ACTION" = "add" ] && [ -f "$SETTINGS_FILE" ];
 then
   while read -r setting;
   do
-    CTRL=$(echo -n "$setting" | cut -d ' ' -f 1)
-    VAL=$(echo -n "$setting" | cut -d ' ' -f 2)
+    CTRL=$(echo "$setting" | tr -d '\n' | cut -d ' ' -f 1)
+    VAL=$(echo "$setting" | tr -d '\n' | cut -d ' ' -f 2)
     v4l2-ctl -d "$DEVNAME" --set-ctrl "${CTRL}=${VAL}"
   done < "$SETTINGS_FILE"
 fi