Coup de pouce lecture automatique de fichiers / boucle avec nom de fichiers

Version imprimable

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:

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

22/03/2014, 14h31
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:

for f in /media/!(SETTINGS)

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

Code:

shopt -s extglob nullglob

NB : un script bash doit commencer par un shebang bash : #!/bin/bash.
22/03/2014, 14h58
nicolas.poulain

Citation:

Envoyé par N_BaH

Bonjour,

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

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

Citation:

ni une boucle for pour parcourir une substitution de commande !

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

Citation:

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

=> remplace ta boucle for par celle-ci :

Code:

for f in /media/!(SETTINGS)

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

Code:

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
22/03/2014, 15h12
nicolas.poulain

Citation:

Envoyé par nicolas.poulain

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

Il faut bash ;o) je confirme !

Code:

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:

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

22/03/2014, 17h07
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 :resolu:

Citation:

Envoyé par nicolas.poulain

Code:

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:

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

pourrait être remplacé par:

Code:

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

Et pourquoi ne pas remplacer le bloc:

Code:

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:

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

:pastaper:

22/03/2014, 21h30
N_BaH

~~ps | grep~~ : ps -C <commande>, ou pidof <commande>, ou encore pgrep <commande>
22/03/2014, 22h09
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...
22/03/2014, 23h05
nicolas.poulain
Re'

Code:

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:

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