[prev in list] [next in list] [prev in thread] [next in thread] 

List:       password-store
Subject:    Re: [PATCH] a single result from find prompts the user
From:       HacKan <hackan () gmail ! com>
Date:       2017-04-19 20:07:58
Message-ID: 02d4aaf5-2203-67c2-16aa-c874187906ce () gmail ! com
[Download RAW message or body]

Better than that is simply making an extension. Go that way, it'll be
appreciated.


On 04/19/2017 03:40 PM, John Gliksberg wrote:
> On 19 April 2017 20:09:38 CEST, Milo Gertjejansen <milo@milogert.com> wrote:
> > Thanks for the feedback. I could try my hand at a plugin instead of a
> > core
> > change if this ends up not making it into pass proper.
> > 
> > I see your point about the structure. Even some of my entries are not
> > structured like that. My main gripe was that it didn't just copy the
> > password for me so any changes beyond clipping the password was
> > probably me
> > "rambling."
> > 
> > Would the patch be more in line with the rest of pass if find accepted
> > a
> > "-c" argument to clip the password if there is only one result? Then
> > the
> > user would have to do something that they would be familiar with to
> > make
> > find act differently. Or even provide a list of files to choose to clip
> > from if many were matched?
> 
> How about a patch with a new "findshow" command?
> 
> It would search like find, and if there is one result (or maybe the first if there \
> are several) act like show. 
> It could take a -c flag like show.
> 
> I don't see why the user would have to specify which entries would be taken into \
> account — show doesn't; the only difference being calling findshow doesn't \
> guarantee what the resulting entry will be. It nonetheless seems a superfluous \
> distinction to introduce in this tool. 

-- 
HacKan || Iván
GPG: 0x35710D312FDE468B

_______________________________________________
Password-Store mailing list
Password-Store@lists.zx2c4.com
https://lists.zx2c4.com/mailman/listinfo/password-store


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic