* [Emacs-diffs] master 15059d2: * configure.ac (emacs_config_features): Add notify backends
@ 2018-10-30 7:53 Eli Zaretskii
2018-11-01 16:17 ` Glenn Morris
0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2018-10-30 7:53 UTC (permalink / raw)
To: Glenn Morris; +Cc: emacs-devel
After this change, I see in system-configuration-features:
... NOTIFY W32NOTIFY ...
This sounds redundant. Should we remove "NOTIFY", or say something
like "NOTIFY/W32NOTIFY" or "NOTIFY(W32NOTIFY)"?
Thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: master 15059d2: * configure.ac (emacs_config_features): Add notify backends
2018-10-30 7:53 [Emacs-diffs] master 15059d2: * configure.ac (emacs_config_features): Add notify backends Eli Zaretskii
@ 2018-11-01 16:17 ` Glenn Morris
2018-11-01 18:33 ` Eli Zaretskii
0 siblings, 1 reply; 3+ messages in thread
From: Glenn Morris @ 2018-11-01 16:17 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: emacs-devel
Eli Zaretskii wrote:
> After this change, I see in system-configuration-features:
>
> ... NOTIFY W32NOTIFY ...
>
> This sounds redundant.
It was intentional to list both the generic feature and the specific backend.
> Should we remove "NOTIFY",
I don't mind if you want to do that.
> or say something like "NOTIFY/W32NOTIFY" or "NOTIFY(W32NOTIFY)"?
I'm not keen on that.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: master 15059d2: * configure.ac (emacs_config_features): Add notify backends
2018-11-01 16:17 ` Glenn Morris
@ 2018-11-01 18:33 ` Eli Zaretskii
0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2018-11-01 18:33 UTC (permalink / raw)
To: Glenn Morris; +Cc: emacs-devel
> From: Glenn Morris <rgm@gnu.org>
> Cc: emacs-devel@gnu.org
> Date: Thu, 01 Nov 2018 12:17:20 -0400
>
> Eli Zaretskii wrote:
>
> > After this change, I see in system-configuration-features:
> >
> > ... NOTIFY W32NOTIFY ...
> >
> > This sounds redundant.
>
> It was intentional to list both the generic feature and the specific backend.
But we cannot have one without the other, so I don't think I
understand the motivation.
Anyway, it's a very minor issue, so feel free to ignore me.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-01 18:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-10-30 7:53 [Emacs-diffs] master 15059d2: * configure.ac (emacs_config_features): Add notify backends Eli Zaretskii
2018-11-01 16:17 ` Glenn Morris
2018-11-01 18:33 ` Eli Zaretskii
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).