Discussion :

[bihouee]

Bonjour

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

Code :

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

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.



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.


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

[Sve@r]

Bonjour

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"...

[disedorgue]

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 )

[bihouee]

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 .

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

Code :

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

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.

[N_BaH]

oui :

Code :

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

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

[Sve@r]

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)

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 bash :

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

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.

[N_BaH]

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