Fix shellcheck issues

- Added shebang to all the scripts
- Adjusted all the scripts to make them compatible with a POSIX shell

easytest.sh

@@ -1,12 +1,12 @@
 #!/bin/sh
+
 status="0"
 OUTPUT_FREQ=434.0
 LAST_ITEM="0 Tune"
 do_freq_setup()
 {
 
-FREQ=$(whiptail --inputbox "Choose output Frequency (in MHZ) Default is 434MHZ" 8 78 $OUTPUT_FREQ --title "Rpitx transmit Frequency" 3>&1 1>&2 2>&3)
+if FREQ=$(whiptail --inputbox "Choose output Frequency (in MHZ) Default is 434MHZ" 8 78 $OUTPUT_FREQ --title "Rpitx transmit Frequency" 3>&1 1>&2 2>&3); then
-if [ $? -eq 0 ]; then
     OUTPUT_FREQ=$FREQ
 fi
 
@@ -20,10 +20,10 @@ do_stop_transmit()
 	sudo killall pifmrds 2>/dev/null
 	sudo killall sendiq 2>/dev/null
 	sudo killall pocsag 2>/dev/null
-	sudo killall piopera 2>/dev/null	
+	sudo killall piopera 2>/dev/null
 	sudo killall rpitx 2>/dev/null
 	sudo killall freedv 2>/dev/null
-	sudo killall pisstv 2>/dev/null	
+	sudo killall pisstv 2>/dev/null
 }
 
 do_status()
@@ -36,7 +36,7 @@ do_status()
 
 do_freq_setup
 
- while [ "$status" -eq 0 ] 
+ while [ "$status" -eq 0 ]
     do
 
  menuchoice=$(whiptail --default-item "$LAST_ITEM" --title "Rpitx on ""$OUTPUT_FREQ""MHZ" --menu "Range frequency : 50Khz-1Ghz. Choose your test" 20 82 12 \
@@ -57,10 +57,10 @@ do_freq_setup
 		RET=$?
 		if [ $RET -eq 1 ]; then
     		exit 0
-		elif [ $RET -eq 0 ]; then	
+		elif [ $RET -eq 0 ]; then
 			case "$menuchoice" in
 			F\ *) do_freq_setup ;;
-			0\ *) "./testvfo.sh" "$OUTPUT_FREQ""e6" >/dev/null 2>/dev/null & 
+			0\ *) "./testvfo.sh" "$OUTPUT_FREQ""e6" >/dev/null 2>/dev/null &
 			do_status;;
 			1\ *) "./testchirp.sh" "$OUTPUT_FREQ""e6" >/dev/null 2>/dev/null &
 			do_status;;
@@ -70,7 +70,7 @@ do_freq_setup
 			do_status;;
 			4\ *) "./testfmrds.sh" "$OUTPUT_FREQ" >/dev/null 2>/dev/null &
 			do_status;;
-			5\ *) "./testnfm.sh" "$OUTPUT_FREQ""e3" >/dev/null 2>/dev/null &   
+			5\ *) "./testnfm.sh" "$OUTPUT_FREQ""e3" >/dev/null 2>/dev/null &
 			do_status;;
 			6\ *) "./testssb.sh" "$OUTPUT_FREQ""e6" >/dev/null 2>/dev/null &
 			do_status;;
@@ -90,7 +90,7 @@ do_freq_setup
 			esac
 		else
 			exit 1
-		fi	
+		fi
     done
 	exit 0
 

fm2ssb.sh

@@ -1,3 +1,8 @@
-echo fm2ssb.sh freq gain
+#!/bin/sh
-rtl_fm -f $1 -s 250k -r 48k -g $2 -  | csdr convert_i16_f | csdr fir_interpolate_cc 2| csdr dsb_fc | csdr bandpass_fir_fft_cc 0.002 0.06 0.01 | csdr fastagc_ff | buffer | sudo ./sendiq -i /dev/stdin -s 96000 -f $3 -t float
+
-# | sox -traw -r48k -es -b16 - -c1 -r 48k -traw -
+echo "FREQ_IN=value-in_MHz GAIN=value-0_to_45 FREQ_OUT=value-in_MHz fm2ssb"
+rtl_fm -f "$FREQ_IN" -s 250k -r 48k -g "$GAIN" - | csdr convert_i16_f \
+  | csdr fir_interpolate_cc 2 | csdr dsb_fc \
+  | csdr bandpass_fir_fft_cc 0.002 0.06 0.01 | csdr fastagc_ff | buffer \
+  | sudo ./sendiq -i /dev/stdin -s 96000 -f "FREQ_OUT" -t float
+# | sox -traw -r48k -es -b16 - -c1 -r 48k -traw -

install.sh

