petit partage de ce que j'ai appris :
ajoute aussi ceci dans le téléchargement des émissions :
wget <lien a dl> | sed -u 's/.*\ ([0-9]+%)\ +([0-9.]+\ [KMB\/s]+)$/\1\n# Vitesse du téléchargement = \2/' | zenity --progress --title="Mise à jour de l'index des périphériques" --text="Je suis en train de télécharger..." --auto-close
ps: n'hésite pas a regarder certains de mes scripts et a t'en inspirer pour la propreté, je ne suis pas non plus un AS du Bash j'apprend comme toi, mais comme tu dis ne réinvente pas la roue et regarde ce qui ce fais de bien un peu partout :
http://forum.ubuntu-fr.org/viewtopic.php?id=117039&p=17
J'ai choisis plusieurs émissions mais il ne m'en a télécharger qu'une seul !
Pour la condition ce qui est plus clair :
if [ "$?" = "0" ]; then # des espaces entre les " et [ = ]
(...)
fi
`true_false ZAP` ZAP "Le zapping"\
a remplacer par
FALSE ZAP "Le zapping" \ # ca marche très bien
if [ ! `which zenity` ] ou alors regarde la fonction requis de mon script ndiswrapper, l'entré est paquet_requis et il se débrouille pour tout installer .
Dans les text pour zenity ajoute des \n pour sauter des lignes
if [[ $? == "0" ]]
a remplacer par
if [ $PIPESTATUS = "0" ] ; then
par contre si tu as une idée sur la différence avec
if [ ${PIPESTATUS[0]} = "0" ] ; then
je suis preneur
on abolis les tests :
if (test $? -eq 1 );
par
if [ $? -eq 1 ] ; then
folder=$( date +%Y-%m-%d -d $date_us )
par
folder=`date +%Y-%m-%d -d $date_us`
if [[ -z $aVideos ]] && [[ -z $video_id ]]
par
if [ -z $aVideos ] && [ -z $video_id ] ; then
( emissions ) | zenity --progress \
par
emissions | zenity --progress \
sans les parenthèse ca marche très bien d(ailleurs je viens d'apprendre quelque chose, merci:
if grep -q GTK .canal_config ; then
echo salu;
else
echo by;
fi
Voila je te laisse améliorer ton script pour le rendre plus lisible mais c'est a toi de voir ca dépend des préférences de chacun, je trouve qu'il deviendrait plus clair, ensuite je mettrais à la pâte si ca peu te motiver.
Ma Bible ou plutot mon Coran
http://abs.traduc.org/