bug#25613: Temporary NEWS markup in still RC

bug#25613: Temporary NEWS markup in still RC

[Glenn Morris]

Package: emacs
Version: 25.2

Temporary NEWS markup is left in the RC, like last time.
I added a check for this in a1ef10e, but clearly it did not work.
Any idea why?

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 406 bytes --]

Glenn Morris <rgm@gnu.org> writes:

> Package: emacs
> Version: 25.2
>
> Temporary NEWS markup is left in the RC, like last time.

Indeed :-/

> I added a check for this in a1ef10e, but clearly it did not work.
> Any idea why?

No, I don't.  However, I think it'd be good to update make-tarball.txt
with some documentation for this check, otherwise I'll just keep
forgetting about it.

WDYT?

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Eli Zaretskii]

> From: Nicolas Petton <nicolas@petton.fr>
> Date: Mon, 06 Feb 2017 14:10:31 +0100
> 
> > Package: emacs
> > Version: 25.2
> >
> > Temporary NEWS markup is left in the RC, like last time.
> 
> Indeed :-/

Can we shoot for another RC in a few days?  Note that a couple of
documentation changes were pushed to the branch since RC1.

Thanks.

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 179 bytes --]

Eli Zaretskii <eliz@gnu.org> writes:

> Can we shoot for another RC in a few days?  Note that a couple of
> documentation changes were pushed to the branch since RC1.

Yes!

Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Glenn Morris]

Nicolas Petton wrote:

> No, I don't.  However, I think it'd be good to update make-tarball.txt
> with some documentation for this check, otherwise I'll just keep
> forgetting about it.

It's documented in admin/release-process section "To be done shortly
before release". There are various things that aren't specifically to do
with making the tarfile that need doing.

bug#25613: Temporary NEWS markup in still RC

[Glenn Morris]

Eli Zaretskii wrote:

> Can we shoot for another RC in a few days?  Note that a couple of
> documentation changes were pushed to the branch since RC1.

IMO no non-critical changes should be made to a release branch after an RC.
Just changing NEWS wouldn't have necessitated another RC. There's only
supposed to be one RC unless something serious happens.

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 269 bytes --]

Glenn Morris <rgm@gnu.org> writes:


> It's documented in admin/release-process section "To be done shortly
> before release". There are various things that aren't specifically to do
> with making the tarfile that need doing.

Oh, right, thanks for the reminder!

Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Cc: Nicolas Petton <nicolas@petton.fr>,  25613@debbugs.gnu.org
> Date: Mon, 06 Feb 2017 23:35:12 -0500
> 
> Eli Zaretskii wrote:
> 
> > Can we shoot for another RC in a few days?  Note that a couple of
> > documentation changes were pushed to the branch since RC1.
> 
> IMO no non-critical changes should be made to a release branch after an RC.
> Just changing NEWS wouldn't have necessitated another RC. There's only
> supposed to be one RC unless something serious happens.

We've been through this discussion before:

  http://lists.gnu.org/archive/html/emacs-devel/2016-09/msg00063.html

Needless to say, I agree with John here, or else I wouldn't be pushing
those changes a few days ago.

P.S. We seem to have very different ideas of what RC is and what
purpose it serves.

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 324 bytes --]

Eli Zaretskii <eliz@gnu.org> writes:

> We've been through this discussion before:
>
>   http://lists.gnu.org/archive/html/emacs-devel/2016-09/msg00063.html
>
> Needless to say, I agree with John here, or else I wouldn't be pushing
> those changes a few days ago.

I'll upload another RC later this week then.

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 204 bytes --]

Eli Zaretskii <eliz@gnu.org> writes:

> Can we shoot for another RC in a few days?  Note that a couple of
> documentation changes were pushed to the branch since RC1.

Are we ready for a second RC?

Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Eli Zaretskii]

> From: Nicolas Petton <nicolas@petton.fr>
> Cc: rgm@gnu.org, 25613@debbugs.gnu.org
> Date: Wed, 15 Feb 2017 14:33:54 +0100
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > Can we shoot for another RC in a few days?  Note that a couple of
> > documentation changes were pushed to the branch since RC1.
> 
> Are we ready for a second RC?

Yes, I think so.

bug#25613: Temporary NEWS markup in still RC

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 154 bytes --]

Eli Zaretskii <eliz@gnu.org> writes:

>> Are we ready for a second RC?
>
> Yes, I think so.

The second RC is out, I'm closing this ticket.

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

bug#25613: Temporary NEWS markup in still RC

[Eli Zaretskii]

> From: Nicolas Petton <nicolas@petton.fr>
> Cc: rgm@gnu.org, 25613@debbugs.gnu.org, 25613-done@debbugs.gnu.org
> Date: Tue, 21 Feb 2017 18:50:31 +0100
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> Are we ready for a second RC?
> >
> > Yes, I think so.
> 
> The second RC is out, I'm closing this ticket.

Thank you.