[Résolu] paramètre de commande

ar barzh paour · Le 06/03/2020, à 14:01

pourquoi ce bash ne fonctionne-t-il pas ?
autrement dit
comment faut-il coder pour avoir le résultat de dpkg -l|grep hplip dans la fonction ?

#!/bin/bash
function lance-commande {
    echo "début de function"
    echo $1
    $($1)
   echo "fin de fonction"
}
lance-commande "dpkg -l | grep hplip"

le résultat

jpb@jpb-MS-7A70:~$ bash ./l-c.sh
    début de function
dpkg -l | grep hplip
dpkg-query: aucun paquet ne correspond à |
./l-c.sh: ligne 5: Souhait=inconnU/Installé/suppRimé/Purgé/H=à: Aucun fichier ou dossier de ce type
    fin de function
jpb@jpb-MS-7A70:~$

Dernière modification par ar barzh paour (Le 07/03/2020, à 10:20)

melixgaro · Le 06/03/2020, à 14:10

desktop$  var="dpkg -l \| grep hplip"
desktop$  $var
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version               Architecture Description
dpkg-query: no packages found matching \|
+++-==============-=====================-============-============================================
ii  grep           3.3-1build1           amd64        GNU grep, egrep and fgrep
ii  hplip          3.19.6+dfsg0-1ubuntu1 amd64        HP Linux Printing and Imaging System (HPLIP)

ne pas mettre de " autour de $var
var contient la commande, suffit de l'appeler Modif : Non !

Dernière modification par melixgaro (Le 06/03/2020, à 16:07)

ar barzh paour · Le 06/03/2020, à 14:13

#!/bin/bash
function lance-commande {
    echo "    début de function"
    echo $1
    $1
    echo "    fin de function"
}
lance-commande "dpkg -l | grep hplip"

oui j'avais essayé mais ce n'est pas bon

jpb@jpb-MS-7A70:~$ bash ./l-c.sh
    début de function
dpkg -l | grep hplip
Souhait=inconnU/Installé/suppRimé/Purgé/H=à garder
| État=Non/Installé/fichier-Config/dépaqUeté/échec-conFig/H=semi-installé/W=attend-traitement-déclenchements
|/ Err?=(aucune)/besoin Réinstallation (État,Err: majuscule=mauvais)
||/ Nom                    Version          Architecture     Description
+++-======================-================-================-==================================================
ii  grep                   3.1-2build1      amd64            GNU grep, egrep and fgrep
ii  hplip                  3.17.10+repack0- amd64            HP Linux Printing and Imaging System (HPLIP)
dpkg-query: aucun paquet ne correspond à |
    fin de function
jpb@jpb-MS-7A70:~$

en fait j'ai essayé aussi $* $@ "$*" "$@"

Dernière modification par ar barzh paour (Le 06/03/2020, à 14:15)

melixgaro · Le 06/03/2020, à 14:17

regarde bien ce que j'indique Modif : Non ! Regarde plutôt ce que kamaris propose

Dernière modification par melixgaro (Le 06/03/2020, à 16:07)

kamaris · Le 06/03/2020, à 15:18

Le pipe « | » est considéré littéralement, à partir du moment où il a été passé comme argument à ta fonction.
Pour t'en convaincre, tu peux utiliser set -x :

set -x
$1
set +x

Tu vas avoir que ce que le shell exécute, c'est :

dpkg -l '|' grep hplip

Si tu veux qu'il interprète « | » comme un pipe, il faut que tu utilises eval :

eval "$1"

melixgaro · Le 06/03/2020, à 15:25

Ou échapper le pipe comme indiqué en 3. Modif : Non !

Dernière modification par melixgaro (Le 06/03/2020, à 16:08)

kamaris · Le 06/03/2020, à 15:31

Non, ça ne fonctionnera pas mieux :

melixgaro a écrit :

dpkg-query: no packages found matching \|

À partir du moment où il y a des choses à interpréter pour le shell, la mise en variables ne peut pas marcher directement : il faut passer par eval, ou bash -c, ou autre procédé du même type.

melixgaro · Le 06/03/2020, à 16:06

Ouh la il me faut des lunettes. Merci kamaris.
Je me suis fait totalement avoir car le message d'erreur n'apparaît pas au même endroit du retour entre deux lancements. Là j'ai refait la même chose qu'au 2 et le message est apparu en 1ère ligne.
Désolé !

kamaris · Le 06/03/2020, à 16:42

No problemo !

Watael · Le 06/03/2020, à 17:31

