Skip to content

Translate library/distutils.po #130

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.

Sign up for GitHub

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

Jump to bottom
Closed
freezed wants to merge 1 commit into from
Closed

Translate library/distutils.po #130

freezed wants to merge 1 commit into from

Conversation

@freezed
Copy link
Contributor

@freezed freezed commented Jan 9, 2018

JulienPalard
JulienPalard requested changes Jan 14, 2018
View reviewed changes
library/distutils.po
"collections of Python packages which include modules coded in both Python "
"and C."
msgstr ""
"Le package :mod:`distutils`fournit le support pour la construction et "
Copy link
Member

@JulienPalard JulienPalard Jan 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il manque un espace entre le backtick et fournit.

library/distutils.po
msgstr ""
"La plupart des utilisateurs de Python ne voudront *pas* utiliser ce module "
"directement, mais plutôt utiliser les outils cross-version maintenus par la "
"Python Packaging Authority. En particulier,`setuptools <https://setuptools."
Copy link
Member

@JulienPalard JulienPalard Jan 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il manque un espace après la virgule.

library/distutils.po
"La plupart des utilisateurs de Python ne voudront *pas* utiliser ce module "
"directement, mais plutôt utiliser les outils cross-version maintenus par la "
"Python Packaging Authority. En particulier,`setuptools <https://setuptools."
"readthedocs.io/en/latest/>`__ est une alternative améliorée à: mod: "
Copy link
Member

@JulienPalard JulienPalard Jan 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L'espace devrait être à gauche des deux points :

- à: mod
+ à :mod

library/distutils.po
"Aux bénéfices des auteurs et utilisateurs d'outils d'empaquetage recherchant "
"une compréhension plus approfondie des détails du système actuel "
"d'empaquetage et de distribution, la documentation utilisateur historique :"
"mod: `distutils`et API de référence restent disponibles:"
Copy link
Member

@JulienPalard JulienPalard Jan 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il y a un espace de trop entre : et backtick, et il en manque un après le dernier backtick, avant le et.

@freezed
Copy link
Contributor Author

freezed commented Jan 14, 2018

On peut faire ce genre de vérif via un script dans poedit peut-être?

@JulienPalard
Copy link
Member

JulienPalard commented Jan 14, 2018

Dans poedit, aucune idée, cependant si tu construit la documentation en local, sphinx-doc te listera les erreurs (ça se fait a la louche juste en clonant cpython au même niveau que ta traduction, puis lancant un make dans le dossier de la traduction).

@freezed
Copy link
Contributor Author

freezed commented Jan 17, 2018

Comme ça?
Je coince avec virtualenv/venv:

user@hal ~/git/python-docs-fr % make merge             
mkdir -p ~/.venvs/python-docs-i18n/
/usr/bin/python3 -m venv ~/.venvs/python-docs-i18n/
Error: Command '['/home/user/.venvs/python-docs-i18n/bin/python3', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1
Makefile:36: recipe for target '/home/user/.venvs/python-docs-i18n//bin/activate' failed
make: *** [/home/user/.venvs/python-docs-i18n//bin/activate] Error 1
zsh: exit 2     make merge

user@hal ~/virtualenvs/python-doc-fr % source bin/activate                 
(python-doc-fr) 23:39 user@hal ~/git/python-docs-fr % make merge                    
mkdir -p ~/.venvs/python-docs-i18n/
/usr/bin/python3 -m venv ~/.venvs/python-docs-i18n/
Error: Command '['/home/user/.venvs/python-docs-i18n/bin/python3', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1
Makefile:36: recipe for target '/home/user/.venvs/python-docs-i18n//bin/activate' failed
make: *** [/home/user/.venvs/python-docs-i18n//bin/activate] Error 1
zsh: exit 2     make merge

@JulienPalard
Copy link
Member

JulienPalard commented Jan 21, 2018

A mon avis tu n'as pas ensurepip car tu as un Python qui vient de anaconda ?

Si c'est le cas, effectivement le classique python3 -m venv ne fonctionne pas, ils ont enlevé ensurepip, qui est necessaire à venv, tu vas devoir le faire à la main:

  • Créer un venv (et l'activer)
  • Y installer blurb et sphinx
  • Créer, dans ton clone de cpython, un dossier Doc/locales/fr/
  • Faire un lien symbolique de Doc/locales/fr/LC_MESSAGES vers ton clone de python-docs-fr
  • Lancer make -C ton_clone_de_cpython/Doc/ SPHINXOPTS='-D locale_dirs=locales -D language=fr -D gettext_compact=0' autobuild-dev-html

Tu peux t'inspirer du Makefile pour les commandes. Ou tu peux essayer de te trouver un python normal (ton gestionnaire de paquets doit en avoir un).

Ou ouvrir une issue (dans ce repo) pour me rappeler qu'il faudrait que je configure un travis-ci þ

@JulienPalard
Copy link
Member

JulienPalard commented Mar 20, 2018

Manually merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JulienPalard JulienPalard JulienPalard requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@freezed @JulienPalard