ls : valeur de retour si vide

Bonjour,

Je souhaite faire un script de sauvegarde, d'une arborescence:

Code:

---

1 2 3 4 5

Dir +-- subdir1 +-- subdir2 +-- subdir3

---

Les 3 subdir sont connus et figés. Je souhaite sauvegarder leur contenu dans l'équivalent de l'arborescence sur un NAS.

J'ai donc un code de ce type:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

LOG=/mon/log DIR=/mon/chemin/dacces/a/Dir for f in `ls $DIR` do if [ -d "$DIR/$f" ]; then NBFILES=`find $DIR/$f -type f | wc -l` echo "Copie de $DIR/$f..." >> $LOG   cp -r $DIR/$f/* /mnt/nas/$DIR/$f/   if [ $? -eq 0 ]; then echo "[ OK ]: $NBFILES fichiers" >> $LOG else echo "[FAIL]: CP command fail" >> $LOG fi fi done

---

Ce code fonctionne très bien, mais dans mon log, une erreur apparait: "Aucun fichier ou répertoire de ce type" quand un des répertoires "subdir" est vide.

Comment puis-je palier à cette erreur? Y a-t-il une option à ajouter à cp pour qu'il ignore les vides?

Merci d'avance.

Bonsoir.

Pour commencer, N_Bah le répète très souvent et à juste titre : pas de for i in $(ls) ou parcours dérivés (faire une recherche pour plus de détails).

Il est préférable d'utiliser le méta-carractère "*" (wildcard) de cette façon (à tester et adapter) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

LOG=/mon/log DIR=/mon/chemin/dacces/a/Dir for f in $DIR/* do if [ -d $f ]; then NBFILES=$(find $f -type f | wc -l) echo "Copie de $f..." >> $LOG cp -r $f/* /mnt/nas/$DIR/   if [ $? -eq 0 ]; then echo "[ OK ]: $NBFILES fichiers" >> $LOG else echo "[FAIL]: CP command fail" >> $LOG fi fi done

---

Cordialement,
Idriss

Bonjour,

as-tu vraiment besoin d'un fichier log, ou est-ce juste pour voir que ton script ne fonctionne pas correctement ? (s'il fonctionnait correctement enregistrerais-tu un log ?)

voilà une ligne qui ne générera pas d'erreurs :

Code:

---

find Dir/ -type f -exec cp -t ./ {} \+

---

Attention! On n'utilise jamais 'ls' dans un script! (sinon tu vas fâcher N_BaH ;) )

Il me paraitrait plus logique d'utiliser une variable 'd' plutôt que 'f' pour désigner un répertoire!

Sinon, tu peux soit tester la vacuité du répertoire soit, comme tu le fais, le code retour du 'cp'.
Dans ce dernier cas, comme tu gères l'erreur toi-même, tu n'as pas besoin du message d'erreur fourni par 'cp' sur la stderr, c'est-à-dire que tu peux le mettre à la poubelle:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

LOG=/mon/log DIR=/mon/chemin/dacces/a/Dir SDIR=$(basename "${DIR}") for d in $DIR/* do if [ -d "$d" ]; then typeset sd=$(basename "${d}") # ou bien sd=${d##*/} # pour les avancés... NBFILES=`$d -type f | wc -l` echo "Copie de $sd dans /mnt/nas/$DIR/$sd..." >> $LOG   cp -pr "$d"/* "/mnt/nas/$SDIR/$sd/" 2> /dev/null   if [ $? -eq 0 ]; then echo "[ OK ]: $NBFILES fichiers" >> $LOG else echo "[FAIL]: CP command fail" >> $LOG fi fi done

---

Attention! si un des répertoires contient un fichier dont le nom commence par '.' il ne sera pas copié.
Si c'est important, tu peux utiliser cpio ou tar:

Code:

---

(cd "$d" ; tar cf - . ) | (mkdir -p "/mnt/nas/$SDIR/$sd" ; cd "/mnt/nas/$SDIR/$sd" ; tar xf -)

---

Merci à tous pour vos réponses.

Je viens de voir que mon titre est faux: c'est le code retour de CP (et non LS) que je voulais analyser.

OK N_BaH, je n'utiliserai plus ls. Je ne le savais pas mais je me suis renseigné. Merci.

Le problème en utilisant la méthode "for d in $DIR/*", est que l'arborescence complète sera dans ma sauvegarde:
DIR=/mon/chemin/dacces/a/Dir
sur le NAS, je voudrais uniquement: /mnt/nas/Dir/....

Je vais donc utiliser:
cd $DIR
for d in ./*

Je souhaite garder les messages d'erreur, mais pas les répertoires vides. Le but de garder mon log est de vérifier en arrivant le matin, que la sauvegarde n'a pas rencontré de problèmes.
Voilà à quoi j'ai pensé:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

LOG=/mon/log DIR=/mon/chemin/dacces/a/Dir cd $DIR for d in ./* do if [ -d "$DIR/$d" ]; then NBFILES=`find $DIR/$d -type f | wc -l`   if [ $NBFILES -ne 0 ]; then echo "Copie de $DIR/$d..." >> $LOG cp -pr "$DIR/$d/*" "/mnt/nas/Dir/$d/" 2>>$LOG   if [ $? -eq 0 ]; then echo "[ OK ]: $NBFILES fichiers" >> $LOG else echo "[FAIL]: CP command fail" >> $LOG fi fi fi done

---

Moi, c'est cd que j'aime pas trop utiliser en scripts...

Que pensez-vous de cette solution?

Bonjour.

Le ./* est inutile. Dans ce cas autant écrire directement :

Code:

---

1 2 3

for d in *; do # ... done

---

Mais la solution du $DIR/* reste préférable. Tu n'aura pas eu de copie du chemin complet si tu avait fait ton "cp" de cette manière :

Code:

---

cp -pr $d/* /mnt/nas/Dir/

---

($d est un chemin complet d'où le fait que tu copiait une arborescence complète :aie: ).

Encore inutile :

Code:

---

if [ -d "$DIR/$d" ]; then

---

=> if [[ -d $d ]]; then.

Cordialement,
Idriss

Bonjour,

Pourquoi ne pas utiliser rsync ?
En dehors du fait qu'il est toujours instructif de faire soi même. :mrgreen:

Citation:

---

Envoyé par dj_benz

Le problème en utilisant la méthode "for d in $DIR/*", est que l'arborescence complète sera dans ma sauvegarde:
DIR=/mon/chemin/dacces/a/Dir
sur le NAS, je voudrais uniquement: /mnt/nas/Dir/....

---

C'est pas faux ! :oops: J'ai corrigé le script que j'avais fourni! :P