- Notifications
You must be signed in to change notification settings - Fork397
Arregla y wrapea library/re.po#2873
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
A msgstr was missing its opening double quotes, which gettext's msgcatcomplained about (but powrap and polib didn't).Seehttps://git.afpy.org/AFPy/powrap/pulls/4 andizimobil/polib#161 for PRs to fix both of those.Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
All entries translated, horray! 🎉 |
Acá hay algo pasando con este archivo en particular que no estaba anticipado, así que estoy dejando este PR en Draft. El problema ocurre en una línea en la que aparecen dos caracteres "especiales" ( |
Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
1d44c60
to4901b52
CompareOK, al parecer era más simple: usando una versión anterior de gettext via Docker obtengo el wrapping que CI obtiene también. Éstas son las versiones involucradas:
Claramente en 0.22 algo pasó que cambió el wrapping. La verdad es que leyendo el Con esto ya dejo el PR listo para revisión. |
Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
Not only a new version is out, powrap has also changed to a new home.Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
tremendo trabajo 👏
a65f03e
intopython:3.12Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
En
library/re.po
había una entrada que no estaba delineada correctamente con comillas dobles (si ven el diff entero es la última entrada en el diff, o pueden ver simplemente el primer commit de este PR). Esto hacía quepowrap --check
se saltara el archivo y no lo validara. Esto, a su vez, ocurría porque la utilidadmsgcat
degettext
identificaba el error de sintaxis, y fallaba al ser ejecutada.powrap
no consideraba esos errores al momento de calcular el exit code del proceso, y por lo tanto el archivo no sólo seguía siendo inválido, sino que tampoco era verificado. De igual forma, el archivo no podía ser wrapeado correctamente usandopowrap library/re.po
.Ya abrí un PR contra
powrap
para cambiar este comportamiento enhttps://git.afpy.org/AFPy/powrap/pulls/4 (actualización: el PR ya fue mergeado, y una nuevs versión de powrap fue publicada, pornlo que también actualicé en este PR nuestra dependencia de powrap, además del pre-commit hook de powrap).Por otro lado, el resto de nuestras herramientasno consideraban este archivo como inválido, Esto es porque
polib
no hacía la validación correspondiente, e incorrectamente parseaba la entrada. También abrí un PR contra polib para esto enizimobil/polib#161. Actualización: en el intertanto también me di cuenta de que el paquetebabel
sufre del mismo problema, yo incorrectamente había asumido que babel dependía de polib; PR creada contra babel:python-babel/babel#1151.Después de corregir el error de sintaxis, ejecuté powrap de tal manera que ahora
library/re.po
está bien formateado.