Bonjour,
 
Je me suis fendu d'un petit script de transcodage de films mkv vers avi.
Je sais, c'est une usine à gaz, c'est pourquoi je suis ouvert à toute remarque constructive. Que ce soit en terme de code ou de fonctionnalités.
A ma décharge, ça ne fait que 2 mois que je suis rentré dans le monde magique de linux, et je n'avais plus rien codé depuis 11 ans...  
 
Script testé avec l'installation standard de Maverick Meerkat.
 
Merci d'avance pour vos réponses
 
Script disponible ici

Une suggestion, par exemple passer par un fifo pour éviter que la décompression en wave ne prenne une place énorme :

Merci pour ta réponse.
Eh bien je vais le renseigner de ce pas pour ces fichiers fifo, que je ne connaissais pas.
Concernant le dossier "film_temp", je l'ai mis en place parce qu'il me permet s'effacer tous les temporaires en une seule commande, du moins c'est la méthode la plus simple que je connaisse. Si c'est son nom qui te dérange, il est ainsi parce que j'ai pris en compte le fait que les personnes nomment souvent leurs dossiers temporaires "tmp" ou "temp" et je ne voulais pas interférer avec leur organisation.
Maintenant si le fichier fifo crée n'est pas trop gros, je vais voir pour tout mettre dans le /tmp de Linux... Qu'en penses tu ?

La taille d'un fichier fifo est de 0. Il s'agit d'un tube nommé, Le fichier audio est décompressé et injecté directement dans Lame qui le convertit en mp3. À propos 'transcode' est le nom d'un autre programme, tu devrais en changer.

Merci pour ces précisions, je fais les corrections des que je reviens de mes courses de Noël Je profiterai des files d'attente aux caisses pour réfléchir à un nouveau nom...

Voilà, corrections faites, utilisation du dossier /tmp et amélioration du logfile

Disponible >>ici<<, les avis sont les bienvenus

Plutôt que :

Pourquoi pas :

Et puis j'aime pas trop l'utilisation de 'path' en variable alors qu'il y a déjà PATH en variable du shell.

Voilà c'est modifié... Encore merci pour tes corrections.
Nouvelle mouture dispo au même endroit

Useless use of cat

 
Ah, ça quand on copie des commandes, parfois on fait des trucs inutiles
 
Merci, correction faite

Petites améliorations envisageables pour éviter les erreurs de saisie :

A faire de la même manière pour le choix des sous-titres.

Je modifierai ça après les digestions de Noël

Par contre, 2 petites questions sur tes corrections :

- pourquoi grep -cF et pas simplement grep -c ?

- le dossier /tmp se réinitialisant au redémarrage, faut il supprimer les fichiers crées ? J'ai remarqué que tu me le proposais systématiquement. Est ce un question d'espace disque, de propreté de programme ou une simple déformation professionnelle ?

C'est sans doute de l'enc***ge de mouches... grep recherche une chaîne de caractères fixe et pas une variable à interpréter, il est plus rapide.

Taratata ! Ubuntu centrisme ! Pas sur toutes les distributions. Et il y a parfois dans mon /tmp de vielles choses qui traînent (crash, redémarrage brutal, bug logiciel).
Quand je fais un script je préfère largement créer un dossier caché dans mon HOME avec des sous-dossiers /tmp et /log.

Bah, comme dit, j'apprends, donc en***ge de mouches ou pas, ces précisions peuvent toujours être utiles. Ici, ok, c'est sans grand intérêt, mais qui sait, dans un prochain script....

Ok, ça je savais pas... Faut que je me détache du "www.chezmoicamarche.com"... Noté

Rhaaa et moi qui avais supprimé mon répertoire film_temp