Re: [dev] new pre-patched version of dwm available

[Britton Kerin]

On Sun, Jul 24, 2016 at 7:26 PM, Ben Woolley  wrote:
> I think it is natural for related changes to be consolidated over time. Think 
> of punctuated equilibrium.
>
> Maybe after it is clear that some patches just go well together, fitting a 
> related niche, they could be consolidated to make maintenance easier.

In some cases this might be good.  In others the larger consolidated
patch will be more likely to conflict with others, making it harder for
users to pick and choose their favorite features.

> On Jul 24, 2016, at 12:51 PM, Jan Christoph Ebersbach  wrote:
>> Personally, I'd recommend newbies that want to use a patched version to
>> find someone who's maintaining a "fork".  Maybe these "forks" could be
>> listed in the patches section of the wiki.  What do you think?

I think yes.  I'd call them 'flavors' or "pre-rolled" rather than "forks" since
they would be friendly and aiming for maximum compatibility with other
patches.

It doesn't matter what I think though, and so far no buy-in from the
suckless admins...

Britton

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[Eric Pruitt]

On Tue, Jul 26, 2016 at 06:16:15PM +0200, Kajetan Jasztal wrote:
> Hash masking [0] password gives you more insight in mistakes you made
> while you are typing since you can remember sequence of changing
> hashes while you type and correct them on the fly. 

I've seen that before, and although I think it's a neat idea, I feel
like it's way too leaky to use in public.

Eric

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[Eric Pruitt]

On Tue, Jul 26, 2016 at 06:09:18PM +0200, Jan Christoph Ebersbach wrote:
> If dmenu could easily be used as a replacement for pinentry my desktop
> would suck a little less.  I'm not aware of many other use cases for
> entering passwords.  Is that possible with your patch?

The pinentry programs for GPG do more than pipe passwords. This patch
could probably be used to create a pinentry program or script that uses
dmenu, but I'm not sure exactly how to do that off the top of my head.
If you just want to use dmenu to enter passphrases and you have control
over how gpg is invoked (i.e. you're not using an inflexible front-end),
you could use "--passphrase-fd 0".

Eric


signature.asc
Description: Digital signature

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[Kajetan Jasztal]

2016-07-25 23:22 GMT+02:00 Eric Pruitt :
> In general, I find showing the asterisks useful because if I
> accidentally press two keys at once or something, I have the opportunity
> to correct it rather than typing something then, hitting Enter and
> wondering what I did wrong. On a couple of occasions, I have also had a
> key break on my keyboard, and the password prompts I dealt with most
> often provided no feedback until after submission. It took me a bit to
> finally figure out what was going on, and having visual feedback
> would've made my life a little easier.

Asterisks are an improvement over blank prompt, and it probably helps in
cases you pointed out, but not that much. Hash masking [0] password
gives you more insight in mistakes you made while you are typing since
you can remember sequence of changing hashes while you type and
correct them on the fly. And probably there are tons of improvements you
can imagine.

I don't think dmenu will ever have such *feature* nor it should.
But that's what patches are for.

[0]: http://mattt.github.io/Chroma-Hash/

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[Jan Christoph Ebersbach]

If dmenu could easily be used as a replacement for pinentry my desktop
would suck a little less.  I'm not aware of many other use cases for
entering passwords.  Is that possible with your patch?

Jan Christoph

On Mon 25-07-2016 13:21 -0700, Eric Pruitt wrote:

> On Mon, Jul 25, 2016 at 11:14:28PM -1100, mikan wrote:
> > I was working on a similar thing but decided that password entry
> > doesn't make sense in dmenu context (since it's a menu!) and made a
> > separate tool instead and glue pinentry code to it.
>
> I don't agree with this sentiment -- dmenu happily supports arbitrary
> text entry when the thing you want to enter isn't in the list of
> presented items. I already use dmenu, so I would much rather maintain
> my ~40 line patch than introduce a new tool to my workflow that may
> eventually diverge from dmenu in ways I consider undesirable or be
> abandoned by its sole maintainer.
>
> Eric
>

--
Jan Christoph Ebersbach
I didn’t want some petty, inferior brand of righteousness that comes
from keeping a list of rules when I could get the robust kind that comes
from trusting Christ - God’s righteousness.  Phil 3:9


signature.asc
Description: PGP signature

[dev] [dmenu][PATCH] Improve spacing in config.h

[Ivan Tham]

---
 config.def.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/config.def.h b/config.def.h
index 1edb647..b53885b 100644
--- a/config.def.h
+++ b/config.def.h
@@ -8,10 +8,10 @@ static const char *fonts[] = {
 };
 static const char *prompt  = NULL;  /* -p  option; prompt to the left 
of input field */
 static const char *colors[SchemeLast][2] = {
-   /* fg bg   */
+   /* fg bg  */
[SchemeNorm] = { "#bb", "#22" },
-   [SchemeSel] = { "#ee", "#005577" },
-   [SchemeOut] = { "#00", "#00" },
+   [SchemeSel]  = { "#ee", "#005577" },
+   [SchemeOut]  = { "#00", "#00" },
 };
 /* -l option; if nonzero, dmenu uses vertical list with given number of lines 
*/
 static unsigned int lines  = 0;
-- 
2.8.3

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[v4hn]

Hey everyone, hey hiro,

I did not yet look at the patch, but it would be a nice
(and imo reasonably small) addition to allow for password entry via dmenu.
If the asterisk-approach is too bloated for the maintainer's taste, I'm unsure
about this myself, I believe `dmenu -p "Enter Password" -g` where -g is a flag
that makes dmenu not print *anything* entered by the user would be
a neat, simple and possibly reusable solution.

If this would be merged upstream, I would like to make use of it in
snotes [0] to support encrypted notes via gnupg - a feature multiple people
asked for over the last years.


v4hn

[0] - https://github.com/v4hn/snotes


signature.asc
Description: PGP signature

Re: [dev] [dmenu] [PATCH] Added option to prompt for passwords

[hiro]

i think the idea was fine, as long as you provide autocompletion for
your password, too.