Discussion :

[abdel_59]

Bonjour,
Je suis débutant en scripting shell.

On m'a donné le script ci-dessous à modifier. Suite à une migration d'environnement (windows > unix) les fichier générés par ce script doivent l'être au format unix (commande unix = dos2unix) et au format texte ISO-8859-1 (commande unix = iconv).
Dans un premier temps j'aimerais comprendre ce que ce script fait effectivement.
J'aimerai ensuite insérer les 2 commande unix précitées dans ce script...
Merci pour votre aide

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
21
22
23
#!/bin/sh
IBM_HEAPDUMPDIR=/product/dumpjvm
IBM_JAVACOREDIR=/product/dumpjvm
SNP_INIT_HEAP=256M
SNP_MAX_HEAP=768M
 
cd $SAV_HOME/WEB-INF/classes
 
for lib in `ls $SAV_HOME/WEB-INF/lib`
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done
 
CP="$CP:/product/ESIS/lib/ojdbc14-10.2.0.4.0.auchan.jar"
 
# lancement du traitement
java -Xmx512M -classpath $CP -DSAV_DATA=$SAV_DATA -DPRODUCT_LOG_HOME=/product_data/SAV/logs $1 $SAV_DATA/conf/batch_params.properties $2 $3 $4 $5
 
res=$?
 
echo Resultat du batch=$res
 
exit $res

[N_BaH]

Bonjour,

quelle est ton interprétation ?

c'est un mauvais script :

• il utilise ls, et ON N'UTILISE PAS ls DANS UN SCRIPT !!!
• il n'y a pas de guillemets autour des développement de variables

de toute façon, si le script est exécuté sur unix/linux les retours de lignes seront au format UNIX
quant à iconv, je ne connais java, donc je ne peux pas dire où/quels sont les fichiers créés.

[Flodelarab]

Bonjour

Chez Auchan, on met ls dans un script ? Oh les branqu' !

Code :

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

