Mon premier script bash: (c correcte ou) c tout a revoir(? [ver 0.002]

Mon premier script bash: (c correcte ou) c tout a revoir(? [ver 0.002] - Multimédia - Linux et OS Alternatifs

Marsh Posté le 19-09-2002 à 20:56:08    

Code :
  1. #!/bin/sh
  3. #Variables
  4. NAME="pure-ftpd"
  5. PATH="/usr/local/sbin/pure-ftpd"
  6. OPTION=" -E -j -lpuredb:/etc/pureftpd.pdb"
  7. PID="/var/run/pure-ftpd.pid"
  9. case "$1" in
  10.     start)
  11.         if [ -e $PID ];
  12.         then
  13.             echo "Pure-FTPd already running"
  14.         elif [ -e $PATH ];
  15.         then
  16.             echo "Lancement de Pure-FTPd"
  17.             $PATH $OPTION &
  18.         else
  19.             echo "Pure-FTPd n'a pas été trouvé"
  20.         fi
  21.     ;;
  23.     stop)
  24.         if [ -e $PID ];
  25.         then
  26.             echo "Arret de pure-FTPd"
  27.             /usr/bin/killall $NAME
  28.         else
  29.             echo "Pure-FTPd already down"
  30.         fi
  31.     ;;
  33.     restart)
  34.         $0 start
  35.         $0 stop
  36.     ;;
  38.     *)
  39.         echo "{start|stop|restart}"
  40.         exit 1
  41. esac
  43. exit 0


 
il fonctionne bien mais je c pas si il est ecrit correctement...


Message édité par leirn le 23-09-2002 à 22:50:35

---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 19-09-2002 à 20:56:08   

Reply

Marsh Posté le 21-09-2002 à 20:40:57    

Oui il est écris correctement MAIS :
 
-J'utiliserais le contenu du fichier pid, pour faire une seconde vérification sur la présence du processus.
-Quand tu fais un stop : tu n'es pas obligé de vérifier si le processus tourne.
 
 
J'ajouterais en fonction du fichier pid et de sa présence, un  
status) ou une fonction status()  qui retourne une valeur

Reply

Marsh Posté le 21-09-2002 à 21:02:53    

joli et propre ce script !  
 
pour le exit , retourne plutot le statut d'une execution dans un des choix .  
 
exemple declarer une globale STATUT , puis lui affecter une valeur dans les choix  , comme dans "pureftpd pas trouvé" : STATUT=1    et en fin de script tu retournes un exit $STATUT ( tu dois dans ce cas là affecter aussi une valeur en cas de succes ou alors affecter 0 d'office dans la déclaration de la variable)

Reply

Marsh Posté le 22-09-2002 à 00:12:30    

Ca ne répond sans doute pas à ta question, mais peut-être à tes préoccupations:
http://sourceforge.net/projects/pftpd-sh/

Reply

Marsh Posté le 22-09-2002 à 19:13:03    

becket a écrit a écrit :

Oui il est écris correctement MAIS :
 
-J'utiliserais le contenu du fichier pid, pour faire une seconde vérification sur la présence du processus.
-Quand tu fais un stop : tu n'es pas obligé de vérifier si le processus tourne.
 
 
J'ajouterais en fonction du fichier pid et de sa présence, un  
status) ou une fonction status()  qui retourne une valeur




 
g pas trouvé comment faire ca... ya une fonction ki le fait ou fo ke je fasse autrement par un moyen detourné kkonk?
 
qd a mes preoccupations, faire un script moi meme, c tout :)


Message édité par leirn le 22-09-2002 à 19:13:40

---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 22-09-2002 à 20:50:18    

Juste un détail:
je doute fort que le "restart" fonctionne correctement avec ton script :D

Reply

Marsh Posté le 22-09-2002 à 21:11:39    

[Albator] a écrit a écrit :

Juste un détail:
je doute fort que le "restart" fonctionne correctement avec ton script :D




je m'en suis appercu apres le post, il a deja ete corrigé :D


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 22-09-2002 à 21:16:24    

Voici ma nouvelle version du script, a vous de jouer!
 

