C'est assez subtil comme comportement... Le coupable est ffmpeg qui, durant son exécution, lit aussi sur l'entrée standard et court-circuite du coup read. Pour arranger ça, rajoute « </dev/null » au bout de la commande ffmpeg :

find . -name "*.mp3" | while read i; 
do ffmpeg -i "$i" -map_meta_data "${i%mp3}flac":"$i" "${i%mp3}flac" </dev/null;
        oggenc -b 128 "${i%mp3}flac";
        rm "${i%mp3}flac";
        echo $i;
done 

Pour la définition de /dev/null : http://fr.wikipedia.org/wiki//dev/null

Est-ce que ça veut dire que ffmpeg "mange" l'entrée standard, c'est-à-dire ce qui est envoyé par find ?

Exactement.

rm "${i%mp3}flac" est censé effacer le fichier de transition FLAC, mais ne le fait pas. WTH ? ça fonctionnait dans l'ancienne version du script...

Pourrais-tu poster les traces d'exécution du script ?

patou355 wrote:En fait j'ai trouvé :
Il cherchait à supprimer un fichier dont le nom finit par .mp3.flac. Donc en remplaçant

rm "$i.flac";

par

rm "${i%.mp3}.flac";

Tout rentre dans l'ordre...

Les comamndes que tu as postées indiquaient pourtant bien « rm "${i%.mp3}.flac" ». Pas bien ça, tu veux faire peur ? ^^

Ce n'est pas fini 🙂. C'est dommage qu'il y ait ici duplication de code, on peut largement condenser :

# Une liste des types de fichiers à convertir, par extension
TYPES_SUPPORTES="mp3 mp4 m4a flac flv wma"

# On va parcourir tous les fichiers de l'arborescence, quels qu'ils soient
find . -type f | while read i; do
   # on récupère l'extension du fichier traité
   ext=${i##*.}

   # Si l'extension n'est pas dans la liste TYPES_SUPPORTES, le fichier est ignoré et on passe au suivant
   grep -q "$ext" <<<"$TYPES_SUPPORTES" || continue

   # On ne convertit évidemment pas les fichiers FLAC avec ffmpeg, c'est déjà fait...
   [ $ext != flac ] && ffmpeg -i "$i" -map_meta_data "${i%$ext}flac":"$i" "${i%$ext}flac" </dev/null

   oggenc -b 128 "${i%$ext}flac"

   rm "${i%$ext}flac"
done

Ainsi, pour convertir d'autres types de fichiers supportés par ffmpeg avec la même commande, tu n'aurais qu'à rajouter l'extension associé à la variable TYPES_SUPPORTES.

Es-tu cependant sûr de vouloir supprimer les fichiers FLAC originaux dans ton bloc « #Conversion depuis flac » ?

patou355 wrote:Bon, là je peux pas lutter 🙂

Par contre j'ai pas compris pk tu dis que la conversion en .flac est déjà faite...

et je vois pas non plus le lien entre TYPES_SUPPORTES et le 'find -type -f'

Je vais détailler le fonctionnement du script. Celui-ci va parcourir tous les fichiers, y compris ceux non multimédia. Mais seuls ceux dont l'extension est dans la variable TYPES_SUPPORTES seront effectivement convertis. Un fichier non multimédia sera ignoré (la commande « continue », si elle est exécutée, fait en sorte que les instructions situées après soient ignorée et fait passer la boucle au fichier suivant).
Par ailleurs, la commande ffmpeg dans le script va convertir ici n'importe quel fichier dont l'extension est déclarée dans TYPES_SUPPORTES en FLAC. Or les fichiers FLAC sont aussi dans TYPES_SUPPORTES. Dès lors pourquoi convertir du FLAC en FLAC ? Aussi pour ces fichiers la commande n'est pas exécutée, on passe directement à l'encodage en OGG.
Le but de ma réécriture était d'une part d'éviter la duplication de code (si tu devais modifier par exemple la commande ffmpeg, tu devrais sans ça la modifier autant de fois qu'elle apparaît), d'autre part de permettre la prise en compte de nouveaux fichiers multimédia sans peine ni copie de code (grâce à la variable TYPE_SUPPORTES).

patou355 wrote: Par contre, cela veut dire que tous les fichiers sont stockés dans la variable, même si on en fait rien.

Quelle variable ? TYPES_SUPPORTES ? Son contenu n'est jamais modifié dans la boucle ; la commande suivante :

grep -q "$ext" <<<"$TYPES_SUPPORTES"

vérifie simplement si l'extension du fichier courant fait partie de celles définies dans la variable TYPES_SUPPORTES. Si tu préfères, une notation "développée" strictement équivalente serait :

echo "$TYPE_SUPPORTES" | grep -q "$ext"

L'opérateur "<<<" (avec 3 chevrons) permet d'envoyer le contenu d'une chaîne sur sur l'entrée standard de la commande qui le précède.