Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix relative paths #5

Open
qchateau opened this issue Mar 8, 2014 · 7 comments
Open

Fix relative paths #5

qchateau opened this issue Mar 8, 2014 · 7 comments

Comments

@qchateau
Copy link
Member

qchateau commented Mar 8, 2014

Actuellement beaucoup de path sont codés comme ceci :
"../../foo/bar"
Le problème c'est que ce path relatif est interprété par rapport à l'endroit ou l'on lance l'interpréteur Python. Il faut donc changer tout les path sous cette forme :

path = os.path.join(os.path.dirname(os.path.abspath(file)),"../../foo/bar/")

Cela permettra d'éviter tout bug de la part des liens relatifs

@furmi
Copy link
Contributor

furmi commented Mar 8, 2014

Utiliser les os.path.join, ya que ça de vrai :)

@schadocalex
Copy link
Member

Euuuuuuh, please not!
Rien que dans ta phrase ça va pas, "il faut donc changer tous les path sous cette forme" ça fait sale. (Selon moi).

Je préférerais qu'on modifie tous les paths comme ça :
path = getPath("../../foo/bar/"); et après on se créé une fonction getPath quelque part dans laquelle on met du code qui permet de modifier l'url (en l'occurence os.path.join(...)).

@trecouvr
Copy link

trecouvr commented Mar 8, 2014

Oo vous faites des math pour quoi ? Si c'est pour importer des modules y'a
une syntaxe spéciale
Le 8 mars 2014 17:07, "Alexis" notifications@github.com a écrit :

Euuuuuuh, please not!
Rien que dans ta phrase ça va pas, "il faut donc changer tous les path
sous cette forme" ça fait sale. (Selon moi).

Je préférerais qu'on modifie tous les paths comme ça :
path = getPath("../../foo/bar/"); et après on se créé une fonction getPath
quelque part dans laquelle on met du code qui permet de modifier l'url.

Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-37101530
.

@qchateau
Copy link
Member Author

qchateau commented Mar 8, 2014

Pour importer des fichiers, genre xml

@alexis : Non, parceque cette fonction faut aussi aller la chercher, etc
etc...

CHATEAU Quentin

Etudiant en Génie Informatique à l'Université de Technologie de Compiègne
Trésorier et Responsable communication de l'association UTCoupe

Mail : quentin.chateau@gmail.com quentin.chateau@gmail.com
Tel : 07 79 82 99 34

Le 8 mars 2014 18:29, Thomas Recouvreux notifications@github.com a écrit :

Oo vous faites des math pour quoi ? Si c'est pour importer des modules y'a
une syntaxe spéciale
Le 8 mars 2014 17:07, "Alexis" notifications@github.com a écrit :

Euuuuuuh, please not!
Rien que dans ta phrase ça va pas, "il faut donc changer tous les path
sous cette forme" ça fait sale. (Selon moi).

Je préférerais qu'on modifie tous les paths comme ça :
path = getPath("../../foo/bar/"); et après on se créé une fonction
getPath
quelque part dans laquelle on met du code qui permet de modifier l'url.

Reply to this email directly or view it on GitHub<
https://github.com/utcoupe/coupe14/issues/5#issuecomment-37101530>
.

Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-37103685
.

@schadocalex
Copy link
Member

J'trouve ça tellement dégueu de copier partout le bout de code à rallonge, alors qu'il suffit de faire une classe Asset et c'est réglé...
C'est quoi ton etc. etc. ? Car aller chercher la fonction c'est pas un problème, sinon ça veut dire qu'on peut aller chercher aucun autre fichier ^^

@qchateau
Copy link
Member Author

qchateau commented Mar 9, 2014

Je trouve ca dégueu de créer un fonction dont le seul but est de raccourcir
le nom d'une autre fonction...

CHATEAU Quentin

Etudiant en Génie Informatique à l'Université de Technologie de Compiègne
Trésorier et Responsable communication de l'association UTCoupe

Mail : quentin.chateau@gmail.com quentin.chateau@gmail.com
Tel : 07 79 82 99 34

Le 8 mars 2014 23:46, Alexis notifications@github.com a écrit :

J'trouve ça tellement dégueu de copier partout le bout de code à rallonge,
alors qu'il suffit de faire une classe Asset et c'est réglé...
C'est quoi ton etc. etc. ? Car aller chercher la fonction c'est pas un
problème, sinon ça veut dire qu'on peut aller chercher aucun autre fichier
^^

Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-37112372
.

@trecouvr
Copy link

trecouvr commented Mar 9, 2014

Aller chercher une fonction c'est pas un problème avec les importés relatifs
Le 8 mars 2014 23:46, "Alexis" notifications@github.com a écrit :

J'trouve ça tellement dégueu de copier partout le bout de code à rallonge,
alors qu'il suffit de faire une classe Asset et c'est réglé...
C'est quoi ton etc. etc. ? Car aller chercher la fonction c'est pas un
problème, sinon ça veut dire qu'on peut aller chercher aucun autre fichier
^^

Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-37112372
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants