unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#42917: 27.1; temporary note included in 27 NEWS
@ 2020-08-18 18:05 Alex Branham
  2020-08-18 18:26 ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Alex Branham @ 2020-08-18 18:05 UTC (permalink / raw)
  To: 42917

Hi all -

The following note:

Temporary note:
+++ indicates that all relevant manuals in doc/ have been updated.
--- means no change in the manuals is needed.
When you add a new item, use the appropriate mark if you are sure it
applies, and please also update docstrings as needed.

is included in Emacs 27.1 NEWS. Judging from previous releases, it
should have been removed, correct?

Thanks!





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-18 18:05 bug#42917: 27.1; temporary note included in 27 NEWS Alex Branham
@ 2020-08-18 18:26 ` Eli Zaretskii
  2020-08-19 18:01   ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-08-18 18:26 UTC (permalink / raw)
  To: Alex Branham; +Cc: 42917

> From: Alex Branham <alex.branham@gmail.com>
> Date: Tue, 18 Aug 2020 14:05:34 -0400
> 
> The following note:
> 
> Temporary note:
> +++ indicates that all relevant manuals in doc/ have been updated.
> --- means no change in the manuals is needed.
> When you add a new item, use the appropriate mark if you are sure it
> applies, and please also update docstrings as needed.
> 
> is included in Emacs 27.1 NEWS. Judging from previous releases, it
> should have been removed, correct?

Yes.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-18 18:26 ` Eli Zaretskii
@ 2020-08-19 18:01   ` Stefan Kangas
  2020-08-19 18:43     ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2020-08-19 18:01 UTC (permalink / raw)
  To: Eli Zaretskii, Alex Branham; +Cc: 42917

close 42917 27.2
thanks

Eli Zaretskii <eliz@gnu.org> writes:

>> The following note:
>>
>> Temporary note:
>> +++ indicates that all relevant manuals in doc/ have been updated.
>> --- means no change in the manuals is needed.
>> When you add a new item, use the appropriate mark if you are sure it
>> applies, and please also update docstrings as needed.
>>
>> is included in Emacs 27.1 NEWS. Judging from previous releases, it
>> should have been removed, correct?
>
> Yes.

Fixed on emacs-27.

Best regards,
Stefan Kangas





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-19 18:01   ` Stefan Kangas
@ 2020-08-19 18:43     ` Eli Zaretskii
  2020-08-19 18:58       ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-08-19 18:43 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 42917, alex.branham

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Wed, 19 Aug 2020 14:01:50 -0400
> Cc: 42917@debbugs.gnu.org
> 
> >> Temporary note:
> >> +++ indicates that all relevant manuals in doc/ have been updated.
> >> --- means no change in the manuals is needed.
> >> When you add a new item, use the appropriate mark if you are sure it
> >> applies, and please also update docstrings as needed.
> >>
> >> is included in Emacs 27.1 NEWS. Judging from previous releases, it
> >> should have been removed, correct?
> >
> > Yes.
> 
> Fixed on emacs-27.

Thanks, but that was a mistake: this part is only removed when
preparing the release tarball.  It cannot be removed in the
repository, because it must be there for people who make changes in
NEWS to see.  And people do make changes in NEWS, even though this is
a release branch.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-19 18:43     ` Eli Zaretskii
@ 2020-08-19 18:58       ` Stefan Kangas
  2020-08-19 19:22         ` Eli Zaretskii
  2020-08-20  9:58         ` Stefan Kangas
  0 siblings, 2 replies; 9+ messages in thread
From: Stefan Kangas @ 2020-08-19 18:58 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42917, alex.branham, Nicolas Petton

reopen 42917
thanks

(I'm copying in Nico into the discussion.)

Eli Zaretskii <eliz@gnu.org> writes:

> Thanks, but that was a mistake: this part is only removed when
> preparing the release tarball.  It cannot be removed in the
> repository, because it must be there for people who make changes in
> NEWS to see.  And people do make changes in NEWS, even though this is
> a release branch.

Oh, I double-checked the emacs-26 branch and saw that it was removed
there (but I guess that was right after 26.3).  I'll revert that change.

I'm not sure what to do about this then.  Should we perhaps add
anything else to admin/release-process to remind about this, for example:

modified   admin/release-process
@@ -79,8 +79,10 @@ so e.g. admin/ is fine.)  This is important for
legal compliance.
 ** Remove temporary +++/--- lines in NEWS.
 But first make sure there are no unmarked entries, and update the
 documentation (or decide no updates are necessary) for those that aren't.

+** Remove "Temporary note" at the top of NEWS.
+
 ** Try to reorder NEWS: most important things first, related items together.

 ** For a major release, add a "New in Emacs XX" section to faq.texi.

Best regards,
Stefan Kangas





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-19 18:58       ` Stefan Kangas
@ 2020-08-19 19:22         ` Eli Zaretskii
  2020-08-20  9:58         ` Stefan Kangas
  1 sibling, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2020-08-19 19:22 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 42917, alex.branham, nico

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Wed, 19 Aug 2020 18:58:57 +0000
> Cc: alex.branham@gmail.com, 42917@debbugs.gnu.org, 
> 	Nicolas Petton <nico@petton.fr>
> 
> I'm not sure what to do about this then.  Should we perhaps add
> anything else to admin/release-process to remind about this, for example:
> 
> modified   admin/release-process
> @@ -79,8 +79,10 @@ so e.g. admin/ is fine.)  This is important for
> legal compliance.
>  ** Remove temporary +++/--- lines in NEWS.
>  But first make sure there are no unmarked entries, and update the
>  documentation (or decide no updates are necessary) for those that aren't.

