SHELLSCRIPT: Syntax error: word unexpected (expecting ")")

[inception]

Hallo,
ich habe eben ein backup shellscript geschrieben und bekomme nach dem ausführen folgende Errormeldung: ./backup.sh: 8: ./backup.sh: Syntax error: word unexpected (expecting ")").

Das script sieht folgendermaßen aus:

Code: Alles auswählen

#!/bin/sh
case "$1" in
jts3)
		cd /home
		tar cpfz /home/backups/jts3/jts3_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz JTS3ServerMod
		;;
		
del jts3)
		cd /home/backups/jts3
		if [ "$(ls /home/backups/jts3/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		;;
		
ts)
		cd /home
		tar cpfz /home/backups/ts3/ts3_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz teamspeak3-server_linux_amd64
		;;
		
del ts)
		fi
		cd /home/backups/ts3
		if [ "$(ls /home/backups/ts3/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		;;
		
mc)
		cd /home
		tar cpfz /home/backups/mc/mc_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz minecraft
		;;
		
del mc)
		cd /home/backups/mc
		if [ "$(ls /home/backups/mc/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		;;
		
sb)
		cd /opt
		tar cpfz /home/backups/sb/sb_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz sinusbot
		;;
		
del sb)		
		cd /home/backups/sb
		if [ "$(ls /home/backups/sb/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		;;
		
all)
		cd /home
		tar cpfz /home/backups/jts3/jts3_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz JTS3ServerMod
		cd /home
		tar cpfz /home/backups/ts3/ts3_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz teamspeak3-server_linux_amd64
		cd /home
		tar cpfz /home/backups/mc/mc_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz minecraft
		cd /opt
		tar cpfz /home/backups/sb/sb_backup_"$(date +%d_%m_%y_%H_%M_%S)".tar.gz sinusbot
		;;
		
del all)
		cd /home/backups/jts3
		if [ "$(ls /home/backups/jts3/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		cd /home/backups/ts3
		if [ "$(ls /home/backups/ts3/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		cd /home/backups/mc
		if [ "$(ls /home/backups/mc/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		cd /home/backups/sb
		if [ "$(ls /home/backups/sb/ | wc -l)" -gt 5 ]; then
				ls -t | sed -e '1, 5d' | xargs -d '\n' rm
		fi
		;;

*)
		echo "Falsche Benutzung, bitte wie folgt verwenden {ts|del ts|jts3|del jts3|mc|del mc|sb|del sb|all|del all}"
		exit 1
		;;
		
esac
exit 0

irgendwie finde ich den fehler nicht -.-' Wenn jemad weiß woran es liegt fällt es mir wahrscheinlich wie Schuppen von den Augen

Mfg,
Daniel

[tobo]

Entweder schreibst du

Code: Alles auswählen

"del jts3")

für einen übergebenen Parameterstring oder

Code: Alles auswählen

del | jts3)

als Auswahl (oder) aus 2 übergebenen Parametern. Wobei "jts3" dann natürlich schon darüber greift.

[inception]

warum ist das so? bei all funktioniert es ohne probleme.

[Meillo]

warum ist das so?

Darum: http://pubs.opengroup.org/onlinepubs/96 ... 8_09_04_05

Oder wie hast du die Frage gemeint?

[tobo]

warum ist das so? bei all funktioniert es ohne probleme.

Gerade mal geschaut - nein tut es nicht!
Du musst die Leerzeichen quoten und dann auch dem Skript als geschlossenen Parameter übergeben. Also alle del XX) zu "del XX") ändern und Aufruf mit script.sh "del XX". In Zeile 21 ist ein fi zuviel.

[inception]

Jap das fi habe ich schon weggemacht jetzt muss ich den Parameter auch mit den Anführungszeichen benutzen. Naja das ist ha halb so schlimm. Danke für die hilfe!

[tobo]

Wenn du nur einen Parameter überprüfst/erwartest, dann könntest du auch im case anstelle "$1" ein "$*" abfragen. Das übergibt dann die komplette Parameterliste als geschlossenen String und du könntest das Skript z.B. mit skript.sh del all aufrufen.

[breakthewall]

irgendwie finde ich den fehler nicht -.-' Wenn jemad weiß woran es liegt fällt es mir wahrscheinlich wie Schuppen von den Augen l

Das ist je nach Coding-Style nicht ungewöhnlich, womit Probleme mal mehr oder mal weniger auffallen.
Zusätzlich würde Ich Dir empfehlen, mal das Programm Shellcheck im Repository anzusehen, was viele gängige Design -und Scripting-Probleme erkennt und entsprechend erklärt warum das so ist.
Kann daher je nach Komplexität eines Shellscriptes nicht schaden, insbesondere für z.B. Anfänger.

Weitere Informationen: https://github.com/koalaman/shellcheck

[seep]

ich habe eben ein backup shellscript geschrieben und bekomme nach dem ausführen folgende Errormeldung: ./backup.sh: 8: ./backup.sh: Syntax error: word unexpected (expecting ")").

Ein "vi +8 backup.sh" hätte Dich übrigens auch gleich direkt in die fehlerhafte Zeile befördert.