@@ -1,3 +1,5 @@
+#!/bin/sh
+
 echo Install rpitx - some package need internet connection -
 
 sudo apt-get update
@@ -8,26 +10,27 @@ sudo apt-get install -y rtl-sdr buffer
 # We use CSDR as a dsp for analogs modes thanks to HA7ILM
 git clone https://github.com/simonyiszk/csdr
 patch -i csdrpizero.diff csdr/Makefile
-cd csdr
+cd csdr || exit
 make && sudo make install
-cd ../
+cd ../ || exit
 
-cd src
+cd src || exit
 git clone https://github.com/F5OEO/librpitx
-cd librpitx/src
+cd librpitx/src || exit
+make
+cd ../../ || exit
 make
-cd ../../
-make 
 sudo make install
-cd ..
+cd .. || exit
 
-echo "\n\n"
+printf "\n\n"
-read -p "In order to run properly, rpitx need to modify /boot/config.txt. Are you sure (y/n) " CONT
+printf "In order to run properly, rpitx need to modify /boot/config.txt. Are you sure (y/n) "
+read -r CONT
 
 if [ "$CONT" = "y" ]; then
   echo "Set GPU to 250Mhz in order to be stable"
    LINE='gpu_freq=250'
-   FILE='/boot/config.txt' 
+   FILE='/boot/config.txt'
    grep -qF "$LINE" "$FILE"  || echo "$LINE" | sudo tee --append "$FILE"
    echo "Installation completed !"
 else

rtlmenu.sh

@@ -1,4 +1,5 @@
-#!/bin/bash
+#!/bin/sh
+
 status="0"
 INPUT_RTLSDR=434.0
 INPUT_GAIN=35
@@ -8,18 +9,15 @@ LAST_ITEM="0 Record"
 do_freq_setup()
 {
 
-FREQ=$(whiptail --inputbox "Choose input Frequency (in MHZ) Default is 434MHZ" 8 78 $INPUT_RTLSDR --title "RTL-SDR Receive Frequency" 3>&1 1>&2 2>&3)
+if FREQ=$(whiptail --inputbox "Choose input Frequency (in MHZ) Default is 434MHZ" 8 78 $INPUT_RTLSDR --title "RTL-SDR Receive Frequency" 3>&1 1>&2 2>&3); then
-if [ $? -eq 0 ]; then
     INPUT_RTLSDR=$FREQ
 fi
 
-GAIN=$(whiptail --inputbox "Choose input Gain (0(AGC) or 1-45)" 8 78 $INPUT_GAIN --title "RTL-SDR Receive Frequency" 3>&1 1>&2 2>&3)
+if GAIN=$(whiptail --inputbox "Choose input Gain (0(AGC) or 1-45)" 8 78 $INPUT_GAIN --title "RTL-SDR Receive Frequency" 3>&1 1>&2 2>&3); then
-if [ $? -eq 0 ]; then
     INPUT_GAIN=$GAIN
 fi
 
-FREQ=$(whiptail --inputbox "Choose output Frequency (in MHZ) Default is 434MHZ" 8 78 $OUTPUT_FREQ --title "Transmit Frequency" 3>&1 1>&2 2>&3)
+if FREQ=$(whiptail --inputbox "Choose output Frequency (in MHZ) Default is 434MHZ" 8 78 $OUTPUT_FREQ --title "Transmit Frequency" 3>&1 1>&2 2>&3); then
-if [ $? -eq 0 ]; then
     OUTPUT_FREQ=$FREQ
 fi
 
@@ -28,8 +26,8 @@ fi
 do_stop()
 {
 	sudo killall rtl_sdr 2>/dev/null
-	sudo killall sendiq 2>/dev/null 
+	sudo killall sendiq 2>/dev/null
-	sudo killall rtl_fm 2>/dev/null 
+	sudo killall rtl_fm 2>/dev/null
 }
 do_status()
 {
@@ -40,7 +38,7 @@ do_status()
 
 do_freq_setup
 
- while [ "$status" -eq 0 ] 
+ while [ "$status" -eq 0 ]
     do
 
  menuchoice=$(whiptail --default-item "$LAST_ITEM" --title "Rpitx with RTLSDR" --menu "Choose your test" 20 82 12 \
@@ -54,18 +52,16 @@ do_freq_setup
         case "$menuchoice" in
 		0\ *) rtl_sdr -s 250000 -g "$INPUT_GAIN" -f "$INPUT_RTLSDR"e6 record.iq >/dev/null 2>/dev/null &
 		do_status;;
-	    1\ *) sudo ./sendiq -s 250000 -f "$OUTPUT_FREQ"e6 -t u8 -i record.iq >/dev/null 2>/dev/null &
+		1\ *) sudo ./sendiq -s 250000 -f "$OUTPUT_FREQ"e6 -t u8 -i record.iq >/dev/null 2>/dev/null &
 		do_status;;
