7za et script Shell (rhel 6.8)

Bonjour

j'ai écris un script Schell tout simple permettant de compresser des dossiers et le contenu via 7za.

Code:

---

1 2 3 4 5 6 7

for i in `find * -maxdepth 0 -type d`   do 7za a -mx=9 -t7z $i.7z $i echo 7za a -mx=9 -t7z $i.7z $i #pour voir la commande exécutée done

---

lorsque je lance le script, le fichier est bien créé mais il m'annonce qu'il ne trouve aucun sous dossiers ou fichiers.

Pièce jointe 626155

Par contre si je lance manuellement la commande afficher par "echo 7za a -mx=9 -t7z $i.7z $i", Il trouve bien les sous-dossiers et les fichiers.
Pièce jointe 626156

j'ai essayé différentes méthodes, sans succès. Je n'ai pas trouvé la bonne syntaxe, tout simplement.
Si vous avez des idées que je pourrais tester, un grand merci par avance pour vos attentions.
Un regard neuf permet parfois de cibler une erreur.

Jérôme

Bonjour

Citation:

---

Envoyé par bihouee

for i in `find * -maxdepth 0 -type d`

---

On ne fait pas de for i in `commande`; do action $i; done car le for utilise l'espace pour séparer ses éléments. Si un fichier possède un espace dans son nom, il n'est alors pas traité => commande |while read i; do action "$i"; done ou bien while read i; do action "$i"; done < <(commande) (et tu remarqueras les guillemets autour de la variable "$i" lorsqu'elle est utilisée).
Pour le reste, le souci vient de la commande "7za" donc il faut regarder ses possibilités et options via le "man"...

Surtout qu'un simple: for i in */ est suffisant ici car dans le find présenté, c'est le * qui liste tous les fichiers et le -type d qui écarte ce qui n'est pas un répertoire (c'est syntaxe n'est pas une syntaxe "standard" pour l'utilisation d'un find, la syntaxe standard serait plutôt find -maxdepth 1 -not -name '.*' -type d )

Bonjour Sve@r

Bon, en lisant ta réponse, je me suis dit, mince c'est pas la réponse que j'attendais.

Je n'avais jamais utilisé la commande while.
Comme je suis curieux, je teste la commande pour comprendre son fonctionnement.
J'utilise mon scripte pour cela.
Bah cela fonctionne maintenant :D.

Voici le script modifié (on peut faire autrement mieux peut être).

Code:

---

1 2 3 4 5 6 7

find * -maxdepth 0 -type d > liste while read i do 7za a -mx=9 -t7z $i.7z $i done < liste rm -rf liste

---

Donc merci pour ton aide.

oui :

Code:

---

for d in ./*/; do 7za a -mx=9 -t7z "$d.7z" "$d"; done

---

Citation:

---

Envoyé par bihouee

Je n'avais jamais utilisé la commande while.

---

Bah, si tu apprends le for, faut aussi apprendre le while (qui continue tant que la commande qu'il invoque est vraie) + le until qui, lui, continue tant qu'elle est fausse (toutefois un peu tombé en désuétude depuis qu'on peut inverser le retour vrai/faux d'une commande via le point d'exclamation)

Citation:

---

Envoyé par bihouee

Voici le script modifié (on peut faire autrement mieux peut être).

---

Oui. Déjà les fichiers de "travail" c'est rarement nécessaire mais si ça l'est, alors à minima on les met dans "/tmp". Ca évite déjà à ton script de planter si tu te trouves dans un dossier qui n'est pas à toi lorsque tu le lances car le fichier "liste" sera créé dans le dossier où tu te trouves. Et on leur donne un nom permettant d'éviter les collisions (quid si deux users exécutent le script en même temps ???) donc on peut inclure la variable "$$" (pid en cours) dans le nom (car un pid est unique à un moment donné) ou utiliser mktemp pour des soucis de noms temporaires plus complexes.
Donc de façon générale

Code:

---

1 2	commande >fichier while read i do; ...; done <fichier

---

... se réduit en while read i do; ...; done < <(commande) et on évite de créer un fichier (temps d'accès disque) pour rien.

Ensuite tu as des raccourcis (comme N_BaH) qui permettent parfois d'éviter les appels de commandes externes.

je n'appellerais pas ça un "raccourci", mais la voie directe. :)