[résolu][BASH]commande for foireuse ?

commande for foireuse ? [résolu][BASH] - Codes et scripts - Linux et OS Alternatifs

Marsh Posté le 02-12-2011 à 13:49:59    

Bonjour à tous, j'ai un petit soucis avec un de mes scripts, chargé de modifier des droits utilisateurs.
Je vous mets la version expurgée de pas mal de commandes subsidiaire :

Code :
  1. #!/bin/bash
  3. # 2010-2011 - GNU GPL
  4. # Script pour placer les ACL plus rapidement que par l'utilisation
  5. # seule de la ligne de commande. Il ne s'agit que de raccourcis et
  6. # d'une demande en boucle de la commande.
  8. entrer_utilisateur() {
  9. echo "### ENTRER UN UTILISATEUR SUR UN DOSSIER ###"
  10. echo "Quel dossier (depuis la racine) voulez-vous modifier ?"
  11. read cible
  12. echo "Avant modification :"
  13. getfacl $cible
  14. echo "Voulez-vous le modifier ? (o/N)"
  15. read lecture
  16.  case $lecture in
  17.   o | O) break
  18.   ;;
  19.   *) echo "Abandon."
  20.   acl_base
  21.  esac
  22. echo "Combien d'utilisateurs voulez-vous entrer ?"
  23. read nbre
  24. for ((i=0; i<$nbre; i++)); do
  25. echo "Nom d'utilisateur ?"
  26. read nom
  27. setfacl -Rm u:$nom:rwx $cible
  28. setfacl -Rm d:u:$nom:rwx $cible
  29. done
  30. setfacl -Rk u:: $cible
  31. setfacl -Rk d:u:: $cible
  32. setfacl -Rx d:u:: $cible
  33. echo "Après modification :"
  34. getfacl $cible
  35. acl_base
  36. }
  37. (... bla bla bla autres commandes du même genre)
  38. acl_base(){
  39. echo "Que voulez-vous faire ?"
  40. echo "1 - entrer l'accès à un ou plusieurs utilisateurs."
  41. echo "2 - supprimer l'accès à un ou plusieurs utilisateurs."
  42. echo "3 - entrer l'accès à un ou plusieurs groupes."
  43. echo "4 - supprimer l'accès à un ou plusieurs groupes."
  44. echo "5 - supprimer un groupe par défaut sur un dossier."
  45. echo "6 - supprimer un utilisateur par défaut sur un dossier."
  46. echo "7 - bloquer un utilisateur sur un dossier."
  47. echo "0 - pour annuler."
  48.  read lecture
  49.  case $lecture in
  50.   1) entrer_utilisateur
  51.   ;;
  52.   2) supprimer_utilisateur
  53.   ;;
  54.   3) entrer_groupe
  55.   ;;
  56.   4) supprimer_groupe
  57.   ;;
  58.   5) supprimer_defautg
  59.   ;;
  60.   6) supprimer_defautu
  61.   ;;
  62.   7) bloquer_util
  63.   ;;
  64.   *) echo "Abandon."
  65.   exit 1
  66.  esac
  67. }
  69. #verification des droits de l'utilisateur (id root = 0)
  70. IDuser=`id -u`
  72. if [ ! $IDuser -eq 0 ]
  73. then
  74.  echo "Vous  n'avez pas les droits ROOT."
  75.  exit 1
  76. fi
  77. acl_base
  78. exit 0


 
Et là c'est le drame, par un sh monscript.sh j'ai en retour :

Citation :

monscript.sh 24: Syntax error: Bad for loop variable


 
WTF ? :heink:


Message édité par bardiel le 09-12-2011 à 23:34:31

---------------
Grippe ? Coronavirus ? Portez votre masque correctement ! :D
Reply

Marsh Posté le 02-12-2011 à 13:49:59   

Reply

Marsh Posté le 02-12-2011 à 14:07:43    

bash monscript.sh

Reply

Marsh Posté le 02-12-2011 à 15:58:55    

Hmm ok, ça fonctionne [:cdtf:5]  
 
Donc je le lance, et paf à un moment pour valider ou non la modification, correspondant à ce bout de code :

Code :
  1. echo "Voulez-vous le modifier ? (o/N)"
  2.  case $lecture in
  3.   o | O) break
  4.   ;;
  5.   *) echo "Abandon."
  6.   acl_base
  7.  esac
  8. (... la suite du script)


J'ai en magnifique réponse si je répond "o" ou "O" (pour oui quoi :o ) :

Citation :

break: ceci n'a un sens que dans une boucle « for », « while » ou « until »


Mais le pire dans l'histoire, c'est que le script continue normalement après :pt1cable:  
 
Pourquoi un foutu script farpaitement fonctionnel sur Debian Etch ne fait que des misères sur Debian Squeeze ? :pfff:


---------------
Grippe ? Coronavirus ? Portez votre masque correctement ! :D
Reply

Marsh Posté le 02-12-2011 à 16:58:19    

Pourquoi tu ne fais pas un if? Ton case est inutile.
 

Code :
  1. if [[  ( $lecture != "o" ) && ( $lecture != "O" ) ]]
  2. then
  3.          echo "Abandon."
  4.          acl_base
  5. fi


Message édité par Kadaj le 02-12-2011 à 16:58:41
Reply

Marsh Posté le 02-12-2011 à 17:04:44    

bah, c'était pour faire "plus propre" et "parce que j'avais appris comme ça", néanmoins très bonne remarque je change :jap:  
 
...
 
Sauf que je ne pourrais retester que lundi, vu qu'il s'agit d'un serveur du boulot, et que je n'ai pas envie d'y retourner de suite :D


---------------
Grippe ? Coronavirus ? Portez votre masque correctement ! :D
Reply

Marsh Posté le 02-12-2011 à 20:08:20    

les joies du shell et ses variantes, moi c'est avec les tests if que j'ai du mal :D


---------------
http://agentoss.wordpress.com/
Reply

Marsh Posté le 09-12-2011 à 21:27:47    

Manque un ;; ma poule :D :

case $lecture in
  o | O) break
  ;;
  *) echo "Abandon."
  acl_base
 ;;
 esac


pareil :

case $lecture in
  1) entrer_utilisateur
  ;;
  2) supprimer_utilisateur
  ;;
  3) entrer_groupe
  ;;
  4) supprimer_groupe
  ;;
  5) supprimer_defautg
  ;;
  6) supprimer_defautu
  ;;
  7) bloquer_util
  ;;
  *) echo "Abandon."
  exit 1
  ;;
 esac


Message édité par twocats le 09-12-2011 à 21:34:29

---------------
La réponse est 42
Reply

Marsh Posté le 09-12-2011 à 23:34:12    

Effectivement, avec le ;; ça fonctionne déjà beaucoup mieux :o
Je l'avais trouvé ce matin... mais pourquoi ça fonctionnait avant sans me mettre cette erreur ? :pt1cable:


---------------
Grippe ? Coronavirus ? Portez votre masque correctement ! :D
Reply

Sujets relatifs:

Leave a Replay

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