Arregla y wrapea library/re.po #2873
Conversation
A msgstr was missing its opening double quotes, which gettext's msgcat complained about (but powrap and polib didn't). See https://git.afpy.org/AFPy/powrap/pulls/4 and izimobil/polib#161 for PRs to fix both of those. Signed-off-by: Rodrigo Tobar <[email protected]>
|
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 <[email protected]>
Signed-off-by: Rodrigo Tobar <[email protected]>
rtobar
force-pushed
the
invalid-pofile-syntax-fixed
branch
from
1d44c60 to
4901b52
Compare
|
OK, 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 <[email protected]>
Not only a new version is out, powrap has also changed to a new home. Signed-off-by: Rodrigo Tobar <[email protected]>
En
library/re.pohabí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 --checkse saltara el archivo y no lo validara. Esto, a su vez, ocurría porque la utilidadmsgcatdegettextidentificaba el error de sintaxis, y fallaba al ser ejecutada.powrapno 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
powrappara cambiar este comportamiento en https://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 herramientas no consideraban este archivo como inválido, Esto es porque
polibno hacía la validación correspondiente, e incorrectamente parseaba la entrada. También abrí un PR contra polib para esto en izimobil/polib#161. Actualización: en el intertanto también me di cuenta de que el paquetebabelsufre 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.poestá bien formateado.