Discussion :

[dj_benz]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[ok.Idriss]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[N_BaH]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[jack-ft]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[dj_benz]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

[ok.Idriss]

Bonjour.

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Encore inutile :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Cordialement,
Idriss

[Winnt]

Bonjour,

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

[jack-ft]

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 ! J'ai corrigé le script que j'avais fourni!