Re: [PATCH] Add support for changing X-Label

2013-04-22 Thread Jiri Denemark
On Tue, Mar 12, 2013 at 10:28:55 -0500, David Champion wrote: > Barring any overt objections from the team, I will push this. I would > like more bug testing, but I guess it's high time to just do it and let > the bugs sort themselves out. Hi David, Any chance this functionality will actually ge

Re: [PATCH] Add support for changing X-Label

2013-03-13 Thread Cameron Simpson
On 13Mar2013 21:55, David Champion wrote: | * On 13 Mar 2013, Cameron Simpson wrote: | > Write a script that prompts for a label, then patches a message | > file with the new label. Extra points for offering any existing | > label as a default. | | But of course, the patches already exist and wo

Re: [PATCH] Add support for changing X-Label

2013-03-13 Thread David Champion
* On 13 Mar 2013, Cameron Simpson wrote: > > Write a script that prompts for a label, then patches a message > file with the new label. Extra points for offering any existing > label as a default. But of course, the patches already exist and work, so doing this is not necessary. My patch even l

Re: [PATCH] Add support for changing X-Label

2013-03-13 Thread Cameron Simpson
On 12Mar2013 16:05, Jiri Denemark wrote: | Sure, but that's for composing new emails. This patch is targeted at | changing the header for emails already present in one's mailbox. Of | course, once could edit that email and change the header but that's far | from convenient and creating a mutt macr

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Vincent Lefevre
On 2013-03-12 12:53:56 -0500, David Champion wrote: > To some degree you can do that now with $editor and edit-message, right? > > set my_editor=$editor > set editor="my-script %s" > > set editor="$my_editor" > > my-script: > #!/bin/sh > sed -e 's/^X-Label: /' ... But it is not possible to prov

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread David Champion
To some degree you can do that now with $editor and edit-message, right? set my_editor=$editor set editor="my-script %s" set editor="$my_editor" my-script: #!/bin/sh sed -e 's/^X-Label: /' ... However that's a workaround, and I see that this isn't really what you're asking about. Perhaps a goo

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Vincent Lefevre
I'd like to be able to execute a script (this can be an option) for changing X-Label, e.g. if the string starts with the | character (or have a different function for that). The script could take the current X-Label value an input and output the new value. -- Vincent Lefèvre - Web:

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Jiri Denemark
On Tue, Mar 12, 2013 at 08:38:40 -0400, Patrick Shanahan wrote: > * Jiri Denemark [03-12-13 05:17]: > > On Mon, Mar 04, 2013 at 23:16:32 +0100, Jiri Denemark wrote: > > > # HG changeset patch > > > # User Jiri Denemark > > > # Date 1362434347 -3600 > > > # Branch HEAD > > > # Node ID a5f8b3b34a5a

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread David Champion
* On 12 Mar 2013, Jiri Denemark wrote: > 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). It has been controversial in the past, but

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Jiri Denemark
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 i

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread David Champion
* On 12 Mar 2013, Jiri Denemark wrote: > On Mon, Mar 04, 2013 at 23:16:32 +0100, Jiri Denemark wrote: > > # HG changeset patch > > # User Jiri Denemark > > # Date 1362434347 -3600 > > # Branch HEAD > > # Node ID a5f8b3b34a5a1780850c85687d22a1883083aacb > > # Parent 4c16c0d1ba9e0d81608a0fbccebd08

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Patrick Shanahan
* Jiri Denemark [03-12-13 05:17]: > On Mon, Mar 04, 2013 at 23:16:32 +0100, Jiri Denemark wrote: > > # HG changeset patch > > # User Jiri Denemark > > # Date 1362434347 -3600 > > # Branch HEAD > > # Node ID a5f8b3b34a5a1780850c85687d22a1883083aacb > > # Parent 4c16c0d1ba9e0d81608a0fbccebd08daa28

Re: [PATCH] Add support for changing X-Label

2013-03-12 Thread Jiri Denemark
On Mon, Mar 04, 2013 at 23:16:32 +0100, Jiri Denemark wrote: > # HG changeset patch > # User Jiri Denemark > # Date 1362434347 -3600 > # Branch HEAD > # Node ID a5f8b3b34a5a1780850c85687d22a1883083aacb > # Parent 4c16c0d1ba9e0d81608a0fbccebd08daa28d76f5 > Add support for changing X-Label > > Mut

[PATCH] Add support for changing X-Label

2013-03-04 Thread Jiri Denemark
# HG changeset patch # User Jiri Denemark # Date 1362434347 -3600 # Branch HEAD # Node ID a5f8b3b34a5a1780850c85687d22a1883083aacb # Parent 4c16c0d1ba9e0d81608a0fbccebd08daa28d76f5 Add support for changing X-Label Mutt supports ~y pattern which matches X-Label content but does not provide an eas