bug#50179: [PATCH v4] Add support for "bright" ANSI colors to ansi-color and term-mode

[Jim Porter]

On 9/22/2021 12:49 PM, Lars Ingebrigtsen wrote:
> Jim Porter <jporterbugs@gmail.com> writes:
>
> > I don't quite know what to do about man.el though. It has overrides
> > for a few of the faces used by ansi-color. I could maintain that
> > behavior fairly easily, but maybe it makes sense to have it use the
> > defaults from ansi-color.
>
> Hm...  I wonder why man.el is overriding the colours?

Since I'm not totally sure, I maintained that behavior. It's not really 
any extra effort, so someone can make that decision later.

> Skimming the patch, it looks good to me, but some trivial comments:
>
> > -(defcustom ansi-color-faces-vector
> > -  [default bold default italic underline success warning error]
>
> Instead of removing the variable, mark it as obsolete instead.  (Users
> may have code that references it in their .emacs files, and that
> shouldn't bug out.)

Ok, I've done this here, and all the other places as well. I also 
obsoleted the old `ansi-color-map' and friends instead of deleting them 
for the same reason. Hopefully all that's right; I tried to make sure 
that they warn the user if they use them, but that warnings don't show 
up when building Emacs.

0001-Add-support-for-bright-ANSI-colors-in-ansi-color.patch
Description: Text document

0002-Add-support-for-bright-ANSI-colors-in-term-mode.patch
Description: Text document