Bug ensure_dir ?

Questions about WAPT Packaging / Requêtes et aides autour des paquets Wapt.
Règles du forum
Règles du forum communautaire
* English support on www.reddit.com/r/wapt
* Le support communautaire en français se fait sur ce forum
* Merci de préfixer le titre du topic par [RESOLU] s'il est résolu.
* Merci de ne pas modifier un topic qui est taggé [RESOLU]. Ouvrez un nouveau topic en référençant l'ancien
* Préciser version de WAPT installée, version complète ET numéro de build (2.2.1.11957 / 2.2.2.12337 / etc.) AINSI QUE l'édition Enterprise / Discovery
* Les versions 1.8.2 et antérieures ne sont plus maintenues. Les seules questions acceptées vis à vis de la version 1.8.2 sont liés à la mise à jour vers une version supportée (2.1, 2.2, etc.)
* Préciser OS du serveur (Linux / Windows) et version (Debian Buster/Bullseye - CentOS 7 - Windows Server 2012/2016/2019)
* Préciser OS de la machine d'administration/création des paquets et de la machine avec l'agent qui pose problème le cas échéant (Windows 7 / 10 / 11 / Debian 11 / etc.)
* Eviter de poser plusieurs questions lors de l'ouverture de topic, sinon il risque d'être ignorer. Si plusieurs sujet, ouvrir plusieurs topic, et de préférence les uns après les autres et pas tous en même temps (ie ne pas spammer le forum).
* Inclure directement les morceaux de code, les captures d'écran et autres images directement dans le post. Les liens vers les pastebin, les bitly et autres sites tierces seront systématiquement supprimés.
* Comme tout forum communautaire, le support est fait bénévolement par les membres. Si vous avez besoin d'un support commercial, vous pouvez contacter le service commercial Tranquil IT au 02.40.97.57.55
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.
Verrouillé