emacs-evil/evil-magit

evil-magit-add-rebase-messages does not insert commands in buffer when locale is different from english

oggers opened this issue · 3 comments

oggers commented

My linux box has spanish locale, so git messages are generated in spanish.

When I do a git rebase the git-rebase-todo file is in spanish. You are inserting rebase key bindings only if this condition is true (line 507 in evil-magit.el):

 (when (and (boundp 'git-rebase-show-instructions)
                        git-rebase-show-instructions
                        (re-search-forward "^# Commands:\n" nil t))

but in the spanish message you will never find the word "Commands", instead you get "Comandos:", therefore it will never insert the key bindings.

Sample git-rebase-todo generated in a spanish linux:

pick 689d051 Working

# Rebase d317ddf..689d051 en c49fa71 (5 comandos)
#
# Comandos:
# p, pick <commit> = usar commit
# r, reword <commit> = usar commit, pero editar el mensaje de commit
# e, edit <commit> = usar commit, pero parar para un amend
# s, squash <commit> = usar commit, pero fusionarlo en el commit previo
# f, fixup <commit> = como "squash", pero descarta el mensaje del log de este co
mmit
# x, exec <commit> = ejecuta comando ( el resto de la línea) usando un shell
# b, break = parar aquí (continuar rebase luego con 'git rebase --continue')
# d, drop <commit> = eliminar commit
# l, label <label> = poner label al HEAD actual con un nombre
# t, reset <label> = reiniciar HEAD a el label
# m, merge [-C <commit> | -c <commit>] <label> [# <oneline>]
# .       crea un commit de fusión usando el mensaje original de
# .       fusión (o la línea de oneline, si no se especifica un mensaje
# .       de commit). Use -c <commit> para reescribir el mensaje del commit.
#
# Estas líneas pueden ser reordenadas; son ejecutadas desde arriba hacia abajo.
#
# Si eliminas una línea aquí EL COMMIT SE PERDERÁ.
#
# Como sea, si quieres borrar todo, el rebase será abortado.
#

I think the same happens in all other languages that uses a different word for "commands".

justbur commented

Thanks. Can you try 253c644 ?

oggers commented

It works, Thank you!!

Do you know when it will be deployed to melpa?

justbur commented