[Bash]Hulp nodig met mijn script.

[Verwijderd]

Ik heb geprobeerd een script te schrijven die de datum en het huidige nummer weergeeft. Het is de bedoeling dat dit script om de tien seconden opnieuw wordt uitgevoerd mits het proces van mijn "window manager" nog niet beeindigd is.

Als ik mijn window manager opstart, start dit script automatisch ook, maar ik krijg de volgende fout terug:
line 25: \#81E234\: command not found
Het volledige script is in de volgende link:
http://pastebin.com/UDKw9mWH

Kan iemand mij duidelijk maken wat precies fout is?

[Osiris]

Eeehhhm..

Mijn vraag aan jou is: wat denk je zélf wat er mis gaat?

Want er staan in je eigen topicstart toch best wel aardig wat goeie hints naar wat fatsoenlijk debugging.

Mocht je er overheen kijken, de error zit in LINE 25 en hij kan het commando `\#81E234\` niet uitvoeren. Hey, guess what er mis gaat?

[HellStorm666]

wss
RED="\\#D7875F\\"
GREEN="\\#81E234\\"
YELLOW="\\#DFAF8F\\"
BLUE="\\#729CFC\\"
MAUVE="\\#AD7FA8\\"
WHITE="\\#BEBEBE\\"
DEFAULT="\\#BEBEBE\\"

vervangen door:
RED="\#D7875F\"
GREEN="\#81E234\"
YELLOW="\#DFAF8F\"
BLUE="\#729CFC\"
MAUVE="\#AD7FA8\"
WHITE="\#BEBEBE\"
DEFAULT="\#BEBEBE\"

[Verwijderd]

Osiris schreef op maandag 28 november 2011 @ 17:54:
Eeehhhm..

Mijn vraag aan jou is: wat denk je zélf wat er mis gaat?

Want er staan in je eigen topicstart toch best wel aardig wat goeie hints naar wat fatsoenlijk debugging.

Mocht je er overheen kijken, de error zit in LINE 25 en hij kan het commando `\#81E234\` niet uitvoeren. Hey, guess what er mis gaat?

Ik begrijp dat dat geen commando is .
Maar wat ik probeerde was een variabele maken voor elke kleur, voor meer overzicht.
Hoe kan ik zorgen dat mijn script mijn variabele niet uitgevoerd wordt als commando?

Want iets als
#!/bin/bash
$FOO="Hallo"
echo $FOO
zou gewoon moeten werken toch?
en ik heb per ongeluk de functie statustext er in gelaten. Ik zal de pastebin even updaten zonder dat erbij.

HellStorm666 schreef op maandag 28 november 2011 @ 17:57:
wss
RED="\\#D7875F\\"
GREEN="\\#81E234\\"
YELLOW="\\#DFAF8F\\"
BLUE="\\#729CFC\\"
MAUVE="\\#AD7FA8\\"
WHITE="\\#BEBEBE\\"
DEFAULT="\\#BEBEBE\\"

vervangen door:
RED="\#D7875F\"
GREEN="\#81E234\"
YELLOW="\#DFAF8F\"
BLUE="\#729CFC\"
MAUVE="\#AD7FA8\"
WHITE="\#BEBEBE\"
DEFAULT="\#BEBEBE\"

Dat zal niet werken omdat mijn window manager de kleuren herkent met twee backslashes.

EDIT: Nieuwe "versie", het enige probleem is dat hij nu niet "looped":
http://pastebin.com/UDKw9mWH

[ Voor 25% gewijzigd door Verwijderd op 28-11-2011 18:17 ]

[Kees]

In je eerste versie miste je domweg een 'echo' voor de tekst. In je tweede versie hergenereer je je texten nooit, dus die blijven altijd het zelfde.

[Verwijderd]

Kees schreef op maandag 28 november 2011 @ 18:29:
In je eerste versie miste je domweg een 'echo' voor de tekst. In je tweede versie hergenereer je je texten nooit, dus die blijven altijd het zelfde.

Is er hiervoor een oplossing?

[Kees]

ja, die dingen hergeneren voor je ze output

[Verwijderd]

Kees schreef op maandag 28 november 2011 @ 18:33:
ja, die dingen hergeneren voor je ze output

Ok, maar hoe pas ik dit aan in mijn script. Bash scripting is redelijk nieuw voor mij .
Erg bedankt voor je hulp!

[Kees]

copy lijn 11,12 pasta voor lijn 16.

[Verwijderd]

Kees schreef op maandag 28 november 2011 @ 19:01:
copy lijn 11,12 pasta voor lijn 16.

Ah, wat een leermoment.
Na wat andere aanpassingen werkt het prima.
Bedankt voor jullie hulp.
Hier is het uiteindelijke, werkende, script:
http://pastebin.com/ScbyrKcA

[Ertepeller]

Alternatief en iets korter:

code:

#!/bin/bash

pkill -U $UID status.sh

declare -A COLOR=(
      [red]='\#d7875f\'
    [green]='\#81e234\'
   [yellow]='\#dfaf8f\'
     [blue]='\#729cfc\'
    [mauve]='\#ad7fa8\'
    [white]='\#bebebe\'
)

COLOR[default]=${COLOR[white]}

while pgrep -U $UID wmfs >&-; do
   wmfs -s "${COLOR[green]} mpd: $(ncmpcpp --now-playing)${COLOR[default]} | ${COLOR[blue]}$(date '+%H:%M - %a %d %b')${COLOR[default]}"
   sleep 10
done

En nee, er hoeven geen dubbele backslashes gebruikt te worden als ze tussen single quotes staan...

pgrep/pkill zijn erg handig als je processen wilt checken/killen, je hoeft dan tenminste niet moeilijk te doen met awk-tjes en grep -v enzo.

[Rainmaker]

Je hebt ook een fout in je shebang:

code:

kill $(ps U $UID | awk '/wmfs/&&/status.sh/' | grep -vi "$$\|grep\|awk" | awk '{print $1}')
#!/bin/bash/Users/Skip/.config/wmfs/status.sh

De shebang moet bovenaan staan en een geldige interpreter zijn.

Tenzij je op jouw systeem /bin/bash vervangen hebt door een directory, klopt deze niet.