Problème script shell

Version imprimable

bonjour,
voici mon script :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#!/bin/sh
# bin/telnet
 
while [ 1 ];do
{
 
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 $MACHINE";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";
 
   while [ $PING ] && [ i!=720 ];do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done;
 
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e "$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e "${CMD[5]}\15";) | telnet $MACHINE $PORT;
 
   while [ ! $PING ];do
   {
      sleep 5;
   }
   done;
 
}
done;
exit 0;

ce script doit me permettre de "pinguer" en continue un switch et si au bout d'une heure une on peu toujours le "pinguer" alors il faut fermer un port du switch.
mon probleme est que mon script ne marche pas.
il me met : too many arguments line 20 line 32.
je ne comprends pas pourquoi ? Auriez vous une idée ? Pouvez vous m'aider s'il vous plait ?

30/06/2006, 09h22
bessonnet
sur ligne 20 et ligne 32 se trouve :
ligne 20
Code:

1 2 3 4 5 6 while [ $PING ] && [ i!=720 ];do { sleep 5; i=`expr $i + 1`; } done;
ligne 32
Code:

1 2 3 4 5 while [ ! $PING ];do { sleep 5; } done;
voila je prends toute les remarques et commentaires constructif qui me permettront d'avancer.
je remercie tous ceux qui ont pris le temps de lire ce topic.
Cordialement
30/06/2006, 09h25
Arnaud F.

Déjà dans dans ta seconde boucle while, sache que la comparaison ne se fait pas avec != mais -ne (pour Not Equal donc ta ligne deviendra

Code:

while [ $PING && i -ne 720 ];do

Et il ne faut pas mettre d'accolades non plus dans la boucle while ;)

[edit] il faut faire l'opérateur logique dans les crochets aussi :oops: (j'ai modifié le code)

[edit2] pas de ; après le done :lol:
30/06/2006, 09h40
bessonnet

et pour :

Code:

while [ ! $PING ];do

j'aurais donc :

Code:

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci
30/06/2006, 09h52
bessonnet

j'ai donc effectué les modifications,
pour la ligne 32, au premier tour l'erreur à disparue, au tour d'après elle est revenue, la meme : "too many arguments".
pour la ligne 20 elle s'est changée en : [:missing`]'
voila je m'y connais pas encore assez pour tout comprendre.
si vous pouviez continiuer à me mettre sur la bonne voie je vous en serais reconnaissante.
merci
30/06/2006, 09h57
gorgonite

pourquoi pas plutôt cela pour le test de ping ???

Code:

[$PING -ne 0]
30/06/2006, 10h05
Arnaud F.

Citation:

Envoyé par gorgonite

pourquoi pas plutôt cela pour le test de ping ???

Code:

[$PING -ne 0]

+1

:oops: j'avais meme pas fait attention à ça ...
30/06/2006, 10h07
Arnaud F.

Citation:

Envoyé par bessonnet

et pour :

Code:

while [ ! $PING ];do

j'aurais donc :

Code:

while [ -ne $PING ];do

si j'ai bien compris ?
je vai essayer
merci

Non car quand tu utilise -ne il faut comparer par rapport à une valeur or si tu fais ça, ce n'est pas le cas ;)

Suis ce qu'a écris gorgonite ;)

P.S: Pense à le modifier dans tes 2 boucles ;)
30/06/2006, 10h25
bessonnet

merci,
j'ai changé dans les deux boucles, mais les erreurs persistes !
peu etre y à t'il des erreurs ailleurs que sur ces deux lignes là ? parce que là je ne vois vraiment pas :? ...
quand je fais : "sh -n ping.sh" il ne m'affiche aucune erreur, c'est quand je le lance (sh ping.sh) qu'il commence son execution et en meme temps il m'affiche ces deux erreurs.
au finale il tourne en rond avec les deux erreurs.

auriez vous d'autres idées ?
30/06/2006, 11h18
Arnaud F.

Oui, une seule, post nous ton code mis à jour maintenant, qu'on voit réellement ce que tu à modifié ;)

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#!/bin/sh
# bin/telnet
 
while [ 1 ];do
{
 
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 @ip";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";
 
   while [ [ $PING ] && [ i -ne 720 ] ]; do
   {
      sleep 5;
      i=`expr $i + 1`;
   }
   done
 
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e 
 
"$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 
 
5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e 
 
"${CMD[5]}\15";) | telnet $MACHINE $PORT;
 
   while [ $PING -ne 0 ]; do
   {
      sleep 5;
   }
   done
 
}
done
 
exit 0;

30/06/2006, 14h08
Arnaud F.

Enlève les accolades dans les boucles while
Et tu veux tester tant que $PING égal à quoi?

Au passage tu n'a pas besoin de mettre tes sous-expression entre crochet:

Code:

while [ $PING && i -ne 720 ];

Voilà essai déjà ça :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#!/bin/sh
# bin/telnet

while [ 1 ]; do
   #declaration des variables globales
   i="0";
   PORT="23";
   PING="ping -c 1 @ip";
   MACHINE="@ip";
   LOGIN="login";
   PWD="pwd";
   CMD[1]="ethernet";
   CMD[2]="portState";
   CMD[3]="8";
   CMD[4]="disable";
   CMD[5]="logout";

   # Tu veux tester le PING pour quelle valeur?
   while [  $PING -eq TaValeur  &&  i -ne 720  ]; do
      sleep 5;
      i=`expr $i + 1`;
   done
  
   #ouverture d'une session telnet sur le switch 3com
   (sleep 2; echo -e "\15"; sleep 2; echo -e "$LOGIN\15"; sleep 2; echo -e 

"$PWD\15"; sleep 2; echo -e "${CMD[1]}\15"; sleep 5; echo -e "${CMD[2]}\15"; sleep 

5; echo -e "${CMD[3]}\15"; sleep 5; echo -e "${CMD[4]}\15"; sleep 5; echo -e 

"${CMD[5]}\15";) | telnet $MACHINE $PORT;

   while [ $PING -ne 0 ]; do
      sleep 5;
   done
done

exit 0;

Voilà, un code qui me semble un peu plus juste, teste déjà ça et dis nous si tu as encore des erreurs ;)

30/06/2006, 15h26
bessonnet

en enlevant les crochet j'ai essayer dejà, ça ne change rien et comme ej trouve ça plus clair avec je les laisse.
ce que je veux c'est "pinguer" le switch et tant qu'il repond attendre 1h et au bout de cette heure fermer le port correspondant du switch.
le port du switch je le connais.

et j'ai vu que j'avais oublié un $ devant mon i :

Code:

while [ $PING -eq TaValeur && $i -ne 720 ]; do

donc je ne sais pas si ta ligne de code est correct, est ce que pour verifier si ça ping je dois donner une valeur à $ping ?
si oui quel genre ?
30/06/2006, 15h38
Arnaud F.

Dl pour le $ du i mais c'est une erreur de copié-coller ;)

En ce qui concerne la valeur du ping à testé je ne peux pas t'aider car ça sort de mes compétences (assez limité comme tu peux le constater :roll: )

Il te renvoi quelles erreurs maintenant? toujours les meme?

[edit] Je viens de voir qqchose

Code:

PING="ping -c 1 @ip";

Tu veux que ton shell script execute la commande ping -c 1 @ip à partir du shell non?
Si c'est le cas, tu dois mettre des backquotes pour executer la commande:

Code:

PING=`ping -c 1 @ip`;
30/06/2006, 15h45
bessonnet

merci pour cette dernière info je ne savais pas cela !
je vai mettre des backquotes tout de suite !
merci