Discussion :

[grunk]

Salut à tous ,

J'ai pour objectif de faire un petit script shell qui, lancé par un cron me permettra de supprimer des fichiers "périmé".
L'idée est là , le script aussi mais j'ai une erreur que je ne sait pas régler (je comprend pas pourquoi y'a une erreur en fait :/

#! /bin/bash
# Configuration
perime=1200
heureserv=`date '+%H'`
minuteserv=`date '+%M'`
secserv=`date '+%S'`
timeserv=$[($heureserv * 3600)+($minuteserv * 60)+$secserv]
# Liste des fichier
listing=`ls`
for i in $listing
do
element="$i"
if [ -d $element ]
then
types=`ls -la -d $element | cut -c0-1`
else
types=`ls -la $element | cut -c0-1`
fi

if [ $types = 'l' ]# si lien symbolique
then
heurefic=`stat -c %x /home/chemin/vers/fichier/$element | awk '{print $4}' |cut -c0-2`
minutefic=`stat -c %x /home/chemin/vers/fichier/$element | awk '{print $4}' |cut -c4-5`
secfic=`stat -c %x /home/chemin/vers/fichier/$element | awk '{print $4}' |cut -c7-8`
timefic=$[($heurefic * 3600)+($minutefic * 60)+$secfic]
resultat=$[$timeserv - $timefic]

if[ $resultat -gt $perime ]
then# /!\ erreur à cette ligne /!\
rm $element
fi
fi
done
exit 0

L'erreur retournée est :
./cleaner.sh: line 28: syntax error near unexpected token `then'
./cleaner.sh: line 28: ` then'

Donc apperement il n'aime pas ma condition if mais pourquoi ?

Merci de votre aide

[becket]

Désolé si je te semble un peu dur mais quel complication !

J'apporte quand même quelques améliorations

1 - Pour tester si c'est un lien symbolique -> if [ -L $fichier ]
2 - Tu faire une if else qui contient le même code


un proposition de script

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#!/bin/sh
fichierstamp=/tmp/coucou
 
if [ -e $fichierstamp ]
then
        for fichier in  $( find /tmp -type l  -not -newer $fichierstamp  )
        do
                echo " Fichier $fichier"
                echo "traitement"
        done
touch $fichierstamp
else touch $fichierstamp
fi

[garga69]

en fait ton erreur et toute bete j'ai eu la meme recemment c'est juste que tu a oublié de mettre un espace entre le if et le crochet qui le suit

Code :

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

1
2
3
4
if [ $resultat -gt $perime ]
then# /!\ erreur à cette ligne /!\
rm $element
fi

[grunk]

Désolé si je te semble un peu dur mais quel complication !

J'apporte quand même quelques améliorations

1 - Pour tester si c'est un lien symbolique -> if [ -L $fichier ]
2 - Tu faire une if else qui contient le même code

Pas de problème , je suis pas bon en shell et tout amélioration et conseils sont toujours les bienvenue

en fait ton erreur et toute bete j'ai eu la meme recemment c'est juste que tu a oublié de mettre un espace entre le if et le crochet qui le suit

Merci Garga, vraiement le genre d'erreur qu'on trouve jamais seul ^^

[MarcG]

hum hum....
le script peut tout a fait fonctionner mais l'usage de la commande find reste probablement meilleur.

Code :

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

1
2
 
find / -type f -utime +X -exec rm {} \;

pourrait faire la même chose : l'otion utime (ou atime ou mtime) celon le but recherché permettra de selectionner un fichier en fonction de son ancienneté (creation, accés, misa ajour) et d'effectuer l'action apres le -exec.

voir le man find pour le detail.

[becket]

hum hum....
le script peut tout a fait fonctionner mais l'usage de la commande find reste probablement meilleur.

Code :

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

1
2
 
find / -type f -utime +X -exec rm {} \;

pourrait faire la même chose : l'otion utime (ou atime ou mtime) celon le but recherché permettra de selectionner un fichier en fonction de son ancienneté (creation, accés, misa ajour) et d'effectuer l'action apres le -exec.

voir le man find pour le detail.

Tout à fait, l'ennui avec l'utilisation des options atime ou ctime est ( a moins de jouer avec un daystart ) il compte un nombre de jours alors que le script original à une périodicité d'une vingtaine de minutes