-		2\ *) source $"$PWD/transponder.sh" "$INPUT_RTLSDR"M $INPUT_GAIN $OUTPUT_FREQ"e6" >/dev/null 2>/dev/null &
+		2\ *) FREQ_IN="$INPUT_RTLSDR"M GAIN="$INPUT_GAIN" FREQ_OUT="$OUTPUT_FREQ"e6 . "$PWD/transponder.sh" >/dev/null 2>/dev/null &
 		do_status;;
-		3\ *) source $"$PWD/fm2ssb.sh" "$INPUT_RTLSDR"M $INPUT_GAIN $OUTPUT_FREQ"e6" >/dev/null 2>/dev/null &
+		3\ *) FREQ_IN="$INPUT_RTLSDR"M GAIN="$INPUT_GAIN" FREQ_OUT="$OUTPUT_FREQ"e6 . "$PWD/fm2ssb.sh" >/dev/null 2>/dev/null &
 		do_status;;
 		4\ *)
 		do_freq_setup;;
 		*)	 status=1
-		whiptail --title "Bye bye" --msgbox "Thx for using rpitx" 8 78
+		whiptail --title "Bye bye" --msgbox "Thanks for using rpitx!" 8 78
 		;;
         esac
-        
     done
-

snap2spectrum.sh

@@ -1,9 +1,12 @@
+#!/bin/sh
+
 raspistill -w 320 -h 256 -o picture.jpg -t 1
 #convert  picture.jpg -flip -colors 16 -colorspace gray -dither -colorspace YUV picture.yuv
 #convert picture.jpg -flip -colors 16 -colorspace gray -colorspace YUV picture.yuv
 #convert BBC-Test-Card-F320x256.jpg -flip -quantize YUV -dither FloydSteinberg -colors 4 -interlace partition picture.yuv
-convert picture.jpg -flip -quantize YUV -dither FloydSteinberg -colors 4 -interlace partition picture.yuv
+convert picture.jpg -flip -quantize YUV -dither FloydSteinberg -colors 4 \
-sudo ./spectrumpaint picture.Y $1 100000
+  -interlace partition picture.yuv
+sudo ./spectrumpaint picture.Y "$1" 100000
 
 #convert -depth 8 picture.jpg picture.rgb
 

snapsstv.sh

@@ -1,6 +1,8 @@
+#!/bin/sh
+
 raspistill -w 320 -h 256 -o picture.jpg -t 1
 convert -depth 8 picture.jpg picture.rgb
 
-sudo ./pisstv picture.rgb $1
+sudo ./pisstv picture.rgb "$1"
 
 

sv1afnfilter.sh

@@ -1,14 +1,16 @@
-#COMMAND A BANDPATH FILTER FROM SV1AFN 
+#!/bin/sh
+
+#COMMAND A BANDPATH FILTER FROM SV1AFN
 
 #GPIO Declaration
 if [ ! -f /sys/class/gpio/gpio26 ]; then
 echo "Create GPIOs"
 
 echo GPIO declaration
-echo 26 > /sys/class/gpio/export 
+echo 26 > /sys/class/gpio/export
-echo 19 > /sys/class/gpio/export 
+echo 19 > /sys/class/gpio/export
-echo 13 > /sys/class/gpio/export 
+echo 13 > /sys/class/gpio/export
-echo 6  > /sys/class/gpio/export 
+echo 6  > /sys/class/gpio/export
 
 
 #GPIO out
@@ -24,8 +26,7 @@ fi
 #Initialization : All at zero (disable filter)
 reset_all()
 {
-	
+	echo 0 > /sys/class/gpio/gpio26/value
-	echo 0 > /sys/class/gpio/gpio26/value 
 	echo 0 > /sys/class/gpio/gpio19/value
 	echo 0 > /sys/class/gpio/gpio13/value
 	echo 0 > /sys/class/gpio/gpio6/value
@@ -36,13 +37,13 @@ case "$1" in
 "10m")
 	echo 1 > /sys/class/gpio/gpio26/value
         ;;
-"15m") 
+"15m")
     echo 1 > /sys/class/gpio/gpio19/value
     ;;
-"20m") 
+"20m")
     echo 1 > /sys/class/gpio/gpio13/value
     ;;
-"40m") 
+"40m")
     echo 1 > /sys/class/gpio/gpio6/value
     ;;
 

testam.sh

@@ -1,3 +1,7 @@
+#!/bin/sh
+
 #TODO using the AM mode from librpitx
 echo Need to implement
