[gelöst] do, done, newline verzweifeln - Bash Script Error

[synapsenstau]

Ich sitze gerade an diesem (38370) Script. Es soll mir Track und Interpret ausgeben, welcher gerade in Vlc gespielt wird. Diese Ausgabe wird dann von einem anderem Programm weiterverwendet.
Immer wenn ich es Testhalber ausführe kommt diese Fehlermeldung:

Code: Alles auswählen

xx@xx:./now_play.sh
./now_play.sh: Zeile 89: Syntaxfehler beim unerwarteten Wort `newline'
./now_play.sh: Zeile 89: `                done'

Ich starre nun schon die ganze Zeit darauf und sehe den Fehler nicht. Das "do...done" sieht aus wie das davor und das funktioniert auch. Ich könnte mal ein paar Hilfsaugen gebrauchen.

[LessWire]

Vielleicht wurde mal mit einem Windows Editor editiert und es hat sich ein CR an der Stelle eingeschlichen? Ich würde mal "dos2unix" über die Datei laufen lassen.

Gruß, LW

[uname]

Kannst evtl. auch einfach deinen eigenen Paste aus NoPaste zurückkopieren. Nutze einen ordentlichen Editor wie z.B. Vim. Im Vim kannst du mal die Option ":set list" probieren.

[CH777]

in Zeile 86 fehlen glaube ich ein $ und ein Leerzeichen.

[synapsenstau]

Vielleicht wurde mal mit einem Windows Editor editiert und es hat sich ein CR an der Stelle eingeschlichen? Ich würde mal "dos2unix" über die Datei laufen lassen.

dos2unix hat nichts geändert, leider. Ich habe das Script aber mit Kate bearbeitet und welches steht "Zeilenende/Unix". Damit sollte das sich schon im vorfeld erledigt haben.

Kannst evtl. auch einfach deinen eigenen Paste aus NoPaste zurückkopieren. Nutze einen ordentlichen Editor wie z.B. Vim. Im Vim kannst du mal die Option ":set list" probieren.

Hat leider auch nicht geholfen das zurückkopieren und mit Vim kann ich mich nicht wirklich anfreunden.

in Zeile 86 fehlen glaube ich ein $ und ein Leerzeichen.

Ja, dem könnte so sein. Habe es mal eingefügt, nur dann sagt er mir, das man die vorangehende for schleife nicht so mit einem do verwenden dürfe. Alles sehr rätselhaft. Das hat man davon wenn man nicht funktionierende example scripte von Programmen zum laufen bringen will. Am besten neuschreiben....

[rendegast]

Code: Alles auswählen

                progressbar="${progressbar}|"
                for((i=1;i<(10-nrate);i++))
                do
                        progressbar="${progressbar}-"
                done

->

Code: Alles auswählen

                progressbar="${progressbar}|"
                for((i=1;i<$((10-nrate));i++))
                do
                        progressbar="${progressbar}-"
                done

oder

Code: Alles auswählen

                progressbar="${progressbar}|"
WERT=$((10-nrate))
                for((i=1;i<WERT;i++))
                do
                        progressbar="${progressbar}-"
                done

(Leerzeichen nach dem for ist schöner, aber nicht nötig)

Warum? k.A.

-----------------------------------------------------------------------
Die Sonderzeichen in den Variablen (Pipe, eckige Klammer) würde ich ersetzen, potentielle Ostereier.

[synapsenstau]

Großen dank dir @rendergast. Dat wars. Es war so klein und unscheinbar das es komplett übersehen habe.

Das mit den Sonderzeichen ist klar, hat ber bis jetzt noch keine Probleme gemacht (was nicht ist, kann ja noch werden). Außerdem sieht es nett aus

Die Ausgabe nun wie gewünscht/gedacht:

Code: Alles auswählen

/me is listening to: Symbolic And Avalon - Modern Age (Modern Age) from 2015 [01:49/08:27] [--|-------] [0 kbps](VLC media player)