Code :
  1. #!/bin/sh
  2. #
  3. # Fichier de lancement/arret de pure ftpd version 0.001
  4. # ecrit par Leirn
  5. #
  7. #
  8. # Liste des statuts:
  9. # 0: arret normal
  10. # 1: fichier introuvable
  11. # 2: serveur deja lancé
  12. # 3: argument incompris
  13. # 4: status
  14. #
  17. #Variables
  18. NAME="pure-ftpd"
  19. PUREPATH="/usr/local/sbin/pure-ftpd"
  20. OPTION=" -E -j -lpuredb:/etc/pureftpd.pdb"
  21. PID="/var/run/pure-ftpd.pid"
  22. STATUT=0
  25. case "$1" in
  26.         start)
  27.                 if [ -e $PID ];
  28.                 then
  29.                         if [ -d /proc/$(<$PID) ];
  30.                         then
  31.                                 echo "Pure-FTPd already running"
  32.                                 STATUT=2
  33.                         fi
  34.                 elif [ -x $PUREPATH ];
  35.                 then
  36.                         echo "Lancement de Pure-FTPd"
  37.                         $PUREPATH $OPTION &
  38.                 else
  39.                         echo "Pure-FTPd n'a pas été trouvé"
  40.                         STATUT=1
  41.                 fi
  42.         ;;
  44.         stop)
  45.                 echo "Arret de pure-FTPd"
  46.                 /usr/bin/killall $NAME
  47.         ;;
  49.         restart)
  50.                 $0 stop
  51.                 $0 start
  52.         ;;
  54.         status)
  55.                 if [ -e $PID ];
  56.                 then
  57.                         echo "Server running
  58.                         echo "/proc/$PID""
  59.                         echo "Pid: "$(< $PID)
  60.                 else
  61.                         echo "Server down"
  62.                 fi
  63.                 STATUT=4
  64.         ;;
  66.         *)
  67.                 echo "{start|stop|restart|status}"
  68.                 STATUT=3
  69. esac
  71. exit $STATUT


Message édité par leirn le 22-09-2002 à 21:38:39

---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 22-09-2002 à 21:29:55    

Je sais pas si c'est volontaire ou pas, mais appeler une variable "PATH" c'est pas une très bonne idée ...
 
De plus, j'utiliserais [ -x "$PATH" ] plutot que [ -e $PATH ] .
 
Sinon, pourquoi utiliser une variable "STATUT" plutot que de faire exit directement ?
 
Enfin, je ne vois pas à quel endroit tu effaces ton fichier PID.

Reply

Marsh Posté le 22-09-2002 à 21:37:10    

[Albator] a écrit a écrit :

Je sais pas si c'est volontaire ou pas, mais appeler une variable "PATH" c'est pas une très bonne idée ...
 
De plus, j'utiliserais [ -x "$PATH" ] plutot que [ -e $PATH ] .
 
Sinon, pourquoi utiliser une variable "STATUT" plutot que de faire exit directement ?
 
Enfin, je ne vois pas à quel endroit tu effaces ton fichier PID.




 
je ne l'efface pas par ce qu'il me semble que pure le fait seul... d'apres les tests ke g pu en faire...
 
j'utilise statut du au fait ke kkun me la proposé avant
 
pour le x, c une eventualité... je le modifie de suite, ainsi ke le nom du path


Message édité par leirn le 22-09-2002 à 21:38:07

---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 22-09-2002 à 21:37:10   

Reply

Marsh Posté le 23-09-2002 à 21:07:47    

Je verrais plutot quelque chose du genre pour status)
 
 
status)
 
if [  $( ps -C $PROGRAMME -o pid=  )  -eq $( cat $PID ) ]
then echo " le programme tourne -> OK"
 
else
 
...
 
fi


Message édité par becket le 23-09-2002 à 21:10:07
Reply

Marsh Posté le 23-09-2002 à 21:30:47    

becket a écrit a écrit :

Je verrais plutot quelque chose du genre pour status)
 
 
status)
 
if [  $( ps -C $PROGRAMME -o pid=  )  -eq $( cat $PID ) ]
then echo " le programme tourne -> OK"
 
else
 
...
 
fi




 
en fait je c pas pkoi c pas la derniere version


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 23-09-2002 à 22:11:26    

