From: Eli Zaretskii <eliz@gnu.org>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: michael.albinus@gmx.de, 54993@debbugs.gnu.org
Subject: bug#54993: 28.1; Editing etc/NEWS on master with Emacs 28 signals an error
Date: Mon, 18 Apr 2022 17:15:42 +0300 [thread overview]
Message-ID: <835yn6wk8h.fsf@gnu.org> (raw)
In-Reply-To: <87tuaqir1c.fsf@gnus.org> (message from Lars Ingebrigtsen on Mon, 18 Apr 2022 13:12:31 +0200)
> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: Michael Albinus <michael.albinus@gmx.de>, 54993@debbugs.gnu.org
> Date: Mon, 18 Apr 2022 13:12:31 +0200
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> > We could add NEWS to auto-mode-alist instead, and that would solve the
> > issue cleanly for everyone. That was one of the original
> > alternatives, but somehow it was forgotten now.
>
> My worry with that was that adding etc/NEWS to auto-mode-alist would be
> false positives (for other files with that name). But I guess we could
> check that it's actually an Emacs NEWS file before enabling the mode.
I looked at several NEWS files in various projects, and they look well
under outline mode. Which doesn't really surprise me, since NEWS is
AFAIR a GNU thing.
> But that would mean removing the mode: setting from NEWS itself, because
> mode: takes precedence over auto-mode-alist, I think?
Yes.
> So then we're back to people editing the file in Emacs 28 in
> fundamental-mode.
Yes. So maybe instead Outline mode in Emacs 29 could turn on
emacs-news-mode automatically when the file's name is NEWS or NEWS.*
or ONEWS.
> > Btw, another issue with emacs-news-mode is that it invokes
> > outline-minor-mode, which means all the headings-related commands are
> > now bound to different keys than they were before. Wouldn't it be
> > better if emacs-news-mode were a minor mode invoked by Outline
> > instead?
>
> emacs-news-mode is quite major-modeish, though, so I think that would be
> awkward.
I don't see why. What do you see in emacs-news-mode that doesn't fit
a minor mode?
I guess my point is that making it a major mode makes this a
backward-incompatible change, when we consider editing of NEWS. If we
can avoid that, it would be better.
Or maybe it's enough to make it be derived from Outline (if that
doesn't interfere with editing NEWS with older Emacsen)?
next prev parent reply other threads:[~2022-04-18 14:15 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-17 17:44 bug#54993: 28.1; Editing etc/NEWS on master with Emacs 28 signals an error Eli Zaretskii
2022-04-17 18:38 ` Lars Ingebrigtsen
2022-04-17 18:49 ` Eli Zaretskii
2022-04-17 18:54 ` Lars Ingebrigtsen
2022-04-17 19:15 ` Eli Zaretskii
2022-04-17 19:37 ` Lars Ingebrigtsen
2022-04-17 20:12 ` Eli Zaretskii
2022-04-18 8:40 ` Michael Albinus
2022-04-18 8:45 ` Lars Ingebrigtsen
2022-04-18 8:48 ` Michael Albinus
2022-04-18 8:56 ` Michael Albinus
2022-04-18 9:35 ` Lars Ingebrigtsen
2022-04-18 9:27 ` Eli Zaretskii
2022-04-18 9:34 ` Lars Ingebrigtsen
2022-04-18 9:36 ` Eli Zaretskii
2022-04-18 10:08 ` Michael Albinus
2022-04-18 10:45 ` Eli Zaretskii
2022-04-18 11:12 ` Lars Ingebrigtsen
2022-04-18 14:15 ` Eli Zaretskii [this message]
2022-04-19 12:04 ` Lars Ingebrigtsen
2022-04-17 20:17 ` Jim Porter
2022-04-17 20:27 ` Lars Ingebrigtsen
2022-04-17 21:32 ` Lars Ingebrigtsen
2022-04-18 8:41 ` Lars Ingebrigtsen
2022-04-18 8:44 ` Lars Ingebrigtsen
2022-04-18 19:05 ` Juri Linkov
2022-04-19 12:10 ` Lars Ingebrigtsen
2022-04-19 13:18 ` Michael Albinus
2022-04-19 13:21 ` Lars Ingebrigtsen
2022-04-19 19:15 ` Juri Linkov
2022-04-20 6:22 ` Kévin Le Gouguec
2022-04-20 10:49 ` Lars Ingebrigtsen
2022-04-18 16:48 ` Jim Porter
2022-04-19 12:08 ` Lars Ingebrigtsen
2022-04-19 13:56 ` Lars Ingebrigtsen
2022-04-19 14:24 ` Lars Ingebrigtsen
2022-04-19 16:06 ` Eli Zaretskii
2022-04-20 6:14 ` Kévin Le Gouguec
2022-04-20 6:46 ` Eli Zaretskii
2022-04-20 10:45 ` Lars Ingebrigtsen
2022-04-20 12:28 ` Eli Zaretskii
2022-04-21 11:32 ` Lars Ingebrigtsen
2022-04-19 16:09 ` Lars Ingebrigtsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=835yn6wk8h.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=54993@debbugs.gnu.org \
--cc=larsi@gnus.org \
--cc=michael.albinus@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.