1
2
3
4
for lib in "${SAV_HOME}"/WEB-INF/lib/*
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done

Non?


Pour ton problème principal, il n'y a que toi qui sache ce que fait réellement ce script puisque du code java est exécuté.


De plus, on ne sait pas

• si on peut intervenir avant le script.
• si on peut intervenir après le script.
• si les fichiers en entrée sont à modifier aussi.

[grilled]

[abdel_59]

1) pardon pour ma question sans doute bête mais pourquoi on n'utilise jamais ls dans un script?

2) que veut dire cette séquence ?

Code :

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

1
2
3
4
for lib in "${SAV_HOME}"/WEB-INF/lib/*
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done

[N_BaH]

parce que c'est une commande externe, dont la syntaxe du shell permet de se dispenser
parce que si les noms de fichiers sont "exotiques" (espaces, tabulation, alinéas...) la boucle va tourner sur chaque élément du nom, et pas sur le nom en entier

on ne va pas faire ton travail.
on veut bien aider, mais il faut que tu fasses ta part.

je te le demande encore : quelle est ton interprétation ? que penses-tu que font les commandes ?

[abdel_59]

Je suppose que

Code :

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

1
2
3
4
for lib in `ls $SAV_HOME/WEB-INF/lib`
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done

effectue une copie de lib mais où est l'emplacement source et l'emplacement cible?

[jack-ft]

Je suppose que

Code :

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

1
2
3
4
for lib in `ls $SAV_HOME/WEB-INF/lib`
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done

effectue une copie de lib mais où est l'emplacement source et l'emplacement cible?

Non, non, ça ne fait pas du tout de copie.

Juste pour clarifier, la variable (super bien nommée (merci l'auteur du script!)) "CP" veut dire "ClassPath". Elle indique une liste de chemins où java doit chercher ses librairies, ainsi qu'on peut le constater dans son utilisation en tant qu'argument de "-classpath" dans la commande qui suit:

Code :

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

java -Xmx512M -classpath $CP -DSAV_DATA=$SAV_DATA -DPRODUCT_LOG_HOME=/product_data/SAV/logs $1 $SAV_DATA/conf/batch_params.properties $2 $3 $4 $5

Au début la variable "CP" ne contient rien, ce qui, en gros (pour simplifier) est équivalent à une chaîne vide.

Si on suppose que "$SAV_HOME/WEB-INF/lib" contient les dossiers libA libB libC, alors la boucle va concaténer chaque lib pour que, finalement, la variable CP contienne:

Code :

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

CP=":$SAV_HOME/WEB-INF/lib/libA:$SAV_HOME/WEB-INF/lib/libB:$SAV_HOME/WEB-INF/lib/libC"

De cette manière, la commande java saura où aller chercher les librairies dont elle a besoin.

[disedorgue]

Code :

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

1
2
3
4
for lib in "${SAV_HOME}"/WEB-INF/lib/*
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done

Il y a une erreur dans ce code, java n'appréciera pas

[N_BaH]

CP n'est pas cp : unix/linux est sensible à la casse.
en plus, une commande n'est jamais immédiatement suivie d'un signe égal.

que font les premières lignes sous le shebang ?

[abdel_59]

Les 2 premières lignes sous le shebang, détermine le chemin des variable nommées
les 2 suivantes la taille des variables nommées

[disedorgue]

Dans ton script tu as une variable non définie: SAV_HOME.
Sinon, on peut simplifier en retirant la boucle for et initialiser la variable CP avec printf, ce qui donnerait:
au lieu de:

Code :

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

1
2
3
4
5
6
for lib in `ls $SAV_HOME/WEB-INF/lib`
do
	CP="$CP:$SAV_HOME/WEB-INF/lib/$lib"
done
 
CP="$CP:/product/ESIS/lib/ojdbc14-10.2.0.4.0.auchan.jar"

faire:

Code :

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

1
2
lib=$(printf "%s:" "${SAV_HOME}"/WEB-INF/lib/*)
CP="${CP}:${lib}/product/ESIS/lib/ojdbc14-10.2.0.4.0.auchan.jar"

PS: je n'ai pas oublié de ":" après ${lib}, la variable fini toujours par un ":".

[N_BaH]

ah, oui, c'est bien printf !
et on peut aller plus loin (toujours plus loin ) : printf -v lib '%s:' /chemin/*.

les premières lignes du script définissent des variables.
c'est ainsi qu'on fait en shell : un nom, un signe égal, et une valeur.
donc, dans la boucle, c'est la même chose; ce n'est pas une commande, c'est une assignation de variable.
la boucle ne sert qu'à ça : construire une variable par concaténation.

[Flodelarab]

Pour revenir au problème principal, les scripts bash articulent des logiciels entre eux.
Ici, le script ne fait rien de particulier.
C'est java qui fait tout le boulot.
C'est donc le fichier jar dont il faudrait connaître les entrées/sorties.

Je repose mes interrogations :

• Peut-on intervenir avant ce script ?
• Peut-on intervenir après ce script ?
• Faut-il modifier le format des fichiers entrants ? (J'ai bien compris que le client ne parle que de la sortie qui le fait baver d'envie mais s'il s'agit d'une migration, c'est ta responsabilité de penser à l'entrée du script, abdel_59)

Il y a une erreur dans ce code, java n'appréciera pas

Pas compris. C'est l'absence des 2 points à la fin ?

[disedorgue]

Dans la variable lib, il y a déjà l'arborescence et donc la concaténer encore à l'arborescence...

Code :

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

ls $SAV_HOME/WEB-INF/lib

ne ramène pas la même chose que:

Code :

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

echo "${SAV_HOME}"/WEB-INF/lib/*

[abdel_59]

Pour revenir au problème principal, les scripts bash articulent des logiciels entre eux.
Ici, le script ne fait rien de particulier.
C'est java qui fait tout le boulot.
C'est donc le fichier jar dont il faudrait connaître les entrées/sorties.

Je repose mes interrogations :

• Peut-on intervenir avant ce script ?
• Peut-on intervenir après ce script ?
• Faut-il modifier le format des fichiers entrants ? (J'ai bien compris que le client ne parle que de la sortie qui le fait baver d'envie mais s'il s'agit d'une migration, c'est ta responsabilité de penser à l'entrée du script, abdel_59)

*Qu'entends tu par intervenir avant/après le script?
*d'après ce que j'ai compris, le commande "iconv" ne permet pas de convertir un fichier vers un fichier de même nom. Le fichier en entrée doit avoir un nom différent du fichier en sortie

[abdel_59]

J'ai oublié de préciser que ce script fonctionne dans son état actuel. Le but n'est pas de réécrire le script mais de le modifier afin de convertir les fichiers que génère ce script.
Comme je le disais dans mon premier post, avez-vous une idée sur la façon dont je peux insérer la commande iconv dans ce script.
Je me suis familiarisé avec la commande iconv et je cherche à l'appliquer au script.
Etant donné que les fichiers en entrées ne peuvent pas porter le même nom que les fichiers en sortie, j'essaie de trouver un moyen de renommer ces fiichiers > les convertir > les supprmier...

[Flodelarab]

*Qu'entends tu par intervenir avant/après le script?

Ce script est lancé par quelque chose. Quelle chose ?
Et le résultat est récupéré par quelque chose. Quelle chose ?
Il travaille sur des fichiers en entrée. Lesquels ? Sont-ils au bon format ?
Il produit des fichiers en sortie. Lesquels ?