On Tue, Mar 12, 2013 at 08:24:36 -0500, David Champion wrote:
> * On 12 Mar 2013, Jiri Denemark wrote: 
> The code looks sound at a glance, and I agree it's a good idea -- but
> I'm afraid that I have a much worse response. :( There's a patch for
> this that's been around for several years.  It's integrated with label
> sorting and with a larger update to label use in general, and while I'd
> like to push it I'd appreciate more feedback on it before doing so, and
> simply haven't had time to follow through on soliciting it.
> 
> https://bitbucket.org/dgc/mutt-keywords

Oh, I didn't know it existed. Mainly because it's not upstream. Is there
any reason why it is not upstream when it it's been around for several
years?

> If there's any gap between the patch sequence at that URL and your code,
> and you want to produce a patch based on my patch sequence to bring them
> even, I'd be interested in including your changes (properly tagged as
> yours, of course).

It looks like your patch is doing all my patch can do and even in a
similar way. That said I don't care which patch gets in as long as mutt
supports this feature is supported in upstream mutt :-) I'm not a fan of
third-party patches or even patchsets especially when this functionality
does not seem to me as a controversial thing (unlike sidebar).

Jirka

Reply via email to