Discussion :

[bihoueej]

Bonjour

J'ai écrit un script qui en fonction des variables d'entrée, créé un autre script, puis l'execute.

Le script est le suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#! /bin/sh
DIRECTORY=/work
ID_JOB=test
PRIORITY=Highest
 
 
cd $DIRECTORY
 
 
for listejob in `cat /work/liste.txt`; do
   FILENAME=`basename "$listejob" .txt`
   echo '#!/bin/sh' > "$FILENAME".sh
   echo 'ligne... >> "$FILENAME".sh
   echo 'ligne... >> "$FILENAME".sh
         ...
   echo 'variable=`/usr/bin/uniq -c ${toto} | /usr/bin/head -1 | /bin/awk '{ print $1 }'`' >> "$FILENAME".sh
         ...
done
echo fin

Mon soucis se situe au niveau de cette ligne :

Code :

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

echo 'variable=`/usr/bin/uniq -c ${toto} | /usr/bin/head -1 | /bin/awk '{ print $1 }'`' >> "$FILENAME".sh

j'obtiens dans le fichier FILENAME.sh

Code :

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

variable=`/usr/bin/uniq -c ${toto} | /usr/bin/head -1 | /bin/awk { print $1 }`

Or je souhaiterais avoir:

Code :

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

variable=`/usr/bin/uniq -c ${toto} | /usr/bin/head -1 | /bin/awk '{ print $1 }'`

Si vous avez des suggestions pour ce cas présent, merci beaucoup pour votre retour.
A votre disposition pour tout complément.

Jérôme

[hizoka]

Salut, perso, quand je veux envoyer plein de " et ' dans un fichier, j'utilise cat et je protège les $

Code :

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

1
2
3
cat << EOF >> "$FILENAME".sh
variable=\$(/usr/bin/uniq -c \${toto} | /usr/bin/head -1 | /bin/awk '{ print \$1 }')
EOF

Et on utilise $() et non plus ``.


EDIT :
Beurk :

Code :

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

1
2
3
4
5
6
7
8
9
for listejob in `cat /work/liste.txt`; do
   FILENAME=`basename "$listejob" .txt`
   echo '#!/bin/sh' > "$FILENAME".sh
   echo 'ligne... >> "$FILENAME".sh
   echo 'ligne... >> "$FILENAME".sh
         ...
   echo 'variable=`/usr/bin/uniq -c ${toto} | /usr/bin/head -1 | /bin/awk '{ print $1 }'`' >> "$FILENAME".sh
         ...
done

=>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while read listejob # Pourquoi faire une substitution ?
do
  FileName="${listejob##*/} .txt" # Pourquoi encore faire une substitution ?
  cat << EOF > "${FileName}.sh"
#!/bin/sh
ligne... 
ligne... 
...
variable=\$(/usr/bin/uniq -c \${toto} | /usr/bin/head -1 | /bin/awk '{ print \$1 }')
...
EOF
done < "/work/liste.txt"

