Discussion :

[volimp]

Bonjour tlm,

JE suis entrain de faire du nouveau du shell après 4 ans et je bloque sur une commande.

But de mon Script : envoyer un mail en attachant tous les fichiers csv contrenus dans un repertoire donné.

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
 
 
#preparing the send mails comand
 for i in $(find $FILE_DIR | grep csv )
   do 
      SENDC=" "uuencode" "$i" "$i" ; "$SENDC 
   done
 
#fixing the Subject of the mail
SUBJECT=$(basename $i)
 
 
 
#Sending the mail
 
FINAL_COMAND="( "$SENDC" ) | mail -s "$SUBJECT" "$TO
 
$FINAL_COMAND
echo $FINAL_COMAND

en lançant le script j'obtient cette erreur :

( uuencode /HDisclosure/files/POSITIONS.csv /home/HDisclosure/files/POSITIONS.csv ; uuencode /HDisclosure/files/POSITIONS1.csv /home/HDisclosure/files/POSITIONS1.csv ) | mail -s POSITIONS me@mail.com

./Hdc_sendMail.ksh[43]: (: introuvable.

je pense que ça provient du ";"

Lorsque je fais un copier coller de la commande dans le terminal ça envoi le mail par contre le script retourne l'erreur ci-dessus.

MErci pour votre aide.

[jmelyn]

Sans rapport direct à la question, uuencode était utilisé pour coder un fichier binaire en texte afin de l'envoyer par mail (qui n'acceptait que le texte). Maintenant on utilise MIME (MIME). Les fichiers csv (Comma Separated Value) sont déjà du texte, alors pourquoi utiliser uuencode?

[volimp]

Sans rapport direct à la question, uuencode était utilisé pour coder un fichier binaire en texte afin de l'envoyer par mail (qui n'acceptait que le texte). Maintenant on utilise MIME (MIME). Les fichiers csv (Comma Separated Value) sont déjà du texte, alors pourquoi utiliser uuencode?

pourquoi utiliser uuencode ... c'est la solution la plus proposée par google

Merci pour les infos.


sinon j'arrive à implémenter la fonctionnalité autrement :

Code :

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

1
2
3
4
5
6
7
8
9
for i in $(find $FILE_DIR | grep csv ) 
do 
echo "handling the file " $(basename $i) 
echo 
uuencode $i $i >> PJ.tmp 
 
done 
 
mail -s "$(basename $i)" -c $CC $TO < PJ.tmp

[jmelyn]

Google ne doit certainement pas proposer d'utiliser uuencode lorsque cela est inutile! On peut directement envoyer le fichier.

En nettoyant le code:

Code :

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

1
2
3
4
5
6
for i in $FILE_DIR/*.csv
do 
  echo -e "handling the file ${i##*/}\n"
  uuencode $i $i >> PJ.tmp 
done 
mail -s "${i##*/}" -c $CC $TO < PJ.tmp