Discussion :

[Fork Bomb]

Bonjour,
Je suis en ce moment en train de réécrire certains de mes utilitaires.
Le problème sur lequel je bute est la génération à l’aide d’une boucle "for" des choix d’une liste de boutons radio en dialog.
Pour bien vous faire comprendre, testez ceci :

Code :

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

for Utilisateur in $(grep /home /etc/passwd|awk -F':' '{print $1}'); do echo "${Utilisateur}"; done

Normalement, la liste des utilisateurs de votre système doit apparaître.
Maintenant, si je formate l’affichage pour le préparer à l’inclusion dans la radiolist de dialog :

Code :

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

for Utilisateur in $(grep /home /etc/passwd|awk -F':' '{print $1}'); do echo "${Utilisateur}" "Effacement de : ${Utilisateur}" "off"; done

Enfin, si je tente de mettre le tout dans un dialog :

Code :

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

1
2
3
4
DeleteAccount=`dialog --stdout --backtitle "${BACK_TITLE}" \
		--title "Effacement d’un utilisateur" \
		--radiolist "Compte à effacer :" 18 65 10 \
		\`for Utilisateur in $(grep /home /etc/passwd|awk -F':' '{print $1}'); do echo "${Utilisateur}" "Effacement de : ${Utilisateur}" "off"; done\``

Ben là, y a pas moyen. Pour 2 utilisateurs enregistrés, j’ai 4 lignes.
La raison m’échappe mais s’il est généré (enfin, de la manière dont je le fait…), le second champ d’un radiolist ne supporte pas de contenir des espaces.
-----
Donc, je m’en remets à vous si vous avez des idées, n’hésitez pas.
Merci.

[Flodelarab]

Bonjour

Déjà, est-ce que c'est pas moins casse-bonbon de faire ceci pour avoir les utilisateurs?

Code :

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

id -u -n

Ensuite tu as un problème d'espace. Ne peux-tu pas faire ceci?

Code :

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

DeleteAccount=$( dialog --stdout --radiolist Comptes 18 65 10 $(for Utilisateur in $(id -u -n); do echo -n $Utilisateur "Effacement:$Utilisateur" off ; done ) )

ou cela?

Code :

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

DeleteAccount=$( dialog --stdout --radiolist Comptes 18 65 10 $(for Utilisateur in $(id -u -n); do echo -n $Utilisateur "Effacement" off ; done ) )

[Fork Bomb]

Bonjour

Déjà, est-ce que c'est pas moins casse-bonbon de faire ceci pour avoir les utilisateurs?

Code :

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

id -u -n

Sur ma machine, cette commande ne fait que renvoyer le nom de l’utilisateur qui l’a lancé. L’idée de ma commande de filtrage est bien de récupérer tous les utilisateurs enregistrés sur la machine.

Ensuite tu as un problème d'espace. Ne peux-tu pas faire ceci?

Code :

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

DeleteAccount=$( dialog --stdout --radiolist Comptes 18 65 10 $(for Utilisateur in $(id -u -n); do echo -n $Utilisateur "Effacement:$Utilisateur" off ; done ) )

ou cela?

Code :

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

DeleteAccount=$( dialog --stdout --radiolist Comptes 18 65 10 $(for Utilisateur in $(id -u -n); do echo -n $Utilisateur "Effacement" off ; done ) )

J’ai bien compris que, généré, le second champ ne supportait pas les espaces (écrit en dur, ça passe). Mais je ne souhaite pas utiliser un contournement. Je souhaite trouver une solution à mon problème.

[Fork Bomb]

Bon ben… Résolu.
Voilà ce qui marche chez moi :

Code :

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

1
2
3
4
5
 
		# Noms des comptes sur le système
		SystemAccounts=`for Utilisateur in $(grep /home /etc/passwd|awk -F':' '{print $1}'); do echo "$Utilisateur" \"Effacement de : $Utilisateur\" off; done`
		# On récupère le nom du compte effacé
		DeleteAccount=`echo $SystemAccounts|xargs dialog --stdout --backtitle "${BACK_TITLE}" --title "Effacement d’un utilisateur" --radiolist "Compte à effacer :" 18 65 10`

Vous pouvez tester, ça marche
Merci à Flodelarab de s’être penché sur mon problème.
[Source]

[jack-ft]

Juste pour info, les backquotes (ou backticks) sont moins recommandés que le '$(' qui gère beaucoup mieux les imbrications (de divers guillemets et autres '$(').

[Fork Bomb]

Juste pour info, les backquotes (ou backticks) sont moins recommandés que le '$(' qui gère beaucoup mieux les imbrications (de divers guillemets et autres '$(').

On m’a dit que "`commande`" était plus portable que "$(commande)". Du coup, je l’utilise en priorité.

[N_BaH]

On m’a dit que "`commande`" était plus portable[...]

c'est faux.
pour savoir ce qui est portable, il faut lire les spécifications POSIX.

et on n'utilise par une boucle for sur la sortie d'une commande, car elle boucle sur chaque mot, et non sur chaque ligne.
ça ne porte pas à conséquence ici, c'est une bonne pratique à adopter au plus tôt.

Code :

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

SystemAccounts=$(awk -F':' '/\/home/{print $1" \"Effacement de : "$1"\" off"}' /etc/passwd)

...

[Fork Bomb]

Wow, ta commande est bien plus concise que la mienne o_0;;;
Visiblement, je ne connais PAS*DU*TOUT awk… :/
Merci encore

[disedorgue]

Bonjour,

Pour le fun, une version juste avec grep (de GNU) :

Code :

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

grep -o -P '^[^:]*(?=.*/home)' /etc/passwd | xargs -IUSER echo 'USER "Effacement de : USER" off'