Discussion :

[pierre_harry]

Bonjour,

C'est la première fois que je fais un script, et j'ai un probleme impossible a résoudre malgré la simplicité du code.

Je suis sous ubuntu 14.04 j'execute le script en ligne de commande. Le script a les droits root:root 777

Voici le script :

Code :

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

1
2
3
4
5
6
7
8
#!/bin/bash
 
if test -z "$1"; then commande=""; else commande="$1"; fi
 
if [ "$commande" = "1" || "$commande" = "2" ] ; then
  echo "Bla ? "
  mkdir "/home/toto"
fi

Ce qui me donne :

Code :

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

1
2
3
4
root@host:/home/user01/Bureau/# sh Script/UserIF.sh
Script/UserIF.sh: 8: [: missing ]
Script/UserIF.sh: 8: Script/UserIF.sh: : Permission denied
root@host:/home/user01/Bureau/#

Même en prenant des exemples sur le net, j'ai toujours le même type d'erreur. Je ne vous met pas les dizaines de variante de if then fi que j'ai essayé, c'est toujours la même chose.
Je suis ca : https://doc.ubuntu-fr.org/tutoriel/s...la_structureif
Ou autre exemple :

Code :

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

1
2
3
4
5
6
7
8
#!/bin/bash
 
if test -z "$1"; then commande=""; else commande="$1"; fi
 
if [ $commande = "1" || $commande = "2" ] ; then
  echo "Bla ? "
  mkdir "/home/toto"
fi

Donne

Code :

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

1
2
3
4
root@host:/home/user01/Bureau/# sh Script/UserIF.sh
Script/UserIF.sh: 8: [: missing ]
Script/UserIF.sh: 8: Script/UserIF.sh: =: not found
root@host:/home/user01/Bureau/#

[disedorgue]

Bonjour,

Selon moi, tu as un problème sur le format de ton fichier car à priori, il nous dit pour toutes les erreurs, que c'est la ligne 8...

Peux-tu nous donner le résultat des 2 commandes suivantes:

Code :

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

file Script/UserIF.sh

et

Code :

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

cat -A Script/UserIF.sh

Sinon, la syntaxe la plus correcte des 2 exemples que tu donnes, est la première car les variable sont protégées par des guillemets, dans le 2éme exemple, une erreur de la commande test (ou [ ] ) sera générée si la variable est vide puisque pas protégée par des guillemets.

[pierre_harry]

Sinon, la syntaxe la plus correcte des 2 exemples que tu donnes, est la première car les variable sont protégées par des guillemets, dans le 2éme exemple, une erreur de la commande test (ou [ ] ) sera générée si la variable est vide puisque pas protégée par des guillemets.

En effet, c'est moi qui pensé que mon premier test permettait de me proteger des variables, en fait non ....

Script/UserIF.sh: Bourne-Again shell script, ASCII text executable

root@host:/home/user01/Bureau/# cat -A Script/UserIF.sh
#!/bin/bash$
$
clear$
$
if test -z "$1"; then commande=""; else commande="$1"; fi$
$
if [ $commande = "1" ] || [ $commande = "2" ] ; then$
echo "Bla ? "$
mkdir "/home/toto"$
fi$
$
$
$
if [ $commande = "1" ] || [ $commande = "2" ]; then$
echo "1 ou 2"$
else$
echo "2 ou" plus$
fi$

root@host:/home/user01/Bureau/#

[Flodelarab]

Bonjour

La commande test est abrégée par des crochets. Si tu veux enchaîner des commandes de test, tu peux utiliser un OU || mais il faut fermer le premier test et ouvrir le second test, conformément au lien que tu cites et que tu as mal recopié.

D'où le message:

Code :

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

Script/UserIF.sh: 8: [: missing ]

Ce n'est pas:

Code :

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

[ $commande = "1" || $commande = "2" ]

Mais

Code :

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

[ $commande = "1" ] || [ $commande = "2" ]

Edit: Je vois que tu as corrigé dans la dernière version du script.

[jack-ft]

Ce n'est pas:

Code :

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

[ $commande = "1" || $commande = "2" ]

Mais

Code :

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

[ $commande = "1" ] || [ $commande = "2" ]

Oui.
Ou bien:

Code :

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

[ $commande = "1" -o $commande = "2" ]

[jlliagre]

ou bien encore:

Code :

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

[[ $commande = "1" || $commande = "2" ]]