Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4a6342888077e6b0755e1d9432183a95@smtp.hushmail.com>
Date: Sat, 15 Dec 2012 19:48:29 +0100
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: bash completion failures

On 15 Dec, 2012, at 19:33 , magnum <john.magnum@...hmail.com> wrote:
> On 15 Dec, 2012, at 13:44 , Frank Dittrich <frank_dittrich@...mail.com> wrote:
>> On 12/15/2012 01:20 PM, magnum wrote:
>>> No hidden options work, other things work correctly. If I just write "../run/john --[tab][tab]" I also get a list lacking the hiddens.
>> 
>> Now I am able to reproduce.
>> 
>> I always started john from the run directory and used ./john.
>> If I move to the src directory and use ../run/john, completion for
>> hidden options doesn't work.
>> 
>> Now I found the culprit in line 142 of the script.
>> Here I concatenate the options from the regular usage output and the
>> output of --list=hidden-options.
>> But instead of the name that is used to invoke john, I use "./john" for
>> --list=hidden-options.
>> This needs to be replaced by  ${first}
>> Otherwise completion only works as long as you are in the run directory
>> and as long as the binary is called john.
>> 
>> Feel free to correct this or wait until I submit a patch.
> 
> Great, I'll fix it.

Done. Now there is another little problem:

../run/john --mem[tab]

expands to

../run/john --mem-file-size=_

where '_' is a space. So to fill in the number I'll need to backspace first. This seems to happen with all hidden options but not the other ones.

magnum

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.