GNU bug report logs - #28817
incorrect NEWS entry about vc-state faces

Previous Next

Package: emacs;

Reported by: charles <at> aurox.ch (Charles A. Roelli)

Date: Fri, 13 Oct 2017 18:30:02 UTC

Severity: minor

Done: charles <at> aurox.ch (Charles A. Roelli)

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28817 in the body.
You can then email your comments to 28817 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#28817; Package emacs. (Fri, 13 Oct 2017 18:30:02 GMT) Full text and rfc822 format available.
Acknowledgement sent to charles <at> aurox.ch (Charles A. Roelli):
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Fri, 13 Oct 2017 18:30:03 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: charles <at> aurox.ch (Charles A. Roelli)
To: bug-gnu-emacs <at> gnu.org
Subject: incorrect NEWS entry about vc-state faces
Date: Fri, 13 Oct 2017 20:28:44 +0200

In the 26.0.90 NEWS file we have:

*** The VC state indicator in the mode line now defaults to more
colorful faces to make it more obvious to the user what the state is.
See the 'vc-faces' customization group.

which was added in these commits:

* | | | | | | | | | | | | | | | | | 65fa442..: Lars Ingebrigtsen 2016-03-01 Add a NEWS entry for the colorful VC indicator
  commit 65fa442f2bae42d3f9b06264ca432073ee3197c3
  Author: Lars Ingebrigtsen <larsi <at> gnus.org>
  Date:   Tue Mar 1 14:31:35 2016 +1100

  Add a NEWS entry for the colorful VC indicator

* | | | | | | | | | | | | | | | | | 2621c29..: Lars Ingebrigtsen 2016-03-01 Use colors in the VC mode lines
  commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b
  Author: Lars Ingebrigtsen <larsi <at> gnus.org>
  Date:   Tue Mar 1 14:25:27 2016 +1100

  Use colors in the VC mode lines

  * lisp/vc/vc-hooks.el: Make the mode line faces default to
  using colors to more clearly tell the user what the status is.

The NEWS entry commit seems to have made it to Emacs 26, but the mode
line faces are still the same as in Emacs 25.  Looks like commit
2621c29 got lost somehow.

Also, not sure why this group is defined (in vc-hooks.el):

(defgroup vc-state-faces nil
  "Faces used in the mode line by the VC state indicator."
  :group 'vc-faces
  :group 'mode-line
  :version "25.1")

then never used.  It looks like the vc-state-*/vc-*-state faces are
meant to inherit from it, but it's not clear.  It also makes the
customization buffers confusing.
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#28817; Package emacs. (Fri, 13 Oct 2017 18:43:02 GMT) Full text and rfc822 format available.

Message #8 received at 28817 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: charles <at> aurox.ch (Charles A. Roelli)
Cc: 28817 <at> debbugs.gnu.org
Subject: Re: bug#28817: incorrect NEWS entry about vc-state faces
Date: Fri, 13 Oct 2017 21:41:55 +0300

> Date: Fri, 13 Oct 2017 20:28:44 +0200
> From: charles <at> aurox.ch (Charles A. Roelli)
> 
> * | | | | | | | | | | | | | | | | | 65fa442..: Lars Ingebrigtsen 2016-03-01 Add a NEWS entry for the colorful VC indicator
>   commit 65fa442f2bae42d3f9b06264ca432073ee3197c3
>   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
>   Date:   Tue Mar 1 14:31:35 2016 +1100
> 
>   Add a NEWS entry for the colorful VC indicator
> 
> * | | | | | | | | | | | | | | | | | 2621c29..: Lars Ingebrigtsen 2016-03-01 Use colors in the VC mode lines
>   commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b
>   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
>   Date:   Tue Mar 1 14:25:27 2016 +1100
> 
>   Use colors in the VC mode lines
> 
>   * lisp/vc/vc-hooks.el: Make the mode line faces default to
>   using colors to more clearly tell the user what the status is.
> 
> The NEWS entry commit seems to have made it to Emacs 26, but the mode
> line faces are still the same as in Emacs 25.  Looks like commit
> 2621c29 got lost somehow.

Some merge snafu, I suppose: this was merged from the emacs-25 branch.
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#28817; Package emacs. (Mon, 16 Oct 2017 14:35:02 GMT) Full text and rfc822 format available.

Message #11 received at 28817 <at> debbugs.gnu.org (full text, mbox):

From: charles <at> aurox.ch (Charles A. Roelli)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 28817 <at> debbugs.gnu.org
Subject: Re: bug#28817: incorrect NEWS entry about vc-state faces
Date: Mon, 16 Oct 2017 16:34:17 +0200

> Date: Fri, 13 Oct 2017 21:41:55 +0300
> From: Eli Zaretskii <eliz <at> gnu.org>
> CC: 28817 <at> debbugs.gnu.org
> Reply-to: Eli Zaretskii <eliz <at> gnu.org>
> 
> > Date: Fri, 13 Oct 2017 20:28:44 +0200
> > From: charles <at> aurox.ch (Charles A. Roelli)
> > 
> > * | | | | | | | | | | | | | | | | | 65fa442..: Lars Ingebrigtsen 2016-03-01 Add a NEWS entry for the colorful VC indicator
> >   commit 65fa442f2bae42d3f9b06264ca432073ee3197c3
> >   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
> >   Date:   Tue Mar 1 14:31:35 2016 +1100
> > 
> >   Add a NEWS entry for the colorful VC indicator
> > 
> > * | | | | | | | | | | | | | | | | | 2621c29..: Lars Ingebrigtsen 2016-03-01 Use colors in the VC mode lines
> >   commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b
> >   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
> >   Date:   Tue Mar 1 14:25:27 2016 +1100
> > 
> >   Use colors in the VC mode lines
> > 
> >   * lisp/vc/vc-hooks.el: Make the mode line faces default to
> >   using colors to more clearly tell the user what the status is.
> > 
> > The NEWS entry commit seems to have made it to Emacs 26, but the mode
> > line faces are still the same as in Emacs 25.  Looks like commit
> > 2621c29 got lost somehow.
> 
> Some merge snafu, I suppose: this was merged from the emacs-25 branch.