Something like that, yes.  There's also admin/make-tarball.txt.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-19 18:58       ` Stefan Kangas
  2020-08-19 19:22         ` Eli Zaretskii
@ 2020-08-20  9:58         ` Stefan Kangas
  2020-08-20 13:26           ` Eli Zaretskii
  1 sibling, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2020-08-20  9:58 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42917, Glenn Morris, alex.branham, Nicolas Petton

Stefan Kangas <stefankangas@gmail.com> writes:

> Oh, I double-checked the emacs-26 branch and saw that it was removed
> there (but I guess that was right after 26.3).  I'll revert that change.

Reverted.  Glenn also pushed this:

Author: Glenn Morris <rgm@gnu.org>
Date:   Wed Aug 19 20:52:08 2020 +0100

    * admin/admin.el (set-version): Trap yet another NEWS error.

Is that sufficient to solve this issue, or is there more to do here?

Best regards,
Stefan Kangas





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-20  9:58         ` Stefan Kangas
@ 2020-08-20 13:26           ` Eli Zaretskii
  2020-08-20 14:19             ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2020-08-20 13:26 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 42917, rgm, alex.branham, nico

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Thu, 20 Aug 2020 02:58:10 -0700
> Cc: alex.branham@gmail.com, 42917@debbugs.gnu.org, 
> 	Nicolas Petton <nico@petton.fr>, Glenn Morris <rgm@gnu.org>
> 
> Stefan Kangas <stefankangas@gmail.com> writes:
> 
> > Oh, I double-checked the emacs-26 branch and saw that it was removed
> > there (but I guess that was right after 26.3).  I'll revert that change.
> 
> Reverted.  Glenn also pushed this:
> 
> Author: Glenn Morris <rgm@gnu.org>
> Date:   Wed Aug 19 20:52:08 2020 +0100
> 
>     * admin/admin.el (set-version): Trap yet another NEWS error.
> 
> Is that sufficient to solve this issue, or is there more to do here?

I think this is sufficient, thanks.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#42917: 27.1; temporary note included in 27 NEWS
  2020-08-20 13:26           ` Eli Zaretskii
@ 2020-08-20 14:19             ` Stefan Kangas
  0 siblings, 0 replies; 9+ messages in thread
From: Stefan Kangas @ 2020-08-20 14:19 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 42917, rgm, alex.branham, nico

close 42917 27.2
thanks

Eli Zaretskii <eliz@gnu.org> writes:

> I think this is sufficient, thanks.

OK; closing.





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-08-20 14:19 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-18 18:05 bug#42917: 27.1; temporary note included in 27 NEWS Alex Branham
2020-08-18 18:26 ` Eli Zaretskii
2020-08-19 18:01   ` Stefan Kangas
2020-08-19 18:43     ` Eli Zaretskii
2020-08-19 18:58       ` Stefan Kangas
2020-08-19 19:22         ` Eli Zaretskii
2020-08-20  9:58         ` Stefan Kangas
2020-08-20 13:26           ` Eli Zaretskii
2020-08-20 14:19             ` Stefan Kangas

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).