Discussion :

[nicolas.poulain]

Salut à tous

Bon je ne suis un cador du script bash.
Je bloque sur une petite broutille :\

J'ai adapté le script suivant pour tourner sur une raspbian elle-même sur un raspberry pi.
L'objectif est de lancer la lecture via omxplayer d'un (ou plusieurs) morceau de musique ou une vidéo en boucle, les morceaux étant sur une clé usb.

Mon problème actuellement :
ça fonctionne... sauf si il y a des espaces dans le nom des fichiers.
ça doit être une histoire de guillemets ou de parenthèses à remettre au bon endroit.
Si qq'un voit comment résoudre ça je suis preneur ;o)

a+
Nicolas

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
#!/bin/sh
 
# get rid of the cursor so we don't see it when videos are running
setterm -cursor off
 
# set here the path to the directory containing your videos
VIDEOPATH="/home/pi/medias" 
 
# you can normally leave this alone
SERVICE="omxplayer"
 
USBKEY="/media/$(ls -I SETTINGS /media)"
 
# now for our infinite loop!
while true; do
        if ps ax | grep -v grep | grep $SERVICE > /dev/null
then
        sleep 1;
else
	for entry in $(ls "$USBKEY")
do
		clear
                omxplayer -o local "$USBKEY/$entry" > /dev/null
#		echo "$USBKEY/$entry"
        done
fi
done

[N_BaH]

Bonjour,

ON N'UTILISE PAS ls DANS UN SCRIPT !!!
ni une boucle for pour parcourir une substitution de commande !
une bouclefor itère sur chaque mot, et non sur chaque ligne.

=> remplace ta boucle for par celle-ci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for f in /media/!(SETTINGS)

en ayant ajouté au début du script cette option de configuration :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

shopt -s extglob nullglob

NB : un script bash doit commencer par un shebang bash : #!/bin/bash.

[nicolas.poulain]

Bonjour,

ON N'UTILISE PAS ls DANS UN SCRIPT !!!

Est-ce que tu peux m'expliquer pquoi stp ?

ni une boucle for pour parcourir une substitution de commande !

Est-ce que tu peux m'expliquer encore pquoi re-stp ?

une bouclefor itère sur chaque mot, et non sur chaque ligne.

=> remplace ta boucle for par celle-ci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for f in /media/!(SETTINGS)

en ayant ajouté au début du script cette option de configuration :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

shopt -s extglob nullglob

NB : un script bash doit commencer par un shebang bash : #!/bin/bash.

Bon je vais me replonger dans la doc pour reregarder la différence entre le bash et le sh :\ (edit : http://fr.wikipedia.org/wiki/Shell_Unix)
Pour utiliser shopt il faut rebasculer en bash ou je peux rester avec /bin/sh ?

Sinon pour ceux qui liront, utilisation de shopt dans le man bash

Merci en tout cas
Nicolas

[nicolas.poulain]

Pour utiliser shopt il faut rebasculer en bash ou je peux rester avec /bin/sh ?

Il faut bash ;o) je confirme !

[nicolas.poulain]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
 
#!/bin/bash
 
# fixe les options de bash - voir man bash
shopt -s extglob nullglob
 
# get rid of the cursor so we don't see it when videos are running
setterm -cursor off
 
# you can normally leave this alone
SERVICE="omxplayer"
 
# now for our infinite loop!
# permet de lire les fichiers sur une clé USB en excluant le dossier /media/SETTINGS
while true; do
        if ps ax | grep -v grep | grep $SERVICE > /dev/null
then
        sleep 1;
else
        for file in /media/!(SETTINGS)/*
do
                clear
                omxplayer -o local "$file" > /dev/null
#               echo "$file"
        done
fi
done

ça roule avec ça. En complément dans /etc/rc.local, un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
bash /home/pi/monscript.sh

Tu vois des choses encore à corriger ?

En tout cas, bien merci
Tu bascules en résolu quand tu as complété qques explications ?
Nicolas

[N_BaH]

ls n'apporte rien à un script : un shell de base sait faire des développements de chemins; bash ajoute des extensions, activées par extglob
quant à obtenir des informations sur les fichiers, il est préférable d'utiliser stat dont la sortie peut être formatée à souhait.

pour la boucle for, j'ai expliqué que cela coupe les données en mots, non en lignes.

si, selon toi, nous avons répondu à tes interrogations, tu peux mettre en résolu en appuyant le le bouton

[disedorgue]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
 
#!/bin/bash
 
# fixe les options de bash - voir man bash
shopt -s extglob nullglob
 
# get rid of the cursor so we don't see it when videos are running
setterm -cursor off
 
# you can normally leave this alone
SERVICE="omxplayer"
 
# now for our infinite loop!
# permet de lire les fichiers sur une clé USB en excluant le dossier /media/SETTINGS
while true; do
        if ps ax | grep -v grep | grep $SERVICE > /dev/null
then
        sleep 1;
else
        for file in /media/!(SETTINGS)/*
do
                clear
                omxplayer -o local "$file" > /dev/null
#               echo "$file"
        done
fi
done

Alors, ici, on voit que l'on initialise une variable (SERVICE) et qu'elle est utilisée dans un cas pas très pratique, surtout qu'après elle n'est pas utilisée alors que cela aurait été bien utile:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ps ax | grep -v grep | grep $SERVICE > /dev/null

pourrait être remplacé par:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ps ax | grep -q '[o]mxplayer'

Et pourquoi ne pas remplacer le bloc:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
while true; do
        if ps ax | grep -v grep | grep $SERVICE > /dev/null
then
        sleep 1;
else
        for file in /media/!(SETTINGS)/*
do
                clear
                omxplayer -o local "$file" > /dev/null
#               echo "$file"
        done
fi
done

Par:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
while sleep 1; do
        ps ax | grep -q '[o]mxplayer' && continue
        for file in /media/!(SETTINGS)/*
        do
                clear
                omxplayer -o local "$file" > /dev/null
#               echo "$file"
        done
done

[N_BaH]

ps | grep : ps -C <commande>, ou pidof <commande>, ou encore pgrep <commande>

[disedorgue]

En fait, je me demande même si c'est utile le passage par une des forme de ps, puisque c'est le script lui-même qui lance omxplayer...

[nicolas.poulain]

Re'

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ps ax | grep -v grep | grep $SERVICE > /dev/null

En fait, cette ligne de test (ou celle que tu proposes ;o) ), d'après ce que je comprends, ne sert que si une instance de omxplayer est déjà lancée en dehors de ce script.
ça serait utile si on mettait la lecture en tâche de fond genre "omxplayer file &"
Mais si de toute façon on attend la fin de chaque processus de lecture (comportement par défaut de omxplayer) pour passer à la suite au final, je ne suis pas très convaincu de sa présence.
Pour le dire autrement on ne fait pas le test tant qu'on est dans la boucle du for. Et quand on est plus dans la boucle du for, c'est qu'il n'y plus de lecture de fichier.

Je suis dans le vrai ou plutôt à côté ?

Des fois est-ce qu'un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
kill omxplayer
while true ; do
        for file in /media/!(SETTINGS)/*
        do
                clear
                omxplayer -o local "$file" > /dev/null
        done
done

ne ferait pas le taf ?

Nicolas