Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 18 Nov 2012 22:57:02 +0100
From: magnum <>
Subject: Re: zsh completion for john 

On 18 Nov, 2012, at 16:57 , Vlatko Kosturjak <> wrote:
> Since I did not find any references regarding zsh completion for john,
> except this one:
> I've made one. Of course, I don't mind if you share it with john:

Great, see below.

> Also, I have found some incosistency in --list= command, so I've made
> small patch:

The inconsistences were that is wrote to stderr (no other --list options do) and it had an extra heading for historical reasons. I accepted this pull request despite it might theoretically break things: NOTE! any scripts/programs depending on this might need to be adjusted. Perhaps Johnny the GUI is affected, I doubt there are any others in this case. Unless the Test Suite is... but I think it checks both stderr and stdout. I'll have a look at it right away.

> Actually what is the best method to submit patches? via github pull
> requests or sending here to the mailing list? 

Some things (especially core changes) could/should be discussed on list first, perhaps with a patch (early, PoC or complete). With that said, I really appreciate pull requests. In fact, I would appreciate a pull request for your zsh completion, complete with whatever is needed (maybe a zsh-completion make target? Or maybe change the existing bash-completion make target to do both (new name shell-completion or something like that).


Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.