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