-(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f | csdr gain_ff 4.0 | csdr dsb_fc | sudo ./rpitx -i - -m IQFLOAT -f $1 -s 48000
+(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f \
+  | csdr gain_ff 4.0 | csdr dsb_fc \
+  | sudo ./rpitx -i - -m IQFLOAT -f "$1" -s 48000

testchirp.sh

@@ -1,2 +1,4 @@
-sudo ./pichirp $1 100000 5
+#!/bin/sh
+
+sudo ./pichirp "$1" 100000 5
 

testfmrds.sh

@@ -1 +1,3 @@
-sudo ./pifmrds -freq $1 -audio src/pifmrds/stereo_44100.wav 
+#!/bin/sh
+
+sudo ./pifmrds -freq "$1" -audio src/pifmrds/stereo_44100.wav

testfreedv.sh

@@ -1,2 +1,3 @@
-sudo ./freedv src/freedv/VCO800XA.rf $1 400
+#!/bin/sh
 
+sudo ./freedv src/freedv/VCO800XA.rf "$1" 400

testfsq.sh

@@ -1 +1,3 @@
-sudo ./pifsq "Test" $1
+#!/bin/sh
+
+sudo ./pifsq "Test" "$1"

testnfm.sh

@@ -1,6 +1,10 @@
+#!/bin/sh
+
 #This is only a Narraw Band FM modulator, for FM broadcast modulation , use PiFMRDS
 # Need to use a direct FM modulation with librpitx and not using IQ : TODO
-echo "If you need to test broadcast FM , use PiFMRDS"
+echo "If you need to test broadcast FM, use PiFMRDS"
 #(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f | csdr gain_ff 2500 | sudo ./sendiq -i /dev/stdin -s 24000 -f 434e6 -t float 1
-(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f | csdr gain_ff 7000 | csdr convert_f_samplerf 20833 | sudo ./rpitx -i- -m RF -f $1
+(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f \
+  | csdr gain_ff 7000 | csdr convert_f_samplerf 20833 \
+  | sudo ./rpitx -i- -m RF -f "$1"
 

testopera.sh

@@ -1,3 +1,5 @@
-sudo ./piopera F5OEO 0.5 $1
+#!/bin/sh
+
+sudo ./piopera F5OEO 0.5 "$1"
 
 

testpocsag.sh

@@ -1 +1,3 @@
-echo -e "1:YOURCALL\n2: Hello world" | sudo ./pocsag -f $1
+#!/bin/sh
+
+printf "1:YOURCALL\n2: Hello world" | sudo ./pocsag -f "$1"

testspectrum.sh

@@ -1,2 +1,5 @@
-convert BBC.jpg -flip -quantize YUV -dither FloydSteinberg -colors 4 -interlace partition picture.yuv
+#!/bin/sh
-sudo ./spectrumpaint picture.Y $1 100000
+
+convert BBC.jpg -flip -quantize YUV -dither FloydSteinberg -colors 4 \
+  -interlace partition picture.yuv
+sudo ./spectrumpaint picture.Y "$1" 100000

testssb.sh

@@ -1,2 +1,7 @@
-(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f | csdr fir_interpolate_cc 2| csdr dsb_fc | csdr bandpass_fir_fft_cc 0.002 0.06 0.01 | csdr fastagc_ff | sudo ./sendiq -i /dev/stdin -s 96000 -f $1 -t float
+#!/bin/sh
+
+(while true; do cat sampleaudio.wav; done) | csdr convert_i16_f \
+  | csdr fir_interpolate_cc 2 | csdr dsb_fc \
+  | csdr bandpass_fir_fft_cc 0.002 0.06 0.01 | csdr fastagc_ff \
+  | sudo ./sendiq -i /dev/stdin -s 96000 -f "$1" -t float
 

testsstv.sh

@@ -1,2 +1,4 @@
+#!/bin/sh
+
 convert -depth 8 BBC.jpg picture.rgb
-sudo ./pisstv picture.rgb $1
+sudo ./pisstv picture.rgb "$1"

testvfo.sh

@@ -1 +1,3 @@
-sudo tune -f $1
+#!/bin/sh
+
+sudo tune -f "$1"

transponder.sh

@@ -1,4 +1,6 @@
-#You need a rtl-sdr dongle in order to run this
+#!/bin/sh
-echo "transponder FreqIn(Mhz) Gain(0-45)"
-rtl_sdr -s 250000 -g "$2" -f "$1" - | buffer | sudo ./sendiq -s 250000 -f $3 -t u8 -i -
 
+#You need a rtl-sdr dongle in order to run this
+echo "FREQ_IN=value-in_MHz GAIN=value-0_to_45 FREQ_OUT=value-in_MHz transponder"
+rtl_sdr -s 250000 -g "$GAIN" -f "$FREQ_IN" - | buffer \
+  | sudo ./sendiq -s 250000 -f "FREQ_OUT" -t u8 -i -