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

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

Date: Mon, 16 Jan 2012 14:16:22 -0500

To: Pieter Praet

Cc: Notmuch Mail

From: Austin Clements

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.

• Previous message (by thread): Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups

Thread:

• Pieter Praet—[PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Daniel Schoepe—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::feature, signed, unread]
    • Jameson Graef Rollins—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, signed, unread]
      • Pieter Praet—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
        • David Bremner—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
          • Pieter Praet—[PATCH v2] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
            • David Bremner—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
              • David Edmondson—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, signed, unread]
                • Pieter Praet—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
              • Pieter Praet—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, unread]
                • Pieter Praet—[PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
                  • Austin Clements—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, notmuch::review, unread]
                    • Jameson Graef Rollins—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
                    • Pieter Praet—Re: [PATCH] emacs: logically group def{custom,face}s [inbox, unread]
                      • Pieter Praet—[PATCH v4] emacs: logically group def{custom,face}s [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
                        • David Edmondson—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, signed, unread]
                        • Austin Clements—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, notmuch::review, unread]
                          • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                            • Austin Clements—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                              • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                                • Pieter Praet—[PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
                                  • Austin Clements—Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, notmuch::review, unread]
                                  • David Bremner—Re: [PATCH] emacs: invert relation between 'notmuch-send and 'message customization groups [inbox, unread]
                        • David Bremner—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
                          • Pieter Praet—Re: [PATCH v4] emacs: logically group def{custom,face}s [inbox, unread]
            • Jameson Graef Rollins—Re: [PATCH v2] emacs: logically group def{custom,face}s [inbox, notmuch::review, signed, unread]