Discussion :

[GLDavid]

Salut

Dans la rubrique "GLDavid = Alzeihmer", voici mon problème.
J'édite un script qui doit être lancé au démarrage. Ce que je voudrais c'est récupérer le pid du processus que je lance.
Voici comment je m'y prend sous FedoraCore3 :

Code :

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

1
2
3
4
5
 
getPID(){
	pidno=pidof("svnserve -d -r /home/gldavid/repository/")
	return $pidno
}

mais à l'exécution :

Code :

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

1
2
 
./subversion_server.sh: line 6: syntax error near unexpected token `('

Scrongneugneu !!! Ke ki va pas ??
Merci d'avance.

@++

[Smortex]

Salut

Code :

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

1
2
3
4
5
NOM
       pidof -- affiche le PID d'un programme
 
SYNOPSIS
       pidof [-s] [-x] [-o omitpid] [-o omitpid..]  program [program..]

Donc je dirais :

Code :

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

1
2
svnserve -d -r /home/gldavid/repository/
return `pidof svnserve`

[ggnore]

pourquoi mettres des parenthèses à l'appel de pidof ?

smortex a sûrement raison ceci dit.

[MarcG]

peut être aussi :

BOGUE
Quand pidof est exécuté avec le chemin complet du programme dont il
doit chercher le PID, son comportement est fiable. Dans le cas con-
traire, les PIDs renvoyés peuvent être ceux de processus portant le
même nom mais qui sont en fait des programmes différents

[ggnore]

peut être aussi :

BOGUE
Quand pidof est exécuté avec le chemin complet du programme dont il
doit chercher le PID, son comportement est fiable. Dans le cas con-
traire, les PIDs renvoyés peuvent être ceux de processus portant le
même nom mais qui sont en fait des programmes différents

Intéressant effectivement, mais son problème vient de la syntaxe de son script, et non pas d'une mauvaise valeur renvoyée.

Ceci dit, ce que tu indiques est très intéressant, ça vaudrait le coup de tester que le processus n'est présent qu'une seule fois dans

Code :

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

`ps aux`

histoire de pas se planter.

[GLDavid]

Bonjour

Veuillez excuser le retard dans ma réponse mais ce vendredi, il y a eu une grosse coupure d'électricité sur Namur. Bref, pas de boulot hors mis papier.
revenons-en à mon problème. Je vous adresse mon 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
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
 
#!/bin/bash
 
#This script is to launch the subversion server with security
 
getPID(){
	echo $1
	pidno=pidof $1
	return $pidno
}
 
start() {
	svnserve -d -r /home/dbourgais/repository/
	RETVAL=$?
	return $RETVAL	
}
 
stop() {
	arg="svnserve"
	echo $arg
	$pid=getPID($arg)
	kill $pid
	RETVAL=$?
	return $RETVAL	
}
 
case "$1" in
	start)
		start
		;;
	stop)
		stop
		;;
	restart)
		stop
		start
		;;
	*)
		echo $"Usage: $prog {start|stop|restart}"
		exit 1
esac
exit $RETVAL

Mon problème est toujours le même :

Code :

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

1
2
3
 
./subversion_server.sh: line 20: syntax error near unexpected token `$arg'
./subversion_server.sh: line 20: `      $pid=getPID($arg)'

Le problème intervient donc à cette ligne :

Code :

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

1
2
 
$pid=getPID($arg)

C'est peut être tout bête mais pourquoi n'accepte t'il pas ma chaîne de caractère ?
Merci d'avance de vos réponse et veuillez excuser encore le retard.

@++

[GLDavid]

Bon, j'ai résolu le problème en le contournant. Je trouve ma méthode "crade" mais fonctionnelle, voici mon correctif :

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
#!/bin/bash
 
#This script is to launch the subversion server with security
#
# chkconfig: 345 91 60
# description: execute subversion server with security
#
 
start() {
	svnserve -d -r /home/dbourgais/repository/
	RETVAL=$?
	return $RETVAL	
}
 
stop() {
	killproc `which svnserve`
	RETVAL=$?
	return $RETVAL	
}
 
case "$1" in
	start)
		start
		;;
	stop)
		stop
		;;
	restart)
		stop
		start
		;;
	*)
		echo $"Usage: $prog {start|stop|restart}"
		exit 1
esac
exit $RETVAL

Ca fait un code plus léger. Bien que je mette le tag Résolu, je reste ouvert à toutes optimisations.

@++

[Smortex]

Le problème intervient donc à cette ligne :

Code :

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

1
2
 
$pid=getPID($arg)

C'est peut être tout bête mais pourquoi n'accepte t'il pas ma chaîne de caractère ?

EN script shell, on met pas de parentheses... sauf pour déclarer une fonction (et certainement pas pour l'utiliser !)

je reste ouvert à toutes optimisations.

Enlever le $ après le echo dans :

Code :

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

echo $"Usage: $prog {start|stop|restart}"

(A moins que ce soit moi qui ai du mal a saisir son rôle)

[GLDavid]

Ok, merci Smortex, je vais apporter quelques retouches à mon script en suivant tes remarques.

@++

[ggnore]

pourquoi mettres des parenthèses à l'appel de pidof ?

[GLDavid]

Mes plus humbles excuses ggnore. J'ai lu trop vite. Bon, allez, je t'autorise :
ggnore--><--GLDavid

@++