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

Actualizar instrucciones del Wiki #235

Closed
jorgesumle opened this issue May 18, 2020 · 12 comments
Closed

Actualizar instrucciones del Wiki #235

jorgesumle opened this issue May 18, 2020 · 12 comments
Assignees

Comments

@jorgesumle
Copy link
Contributor

jorgesumle commented May 18, 2020

Las instrucciones de Generar diccionario corrector ya no funcionan debido a los cambios de db9b14c.

@olea olea self-assigned this May 18, 2020
@RickieES
Copy link
Collaborator

RickieES commented Sep 8, 2021

@olea, esto estaría bien tenerlo, porque si resolvemos los issues que he marcado, tendré que generar los OXT y, la verdad, no me acuerdo bien de cómo funcionaba tu script. 😊

@RickieES
Copy link
Collaborator

Buenas, he actualizado la página del wiki y creo que ya tiene la información correcta. @olea , @jorgesumle , si podéis echarle un ojo y la veis bien, se podría cerrar el issue (de nada, Ismael). 😉

@jorgesumle
Copy link
Contributor Author

he actualizado la página del wiki y creo que ya tiene la información correcta

En la página del wiki pone «Ricardo Palomares edited this page on Feb 24, 2017». Me parece que olvidaste subir los cambios.

@RickieES
Copy link
Collaborator

He actualizado la página. Si alguien no familiarizado con la generación del diccionario corrector quiere probar a seguirlas y comentar qué problemas encuentra, estaría bien.

Aclaro también que tengo un cambio local no subido al repositorio en el script. Si os falla el script, decídmelo, subo el cambio y volvéis a probar.

@jorgesumle
Copy link
Contributor Author

jorgesumle commented Jul 13, 2023

El script falla al tomar el argumento posicional --localizacion=es_ES (el que indica la wiki), ya que compara ese argumento con --localizacion (y no coincide, porque tiene el =es_ES también). Si ejecutas ./herramientas/make_dict.sh --localizacion es_ES, no falla. Lo ideal sería usar la instrucción integrada de Bash getopts para los parámetros posicionales, ya que se encarga de todo lo relativo a parámetros pasados a un script. En Internet hay unos cuantos ejemplos de código de cómo usar getopts, no es muy complicado y creo que simplificaría el código.

Aclaro también que tengo un cambio local no subido al repositorio en el script.

¿En qué consiste el cambio? Si haces un pull request, podría revisarlo.

@jorgesumle
Copy link
Contributor Author

Otra cosa: en la wiki habría que actualizar también el comando para generar el diccionario que aparece en Diccionario corrector para Vim/Neovim.

@RickieES
Copy link
Collaborator

El cambio del diccionario consiste solo en líneas como esta:

rm -f "$LO_DICTIONARIES_GIT"Dictionary_es.mk
git add "$LO_DICTIONARIES_GIT"Dictionary_es.mk "$DESTINO"

en las que se añade una barra de separación:

rm -f "$LO_DICTIONARIES_GIT"/Dictionary_es.mk
git add "$LO_DICTIONARIES_GIT"/Dictionary_es.mk "$DESTINO"

Puede que tenga que ver con el valor que se da a la variable LO_DICTIONARIES_GIT, pero creo que o se pone la barra o se añade alguna validación que la añada automáticamente si el usuario no la escribe al ejecutar la opción de configuración de make_dict.sh.

La modificación en el script make_dict.sh la hizo @olea. Antes de tocar cosas sobre sus cambios, prefiero esperar un poco a ver si nos escucha.

La parte de Vim/NeoVim también es cosecha de Ismael y, sinceramente, no tengo conocimientos de esa parte.

@olea
Copy link
Collaborator

olea commented Aug 10, 2023

Estoy listo para actualizar las instrucciones, en cuanto se aprueben las mejoras para make_dict.sh.

procedimiento para publicar:

# antes, actualizar el número de versión en .versiones.cfg

herramientas/make_dict.sh --changelog

# actualizar a mano Changelog.txt

herramientas/make_dict.sh --publicar-version
herramientas/make_dict.sh --todas

# en Github creamos la nueva versión

herramientas/make_dict.sh  --subir-a-LibreOffice

cd $RUTA_LIBREOFFICE/dictionaries/
git push origin hunspell-es-VERSION:refs/for/master

Demo de funcionamiento: https://asciinema.org/a/j387caIQw9ctIYCTx4KVCIA2w

Y publicación de prueba en mi repo: https://github.com/olea/rla-es/releases/tag/v2.8

@olea
Copy link
Collaborator

olea commented Aug 21, 2023

Hecho, por fin: ae768c2.

@olea olea closed this as completed Aug 21, 2023
@olea
Copy link
Collaborator

olea commented Aug 24, 2023

Más hecho aún: cc69f59.

Si alguien le da una revisada, aún mejor. FYI.

@RickieES
Copy link
Collaborator

Revisado. Lo de que, en general, el proceso de publicación lo hago yo me ha hecho gracia. 😃 Lo retocaré para que se entienda que lo puede hacer cualquiera con permisos de escritura en el repo (que, por cierto, ¿sabemos quiénes somos?).

Lo relativo a los XPI de Mozilla habrá que aclararlo mejor porque, precisamente, las cuatro variantes que mencionas que ya están publicando el diccionario posiblemente son todas anteriores a las mejoras en el script make_dict.sh y, por ello, tendrán casi seguro un ID de extensión distinto, por lo que no se podrá usar el script tal cual. Cuando pueda lo toco en el wiki.

Y la parte de publicar en dictionaries de LO te diré si está todo guay pronto, en cuanto lo siga a pies juntillas para la versión 2.8. 😉

@olea
Copy link
Collaborator

olea commented Aug 24, 2023

cualquiera con permisos de escritura en el repo (que, por cierto, ¿sabemos quiénes somos?).

Que yo sepa, del grupo, sólo nosotros dos. Estaré encantado de corregir el dato si procede.

tendrán casi seguro un ID de extensión distinto,

ofrecí añadir los ID automáticamente, pero entendí que preferías que no fuera así.

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

3 participants