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

Installation - branche 3.10 - remarques #1834

Closed
eviau opened this issue Mar 26, 2022 · 14 comments
Closed

Installation - branche 3.10 - remarques #1834

eviau opened this issue Mar 26, 2022 · 14 comments

Comments

@eviau
Copy link
Contributor

eviau commented Mar 26, 2022

Bonjour !

Comme discuté précédemment, je suis en train de tout ré-installer à partir de rien question de voir où on pourrait améliorer/mettre-à-jour le guide de contribution.

J'ai donc mis à jour mon fork de ce dépôt, et j'y ai ajouté la branche 3.10 à partir de upstream - en en créant une nouvelle.

Pour le moment, je n'arrive pas à tout faire rouler - voici quelques remarques en espérant qu'elles seront utiles au projet. Je suis un peu embêtée car à la relecture, je pense que ces remarques doivent être remontées aux projets respectifs et non pas à ce projet, mais voilà - je me suis dit que j'allais vous demander conseil :)

  1. Réservation de fichiers qui ont plusieurs . dans leur nom
    Il me semble que potodo ne prend pas en compte les fichiers qui ont été réservés et qui ont plus d'un . dans leur nom. J'ai une réservation sur library/http.server.po Je travaille sur library/http.server.po #1833 qui n'affiche pas, et je vois que library/xml.etree.elementtree.po Je travaille sur library/xml.etree.elementtree.po #1811 ne s'affiche pas non plus.

  2. make verifs
    Lorsque je teste ma branche sans avoir fait aucun changement, make verifs me retourne une longue série de fichiers à re-wrapper. Ensuite, je tente de faire cette opération avec powrap - je dois mal m'y prendre car make verifs me mentionne la même info. Serait-ce en lien avec les changements sur l'algo de wrap de gettext ?

  3. license_url n'est pas supportée pour ce thème
    N'ayant crainte de rien, je tente tout de même un make, me disant que cela devrait fonctionner puisque je n'ai pas fait de changements sur la branche. J'obtiens alors que L'option 'license_url' n'est pas supportée pour ce thème en lien avec la doc de cpython sur /python-docs-fr/venv/cpython/Doc. Je vois cette option dans le fichier conf.py de ce dossier, donc c'est probablement un truc avec le thème de Sphinx...

C'est tout pour le moment - merci de me mentionner si je dois faire quoique ce soit - j'ai fouillé un peu dans le code et il me ferait plaisir de contribuer en code mais je ne sais pas trop par où commencer 😅

C'est fort possible que c'est moi qui aie mal compris quelque chose aussi...!

Dans tous les cas, merci ✨

  • Edith
@christopheNan
Copy link
Contributor

christopheNan commented Mar 26, 2022

Pour le 3., il semble que ce soit le dernier 'make merge' qui produit l'erreur.
Dans le Makefile, nous avons :

  -CPYTHON_CURRENT_COMMIT := 1b37268ef10bd20c30d349b8401c88215c8a6be8
  +CPYTHON_CURRENT_COMMIT := 6fd9737373f2bed03f409440b4fd50b9f8f121cb

ce qui se traduit dans venv/cpython/Doc/conf.py par :

  @@ -69,7 +69,8 @@
   html_theme_path = ['tools']
   html_theme_options = {
       'collapsiblesidebar': True,
  -    'issues_url': 'https://docs.python.org/3/bugs.html',
  +    'issues_url': '/bugs.html',
  +    'license_url': '/license.html',
       'root_include_title': False   # We use the version switcher instead.
   }

et dans venv/cpython/Doc/requirements.txt par :

@@ -13,4 +13,4 @@ blurb
   
   # The theme used by the documentation is stored separately, so we need
   # to install that as well.
  -python-docs-theme
  +python-docs-theme>=2022.1

@christopheNan
Copy link
Contributor

Pour régler le point 3, il faut mettre à jour ta version de python-docs-theme :

[user@localhost python-docs-fr (...)] source venv/bin/activate # entrée dans le virtualenv
(python-docs-fr) [user@localhost python-docs-fr (...)] pip list | grep python-docs-theme # quelle version actuelle du thème ?
python-docs-theme             2021.8
(python-docs-fr) [user@localhost python-docs-fr (...)] pip install -U python-docs-theme # mettre à jour vers >= 2022.1
(python-docs-fr) [user@localhost python-docs-fr (...)] pip list | grep python-docs-theme # vérification mise à jour 
python-docs-theme             2022.1
(python-docs-fr) [user@localhost python-docs-fr (...)] make
...
La compilation a réussi.

Les pages HTML sont dans build/html.
Writing glossary.json

Build finished. The HTML pages are in build/html.

@eviau
Copy link
Contributor Author

eviau commented Mar 26, 2022

Merci @christopheNan , cela a fonctionné et je peux maintenant générer la documentation 🎉

@jeanas
Copy link
Collaborator

jeanas commented Apr 4, 2022

  1. Bien vu. Cela semble provenir d'ici :

https://github.com/AFPy/Potodo/blob/a23216400488aa3a845d9c1ace5f9e14a41df772/potodo/github.py#L80

Dans les expressions régulières, \w matche un caractère alphabétique, un chiffre ou un souligné, mais pas un point. @Seluj78 ?

  1. Je ne pense pas, puisque make verifs ne fait qu'exécuter powrap, le même powrap que tu utilises toi-même pour reformater les fichiers. Comment t'y prends-tu pour lancer powrap ? Je fais
powrap **/*.po

@eviau
Copy link
Contributor Author

eviau commented Apr 4, 2022

Merci @jean-abou-samra !

Pour la regex - il semble que la regex suivante fonctionne: r"\w*/(\w|\.)*\.po"

@jeanas
Copy link
Collaborator

jeanas commented Apr 4, 2022

Encore plus simple: a priori, r".*\.po" devrait suffire :-)

@Seluj78
Copy link
Collaborator

Seluj78 commented Apr 5, 2022

Encore plus simple: a priori, r".*\.po" devrait suffire :-)

Non, l'objectif c'était de valider que la réservation était du format FOLDER/FILE.po mais ce qui a entrainé AFPy/Potodo#50

Ca fait longtemps que je me suis pas plongé dans potodo mais n'hésitez pas a faire une PR

@ghost
Copy link

ghost commented Apr 20, 2022

Bonjour, je viens d'avoir le même problème que @eviau .
En essayant la mise à jour du thème comme suggéré par
#1834 (comment)
même soucis.

Sphinx v3.2.1 en cours d'exécution
Chargement des traductions [fr]... fait
construction en cours [mo]:cibles pour les fichiers po 0 qui sont périmées
construction [html]:cibles pour les fichiers sources 488 qui sont périmées
mise-à-jour de l'environnement :[nouvelle configuration] 488 ajouté, 0 modifié, 0 supprimé
lecture des sources... [100%] whatsnew/3.1 .. whatsnew/index                                                                                
En attente des processus parallélisés...
recherche des fichiers périmés... aucun résultat
environnement de sérialisation... fait
vérification de la cohérence... fait
document en préparation... échoué

Warning, treated as error:
L'option 'license_url' n'est pas supportée pour ce thème
make[1]: *** [Makefile:51: build] Error 2
make[1] : on quitte le répertoire « /home/thierry/Dépôts/python-docs-fr/venv/cpython/Doc »
make: *** [Makefile:92: all] Error 2

bien que

(venv) thierry@Rabelais ~/D/python-docs-fr> pip list | grep python-docs
python-docs-theme  2022.1   

Une idée?

(Post édité par @jean-abou-samra pour corriger le Markdown.)

@jeanas
Copy link
Collaborator

jeanas commented Apr 28, 2022

On dirait que la version de python-docs-theme utilisée par Sphinx provient d'autre part que le venv. À tout hasard, as-tu essayé de supprimer complètement ce venv et de le réinstaller ?

Tiens, une idée au hasard : comment est-ce que Sphinx prend ses dépendances ? Si le paquet Sphinx est installé sur le système alors que le python-docs-theme récent provient du venv, je parie qu'il prendra un python-docs-theme ancien qui vient du système. Est-ce que Sphinx a bien été installé à l'intérieur du venv ? Si tu fais pip install sphinx dans le venv, qu'est-ce qu'il te dit ?

@ghost
Copy link

ghost commented Apr 28, 2022

Oups j'avais pas vu ta réponse...

Si tu fais pip install sphinx dans le venv, qu'est-ce qu'il te dit ?

Effectivement, ton hypothèse était correcte (sphinx local v3.2.1/nouveau sphinx installé sur le venv v4.5.0).

Toutes mes excuses, ça m'apprendra à tout lire...
Maintenant ça fonctionne de nouveau.
Merci!

@jeanas
Copy link
Collaborator

jeanas commented Apr 28, 2022

Oups j'avais pas vu ta réponse...

Mais non :-) Le fait que tu ais ouvert #1857 m'a rappelé ce problème et m'a poussé à creuser un peu plus.

Qu'est-ce que tu as fait exactement pour régler le problème, finalement ? Il faudrait se préoccuper de mettre à jour le CONTRIBUTING.rst…

@ghost
Copy link

ghost commented May 1, 2022

J'ai fait exactement ce qui était demandé en agrémentant à ma sauce:

Supprimer l'environnement virtuel existant puis le recréer.
Cela n'est cependant pas suffisant puisqu'il faut installer Sphinx dans le venv à la main (dans mon cas c'est la version du système global qui posait problème).
Ce dernier point me pose cependant un soucis : il n'y a pas une incohérence à tenir compte du système global dans le venv? L'installation de Sphinx impérative ne serait-elle pas mieux?

Comme je ne maîtrise pas tous les concepts sous-jacents j'ai un peu de problème à m'exprimer.

Il faudrait se préoccuper de mettre à jour le CONTRIBUTING.rst…

Je vais essayer de m'y atteler la semaine prochaine, mais je suis actuellement un peu surchargé.

@JulienPalard
Copy link
Member

Le repo bouge sur https://git.afpy.org/AFPy/python-docs-fr/.

Je ferme toutes les issues côté github, sens-toi libre de réouvrir côté gitea au besoin.

@JulienPalard JulienPalard closed this as not planned Won't fix, can't repro, duplicate, stale Nov 17, 2022
@ghost
Copy link

ghost commented Nov 18, 2022 via email

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

5 participants