[PATCH v4] emacs: logically group def{custom,face}s
Pieter Praet
pieter at praet.org
Thu Jan 19 11:10:42 PST 2012
On Wed, 18 Jan 2012 13:44:08 -0500, Austin Clements <amdragon at MIT.EDU> wrote:
> Quoth Pieter Praet on Jan 18 at 11:00 am:
> > On Mon, 16 Jan 2012 14:16:22 -0500, Austin Clements <amdragon at MIT.EDU> wrote:
> > > LGTM.
> > >
> > > Quoth Pieter Praet on Jan 16 at 11:38 am:
> > > > - Add `notmuch-send' to the `message' group, as per Austin's
> > > > recommendation [1].
> > >
> > > This is actually the reverse of what I was suggesting (though is also
> > > reasonable). Since notmuch uses message-mode, I was saying that the
> > > notmuch group (or the notmuch-send group) should link to the message
> > > group. That way, if you customize-group notmuch, you get all of the
> > > options that affect notmuch, even those that aren't strictly notmuch
> > > variables.
> >
> > Sorry, failed to mention; I tried that, but the additional
> >
> > :link '(custom-group-link :tag "Message" message)
> >
> > didn't seem all that useful compared to (or combined with)
> > adding `message' as parent group to `notmuch-send':
> >
> > #+begin_example
> > Parent groups: Message Notmuch
> >
> > /- Notmuch Send group: Sending messages from Notmuch.--------------------\
> > State: something in this group has been changed outside customize.
> > #+end_example
> >
> > vs.
> >
> > #+begin_example
> > Parent groups: Message Notmuch
> >
> > /- Notmuch Send group: Sending messages from Notmuch.--------------------\
> > State: something in this group has been changed outside customize.
> > See also Message.
> > #+end_example
> >
> > ... so I ripped it out again. Should I add it back?
>
> Hmm. You're right that the :link doesn't do much.
>
Indeed. I'll leave it out, in favour of:
> One other possibility to consider is
> (custom-add-to-group 'notmuch-show 'message 'custom-group)
Agreed! (though you probably meant 'notmuch-send).
> or even
> (custom-add-to-group 'notmuch 'message 'custom-group)
Hmm, that might be a tad much. `message-mode' config is only relevant
when *sending* mail, so adding its group to 'notmuch-send should be
sufficient IMO.
Patch follows (new one, no amendment; David's on a merging-spree :).
Peace
--
Pieter
More information about the notmuch
mailing list