Bug ensure_dir ?

Questions about WAPT Packaging / Requêtes et aides autour des paquets Wapt.
Règles du forum
* Questions can be asked in English or in French. If the thread is started in English, stick to English, if it is started in French, stick to French.
* Vous pouvez poser vos questions en Français ou en Anglais. Si un sujet est commencé en français, merci de répondre en français. Si un sujet est commencé en anglais, merci de répondre en anglais.
* Please prefix of the title of the thread to [SOLVED] if it is solved.
* Merci de préfixer le titre du thread par [RESOLU] si il est résolu.
erict
Messages : 56
Inscription : 22 déc. 2017 - 19:09

22 juil. 2018 - 13:22

Bonjour,

Version wapt 1.5

J'ai essayé d'utiliser le setuphelper ensure_dir dans un package, et j'ai constaté un dysfonctionnement :

En fait, la comande ensure_dir('C:\Programmes\Ape') ne faisait rien. J'ai testé dans une console python avec toutes les combinaisons possibles (en mettant "r" devant 'C:\Pro...', en utilisant Program Files au lieu de Programmes..., rien à faire.

Du coup, j'ai été voir la code de la fonction, et j'ai fait des tests en python directement :
(bien sur le repertoire C\:Program rrr n'existe pas)

Code : Tout sélectionner

>>> ## Test 1 :
>>> d=path.dirname('C:\program rrr')
>>> d
'C:\\'
>>> path.isdir(d)
True

>>> ## Test 2 :
>>> d="C:\Program rrr"
>>> d
'C:\\Program rrr'
>>> path.isdir(d)
False

>>> ##Test 3 :
>>> d="C:\Program Files"
>>> d
'C:\\Program Files'
>>> path.isdir(d)
True

Dans le test 1, on voit que si d est déclaré en tant que path.dirname, la fonction isdir() répond toujours True, même si le chemin n'existe pas. (La réponse obtenue quand je lui demande d'afficher d est un peu étonnante, mais dans la suite, on voit que ça ne semble pas être la cause du probleme, même si c'est déja problematique)
Dans le test 2, on voit que si d est déclaré en tant que string et que le chemin n'existe pas, on obtient bien False.
Dans le test 3, on voit que par contre, si le chemin existe quand d est déclaré comme string, on obtient bien un True.

Il me semble donc que dans ensure_dir(), plutot que d'utiliser

Code : Tout sélectionner

d = os.path.dirname(filename)
Vous devriez utiliser

Code : Tout sélectionner

d = filename
Ou directement utiliser la variable filename, puisque d, dans ce cas ne me semble plus servir à grand chose, à part, peut être pour convertir filename en string:

Code : Tout sélectionner

d = str(filename)
Cordialement
E. Trezel
erict
Messages : 56
Inscription : 22 déc. 2017 - 19:09

22 juil. 2018 - 17:10

Re-bonjour,

J'ai poussé un peu plus mes investigations, très étonné que j'étais de trouver un bug dans une fonction qui doit être pas mal utilisée, et aussi par l'affichage de la variable d dans mon test 1, qui est en fait bien la cause du probleme :

Code : Tout sélectionner

>>> ## Test 1 :
>>> d=path.dirname('C:\program rrr')
>>> d
'C:\\'
>>> path.isdir(d)
True
La réponse de isdir() est logique, puisque d est égal à C:, et que C: existe bien.
Du coup, j'ai cherché avec quelle(s) syntaxe(s), la valeur de d était celle recherchée.
Avec les valeurs "intuitives" (chemin windows normal, avec ou sans "\" à la fin), en fait, ça ne marche pas :

Code : Tout sélectionner


>>> d = path.dirname('C:\Program Files')
>>> d
'C:\\'

>>> d = path.dirname('C:\ProgramFiles\')
	File "<stdin>", line 1
		d = path.dirname('C:\ProgramFiles\')
									^
SyntaxError:  EOL while scanning string literal

Remarque : Le fait de rajouter 'r' devant le parametre ( ex : d = path.dirname(r'C:\ProgramFiles\') ), n'a rien changé à mes résultats.
Les valeurs pour lesquelles j'ai réussi à faire fonctionner cette fonction correctement (d a la valeur voulue, et path.isdir() renvoie la bonne valeur) sont :

Code : Tout sélectionner

## Chemin Windows "normal", et on échappe le dernier "\" :
d = path.dirname('C:\Program Files\\')
## Chemin Windows "normal", et on ajoute un espace après le dernier "\" :
d = path.dirname('C:\Program Files\ ')
## Chemin Windows "normal", et on remplace le dernier "\" par un "/" :
d = path.dirname('C:\Program Files/')
## Chemin style "Linux", avec des "/" et non des "\" (et "/" à la fin obligatoire) :
d = path.dirname('C:/Program Files/')
## 
Ceci n'est pas signalé dans la doc de ensure_dir() (https://wapt.tranquil.it/wapt/nightly/a ... ensure_dir), et n'est pas pris en compte dans le code, à moins que j'ai raté quelque chose.
Je n'ai d'ailleurs rien trouvé non plus dans la doc de path.dirname() ou isdir() du coté de python.

En esperant que ça aide...
E.T.
Répondre