Pourquoi utiliser x echo ? A part à perdre du temps (surtout lors de l'écriture) et à alourdir le code ?

Code :

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

1
2
3
4
5
6
7
8
9
time for ((x=1; x < 100000; x++)); do echo "oui"; echo "non"; echo "je sais pas"; done
real    0m2,943s
user    0m2,078s
sys     0m0,864s
 
time for ((x=1; x < 100000; x++)); do echo -e "oui\nnon\nje sais pas"; done
real    0m1,864s
user    0m1,250s
sys     0m0,593s

[N_BaH]

ou, on ne protège pas les $, mais on met le eof entre apostrophes.

sinon :
- les noms de variables en majuscules sont reservés aux variables d'environnement; les utilisateurs peuvent utiliser un panachage :varName,
- on ne lit pas un fichier comme ça, mais comme ça,
- et, comme l'a montré hizoka, on n'utilise plus ` depuis bien longtemps.

[hizoka]

ou, on ne protège pas les $, mais on met le eof entre apostrophes.

Merci ! Carrément plus pratique !!!
Ça valait vraiment le coup que je réponde

[bihoueej]

Merci pour vos retours et remarques.
Je vais adapter cela à mon projet (du moins essayer )
Par contre, j'ai oublié de préciser que durant la création du second script FILENAME.sh, des variables (issue du script initial) seront écrites dans celui-ci.

j'ai donc

Code :

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

echo 'SBATCH --qos='$priority >> "$FILENAME".sh

(j'ai volontairement laisser ma synthaxe initiale)

Pour obtenir dans FILENAME.sh :

Code :

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

SBATCH --qos=Normal

Cela change quelque chose compte tenu de vos commentaires?

Encore merci pour vos apports

[Escapetiger]

Et on utilise $() et non plus ``.

- et, comme l'a montré hizoka, on n'utilise plus ` depuis bien longtemps.

Oui, et on voit parfois des scripts anciens chez des clients qui l'utilisent encore (moi-même parfois par mauvaise habitude dans un premier temps).

C'est toujours intéressant de savoir pourquoi :

bash - What is the difference between $(command) and `command` in shell programming? - Stack Overflow

Why is $(...) preferred over `...` (backticks)? BashFAQ/082 - Greg's Wiki

bash - Have backticks (i.e. `cmd`) in *sh shells been deprecated? - Unix & Linux Stack Exchange

etc,

[hizoka]

Code :

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

echo 'SBATCH --qos='$priority >> "$FILENAME".sh

=>

Code :

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

echo "SBATCH --qos=$priority" >> "$FILENAME".sh

du coup, il ne faut pas utiliser 'EOF' mais EOF et protéger les $ qui en ont besoin.

ex :

Code :

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

1
2
3
cat << EOF > fichier # EOF pour End Of File mais tu peux écrire ce que tu veux
echo "regarder ma belle variable ${variable}, et la voila la variable non convertie \${variable}"
EOF

Mais peut être vais je apprendre une autre façon de faire

[bihoueej]

Bonjour

Depuis vos dernières réponses, j'ai pris le temps de lire vos commentaires et les différents lien proposés.
J'ai donc ré-écris le script en fonction de vos remarques (suppression des echo et des ``) .

Néanmoins je bloque sur certains passages.

Voici la partie de mon script qui me pose des problèmes de mise en forme:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
cat << fincat > "${FileName}.sh"
#!/bin/sh
...
for NODE in `/usr/bin/uniq \$TOTALCAE_NODEFILE`; do
   mp_host_list="\$mp_host_list[\"\${NODE}\",\${PROCS_PER_NODE}],"
done
export mp_host_list=`echo \${mp_host_list} | sed -e "s/,$/]/"`
...
fincat

J'obtiens ceci dans mon fichier, mes problèmes sont au niveau des ligne "for" et "export" :

Code :

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

1
2
3
4
5
 
for NODE in ; do
   mp_host_list="$mp_host_list[\"${NODE}\",${PROCS_PER_NODE}],"
done
export mp_host_list=

Or je cherche a avoir ceci :

Code :

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

1
2
3
4
5
 
for NODE in `/usr/bin/uniq $TOTALCAE_NODEFILE`; do
   mp_host_list="$mp_host_list[\"${NODE}\",${PROCS_PER_NODE}],"
done
export mp_host_list=`echo ${mp_host_list} | sed -e "s/,$/]/"`

Les commandes entre les `` sont interprétées par le script de base.
Avez vous des idées et pourrait-on supprimer les `` dans ces deux cas précis.
En effet, pour le fichier sh que j'obtiens, les commande devront, par contre, être interprétées.

J'ai trituré ce passage dans tous les sens à partir de vos remarques précédentes mais sans résultats.

Merci pour votre retour.

[hizoka]

heu.... remplace les `` par $().... puis échappe le $...
en gros c'est ce qu'on à déjà dit :p

mais comme signalé, si tu utilises cat << 'EOF', tu n'a plus besoin d'échapper les $ (mais aucune variable n'est plus interprétée).

[bihoueej]

Effectivement, je me focalisais sur la forme résultante et j'ai totalement oublié l’exécution futur du script.
Après modifications, le script généré semble fonctionner. il ne reste plus qu'à le mettre en page.

Merci beaucoup pour votre aide.

Bonne fêtes de fin d'années.

Jérôme

[srvremi]

Salut.

Je ne connaissais pas le coup du

Code :

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

cat << 'EOF'

Cool !

@+
Rémi

[ggnore]

Sinon ansible c'est une approche un peu moins roots.