[Linux] Script op dreambox - syntax error

[Simon VDS]

Hallo,

Ik probeer een script te draaien op mijn Dreambox 800HD dat automatisch een mailtje zo moeten sturen als mijn harde schijf 90% vol is maar krijg volgende melding:

root@sardonis:/etc# diskAlert2.sh: line 10: syntax error: unexpected "done" (expecting "then")

code:

#!/bin/sh
df -H | grep -vE '^Filesystem|tmpfs|cdrom|none|root' | awk '{ print $5 " " $1 }' | while read output;
do
  echo $output
  usep=$(echo $output | awk '{ print $1}' | cut -d'%' -f1  )
  partition=$(echo $output | awk '{ print $2 }' )
  if [ $usep -ge 90 ]; then
    send-mail -s "WARNING! Disk space of DM800 has reached $usep%!" -f [e-mail] -b /etc/mail.txt -S relay.dommel.be -P 25 [e-mail]
  fi
done

Ik zie niet direct waar ik iets verkeerd doe?

[ Voor 3% gewijzigd door Simon VDS op 07-02-2011 12:37 ]

[Onno]

Gokje: DOS newlines.

PS: Gebruik [code] of [cmd], dat is een stuk leesbaarder.

[Simon VDS]

DOS newlines zegt me niks..

Wat is dat voor een beest?

[Manuel]

Sardonis schreef op maandag 07 februari 2011 @ 12:37:
DOS newlines zegt me niks..

Wat is dat voor een beest?

Zie ook: Wikipedia: Newline

[Simon VDS]

Mja ik kan ook wel googelen maar ik snap niet goed waarom de shell niet met deze code overweg kan..
Op ubuntu draait dit perfect maar op busybox niet

Formatting lijkt me toch juist?

[ Voor 15% gewijzigd door Simon VDS op 07-02-2011 12:49 ]

[igmar]

Omdat busybox geen (volledige) shell is waarschijnlijk

[Kees]

probeer sowieso eens het met '/bin/bash' ipv /bin/sh' want veel distro's gebruiken tegenwoordig dash, wat een verschikkelijke shell is als standaard 'sh'.

Verder, controleer op juiste lineendings, met bv 'file'; maar ook mcedit zie je ze wel.

[Onno]

Het aardige van geen bash als /bin/sh gebruiken is dat scripts die bash-extensies gebruiken niet langer werken met /bin/sh als interpreter, waardoor men niet langer per ongeluk bash-scripts kan shippen als sh-scripts. (iets wat helaas in veel Linux-software gebeurt, en slecht voor de portabiliteit daarvan is)

Ik kan me overigens niet voorstellen dat busybox en dash deze code niet begrijpen, ik zie zo niets wat geen standaard sh is.

Sardonis schreef op maandag 07 februari 2011 @ 12:48:
Mja ik kan ook wel googelen maar ik snap niet goed waarom de shell niet met deze code overweg kan..

Met wat voor editor heb je het script gemaakt, en hoe heb je het daarna op dat apparaat gezet? (als het geen lokale editor was)

[ Voor 28% gewijzigd door Onno op 07-02-2011 13:36 ]

[Simon VDS]

Het is een zwaar gestripte versie speciaal gemaakt voor Dreambox satelliet ontvangers.

Nu schedule ik veel opnames en af en toe loopt de disk eens vol als ik vergeet te kijken


Eens kijken of dat ding snmp ondersteunt

[deadinspace]

Sardonis schreef op maandag 07 februari 2011 @ 12:20:
root@sardonis:/etc# diskAlert2.sh: line 10: syntax error: unexpected "done" (expecting "then")

Probeer het eens met die "then" op een nieuwe regel (vervang de voorgaande puntkomma door een newline). Ik kan me voorstellen dat busybox wat shortcuts maakt bij het parsen waardoor hij struikelt over sommige dingen die toch correct zijn.

Als dat niet helpt, probeer dan het script eens zoveel mogelijk te vereenvoudigen. Zet de "do" ook op een nieuwe regel, vervang die send-mail regel door "echo hoi", etc. Net zo lang tot je hebt gevonden wat het verschil maakt.

Onno schreef op maandag 07 februari 2011 @ 12:35:
Gokje: DOS newlines.

Hmm, al de hele file DOS newlines heeft, dan krijg je toch een ander probleem (namelijk dat hij de interpreter niet kan vinden). Maar wie weet verderop in de file inderdaad.

Kees schreef op maandag 07 februari 2011 @ 13:27:
probeer sowieso eens het met '/bin/bash' ipv /bin/sh' want veel distro's gebruiken tegenwoordig dash, wat een verschikkelijke shell is als standaard 'sh'.

Wat is er verschrikkelijk aan dash? Het is een prima posix sh implementatie.

Als je bashisms in je scripts wil gebruiken, dan stel je ook maar bash als interpreter in

[Kees]

deadinspace schreef op maandag 07 februari 2011 @ 14:13:
[...]

Als je bashisms in je scripts wil gebruiken, dan stel je ook maar bash als interpreter in

Dat doe ik nu ook, maarja, teveel (oude) scripts gingen uit van bash als default shell, en dat heeft mij wel eens gebeten. Sindsdien maak ik geen assumpties meer over welke shell de host zal draaien, /bin/bash staat er boven.