Besoin d'aide pour script bash : amélioration

Version imprimable

Bonjour à tous,

J'ai fait un petit script bash me permettant de déplacer les fichiers d'un répertoire de dépôt (ftp) vers un répertoire de travail. Ce script vérifie que le fichier n'est pas en cours d'envoi et si ce n'est pas le cas, déplace ce fichier.

Le script fonctionne bien, sauf quand un utilisateur me mets des espaces dans un nom de fichier ou pire me créé un sous-repertoire. Malgré mes avertissement cela arrive encore.

Quelqu'un aurait une idée pour contourner ce problème ?

Merci beaucoup !!

Voici le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
# Date
DATE_JOUR=`date +%y%m%d_%H%M%S` # Date du jour
 
# Chemin des répertoires
PATH_INIT="/var/***/depot/" # Répertoire de base
PATH_DEST="/var/***/recupdepot/" # Répetoire de destination
 
if [ $(ls ${PATH_INIT}/|wc -l) != 0 ]; then
        for i in $( ls ${PATH_INIT}/* -R )
                do
                        /usr/sbin/lsof $i
                        if [ $? = "1" ]; then
                                # Fichier n'est pas utilisé"
                                cp "$i" "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                chmod 777 "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                chown -R stabo "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                echo "Fichier reçu : ${DATE_JOUR}_$(basename $i)" | mail -s "Nouveau fichier sur DEPOT FTP" xxx@yyy.zzz
                                rm -f $i
                        fi
        done
fi

remplace

Code:

1
2
3
 
if [ $(ls ${PATH_INIT}/|wc -l) != 0 ]; then
        for i in $( ls ${PATH_INIT}/* -R )

par

Code:

1
2
 
ls -1 ${PATH_INIT}/* | while read i

Merci!
Ce qui donne donc :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
# Date
DATE_JOUR=`date +%y%m%d_%H%M%S` # Date du jour
 
# Chemin des répertoires
PATH_INIT="/var/***/depot/" # Répertoire de base
PATH_DEST="/var/***/recupdepot/" # Répetoire de destination
 
ls -1 ${PATH_INIT}/* | while read i
        do
                        /usr/sbin/lsof $i
                        if [ $? = "1" ]; then
                                # Fichier n'est pas utilisé"
                                cp "$i" "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                chmod 777 "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                chown -R stabo "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
                                echo "Fichier reçu : ${DATE_JOUR}_$(basename $i)" | mail -s "Nouveau fichier sur DEPOT FTP" xxx@yyy.zzz
                                rm -f $i
                        fi
        done
fi

Mais Je dois aussi retirer la partie en rouge on est d'accord ?

24/05/2012, 17h17
mvvvv
oui pour le if

par contre ... si ton répertoire PATH_INIT est vide le ls te fera une belle erreur (c'est ce qu'on lui demande de faire) ... si le message te dérange tu peux l'envoyer dans les limbes
Code:

1 2 ls -1 ${PATH_INIT}/* 2> /dev/null | while ...
24/05/2012, 19h22
N_BaH
Bonjour,

quand la sortie standard de ls n'est pas un terminal (soit quand il est pipé vers une autre commande, ou redirigée vers un fichier), il affiche son résultat sur une seule colonne.
mais pourquoi utiliser une commande externe ?
si ton shell est bash :
Code:

1 2 3 4 shopt -s nullglob for f in ./* do echo "$f" done
si tu as besoin de descendre dans les sous-répertoires, avec BASH_4 :
Code:

1 2 3 4 shopt -s nullglob globstar for f in ./**/* do echo "$f" done
24/05/2012, 19h47
mvvvv

Je découvre le shopt ça permet en effet de régler le problème du répertoire vide sans rentrer dans la boucle (le if n'était pas la solution au cas ou le répertoire se vide entre le if et le for)

+1

Bonjour, merci à tous pour vos réponses !

Voici donc le script avec vos recommandations :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
# Date
DATE_JOUR=`date +%y%m%d_%H%M%S` # Date du jour
 
# Chemin des répertoires
PATH_INIT="/var/***/depot/" # Répertoire de base
PATH_DEST="/var/***/recupdepot/" # Répetoire de destination
 
shopt -s nullglob
for i in ${PATH_INIT}/*
	do
		cp "$i" "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
		chmod 777 "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
		chown -R stabo "${PATH_DEST}/${DATE_JOUR}_$(basename $i)"
		echo "Fichier reçu : ${DATE_JOUR}_$(basename $i)" | mail -s "Nouveau fichier sur DEPOT FTP" xxx@yyy.zzz
		rm -f $i
	done

Le script ne fonctionne hélas pas quand il y a un espace dans le nom de fichier :?

25/05/2012, 10h28
mvvvv

à chaque fois que tu utilises $i pense à rajouter les quotes "$i"

tu peux aussi éviter d'invoquer basename plus d'une fois en utilisant une variable

Ok merci !

Pour les problèmes d'espaces c'est réglé.
Subsiste encore le problème de l'utilisateur qui créé un répertoire, là j'ai encore droit a une belle erreur. Je n'ai pas bash4 et ne peux donc pas utiliser la solution de N_BaH. :cry:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
# Date
DATE_JOUR=`date +%y%m%d_%H%M%S` # Date du jour
 
# Chemin des répertoires
PATH_INIT="/var/***/depot/" # Répertoire de base
PATH_DEST="/var/***/recupdepot/" # Répetoire de destination
 
shopt -s nullglob
for i in ${PATH_INIT}/*
        do
                j="$(basename "$i")"
                cp "$i" "${PATH_DEST}/${DATE_JOUR}_${j}"
                chmod 777 "${PATH_DEST}/${DATE_JOUR}_${j}"
                chown -R stabo "${PATH_DEST}/${DATE_JOUR}_${j}"
                echo "Fichier reçu : ${DATE_JOUR}_${j}" | mail -s "Nouveau fichier sur DEPOT FTP" xxx@yyy.zzz
                rm -f "$i"
        done

sinon tu peux aussi essayer le find (qui remplace le for) en créant une fonction

cela me semble plus clair personnellement :
Code:

1 2 3 4 5 6 7 8 9 10 function envoiFichier { i=$1 j=$(basename ... ... } .... find ${PATH_INIT} -name "*" -type f -exec envoiFicher {} \;
et si tu ne veux pas de fonction
Code:

1 2 3 4 5 6 find ${PATH_INIT} -name "*" -type f | while read i do j=$(basename ... ... done
sinon tu peux essayer la syntaxe proposée par Nbha

il te faut y rajouter un

Code:

if test -f "$i"

pour ne garder que les fichiers et exclure les répertoires
Code:

1 2 3 4 5 6 7 8 9 10 11 shopt -s nullglob globstar for i in ./**/* do if test -f "$i" then j= ... .... fi done
sinon ...

25/05/2012, 13h26
N_BaH

Citation:

PATH_INIT="/var/***/depot/" # Répertoire de base

Tiens ? pourquoi trois astérisques ? pourquoi pas quatre ?
un seul astérisque suffit, il remplace zéro, un, ou plusieurs caractères.

basename n'est pas indispensable : j="${f##*/}".

Pour ce qui est de descendre dans les sous-répertoires sans BASH4, il va falloir utiliser une fonction récursive, qui s'appellera si le fichier est en fait un répertoire, ou qui traitera le fichier si c'est bien un fichier.

les noms de variables entièrement en majuscules sont, par convention, réservés aux variables environnementale (PATH, PWD...)
25/05/2012, 13h31
stabo

Citation:

Envoyé par N_BaH

Tiens ? pourquoi trois astérisques ? pourquoi pas quatre ?
un seul astérisque suffit, il remplace zéro, un, ou plusieurs caractères.

En fait c'était pas mettre le chemin complet sur le forum, tout simplement.

Je vais essayer de m'en sortir avec vos commentaires et je reviendrais vers vous...

En attendant merci à tous !