script shell : comparer 2 répertoires entre eux

Version imprimable

Bonjour,

voila j'ai realisé un petit script shell qui permet, en passant 2 repertoire en argument, de les comparer et de renvoyer un code annonçant s'ils sont identique ou non (dans leur contenu aussi).
Le probleme est que j'ai besoin de me deplacer dans les repertoire dans le script, et que ça me renvoi cette erreur "can't cd to..."

Voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
# si le nombre d'arguments n'est pas egal a 2
if [ $# -ne 2 ]; then
 
    echo "Erreur : Il faut 2 arguments."
 
# sinon, si le nombre d'arguments est egal a 2
elif [ $# -eq 2 ]; then
 
	path=`pwd`
 
    # si les deux arguments sont des repertoires
    if [ -d $1 ] && [ -d $2 ]; then
 
	# on liste le contenu du repertoire qui contient le plus d'elements
	if [ `ls -l $1 | wc -l` -gt `ls -l $2 | wc -l` ]; then
	    rep1=$1
	    rep2=$2
	else
	    rep1=$2
	    rep2=$1
	fi	
 
	# listage du repertoire qui a le plus d'elements
	contentrep1=`ls $rep1`
 
	# on les elements de rep1
	for var in $contentrep1
	  do
	  # on se place dans $rep1
	  cd $rep1
	  # si $var est un fichier dans $rep1
	  if [ -f $var ]; then
	      # on se place dans rep2
	      cd $rep2
	      # si $var n'est pas un fichier dans $rep2
	      if [ ! -f $var ]; then
		  echo "Code de retour : 1"
		  exit 1
	      fi
	  # sinon, si $var est un repertoire dans $rep1
	  elif [ -d $var ]; then
	      # on se place dans $rep2
	      cd $rep2
	      # si $var n'est pas un repertoire dans $rep2
	      if [ ! -d $var ]; then
		  echo "Code de retour : 1"
		  exit 1
	      fi
	  fi
	done
	# si on arrive ici, c'est que les deux repertoires sont identiques
	echo "Code de retour : 0"
	exit 0
 
 
    fi
fi

Voila, j'ai essayé plein de bidouillages syntaxique mais je n'y arrive pas :(
Si quelqu'un connait une solution ou une alternative merci d'avance :mouarf:

++

04/12/2007, 10h18
gangsoleil
Bonjour,
Citation:
Envoyé par chicna

Code:

1 2 3 4 5 6 7 8 9 ... # on se place dans $rep1 cd $rep1 # si $var est un fichier dans $rep1 if [ -f $var ]; then # on se place dans rep2 cd $rep2 ...
Avec le premier cd, tu vas dans $rep1 - cela fonctionne sans problème.
Tu es maintenant dans $rep1, et tu cherches à faire : cd $rep2
Ca ne peut pas fonctionner, puisque $rep2 n'est pas forcément inclus dans $rep1

Pour débugger ton script, je te conseil d'ajouter, en tant que première ligne :
Code:

1 2 set -x
Ensuite, mets des 'pwd' un peu partout, ça te montrera que tu as des soucis de répertoire.

Par la suite, tu déplaceras le set -x plus bas dans ton script, là où tu as un soucis.

Ah oui, un dernier point : un script commence par le shell qu'il utilise, avec par exemple la syntaxe suivante, en supposant que tu utilises bash :

Code:

#!/bin/bash

Code:

1 2 3 4 5 6 7 8 9 10 #!/bin/bash file1="/tmp/file1.$$" for i in `find ${1} | sed 1d` ; do echo "${i##${1}}" >> ${file1} ; done file2="/tmp/file2.$$" for i in `find ${2} | sed 1d` ; do echo "${i##${2}}" >> ${file2} ; done diff ${file1} ${file2} > /dev/null
Dans l'état, il n'y a pas de contrôle de paramètres.
Ca crée deux fichiers temporaires dans /tmp (donc suppose un droit en écriture), récupère dans chacun de ces fichiers le contenu de chaque répertoire (excepté le nom du répertoire lui même, qui peut être différent, d'où le sed 1d). Dans chaque fichier, on récupère le chemin complet de chaque sous répertoire et fichier

Si par exemple j'ai

Citation:

a
a/b
a/b/c
a/b/c/d

Alors, j'aurai dans mon fichier

Citation:

/b
/b/c
/b/c/d

Puis enfin diff pour comparer (en jetant la sortie, seul le code de retour intéresse).

Tel quel, le script retourne directement la valeur de diff, donc 0 si OK, 1 sinon.

Il faudrait rajouter des contrôles sur les paramètres, éventuellement supprimer les fichiers temporaires, etc. Et modifier find si tu veux chercher de façon plus fine, bien sûr.
Je me contente de te montrer une base potentielle :)

Pose des questions si tu ne comprends pas.

<edit>
Petit conseil supplémentaire : plutôt que d'utiliser cd dans les scripts, je te conseille fortement le couple pushd/popd plus facile à gérer

04/12/2007, 19h05
Chicna

ok, c'etait donc une simple erreur de logique. Merci ^^

Ma solution :
path=`pwd` au debut du script ;
cd $path avant chaque cd ;
ça marche :D.
18/09/2011, 07h39
bigs3232

Citation:

Envoyé par Pouic

Code:

for i in `find ${2} | sed 1d` ; do echo "${i##${2}}" >> ${file2} ;

désolé de remonter ce topic mais je ne comprends pas trop ce code mais c'est quoi ${2}.a quoi ce là correspond car je ne vois pas qu'il est défini nulle part.merci
18/09/2011, 09h26
Sve@r
Citation:

Envoyé par bigs3232

désolé de remonter ce topic mais je ne comprends pas trop ce code mais c'est quoi ${2}.a quoi ce là correspond car je ne vois pas qu'il est défini nulle part.merci

Salut

Les $1 $2 $3 $4 ... (jusqu'à $9) correspondent aux arguments du script.
Si tu lances le script de cette manière: script toto titi tata<return> alors $1 contiendra toto, $2 contiendra titi et $3 contiendra tata. Cette affectation est faite automatiquement par le shell donc tu n'as pas besoin de les définir, mais seulement de les utiliser si tu en as envie/besoin. On les nomme les "paramètres positionnels".

Les accolades servent à isoler le nom de la variable en cas d'ambigüité possible.
Exemple
Code:

1 2 prix=10 echo "Mon carambar coûte $prixcts"
Ici, le shell essaiera d'afficher la variable "prixcts" (qui n'existe pas) et ne comprendra pas que tu veux afficher $prix suivi des lettres "cts"

Pour éviter ce pb, on encapsule le nom de la variable avec des accolades
Code:

1 2 prix=10 echo "Mon carambar coûte ${prix}cts"
18/09/2011, 09h51
zipe31

Salut,

Citation:

Envoyé par Sve@r

Les $1 $2 $3 $4 ... (jusqu'à $9) correspondent aux arguments du script.

Et au-delà si tu les entoures avec des accolades ${10}, ${25}, ${101}, etc.

;)
18/09/2011, 12h08
N_BaH

il ne faut pas utiliser ce code (for elem in `commande`; do), c'est générateur d'erreur.

Si ce que retourne la commande contient un espace, la boucle for itérera sur chaque élément retourné.

find peut se suffire à lui-même, en utilisant l'option -exec, ou en pipant sa sortie vers xargs
18/09/2011, 14h06
chardclo

Bonjour,

Pourquoi ne pas utiliser tout simplement diff ?
19/09/2011, 07h56
Sve@r

Citation:

Envoyé par zipe31

Salut,

Et au-delà si tu les entoures avec des accolades ${10}, ${25}, ${101}, etc.

;)

Yep, mais uniquement dans certains shells (bash oui, ash ou zsh ou ksh je sais pas, sh non) ;)

Citation:

Envoyé par chardclo

Bonjour,

Pourquoi ne pas utiliser tout simplement diff ?

Oui, diff ou dircmp. Mais bon, le topic date de 2007 et l'auteur (qui voulait peut-être juste s'exercer) a dû l'oublier depuis donc je ne pense pas que tu auras de réponse à ce sujet. Et là on répondait juste à bigs3232 sur une question de syntaxe... 8-)
19/09/2011, 09h43
chardclo

Citation:

Envoyé par Sve@r

Oui, diff ou dircmp. Mais bon, le topic date de 2007 et l'auteur (qui voulait peut-être juste s'exercer) a dû l'oublier depuis donc je ne pense pas que tu auras de réponse à ce sujet. Et là on répondait juste à bigs3232 sur une question de syntaxe... 8-)

arg oui, j'ai pô fait gaffe aux dates ;):oops:
19/09/2011, 14h24
zipe31

Citation:

Envoyé par Sve@r

Yep, mais uniquement dans certains shells (bash oui, ash ou zsh ou ksh je sais pas, sh non) ;)

Pour les récalcitrants il ne reste plus qu'à prendre un bon petit shift :mouarf:
19/09/2011, 18h42
Sve@r

Citation:

Envoyé par zipe31

Pour les récalcitrants il ne reste plus qu'à prendre un bon petit shift :mouarf:

Ben oui, parfois certaines personnes travaillent dans des environnements plus restrictifs/anciens et tous n'ont pas forcément la chance d'avoir du bash :cry: :ccool:

Citation:

Envoyé par zipe31

Et au-delà si tu les entoures avec des accolades ${10}, ${25}, ${101}, etc.
;)

D'un autre coté je ne me souviens pas avoir dû gérer un jour plus de 9 arguments différents (j'en gère parfois des dizaines mais comme ils sont de même nature je fais une boucle). Alors avant d'avoir à en gérer 101... :D