c'est quoi le gain de faire exécuter une commande passée en argument à une fonction, plutôt que d'exécuter la commande ?

ar barzh paour · Le 06/03/2020, à 17:45

ok merci à tous , je crois que j'aurais cherché un moment avant d'utilise eval !!!!

#!/bin/bash
function lance-commande {
    echo "    début de function"
    echo $1
    #$(eval "$1")
    eval "$1" 
    echo "    fin de function"
}
lance-commande "dpkg -l | grep hplip"

jpb@jpb-MS-7A70:~$ bash ./l-c.sh
    début de function
dpkg -l | grep hplip
ii  hplip                                      3.17.10+repack0-5                                amd64        HP Linux Printing and Imaging System (HPLIP)
ii  hplip-data                                 3.17.10+repack0-5                                all          HP Linux Printing and Imaging - data files
    fin de function
jpb@jpb-MS-7A70:~$

ar barzh paour · Le 06/03/2020, à 17:54

Watael a écrit :

roll c'est quoi le gain de faire exécuter une commande passée en argument à une fonction, plutôt que d'exécuter la commande ?

je n'avais pas vu ta remarque
j'ai plusieurs commande à passer et je veux à chaque les faire précéder et suivre d'une marque comme j'ai mis "début de fonction" "fin de fonction"

#!/bin/bash
function lance-commande {
    echo "    début de function"  >> "$fich"
    echo $1                                >> "$fich"
    eval "$1"                              >> "$fich"
    echo "    fin de function"      >> "$fich"
}
# un peu comme ça , je n'ai pas encore testé la syntaxe liste étant une liste de commandes
fich="/tmp/tmp.txt"
echo "" >"$fich"
for cmd in "$liste"
do
        lance-commande "$cmd"
done

Dernière modification par ar barzh paour (Le 06/03/2020, à 17:59)

Watael · Le 06/03/2020, à 18:24

je prendrais le problème dans l'autre sens :

$ echoUn() { echo "un";}
$ echoDeux() { echo "deux";}
$ echoTrois() { echo "trois";}
$ for i in echo{Un,Deux,Trois}; do echo "début $i"; $i; echo "fin $i"; done
début echoUn
un
fin echoUn
début echoDeux
deux
fin echoDeux
début echoTrois
trois
fin echoTrois

ar barzh paour · Le 06/03/2020, à 21:08

@Watael
(je sais que je ne code pas corectement)
tu peux me donner ton avis sur
https://forum.ubuntu-fr.org/viewtopic.php?id=2047694

@kamaris
ça ne fonctionne pas pour cette commande alambiquée

find /etc/apt -name '*.list' -exec bash -c 'echo -e "\n$1\n"; nl -ba "$1"' _ '{}' \;

(voir le lien ci-dessus
utilisation dans
function choixImp {
choix sS

normalement j'aurais du utiliser

cmd="find /etc/apt -name '*.list' -exec bash -c 'echo -e "\n$1\n"; nl -ba "$1"' _ '{}' \;" ; l-c "$cmd"

mais ça ne marche pas , certainement le \n$1n et/ou le "$1" qui mettent le bazar

Dernière modification par ar barzh paour (Le 06/03/2020, à 21:15)

melixgaro · Le 06/03/2020, à 22:00

Mais ce que tu veux faire n'est une bonne pratique !
Relis la signature de Watael

Zakhar · Le 06/03/2020, à 22:24

Tu sais qu'il y a aussi simplement sh -x qui va te faire tourner ton script en mode debug, c'est à dire afficher chaque ligne avant de l'exécuter, en montrant en plus les empilement de sub-process dus aux pipe et autres.

Watael · Le 06/03/2020, à 22:42

moi, j'enregistrerais le résultat des commandes en les concaténant dans une variable, ou un tableau, que j'afficherais après le case :

sep() { printf '=%.s' {1..80}; echo;}

case $(zenity ... ) in
[eE]) aAfficher=( "imprimante Espon" ) 
        aAfficher+=( "$(dpkg -l | grep -i 'epson')" )
;;
[...]) aAfficher...
;;
esac
sep
printf '%s\n' "${aAfficher[@]}"
sep

KISS!!!*

--
* Keep It Simple Stupid!!!

kamaris · Le 07/03/2020, à 01:37

ar barzh paour a écrit :

@kamaris
ça ne fonctionne pas pour cette commande alambiquée
find /etc/apt -name '*.list' -exec bash -c 'echo -e "\n$1\n"; nl -ba "$1"' _ '{}' \; 
normalement j'aurais du utiliser
cmd="find /etc/apt -name '*.list' -exec bash -c 'echo -e "\n$1\n"; nl -ba "$1"' _ '{}' \;" ; l-c "$cmd"
mais ça ne marche pas , certainement le \n$1n et/ou le "$1" qui mettent le bazar

Il faut que tu échappes les guillemets doubles lorsque tu veux les inclure dans des guillemets doubles englobants :

cmd="find /etc/apt -name '*.list' -exec bash -c 'echo -e \"\n$1\n\"; nl -ba \"$1\"' _ '{}' \;"

PS : ceci n'est pas un encouragement à faire du eval, simplement une réponse apportée à une question sur le bon usage des guillemets.

Watael · Le 07/03/2020, à 08:54

PS : ceci n'est pas un encouragement à faire du eval, simplement une réponse apportée à une question sur le bon usage des guillemets.

tu as raison. la chasse est ouverte; il fait bon porter une chasuble orange dans les bois.

ar barzh paour · Le 07/03/2020, à 10:02

@kamaris

Moi a écrit :

ça ne fonctionne pas pour cette commande alambiquée
find /etc/apt -name '*.list' -exec bash -c 'echo -e "\n$1\n"; nl -ba "$1"' _ '{}' \;

au temps pour moi
j'ai bêtement appliqué la méthode pour des autres commandes , sans me poser de question
d'où l'oubli des \ devant les "

mais il a fallu que protège aussi $ pour que cela fonctionne ce qui donne

cmd="find /etc/apt -name '*.list' -exec bash -c 'echo -e \"\n\$1\n\"; nl -ba \"\$1\"' _ '{}' \;" ;  l-c "$cmd"

vu le temps qui me reste je vois que je n'atteindrai jamais votre niveau !!!!!

je passe la discussion en Résolu

ce que j'espère en retenir ...

post #5 Kamaris

eval "$1"

(bien que "eval c'est mal")
post #18 Kamaris
penser aux caractères de protection

post #13 Watael

for i in echo{Un,Deux,Trois};

post #17 Watael

sep() { printf '=%.s' {1..80}; echo;}

aAfficher+=( "$(dpkg -l | grep -i 'epson')" )

printf

commande à la quelle je ne pense pas

post #16 Zakhar

sh -x

à essayer , je ne connais pas

etc etc
merci à tous

et pour faire le pendant à Watael , vu le temps , je vais m'occuper de mon jardin !!!

Dernière modification par ar barzh paour (Le 07/03/2020, à 20:45)

Ubuntu-fr

Navigation

Liens de recherche

Annonce

#1 Le 06/03/2020, à 14:01

[Résolu] paramètre de commande

#2 Le 06/03/2020, à 14:10

Re : [Résolu] paramètre de commande

#3 Le 06/03/2020, à 14:13

Re : [Résolu] paramètre de commande

#4 Le 06/03/2020, à 14:17

Re : [Résolu] paramètre de commande

#5 Le 06/03/2020, à 15:18

Re : [Résolu] paramètre de commande

#6 Le 06/03/2020, à 15:25

Re : [Résolu] paramètre de commande

#7 Le 06/03/2020, à 15:31

Re : [Résolu] paramètre de commande

#8 Le 06/03/2020, à 16:06

Re : [Résolu] paramètre de commande

#9 Le 06/03/2020, à 16:42

Re : [Résolu] paramètre de commande

#10 Le 06/03/2020, à 17:31

Re : [Résolu] paramètre de commande

#11 Le 06/03/2020, à 17:45

Re : [Résolu] paramètre de commande

#12 Le 06/03/2020, à 17:54

Re : [Résolu] paramètre de commande

#13 Le 06/03/2020, à 18:24

Re : [Résolu] paramètre de commande

#14 Le 06/03/2020, à 21:08

Re : [Résolu] paramètre de commande

#15 Le 06/03/2020, à 22:00

Re : [Résolu] paramètre de commande

#16 Le 06/03/2020, à 22:24

Re : [Résolu] paramètre de commande

#17 Le 06/03/2020, à 22:42

Re : [Résolu] paramètre de commande

#18 Le 07/03/2020, à 01:37

Re : [Résolu] paramètre de commande

#19 Le 07/03/2020, à 08:54

Re : [Résolu] paramètre de commande

#20 Le 07/03/2020, à 10:02

Re : [Résolu] paramètre de commande

Pied de page des forums