I'm not sure I understand: the commit was made in the master branch.
See:

https://lists.gnu.org/archive/html/emacs-diffs/2016-02/msg00709.html

Ah, I think I understand now.  Commit 2621c29 was reverted later:

* | | | | | | | | | | | | | | | | | 7882dc6..: John Wiegley 2016-03-06 Revert "Use colors in the VC mode lines"
  commit 7882dc625e1ec562fcd0e1b743ef11b160cae18e
  Author: John Wiegley <johnw <at> newartisans.com>
  Date:   Sun Mar 6 12:51:44 2016 -0800

  Revert "Use colors in the VC mode lines"

  This reverts commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b.

and I can't find a revert for commit 65fa442 (the NEWS entry).
Reply sent to charles <at> aurox.ch (Charles A. Roelli):
You have taken responsibility. (Tue, 21 Nov 2017 19:40:01 GMT) Full text and rfc822 format available.
Notification sent to charles <at> aurox.ch (Charles A. Roelli):
bug acknowledged by developer. (Tue, 21 Nov 2017 19:40:02 GMT) Full text and rfc822 format available.

Message #16 received at 28817-done <at> debbugs.gnu.org (full text, mbox):

From: charles <at> aurox.ch (Charles A. Roelli)
To: charles <at> aurox.ch (Charles A. Roelli)
Cc: eliz <at> gnu.org, 28817-done <at> debbugs.gnu.org
Subject: Re: bug#28817: incorrect NEWS entry about vc-state faces
Date: Tue, 21 Nov 2017 20:40:20 +0100

> Date: Mon, 16 Oct 2017 16:34:17 +0200
> From: charles <at> aurox.ch (Charles A. Roelli)
> 
> > Date: Fri, 13 Oct 2017 21:41:55 +0300
> > From: Eli Zaretskii <eliz <at> gnu.org>
> > CC: 28817 <at> debbugs.gnu.org
> > Reply-to: Eli Zaretskii <eliz <at> gnu.org>
> > 
> > > Date: Fri, 13 Oct 2017 20:28:44 +0200
> > > From: charles <at> aurox.ch (Charles A. Roelli)
> > > 
> > > * | | | | | | | | | | | | | | | | | 65fa442..: Lars Ingebrigtsen 2016-03-01 Add a NEWS entry for the colorful VC indicator
> > >   commit 65fa442f2bae42d3f9b06264ca432073ee3197c3
> > >   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
> > >   Date:   Tue Mar 1 14:31:35 2016 +1100
> > > 
> > >   Add a NEWS entry for the colorful VC indicator
> > > 
> > > * | | | | | | | | | | | | | | | | | 2621c29..: Lars Ingebrigtsen 2016-03-01 Use colors in the VC mode lines
> > >   commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b
> > >   Author: Lars Ingebrigtsen <larsi <at> gnus.org>
> > >   Date:   Tue Mar 1 14:25:27 2016 +1100
> > > 
> > >   Use colors in the VC mode lines
> > > 
> > >   * lisp/vc/vc-hooks.el: Make the mode line faces default to
> > >   using colors to more clearly tell the user what the status is.
> > > 
> > > The NEWS entry commit seems to have made it to Emacs 26, but the mode
> > > line faces are still the same as in Emacs 25.  Looks like commit
> > > 2621c29 got lost somehow.
> > 
> > Some merge snafu, I suppose: this was merged from the emacs-25 branch.
> 
> I'm not sure I understand: the commit was made in the master branch.
> See:
> 
> https://lists.gnu.org/archive/html/emacs-diffs/2016-02/msg00709.html
> 
> Ah, I think I understand now.  Commit 2621c29 was reverted later:
> 
> * | | | | | | | | | | | | | | | | | 7882dc6..: John Wiegley 2016-03-06 Revert "Use colors in the VC mode lines"
>   commit 7882dc625e1ec562fcd0e1b743ef11b160cae18e
>   Author: John Wiegley <johnw <at> newartisans.com>
>   Date:   Sun Mar 6 12:51:44 2016 -0800
> 
>   Revert "Use colors in the VC mode lines"
> 
>   This reverts commit 2621c293d82c15c00d9e73a8db75d70da7d0a23b.
> 
> and I can't find a revert for commit 65fa442 (the NEWS entry).

It's fixed, closing.

  commit 11db253c082a06629d9e8baf30339b6024a0af4f
  Author: Charles A. Roelli <charles <at> aurox.ch>
  Date:   Tue Nov 21 20:36:19 2017 +0100

  Remove incorrect NEWS item about VC state indicator (Bug#28817)

  * etc/NEWS (VC and related modes): Remove news item about the VC
  state indicator.  This should have been done with commit 7882dc6
  of 2016-03-06 'Revert "Use colors in the VC mode lines"'.
bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 20 Dec 2017 12:24:05 GMT) Full text and rfc822 format available.
This bug report was last modified 7 years and 183 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.