Discussion :

[Olivier Regnier]

Bonjour

Je rencontre un petit soucis avec le code suivant qui permet de vérifier si le serveur X est lancé:

Code :

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

1
2
3
4
if [ -s `ps -aux | grep X` ]; then
   echo "Attention, le serveur X n'est pas lancé."
   exit 0
fi

Il m'indique le message suivant:

Code :

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

[: 32651: unexpected operator/operand

Qu'en pensez-vous ?

Merci

[gerald3d]

Sans vraiment répondre à ta question, j'ai plutôt ca comme erreur avec ton code :

Code :

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

1
2
Warning: bad ps syntax, perhaps a bogus '-'? See http://procps.sf.net/faq.html
./test_bash: line 1: [: too many arguments

P.S.: enlève le "-" avant aux. Ca ira déjà mieux.

[Olivier Regnier]

Bonjour

Ta solution ne change pas le message à l'écran

Puis, j'utilise KSH

[Davidbrcz]

Code :

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

1
2
3
4
5
6
7
8
9
 
#!/bin/ksh
 
if [ -s 'ps aux | grep X' ]; then
   echo "Attention, le serveur X n'est pas lancé."
   exit 0
else
   echo "X est lancé."
fi

Ce code marche chez moi et me donne "X est lancé".

[Olivier Regnier]

Davidbrcz: désolé mais chez moi, cela ne fonctionne pas.

Essaye ce script en mode console:

Code :

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

1
2
3
4
5
6
7
 
if [ -s `ps aux | grep X` ]; then
   echo "Attention, le serveur X n'est pas lancé."
   exit 0
fi
 
echo "coucou"

Logiquement, tu verras apparaître coucou donc le script n'est pas bon.

Teste-le ensuite en mode graphique et comme par hasard, coucou s'affiche aussi

En revanche, tu peux tester le script suivant:

Code :

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

1
2
3
4
5
6
if [ ! -f /tmp/.X0-lock ]; then
	echo "Attention, le serveur X n'est pas démarré."
	exit 0
fi
 
echo "coucou"

Se basé sur le fichier temporaire n'est pas non plus la meilleure solution, mais il a le mérite de fonctionner

D'autres avis ?

[tamiel]

Code :

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

1
2
 
ps $(cat /tmp/.X0-lock ) &>/dev/null && echo "X running" || (echo "X not running"; exit 1;)

[Olivier Regnier]

Code :

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

1
2
 
ps $(cat /tmp/.X0-lock ) &>/dev/null && echo "X running" || (echo "X not running"; exit 1;)

Quel est l'avantage de ce code par rapport à celui que j'ai fourni un peu plus haut. Il est un peu plus long mais à part cela ?

Merci

[tamiel]

Je teste le pid du serveur X pour savoir s'il tourne réellement.

[barnab2]

Deux remarques :
1)

Code :

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

"ps aux | grep X"

vas te montrer aussi ta commande "grep X", car "ps aux" affiche les arguments de "grep". Donc cette commande retourne toujours "vrai" que X soit là ou pas.

tu pourrais faire

Code :

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

ps aux | grep -v grep | grep X

mais c'est moche

le mieux c'est que ps ne montre pas les arguments.
utilises plutot

Code :

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

 "ps -e"

equivalent BSD

Code :

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

 ps ao pid,comm

2) "if" sert à tester le retour d'une commande.
Le "[" est l'equivalent de la commande "test" qui permet de comparer des variables et retourne vrai ou faux en fonction du test

pour tester le retour d'un "grep", pas la peine de mettre des "[...]" puisque grep est une commande.

Code :

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

1
2
3
if ps ao comm | grep X >/dev/null; then
    echo X est la
fi

ou plus joli en rendant grep silencieux

Code :

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

1
2
3
if ps ao comm| grep -q X ; then
    echo X est la
fi

ou carrement plus court

Code :

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

ps ao comm | grep -q X && echo X est la

[Olivier Regnier]

Merci pour toutes ces informations