[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