voici la version 0.002 avec status, et normaleent tout fonctionne... suaf si g encore deconner...
 

Code :
  1. #!/bin/sh
  2. #
  3. # Fichier de lancement/arret de pure ftpd version 0.002
  4. # ecrit par Leirn
  5. #
  7. #
  8. # Liste des statuts:
  9. # 0: arret normal
  10. # 1: fichier introuvable
  11. # 2: serveur deja lancé
  12. # 3: argument incompris
  13. # 4: status
  14. #
  17. #Variables
  18. NAME="pure-ftpd"
  19. PUREPATH="/usr/local/sbin/pure-ftpd"
  20. OPTIONS="-E -j -lpuredb:/etc/pureftpd.pdb"
  21. PID="/var/run/pure-ftpd.pid"
  22. STATUT=0
  25. case "$1" in
  26.         start)
  27.                 if [ -e $PID ];
  28.                 then
  29.                         if [ -d /proc/$(<$PID) ];
  30.                         then
  31.                                 echo "Pure-FTPd already running"
  32.                                 STATUT=2
  33.                         fi
  34.                 elif [ -x $PUREPATH ];
  35.                 then
  36.                         echo "Lancement de Pure-FTPd"
  37.                         $PUREPATH $OPTIONS &
  38.                 else
  39.                         echo "Pure-FTPd n'a pas été trouvé"
  40.                         STATUT=1
  41.                 fi
  42.         ;;
  44.         stop)
  45.                 echo "Arret de pure-FTPd"
  46.                 /usr/bin/killall $NAME
  47.         ;;
  49.         restart)
  50.                 $0 stop
  51.                 $0 start
  52.         ;;
  54.         status)
  55.                 if [ -e $PID ];
  56.                 then
  57.                         if [ -d /proc/$(<$PID) ];
  58.                         then
  59.                                 echo "Server running"
  60.                                 echo "Pid: "$(<$PID)
  61.                         fi
  62.                 else
  63.                         echo "Server down"
  64.                 fi
  65.                 STATUT=4
  66.         ;;
  68.         *)
  69.                 echo "{start|stop|restart|status}"
  70.                 STATUT=3
  71.         ;;
  72. esac
  74. exit $STATUT


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 23-09-2002 à 22:45:55    

la je capte pas:
 
 

Code :
  1. #!/bin/sh
  2. #
  3. # Fichier de lancement/arret de pure ftpd version 0.002
  4. # ecrit par Leirn
  5. #
  7. #
  8. # Liste des statuts:
  9. # 0: arret normal
  10. # 1: fichier introuvable
  11. # 2: serveur deja lancé
  12. # 3: argument incompris
  13. # 4: status
  14. #
  16. #Variables
  17. NAME="pure-ftpd"
  18. PUREPATH="/usr/local/sbin/pure-ftpd"
  19. OPTION="-E -j -lpuredb:/etc/pureftpd.pdb"
  20. PID="/var/run/pure-ftpd.pid"
  21. STATUT=0
  24. case "$1" in
  25.         start)
  26.                 if [ -e $PID ];
  27.                 then
  28.                         if [ -d /proc/$(<$PID) ];
  29.                         then
  30.                                 echo "Pure-FTPd already running"
  31.                                 STATUT=2
  32.                         fi
  33.                 elif [ -x $PUREPATH ];
  34.                 then
  35.                         echo "Lancement de Pure-FTPd"
  36.                         $PUREPATH $OPTION &
  37.                 else
  38.                         echo "Pure-FTPd n'a pas été trouvé"
  39.                         STATUT=1
  40.                 fi
  41.         ;;
  43.         stop)
  44.                 echo "Arret de pure-FTPd"
  45.                 /usr/bin/killall $NAME
  46.         ;;
  48.         restart)
  49.                 $0 stop
  50.                 $0 start
  51.         ;;
  53.         status)
  54.                 if [ -e $PID ];
  55.                 then
  56.                         if [ -d /proc/$(<$PID) ];
  57.                         then
  58.                                 echo "Server running"
  59.                                 echo "Pid: "$(<$PID)
  60.                         fi
  61.                 else
  62.                         echo "Server down"
  63.                 fi
  64.                 STATUT=4
  65.         ;;
  67.         *)
  68.                 echo "{start|stop|restart|status}"
  69.                 STATUT=3
  71. esac
  73. exit $STATUT


 
il est tel kel, le stop fonctionne, le start fonctionne, mais le restart n'effectu k'un stop :/
 
vous savez pkoa?


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 23-09-2002 à 23:22:10    

et si tu met un sleep entre les 2 ( peut etre que le stop/start va trop vite )  
 
faut faire gaffe a un truc aussi avec pureftpd toute nouvelle connection sur le ftp recrée un process ( IDLE UPLOAD ou DOWNLOAD )
et dans le cas d un reload il est plus judicieux de faire un kill -HUP  .

Reply

Marsh Posté le 23-09-2002 à 23:42:42    

houplaboom42 a écrit a écrit :

et si tu met un sleep entre les 2 ( peut etre que le stop/start va trop vite )  
 
faut faire gaffe a un truc aussi avec pureftpd toute nouvelle connection sur le ftp recrée un process ( IDLE UPLOAD ou DOWNLOAD )
et dans le cas d un reload il est plus judicieux de faire un kill -HUP  .




 
ke dalle, g l'impression kil appelle meme pas start


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 24-09-2002 à 06:53:53    

1-
 
/etc/init.d/$0 stop
/etc/init.d/$0 start
 
 
2- Ton status) ne vérifie toujours pas si le numero de pid contenu dans ton fichier pid correspond bien au pid de ton process  
 
3- Comme déjà dit auparavant, quand tu fais stop), tu dois vérifier   la supression ou supprimer via ton script le fichier  pid

Reply

Marsh Posté le 24-09-2002 à 10:50:53    

A tous les coups, ton script de démarrage porte le même nom que l'éxécutable qu'il lance, et du coup quand tu fais stop, il se tue lui-même en appelant killall .
 
C'est pas une bonne idée d'utiliser killall  :hello:

Reply

Marsh Posté le 24-09-2002 à 16:50:58    

[Albator] a écrit a écrit :

A tous les coups, ton script de démarrage porte le même nom que l'éxécutable qu'il lance, et du coup quand tu fais stop, il se tue lui-même en appelant killall .
 
C'est pas une bonne idée d'utiliser killall  :hello:  




 
c ke t pas con toa....
 
un petit kill $(<$PID) alors...?


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 24-09-2002 à 16:52:06    

becket a écrit a écrit :

1-
 
/etc/init.d/$0 stop
/etc/init.d/$0 start
 
 
2- Ton status) ne vérifie toujours pas si le numero de pid contenu dans ton fichier pid correspond bien au pid de ton process  
 
3- Comme déjà dit auparavant, quand tu fais stop), tu dois vérifier   la supression ou supprimer via ton script le fichier  pid  




 
comme je te l'ai deja dit le fichier pid est supprimé deja, pas par moi


---------------
"Je brandirai une épée d'orichalque, je m'assouvirai sur des Templiers." | "Avec dans son sillage l'Ombre du Diable, Leirn appelait les morts pour une danse macabre et déchainaît les horreurs de la nuit..."
Reply

Marsh Posté le 24-09-2002 à 19:16:45    

Rappeler ton script dans le restart, c'est pas une bonne idée. Ecrit plutôt une fonction qui fait le start, une fonction qui fait le stop, et appelle ces deux fonctions dans le restart. Ca donnerait qq chose du genre :

Code :
  1. [...]
  3. pftpd_start () {
  4.         [...]
  5. }
  7. pftpd_stop () {
  8.         [...]
  9. }
  11. case "$1" in
  12.         start)
  13.                 pftpd_start
  14.                 ;;
  15.         stop)
  16.                 pftpd_stop
  17.                 ;;
  18.         restart)
  19.                 pftpd_stop
  20.                 pftpd_start
  21.                 ;;
  22. [...]


 
Austre chose : tu n'est pas obligé de mettre de ";" au bout de ta ligne si tu met le then sur une autre ligne. Par contre si tu le met, tu peux mettre le le then à la suite.


Message édité par matafan le 24-09-2002 à 19:18:04
Reply

Marsh Posté le 24-09-2002 à 21:19:33    

oki, je fais ca tout al'heure, merci

Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed