Discussion :

[mrttlemonde]

Bonjour à tous,

voici mon problème : j'ai écrit un script qui me permet de reconstruire un fichier (rar, xtm ou découpé simplement) donné. L'utilisateur n'a qu'a glisser/déposer le fichier sur le terminal et il verra s'afficher le chemin du fichier. Le problème est que je ne parviens pas à utiliser ce chemin! Il m'indique des erreurs incompréhensibles... exemple :

Sortie du script :
----------------
Fichier à reconstruire : '/home/salon/Bureau/liens textes.txt'
if [[ ! -e '/home/jon/Bureau/liens av.txt' ]]
Erreur : fichier inexistant
basename '/home/jon/Bureau/liens av.txt'
dirname: opérande surnuméraire `av.txt\''

Voici une partie du code qui pose problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
read leFichier
...
echo "if [[ ! -e $leFichier ]]" #Debugging
if [[ ! -e $leFichier ]]; then
	echo "Erreur : fichier inexistant"
	#exit 2
fi
echo "basename $leFichier" #Debugging
fichierSource=$(basename $leFichier)

Pourtant quand j'exécute le if "à la main", ça fonctionne!
Ou est ce que ça ne va pas? Je ne comprends pas!

[chaval]

bonjour,

C'est sans doute les espaces dans ton nom de fichier qui posent problème.
As-tu essayé des les échapper avec un backslash ?

[N_BaH]

Bonjour mrttlemonde


Pour moi, la condition fonctionne correctement également dans un script.

Cependant, avant tout, pour debugguer un script, utilise set -x :
tu peux le placer au tout début de ton script, ou juste au-dessus de la partie que tu souhaites debugguer, et que tu "fermeras" par set +x
...

Ensuite, à quoi sert ta condition si de toute façon le script poursuit son exécution alors qu'il n'y a pas de fichier valide ?


Et, utilise davantage de guillemets :

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
 
#!/bin/bash
 
#
set -x
 
read -p 'Entrez un nom de fichier : ' fic
if [[ ! -e $fic ]]
# ou if [ ! -e "$fic" ]
 then
   echo "$fic KO"
else
   echo "$fic OK"
   basename "$fic"
   dirname "$fic"
fi

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$ ./testSpacefile.sh
+ read -p 'Entrez un nom de fichier : ' fic
Entrez un nom de fichier : space file
+ [[ ! -e space file ]]
+ echo 'space file OK'
space file OK
+ basename 'space file'
space file
+ dirname 'space file'
.

PS: ce n'est sans doute pas la meilleure rubrique de développez.net pour obtenir une réponse concernant un shell script

[mrttlemonde]

Merci pour vos réponses!

@ chaval,
Je pense aussi que ça doit être du aux espaces qui se trouvent dans le chemin du fichier... mais après, quand j'arrive à la commande basename, voici ce que ça donne au debugging :

++ basename ''\''/home/salon/Bureau/liens' 'texte.txt'\'''
+ fichierSource=liens

On aurait dit que les apostrophes font partie de la chaine de caractères. Il faudrait donc
- supprimer les deux apostrophes
- ajouter un backslash avant le/les espaces qui pourraient se trouver dans le chemin/nom du fichier...
____________

@ N_BaH,
Tu as essayé la condition dans un script en glissant le fichier sur le terminal? Parce que la condition fonctionne également chez moi si j'écris le chemin du fichier à la main c-a-d avec les backslash si il y a des espaces et sans apostrophe au début et à la fin.

Merci pour l'astuce du debugging! C'est beaucoup plus facile comme ca.
C'est vrai que cette condition ne sert à rien pour le moment mais j'ai commenté la commande exit pour voir la suite (notamment le résultat de basename).

[N_BaH]

Je ne comprend pas bien le coup du drag'n'drop...

- je lance le script dans un terminal (j'atteint le read)
- j'ouvre un post-it, ou un quelconque éditeur de texte fenêtré (mousepad, gedit...)
- j'écris le chemin et le nom du fichier (/home/monuser/space file)
- je surligne (<ctrl-a>)
- je glisse ce que j'ai surligné (ou même une icône de mon bureau pointant vers un fichier texte) dans le terminal ouvert
et le script se déroule de la même manière que précédemment...
?

[mrttlemonde]

En fait, le drag/drop c'est une facilité pour les futurs utilisateurs de mon script (ceux qui ont du mal avec l'informatique... tu vois ce que je veux dire?) et donc je leur expliquerai qu'il faudra simplement glisser le fichier sur la console (une fois qu'ils auront cliqué sur l'icone du script).

Mais apparement tu as aussi fait ça? Il paraitrait que les test tel que -e, -f etc. ne s'utilisent que pour les fichiers dans le dossier courant? Si c'est le cas, ça change tout!

Je comprends pas comment ça peut fonctionner chez toi... tu utilises aussi bash? Quelle version? enfin je ne pense pas que le problème vienne de là...

c'est vraiment frustrant!

[exodev]

Hello,

Il paraitrait que les test tel que -e, -f etc. ne s'utilisent que pour les fichiers dans le dossier courant? Si c'est le cas, ça change tout!

non

Dans ton rapport d'erreur du 1er post:

dirname: opérande surnuméraire `av.txt\''

Ce serait possible de voir ce dirname ?

[mrttlemonde]

Salut,

voici le résultat du script :

'/home/salon/Bureau/liens av.txt'
+ '[' '!' -f ''\''/home/salon/Bureau/liens av.txt'\''' ']'
+ echo 'Erreur : fichier inexistant'
Erreur : fichier inexistant
++ basename ''\''/home/salon/Bureau/liens' 'av.txt'\'''
+ fichierSource=liens
++ dirname ''\''/home/salon/Bureau/liens' 'av.txt'\'''
dirname: opérande surnuméraire `av.txt\''
Pour en savoir davantage, faites: « dirname --help ».

Alors je ne comprends plus...

AH ! Il y a une amélioration :
Si je supprime les apostrophes, le fichier est reconnu (il passe le test if [ ! -f ... ]) PAR CONTRE, le probleme de basename persiste... à cause de l'espace.

[N_BaH]

GNU bash, version 3.2.39(1)-release (i486-pc-linux-gnu) sur Debian GNU/Linux Stable

si e n''est pas le shell, ce serait le terminal ? l'environnement graphique ? Quel terminal, et quel GUI utilises-tu ?

J'ai fait un essai en tirant un fichier depuis Thunar, et j'ai obtenu, dans la console, le chemin et le nom du fichier entre guillemets simples ('/chemin/fichier'), et évidemment le script ne fonctionne plus. J'ai dû ajouter une ligne après le read :

Code :

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

fic=$(sed "/^'.*'$/s/^'\(.*\)'$/\1/g" <<<"$fic")

pour le faire fonctionner à nouveau.

[mrttlemonde]

Oui c'est bien ça! Je suis arrivé à la même chose...
C'est déjà une bonne chose!
Par contre pour les espaces, il faudra sans doute renommer le fichier non? Dans ce cas là je risque de rencontrer pas mal de problèmes puisque je devrais aussi renommer les autres parties nécessaires à la reconstruction du fichier. Vous me suivez? Ca risque d'être compliqué...

[mrttlemonde]

Voila, le problème est finalement résolu...
j'avais oublié d'ajouter des guillements (comme tu me l'avais conseillé N_BaH)
basename "$fichier"

Merci beaucoup pour votre aide!