Subject: bug#41145: 27.0.91; small issues with `display-fill-column-indicator' Customization group

From: Eli Zaretskii <[email-masked]>
Date: Sun, 30 Aug 2020 16:50:04 +0300

> From: Stefan Monnier <[email-masked]>
> Cc: Eli Zaretskii <[email-masked]>,  Philipp Stephani <[email-masked]>,
>   [email-masked]
> Date: Sat, 29 Aug 2020 23:58:04 -0400
> 
> >>>> Or if eventually a defface or a defcustom makes its way into
> >>>> display-fill-column-indicator.el, of course.
> >>
> >> Actually, there is a defcustom there (in the expansion of
> >> `define-globalized-minor-mode`).
> >
> > Sorry, I wasn't clear enough.  There is no defcustom or defface whose
> > :group is display-fill-column-indicator.
> 
> `global-display-fill-column-indicator-mode` should definitely belong to
> the `display-fill-column-indicator` group (because the `defcustom`
> doesn't have an explicit `:group` so it should fallback to using the
> last-defined group).

So you are saying this should have worked as it is?

> BTW, another quick-fix might be to move the defface definition to
> display-fill-column-indicator.el.

The idea was not to require display-fill-column-indicator.el be loaded
to activate the feature.