[bash] Problème de boucle

jp@MDK:~/tmpfs ssh$ ls rep/
paquet1  paquet2  paquet3  paquet4

jp@MDK:~/tmpfs ssh$ cat foo.sh
#! /bin/sh

#set -xv

echo "Liste des paquets : "
ls -1 rep/ | nl
read -p "Choisissez le paquet à installer (entrez son numéro) : " choix
nom="$(ls -1 rep/ | sed -n "${choix}p")"
echo "Installation de ${nom} en cours..."

jp@MDK:~/tmpfs ssh$ ./foo.sh
Liste des paquets :
     1  paquet1
     2  paquet2
     3  paquet3
     4  paquet4
Choisissez le paquet à installer (entrez son numéro) : 3
Installation de paquet3 en cours...

jp@MDK:~/tmpfs ssh$

;-))

ls -1 rep/
man ls pour plus d'explications.
-1 Présenter un fichier par ligne

|
Pipe. On envoie le résultat de "ls -1" à travers un "pipe" (filtre) à la prochaine commande

nl
man nl. Afficher un fichier en numérotant ses lignes.

nom="$(ls -1 rep/ | sed -n "${choix}p")"
On récupère dans une variable le fichier correspondant au numéro choisi à l'aide de la commande "sed"

Salut,

Il faut des espaces autour du signe "égale" !

if [ "${conf]" == 'o' ]; then 

merci jipicy

j'essai tout de suite

Sa ne marche pas, je met mon code en entier

#!/bin/bash
demande2=non

for liste in `ls /home/mimic/Bureau/packages/`
do
echo $liste
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

if [ "$choix" == 'o' ]; then
#until [ $demande2="non"]; do
echo "Vous avez donc choisit Direct News en version pro ? [o/n]"
read conf

if [ "$conf" == 'o' ]; then
echo "L'installation de unzip va demarrer pour verifier si le paquet est installé"
sudo apt-get install unzip
cd /home/mimic/Bureau/packages
unzip directnews-4.10.1-pro.zip -d /home/mimic/Bureau/packages/DirectNewsPro/
echo "Archive decompressé"
#$demande = 'oui'
$demande2= oui

#elif [ $conf='n' ]; then
#$demande = 'non'
#$demande2=oui

else
echo "Reponse indisponible"
$demande2 = non
fi
#done
fi

if [ $choix=="2" ]; then
$demande2 = "non"
#until [ $demande2="non"]; do
echo "Vous avez donc choisit Direct News en version Open ? [o/n]"
read conf

if [ $conf=="o" ]; then
echo "L'installation de unzip va demarrer pour verifier si le paquet est installé"
sudo apt-get install unzip
cd /home/mimic/Bureau/packages
unzip directnews-4.10-open-20070618.zip -d /home/mimic/Bureau/packages/DirectNewsOpen/
echo "Archive decompressé"
$demande = "oui"
$demande2= "oui"

elif [ $conf=n ]; then
$demande = "non"
$demande2= "oui"

else
echo "Reponse indisponible"
$demande2 = "non"
fi
#done
fi

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

if [ "$choix" == 'o' ]; then

Forcément la condition est toujours fausse ;-((

la honte j'ai mal recopier le code que tu m'avais donné (j'evite le copier coller quand j'apprend)
j'essai et je dit le resultat que j'obtient

sa marche beaucoup mieux :)
pourrait tu me dire ce que je pourrai faire pour optimiser mon script :
j'utilise deux grandes boucles if avec les memes choses dedans
je pourrai pas faire un for qui choisirait de faire tel instructions en fonction de tels choix
par exemple :

#!/bin/bash
demande2=non
i=0

for liste in `ls /home/mimic/Bureau/packages/`
do
echo "$liste"
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

for liste in `ls /home/mimic/Bureau/packages/`
do
///INSTRUCTION
done

exit;

relis ton code, tu met une valeur alphanumérique dans une variable, mais cette valeur n'est pas entre ""

merci du conseil

Il y a quoi dans "/home/mimic/Bureau/packages/" ?
S'il n'y a que 2 objets (apparemment connus d'avance), pourquoi faire un listing depuis une boucle ?

j'ai fait ce code...

#!/bin/bash
demande2=non
i=1

for liste in `ls /home/mimic/Bureau/packages/`
do
	echo "$((i++)) $liste"
done

echo Taper 1 Pour installer le premier item, tapez 2 pour installer le deuxieme...
read choix

i=0
for liste in `ls /home/mimic/Bureau/packages/`
do	
	$((i++))
	
	if [ $choix == $i ]; then
		echo $liste
	else
		continue
	fi
done

exit;

Mais sa me retourne un probleme :

./script.sh: line 16: 0 : commande introuvable
directnews-4.10.1-pro.zip
./script.sh: line 16: 1 : commande introuvable

Que faire ?

fait ;)

merci beaucoup c'est une bonne methode
Pourrait tu juste m'expliquer ces lignes :

ls -1 rep/ | nl

et

nom="$(ls -1 rep/ | sed -n "${choix}p")"

Je ne te remercierai jamais assez pour toutes ces informations precieuses qui me font progresser
juste pour sa :

nom="$(ls -1 rep/ | sed -n "${choix}p")"

je ne comprend pas a quoi sert le p a la fin de la commande

merci encore