[PATCH v4] emacs: logically group def{custom,face}s

Pieter Praet pieter at praet.org
Wed Jan 18 02:00:14 PST 2012


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?


Peace

-- 
Pieter


More information about the notmuch mailing list