Passage à Python3

[paternal]

• Abandon de PlasTeX
• Passage à Python3
  • Faire fonctionner songbook
  • Vérifier qu'il n'y a plus de problèmes d'unicode
  • Installation : setup.py
  • Installation : stdeb => Abandonné
  • Supprimer tous les commentaires # -*- coding: utf-8 -*- pour faire échouer l'interpréteur Python2
• Réorganisation des plugins de lecture de type de fichiers
  • Plugin intersong
  • Plugin latex
  • Plugin chordpro => Repoussé à une version ultérieure
• Répercussion des conséquences
  • Documentation => Dire que patacrep utilise Python3 patacrep-doc#8
  • Conversion des chansons de patadata => Repoussé à une version ultérieure (avec chordpro)
  • Mise à jour de patanet => On n'attend pas patanet

Edit Luthaf : Ajout des évolution avec l'arrêt du support Python 2.
Edit Spalax : Changement de nom, de « Nouveau format » à « Passage à Python3 », et modifications en conséquence.

Closes #64 #60

[paternal]

Un analyseur syntaxique LaTeX fonctionne. Un brève description des restrictions est ici : patacrep/patacrep-doc#6.

[Luthaf]

Beau boulot, je vais tester d'ici peu.

Concernant le point

Passage à Python3

pour patanet, on dépends de GitPython lequel dépends de gitdb, lequel est en cours de transition. Il risque donc d'y avoir un petit moment ou l'on reste en python 2.

[Luthaf]

Il risque donc d'y avoir un petit moment ou l'on reste en python 2.

Question en lien avec ça : est-ce que l'on dit que l'on supporte Python2.7 et Python 3 (> 3.3) en même temps ?

[Luthaf]

Alors tout semble fonctionner pour le moment chez moi dans un virtualenv tout propre. J'ai simplement des logs un poil trop détaillés :

WARNING:patacrep.songs:Could not use cached version of /patacrep/data/examples/songs/example-en.sg.
Generating LALR tables
WARNING: 20 shift/reduce conflicts

Le premier warning est normal, et ne devrais pas trop apparaitre. Les lignes suivantes sont plus étranges dans ce cas. Il faudrait aussi ajouter les fichiers parser.out et parsetab.py à l'étape de nettoyage.

Par contre sous python2, la fonction qui charge les plugins plante.

[paternal]

Il faudrait aussi ajouter les fichiers parser.out et parsetab.py à l'étape de nettoyage.

Ça alors. Il me semblait qu'avec le commit 7895aca, ils n'étaient plus générés. Tu confirmes que le test que tu as fait intégrait ce commit ?

[Luthaf]

Oui :

$ git log -n 5 | cat 
commit a17519bbea0677db8e085ae7c094d3d1aecda237
Author: Luthaf <luthaf@luthaf.fr>
Date:   Wed Oct 1 15:34:53 2014 +0200

    Start Python2 compatibility mode

commit 9bc740e096d23338447aad5a58677487696fc632
Author: Luthaf <luthaf@luthaf.fr>
Date:   Wed Oct 1 15:34:42 2014 +0200

    Update setup.py

commit 04f2554759e0eb5e30645cdc8c89649922981699
Author: Louis <spalax@gresille.org>
Date:   Wed Oct 1 13:16:58 2014 +0200

    Simplification

commit d34441fbae3523fb558b6a891e389f4e15552eb6
Author: Louis <spalax@gresille.org>
Date:   Wed Oct 1 11:12:47 2014 +0200

    Added intersong example

commit 7895acad0f3099121af9fe2dc9eb4ee9cdf1cf02
Author: Louis <spalax@gresille.org>
Date:   Wed Oct 1 10:58:20 2014 +0200

    Deleting output and temporary files generated by yacc

[paternal]

Oui

Zut. À creuser…

[paternal]

Il faudrait aussi ajouter les fichiers parser.out et parsetab.py à l'étape de nettoyage.

Je viens de réessayer chez moi, et il ne sont pas créés. Je n'ai pas le temps de me pencher là dessus maintetant, mais il faudra s'en occuper.

[paternal]

On peut se dire que les gens installerons Python 3 s'ils veulent utiliser Patacrep alors.

Ça me va : Python3 est une dépendance comme une autre à Patacrep.

Par contre, est-ce que on attend ce passage pour sortir la 4.0 ?

Ça me parait bien, dans la mesure où c'est un changement majeur (qui, entre autres, change les dépendances).

Par contre une q

Oui ? :P

[Luthaf]

Par contre une q
Oui ? :P

Rien, juste un message posté trop tard ...

[paternal]

Le premier warning est normal, et ne devrais pas trop apparaitre. Les lignes suivantes sont plus étranges dans ce cas. Il faudrait aussi ajouter les fichiers parser.out et parsetab.py à l'étape de nettoyage.

J'ai tenté d'annuler la génération de ces fichiers (5084a0b). C'est quelque chose que j'ai du mal à reproduire, donc je ne suis pas sûr que ça fonctionne. On verra à l'usage…

[Luthaf]

Ça a l'air bon chez moi aussi. Désolé, je trouve tout juste le temps de tester ça =)

[paternal]

J'ai l'impression qu'on est quasiment prêts à merger. Restent :

• Supprimer tous les commentaires # -- coding: utf-8 -- pour faire échouer l'interpréteur Python2
• Documentation : il suffit de voir où est-ce qu'on mentionne Python2, ou Python sans préciser la version.
• Mise à jour de Patanet. Je propose de ne pas attendre ça pour merger : j'ai l'impression qu'on ne fait plus de modifications sur la branche master, donc autant merger python3, et pour patanet, vous n'aurez qu'à utiliser un de dépôt avant le merge, tant que ça n'est pas compatible python3.

Si ça vous convient, je m'attaque à ces trois points, et propose la fermeture du ticket.

[Luthaf]

Mise à jour de Patanet. Je propose de ne pas attendre ça pour merger : j'ai l'impression qu'on ne fait plus de modifications sur la branche master, donc autant merger python3, et pour patanet, vous n'aurez qu'à utiliser un de dépôt avant le merge, tant que ça n'est pas compatible python3.

Dans mes derniers tests, patanet était presque compatible python3, sauf pour un patch remonté, mais pas encore mergé.

Tout le reste me convient !

[paternal]

Pour moi, c'est prêt à merger.

[Luthaf]

Let's go !

[paternal]

Super !