Discussion :

[Fanny28]

Bonjour,

je dois modifier un script shell et je suis complètement novice dans le domaine.

Je m'arrache les cheveux sur quelque chose qui me semblait pourtant être d'une facilité déconcertante... cela montre mon niveau...

Je veux simplement exécuter dans mon script shell une commande qui est positionnée dans une variable. Je précise que la commande est une commande "complexe" (enchainement de commandes avec pipe).

Pour simplifier, prenons l'exemple suivante :

Code :

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

1
2
TOTO="ls | cat"
$TOTO

Je pensais que la ligne "$TOTO" allait permettre d'exécuter ma commande "ls | cat" mais apparament non.
Cela fonctionne pour des commandes non enchainées (sans pipe) mais pas pour des commandes enchainées.

J'ai également essayé avec

Code :

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

1
2
TOTO="ls | cat"
`$TOTO`

mais cela ne fonctionne pas non plus.

Est-ce ce quelqu'un est en mesure de m'expliquer pourquoi cela ne fonctionne pas, et surtout, comment faire pour y parvenir...

J'avais prévenu que ça n'était pas d'un haut niveau

Merci par avance

[N_BaH]

Bonjour,


c'est, ainsi que le montre la sortie de

Code :

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

set -x

, comme si tu tapais :

Code :

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

cmd1 '|' cmd2

; or, les apostrophes protègent la commande pipe (|) d'être interpétée.

le conseil commun, pour les commandes complexes, est d'utiliser une fonction :

Code :

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

mesCmd() { cmd1 | cmd2 ;}

[Fanny28]

Bonjour,

merci beaucoup pour ta réponse

Effectivement, cela fonctionne comme cela...

En revanche, cela va compliquer mon code. En gros, je gérais des appels à find avec des commandes complexes différentes.

Code :

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

1
2
3
4
5
6
 
case $TOTO in
	1) MON_FIND="find ... | ..."
        2) MON_FIND="find ... | ..."
        ...
$MON_FIND > toto

Cela veut dire que je dois créer autant de fonctions que de cas :

Code :

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

1
2
3
4
5
 
case $TOTO in
	1) ma_fonction_1 > toto
        2) ma_fonction_2 > toto
        ...

Confirmes-tu que c'est comme cela que je dois m'y prendre ? Cela me semble faire beaucoup de fonctions à créer pour un code si simple

[Luc1an0]

Salut,

Tu peux aussi faire d'une autre manière comme ça :

Code :

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

1
2
3
 
MesDossiers=`ls`
echo $ls

Ou encore une autre :

Code :

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

1
2
3
 
MesDossiers="ls"
eval $MesDossiers

[N_BaH]

Je ne vois pas ce que ça change :

Code :

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

1
2
3
4
5
case $TOTO in
	1) monFind() { find ... | ...;}
        2) monFind() { find ... | ...;}
        ...
monFind > toto

si éventuellement tu ne te ressers pas de la fonction plus tard dans ton script, et que ton case n'affiche rien d'autre sur la sortie standard :

Code :

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

1
2
3
4
5
case $TOTO in
	1) find ... | ...
        2) find ... | ... 
        *) echo "erreur" >&2
esac > toto

[zipe31]

Salut,

Cela me semble faire beaucoup de fonctions à créer pour un code si simple

Sans voir ton code dans sa totalité et sans en connaitre la finalité, difficile de dire si c'est trop lourd et de t'aider à optimiser le tout

[lennelei]

Autre solution si tes commandes sont similaires, alimenter une ou plusieurs variables dans le case qui sera(ont) utilisée(s) après coup pour comme argument de tes commandes :

Code :

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

1
2
3
4
5
case $TOTO in
	1) MON_FIND="..."
        2) MON_FIND="..."
        ...
find ... $MON_FIND ... | ... > toto

Attention, car selon les options, tu pourrais avoir à utiliser xargs pour construire les arguments de find :

Code :

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

1
2
3
4
5
case $TOTO in
	1) MON_FIND="..."
        2) MON_FIND="..."
        ...
echo $MON_FIND | xargs find | ... > toto

Mais bon, avec un exemple complet, on pourrait te donner une vraie bonne méthode, car cela dépend beaucoup de ce que tu cherches à faire exactement

Si cela se trouve, tu n'as peut-être même pas besoin d'un pipe ?!

[FRUiT]

Veuillez pardonner mon ignorance, mais :

Code :

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

1
2
TOTO="ls | cat"
$TOTO

Pourquoi ne pas se contenter d'un :

Code :

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

1
2
TOTO="ls | cat"
eval "$TOTO"

Ce qui semble très bien marcher chez moi ?

[Fanny28]

Ouah, je ne pensais pas avoir autant de réponses, merci à tous .

Je n'ai pas donné trop de détails car je ne souhaitais pas que des gens perdent trop de temps à effectuer mon travail
Je voulais donc une petite aide sur quelque chose qui me semblait basique, ce que j'ai largement obtenu, et après, c'est à moi de m'y coller quand même.

La solution proposée par FRUiT semble fonctionner (je ne l'ai pas encore testée dans mon script mais sur des petits tests à côté).
Y-a-t-il une raison pour que les autres membres n'aient pas proposé cette solution ?
Est-ce que vous n'y aviez pas pensé ou est-ce volontaire parce qu'il faut éviter d'utiliser eval (je ne sais pour quelle raison).

Du coup, je passe le problème à résolu même si FRUiT et moi attendons de savoir si l'utilisation de eval est correcte. J'espère que c'est la bonne procédure.

En tout cas, encore un grand merci à tous !

[N_BaH]

en l'occurrence, je ne l'ai pas proposé, parce que je ne vois pas de bonne raison de réactiver un comportement qui a été inhibé.

l'utilisation d'une fonction est beaucoup plus simple

[FRUiT]

La solution proposée par FRUiT semble [...]

Rendons à César etc... En fait ce serait plutôt la solution qu'a proposé Luc1an0. Comme je l'utilise aussi dans un de mes scripts qui extrait des commandes dans des fichiers (et donx, comportant éventuellement des pipes), je me demandais ce qu'elle avait de non orthodoxe.

Après pour la question de réactiver un comportement inhibé, c'est pas faux encore que, je ne sais pas quelles raisons poussent le shell à inhiber le comportement du pipe... On peut aussi se demander en quoi c'est dangereux ou mal en dehors du fait que c'est algorithmiquement amoral/paradoxal.

En tout cas merci N_BaH pour l'explication.

[lennelei]

Je n'ai pas donné trop de détails car je ne souhaitais pas que des gens perdent trop de temps à effectuer mon travail

C'est tout à ton honneur Tu peux toujours poster ta solution une fois prête si tu veux que l'on te propose quelque chose d'éventuellement plus adapté :p

Sinon, outre l'explication de N_BaH, l'utilisation d'eval est parfois déconseillée à cause des problèmes potentiels de sécurité que cela peut potentiellement poser.

Enfin, il y a également la possibilité d'utiliser 2 variables distinctes :

Code :

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

1
2
3
4
5
6
$ MON_LS="ls -l"
$ MON_CAT="tail -3"
$ $MON_LS | $MON_CAT
-rw-r--r--  1 user users       538 Mar  9 16:11 test_sort.xml
drwxr-xr-x  4 user users      4096 May  6 09:56 tmp
drwxr-xr-x  2 user users      4096 Feb 18 09:46 vpn

Avec toujours le risque d'avoir à utiliser xargs