Discussion :

[GLDavid]

Salut

J'étais persuadé que mon script était exempt de bugs, que nenni !
Je me suis rendu compte de ce petit cas. Soit les lignes suivantes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
if [ $VERBOSE -eq 1 ]
	then
		echo "`date +"%d/%m/%Y-%H:%M:%S"` : $1 in dump"
		svnadmin dump $1 > $DUMP
	else
		svnadmin dump $1 > $DUMP 2>&1
	fi
	RETURN=$?
	SIZE=`du $DUMP | cut -f1`
	if [ $RETURN -eq 0 ]

Le fait est que quelque soit la valeur de $RETURN, le script le voit comme toujours égal à 0. Ce qui fausse bien entendu le test en dernière ligne.
Voyez-vous où est mon erreur ?

Merci d'avance de vos réponses.

@++

[ggnore]

As tu essayé d'afficher ton $?

Si ça se trouve il a tout le temps la même valeur.
Peut être qu'il prend la valeur de retour de ton if...fi ?

Je te propose de mettre ton RETURN= ... dans le if, c'est à dire, de le mettre deux fois (une fois dans le if, une fois dans l'else).

[GLDavid]

Hello Ggnore

J'ai mis mon RETURN=$? dans mes blocs if et else. Malheureusement, l'effet est le même. Le plus rigolo, c'est que $RETURN change de valeur une fois sorti du bloc if-else.

@++

[ggnore]

Peut être que la variable de nom RETURN est spéciale. Tu as essayé de la renommer ?

[GLDavid]

Non, hélas, quand je change RETURN en RETOUR, l'effet est le même
Merci en tout cas de ton aide.

@++

[ggnore]

Voilà un source qui synthétise ton problème.
Effectivement c'est bizarre.

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
#!/bin/sh
retour=0
if [ $1 = "a" ]
        then
        ls -l
        echo ">>>"$?
        retour=$?
else
        azfzefz
        echo ">>>"$?
        retour=$?
fi
 
echo ">>>$?<<<"
echo $retour

Voici un lien en français concernant les codes retour en script shell.
http://okki666.free.fr/docmaster/articles/linux131.html

[GLDavid]

Je ne te le fais pas dire !
Apparemment, pour bash, 1=0 !

@++

[ggnore]

Je ne te le fais pas dire !
Apparemment, pour bash, 1=0 !

@++

Chuis pas d'accord, c'est $retour qui perd sa valeur à la sortie du if comme tu l'as dit toi même.

[GLDavid]

Considères le code suivant :

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
 
if [ $VERBOSE -eq 1 ]
	then
		echo "`date +"%d/%m/%Y-%H:%M:%S"` : $1 in dump"
		svnadmin dump $1 > $DUMP
		RETOUR=$?
		echo ">>>$RETOUR<<<" 
	else
		svnadmin dump $1 > $DUMP 2>&1
		RETOUR=$?
		echo ">>>$RETOUR<<<"
	fi
	SIZE=`du $DUMP | cut -f1`
	echo ">>>$RETOUR<<<"
	if [ $RETOUR -eq 0 ]

Je m'arrange pour que svnadmin dump ne marche pas. Le code de retour est de 1 et la valeur $RETOUR est toujours de 1 à chaque echo.
Bizarre, non ?

[ggnore]

Normalement

• le code "ça s'est bien passé", c'est 0
  le code "arg ça va pas", c'est différent de 0

donc je vois pas de problème

nouveau source :

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
 
#!/bin/sh
retour=0
if [ $1 = "a" ]
then
        ls -l
else
        azfzefz
fi
retour=$?
 
echo ">>>$retour<<<"
ls -l
 
echo ">>>$retour<<<"

Et là, je ne vois plus de problème.
Les retours de week end prolongé ça n'a pas l'air de me réussir

[GLDavid]

Ok, j'ai trouvé !
Le problème venait de la suite du code :

Code :

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

1
2
3
4
 
if [ $RETOUR -eq 0 ]
echo "`date +"%d/%m/%Y-%H:%M:%S"` : svndump successed." >> $LOG
then

Il aurait fallu écrire :

Code :

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

1
2
3
4
 
if [ $RETOUR -eq 0 ]
then
echo "`date +"%d/%m/%Y-%H:%M:%S"` : svndump successed." >> $LOG

Merci encore à toi de ton soutien !

@++

[ggnore]

Ok, j'ai trouvé !
Le problème venait de la suite du code :

Code :

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

1
2
3
4
 
if [ $RETOUR -eq 0 ]
echo "`date +"%d/%m/%Y-%H:%M:%S"` : svndump successed." >> $LOG
then

Il aurait fallu écrire :

Code :

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

1
2
3
4
 
if [ $RETOUR -eq 0 ]
then
echo "`date +"%d/%m/%Y-%H:%M:%S"` : svndump successed." >> $LOG

Merci encore à toi de ton soutien !

@++

Pile la ligne juste après le code que tu donnes ...

J'avais pas beaucoup de chances de trouver

[GLDavid]

Exact,
Le pire, c'est que j'étais sûr de l'écriture de mes lignes.
Bon, comme quoi les scripts shell sont très permissifs (trop !).

@++