erreur en bash

Bonjour,
J'ai un script a écrire pour un travail de session. J'obtiens l'erreur suivante quand j'execute le script:

line 6: syntax error near unexpected token `do

Voici le code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#!/bin/sh   while [ -n $1 ]   do   list= ls $1   for i in $list   do   cat $i >> basename $i   rm $i   done   shift 1   done

---

Je suis certain que c'est une erreur de syntaxe mais j'arrive pas a trouver.

Merci

Ah, oui, mais là, non !

Allez, je vais pas être plus méchant, prend le temps de lire ce tutoriel.

Citation:

---

Envoyé par trealais

Je suis certain que c'est une erreur de syntaxe mais j'arrive pas a trouver.Merci

---

Comment veux-tu que le shell fasse la différence entre

Code:

---

couleur=bleu

---

et

Code:

---

list=ls $1

---

???

Accessoirement, tu peux remplacer

Code:

---

1 2 3 4 5

while [ -n $1 ] do .... $1 shift done

---

par

Code:

---

1 2 3 4

for file in $* do .... $file done

---

Je viens de commencer a programmer en shell. En fait ca fait 1 semaine.
Faut me laisser une chance...

Citation:

---

Envoyé par Sve@r

Comment veux-tu que le shell fasse la différence entre

Code:

---

couleur=bleu

---

et

Code:

---

list=ls $1

---

???

Comment écrirais-tu list=ls $1 alors?

---

Je cherches encore une solution

Merci

J'obtiens toujours l'erreur même si j'écris avec des apostrophes.

comme suit:

Code:

---

list='ls $1'

---

Citation:

---

Envoyé par trealais

Je viens de commencer a programmer en shell. En fait ca fait 1 semaine.
Faut me laisser une chance...

---

ok

Citation:

---

Envoyé par trealais

J'obtiens toujours l'erreur même si j'écris avec des apostrophes.

comme suit:
list='ls $1'

---

Ce que tu cherches à faire est le mécanisme le plus important du shell et ce qui fait sa force. Ca permet au shell de déléguer à d'autres programmes (comme "ls") ce qu'il ne sait pas faire lui-même (comme lister le contenu du répertoire).

Il faut encadrer la commande d'accent graves (ou backquottes). A ce moment là, le shell exécute la commande mais récupère pour son propre usage le texte qui aura été affiché. Et donc ici, ce texte sera stocké dans la variable "list"

Code:

---

list=`ls $1`

---

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code:

---

list=$(ls $1)

---

Ainsi, quel que soit ton besoin, t'auras toujours un programme qui te donnera ce que t'attends (au pire tu l'écris toi-même en C ou autre) puis tu demandes au shell d'exécuter ledit programme et grâce à ce mécanisme, tu récupères dans ton script le résultat dudit programme. Bref tu fais tout faire par ce qui existe au lieu de le réinventer à chaque fois.

S'il y a une seule chose que tu dois apprendre à faire en shell et que tu dois étudier hyper attentivement, c'est ça parce que grâce à ça, tous tes besoins en programmation shell sont à 90% déjà résolus. T'as simplement qu'à appeler le bon programme à chaque fois que t'as un truc à faire.

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code:

---

list=$(ls $1)

---

Merci pour les infos.
C'est bizzare j'obtiens toujours la même erreur et même ce après avoir remplacé:

Code:

---

list=`ls $1`

---

par

Code:

---

list=$(ls $1)

---

Le problème semble être avec:

Code:

---

1 2 3

for i in $list   do

---

Merci

Il y a un problème avec le while que j'arrive à isoler mais je ne sais pas d'où cela vient, ça fait trop longtemps que je n'ai pas fait de bash pour ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#!/bin/bash while [ -n $1 ] do if [[ -n $1 ]] then echo X$1X not null ; else echo X$1X null ; fi shift 1 done

---

Il rentre dans le while alors que $1 est vide...

en utilisant for file comme indiqué ça fonctionne mais cela m'intrigue quand même.

Citation:

---

Envoyé par Zartan

Il y a un problème avec le while que j'arrive à isoler mais je ne sais pas d'où cela vient, ça fait trop longtemps que je n'ai pas fait de bash pour ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#!/bin/bash while [ -n $1 ] do if [[ -n $1 ]] then echo X$1X not null ; else echo X$1X null ; fi shift 1 done

---

Il rentre dans le while alors que $1 est vide...

en utilisant for file comme indiqué ça fonctionne mais cela m'intrigue quand même.

---

Remplace while [ -n $1 ] par while [ -n "$1" ] et t'auras plus ce soucis.

Citation:

---

Envoyé par trealais

Dans les shells récents, on peut remplacer les backquottes par $(...) plus lisible

Code:

---

list=$(ls $1)

---

Merci pour les infos.
C'est bizzare j'obtiens toujours la même erreur et même ce après avoir remplacé:
list=`ls $1`
par
list=$(ls $1)

Le problème semble être avec:
for i in $list

do

Merci

---

Le meilleur débugging c'est le "echo". Rajoute un echo "[$list]" juste avant le for histoire de voir avec exactitude ce que contient ta variable...

Merci pour votre aide.
J'obtiens toujours l'erreur même avec: while [ -n "$1" ]
Je me demande maintenant comment implanter le: for file in $*

Est-ce comme ceci?:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#!/bin/sh   for file in $*   do   for i in $file do   cat $i >> `basename $i`   rm $i   done   shift 1   done

---

Si c'est le cas j'obtiens toujours la même erreur.
Il y a un mec sur un autre forum qui dit que mon code semble fonctionner sur Mac OS X.
Bizarre...

Merci pour votre aide.

Ok, avant de continuer, j'aimerais être sûr de ce que tu veux faire. Il me semble en lisant ton script que tu cherches à lister le contenu de répertoires et à recopier les fichiers desdits répertoires dans ton dossier courant.

Si c'est ça, alors ton script a une erreur dans le "for i in $file". Il te faut mettre à la place "for i in $file/*" afin d'avoir le contenu de "$file" et non son simple nom.

Accessoirement le shift 1 devient inutile. De plus, il peut être judicieux de choisir ses noms de variable en rapport avec leur rôle. $file semble être destiné à recevoir un nom de dossier donc autant l'appeler "$dir".

De plus, l'erreur de ton premier script devient claire. Quand tu fais "list=`ls $1`", la variable "list" ne contient que les noms des fichiers situés dans "$1" et non leur chemin d'accès. Exemple: si t'as un répertoire "aaa" qui contient "toto", ta variable "list" contiendra "toto" et non "aaa/toto".
Il s'ensuit que pour que ton premier script fonctionne, il suffisait de remplacer

Code:

---

1 2	cat $i >> basename $i rm $i

---

par

Code:

---

1 2	cat $1/$i >> $i rm $1/$i

---

A titre d'exemple, voici ton premier script que j'ai corrigé dans le but de le faire correspondre à ce que je crois que tu cherches à faire

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#!/bin/sh while test -n "$1" do test ! -d "$1" && shift 1 && continue   for file in `ls "$1"` do cat "$1/$file" >> "$file" rm "$1/$file" done shift 1 done

---

Et le même avec un for à la place d'un while

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#!/bin/sh for dir in $* do test ! -d "$dir" && continue   for file in $dir/* do cat "$file" >> `basename "$file"` rm "$file" done done

---

Tu remarqueras qu'il y a une différence notable entre "ls <nom_de_dossier>" et "<nom_de_dossier>/*"

trealais, il serait important que tu dises exactement ce que doit faire ton programme ! Et ce dès le début, ça éviterait de se poser trop de questions et ça permettrait de trouver plus rapidement la réponse...
Ainsi que les retours demandés par Sve@r, il serait bien de les donner ! Ce qu'il dit me semble cohérent bien que je ne sois pas un expert shell.

Citation:

---

Envoyé par troumad

trealais, il serait important que tu dises exactement ce que doit faire ton programme ! Et ce dès le début, ça éviterait de se poser trop de questions et ça permettrait de trouver plus rapidement la réponse...
Ainsi que les retours demandés par Sve@r, il serait bien de les donner ! Ce qu'il dit me semble cohérent bien que je ne sois pas un expert shell.

---

Pas de problèmes voici l'énoncé de la question à laquelle je dois répondre:

Faire un script en Shell avec un nombre variable de paramètres qui sont des
directories existants dans le directory courant. Pour chacun des directories nommés, le script doit déplacer tous ses fichiers au directory courant. Si les noms des fichiers les mêmes, le script doit concaténer les fichiers sous un même nom dans le directory courant.


Merci pour votre aide!

Citation:

---

Envoyé par trealais

Pas de problèmes voici l'énoncé de la question à laquelle je dois répondre:

Faire un script en Shell avec un nombre variable de paramètres qui sont des
directories existants dans le directory courant. Pour chacun des directories nommés, le script doit déplacer tous ses fichiers au directory courant. Si les noms des fichiers les mêmes, le script doit concaténer les fichiers sous un même nom dans le directory courant, SAUF s'il s'agit d'un fichier
de type PDF où le programme élimine simplement le fichier.

Quand les directories nommés en paramètre sont vidés, ils doivent être détruits.

N.B. On suppose que les directories nommés ne contiennent que des fichiers (pas de sous-directories), et que lorsque votre programme trouve un nom identique au directory courant, ce dernier est un fichier valide à concaténer.

Merci pour votre aide!

---

Ouais ok. Mes exemples correspondent quasiment à ce que tu dois faire. Faut juste rajouter la commande d'effacement du dossier devenu vide (rmdir) et le filtre du pdf => if echo $file |grep -y "\.pdf$" 1>/dev/null renverra vrai si ton nom de fichier porte l'extension ".pdf" en minuscules ou majuscules.

Toutefois tu triches un peu car tu fais une concaténation systématique alors que ton énoncé suppose à un moment donné un examen du fichier trouvé pour voir s'il n'existe pas déjà. Ca marche mais si c'est un TP, alors peut-être que ton prof attend de toi que tu écrives ce test afin que tu lui montres que t'es bien à l'aise à la fois dans ta tête (ça ça semble ok) et à la fois dans ton shell (un peu d'expérience et ça viendra aussi).

Appuies-toi sur mes scripts pour comprendre leur façon d'agir et sur mes remarques à propos de la façon dont tes variables sont remplies (le coup des chemins d'accès présents ou pas) et complète ton tp.