"bash: [: too many arguments".

Fermé
michel42 - 23 juin 2014 à 13:08
zipe31
Messages postés
36317
Date d'inscription
dimanche 7 novembre 2010
Statut
Contributeur
Dernière intervention
27 janvier 2021
- 23 juin 2014 à 15:15
Bonjour,
Le second IF me donne "bash: [: too many arguments".

#!/bin/bash
#Script nettoyage.sh qui a pour action de purger les dossiers "/usr/local/applications/nettoayge" et "/usr/local/applications/*/nettoyage" lorsque le timestamp de change a plus de 10 jours ##

# Variables
REP="/usr/local/applications/nettoyage"
REP2="/usr/local/applications/*/nettoyage"

# Vérification que les répertoires sont présents

if [ -d $REP ]; then
find "$REP" -maxdepth 1 -ctime +10 -exec rm -rf {} \;
else
echo "Le répertoire $REP n'est pas présent sur ce serveur"
fi
if [ -d $REP2 ]; then
find "$REP2" -maxdepth 1 -ctime +10 -exec rm -rf {} \;
else
echo "Le répertoire $REP2 n'est pas présent sur ce serveur"
fi

Merci à vous,

2 réponses

zipe31
Messages postés
36317
Date d'inscription
dimanche 7 novembre 2010
Statut
Contributeur
Dernière intervention
27 janvier 2021
6 568
23 juin 2014 à 13:49
Salut,

L'erreur vient du fait que ta syntaxe
if [ -d $REP2 ]
une fois développée par le shell ne correspond plus à l'attente du test.

En fait c'est comme si tu écrivais :
test -d /usr/local/applications/toto/nettoyage /usr/local/applications/tata/nettoyage /usr/local/applications/titi/nettoyage etc.
La commande
test -d
n'attend qu'un paramètre derrière et non pas plusieurs, sauf si on emploie l'option
-a
ou
-o
;-(

De plus, ton test est inutile, la commande
find
seule suffit, elle fera le tri d'elle même. Si elle trouve un dossier nettoyage elle exécutera le reste de la commande, dans le cas contraire elle ne fera rien.
0
Je dois donc remplacé la commande -d par la commande -a ou -o ? ou dois-je mettre la commande avec le -d aussi.

Merci de votre réponse
0
zipe31
Messages postés
36317
Date d'inscription
dimanche 7 novembre 2010
Statut
Contributeur
Dernière intervention
27 janvier 2021
6 568
23 juin 2014 à 14:57
Euh... non, pas tout a fait ;-\

La syntaxe pour tester plusieurs paramètres est la suivante :

if [ -d toto -o -d titi -o -d tata ]; then ... else ... fi

Mais comme précisé dans ma réponse précédente, à mon avis ces tests ne servent à rien. La commande
find
devrait faire le boulot toute seule :

find ${REP} ${REP2} -maxdepth 1 -ctime +10 -exec rm -rf {} \; 2>/dev/null
0
oui mais la commande dans mon cas serait :

if [ -d -o /usr/local/applications/*/nettoyage]; then ... else ... fi
0
zipe31
Messages postés
36317
Date d'inscription
dimanche 7 novembre 2010
Statut
Contributeur
Dernière intervention
27 janvier 2021
6 568
23 juin 2014 à 15:11
Non ! Un répertoire par argument :

if [ -d /usr/local/applications/rep1/nettoyage -o -d /usr/local/applications/rep2/nettoyage -o -d etc.
0
ok d'accord, mais j'ai énormément de répertoire, quand je pourrai optimiser çà ?
0
zipe31
Messages postés
36317
Date d'inscription
dimanche 7 novembre 2010
Statut
Contributeur
Dernière intervention
27 janvier 2021
6 568
23 juin 2014 à 15:15
0