Discussion :

[stabo]

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 :

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
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

[mvvvv]

remplace

Code :

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

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

par

Code :

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

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

[stabo]

Merci!
Ce qui donne donc :

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
# 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 ?

[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 :

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

1
2
 
ls -1 ${PATH_INIT}/* 2> /dev/null | while ...

[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 BASH :

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

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 BASH :

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

1
2
3
4
shopt -s nullglob globstar
for f in ./**/*
do echo "$f"
done

[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

[stabo]

Bonjour, merci à tous pour vos réponses !

Voici donc le script avec vos recommandations :

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
# 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

[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

[stabo]

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.

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
# 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

[mvvvv]

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

cela me semble plus clair personnellement :

Code :

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

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 :

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

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 :

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

if test -f "$i"

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

Code :

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

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 ...

[N_BaH]

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...)

[stabo]

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 !