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

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

Date: Wed, 18 Jan 2012 11:00:14 +0100

To: Austin Clements

Cc: Notmuch Mail

From: Pieter Praet


On Mon, 16 Jan 2012 14:16:22 -0500, Austin Clements <amdragon@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

Thread: