nach Wiederintriebnahme "unary operator expected" Was tun???

Moin!

Was steht denn in der Datei an den betreffenden Stellen? Am besten mit ein paar Zeilen drum rum posten oder gleich die ganze Datei, wenn sie nicht zu lang ist.

mini.

Das hatte ich auch, die "==" müssen durch "=" ersetzt werden.
Vermutlich wurde die Shell durch eine leicht inkompatible Version ersetzt.

Zitat

Original von Stalker
Das hatte ich auch, die "==" müssen durch "=" ersetzt werden.

Das ist dann aber eine Zuweisung und keine Vergleich mehr !?
Vielleicht sollte man besser jeweils ein x davorsetzen wie es bei DMA_ONLY_FOR_DISKS gemacht wurde: x$media == "xcdrom"

Moin!

Ja, entweder das mit dem x, also z.B.

if [ x$media == "xdisk" ]; then

oder um die Variable das doppelte Anführungszeichen

if [ "$media" == "disk" ]; then

Stalker
Du solltest dein Script auch anpassen, nur ein Gleichheitszeichen ist definitiv falsch.

Zum Hintergrund:
In /proc/ide gibt's für jedes Laufwerk einen Eintrag, z.B. /proc/ide/hda für die erste Festplatte. Das ist ein Link auf ein Verzeichnis z.B. /proc/ide/ide0/hda, wenn die erste Festplatte am ersten IDE-Controller hängt. Darin gibt es eine Datei "media", die mit dem Typ des Laufwerks gefüllt ist ("disk" bzw. "cdrom").
Wenn es eine Festplatte ist, wird DMA eingeschaltet (execute hdparm...), wenn es ein CD/DVD-Laufwerk ist, wird DMA nur eingeschaltet, falls DMA_ONLY_FOR_DISKS nicht auf "yes" steht (wird vermutlich in /etc/sysconfig o.ä. gesetzt).
Aus irgendeinem Grund ist die "media"-Datei eines deiner IDE-Laufwerke leer (was nicht sein sollte), weshalb der Vergleich stolpert: an der Stelle von $media steht plötzlich nichts und damit fehlt die linke Seite des Vergleichs.

Funktionieren noch alle Platten und Laufwerke? Oder wurde eins ausgebaut? Was gibt "ls -la /proc/ide" aus, vielleicht finden wir ja das schuldige Laufwerk...

mini.

Zitat

Originally posted by mini73
Stalker
Du solltest dein Script auch anpassen, nur ein Gleichheitszeichen ist definitiv falsch.

Reden wir nicht von der Bourne-Shell bzw. einem Nachfolger?
Wichtig sind die Leerzeichen, x="abc" ist eine Zuweisung, $x = "abc" ist ein Vergleich.

Siehe "man sh":
s1 = s2 True if the strings s1 and s2 are identical.

Der Trick mit X$x ist sinnvoll um Syntaxfehler bei undefinierten Variablen und ein Sicherheitsloch zu vermeiden.

Nachtrag:
Es handelt sich um eine Inkompatibilität zwischen bash und dash, die je nach Linux-Distribution und dessen Release als sh verwendet werden. Das wird noch für einigen Spaß sorgen.

Moin!

Stimmt, es gibt ja dupzig verschiedene Shells... Ok, dann müssen wir nur herausfinden, welche steph@vdr benutzt.

sh --help

müsste Aussage genug liefern.

mini.