GNU bug report logs - #54521
[PATCH] dircolors: colorize backup files with bright black

Previous Next

Package: coreutils;

Reported by: Ville Skyttä <ville.skytta <at> iki.fi>

Date: Tue, 22 Mar 2022 18:25:02 UTC

Severity: normal

Tags: patch

Done: Pádraig Brady <P <at> draigBrady.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ville Skyttä <ville.skytta <at> iki.fi>
Subject: bug#54521: closed (Re: bug#54521: [PATCH] dircolors: colorize
 backup files with bright black)
Date: Sun, 03 Apr 2022 20:25:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#54521: [PATCH] dircolors: colorize backup files with bright black

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 54521 <at> debbugs.gnu.org.

-- 
54521: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=54521
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Pádraig Brady <P <at> draigBrady.com>
To: Ville Skyttä <ville.skytta <at> iki.fi>,
 54521-done <at> debbugs.gnu.org
Subject: Re: bug#54521: [PATCH] dircolors: colorize backup files with bright
 black
Date: Sun, 3 Apr 2022 21:24:41 +0100
On 23/03/2022 23:53, Pádraig Brady wrote:
> On 22/03/2022 18:24, Ville Skyttä wrote:
>> Useful as it makes them stand out less than non-backup files.
>> ---
>>    src/dircolors.hin | 19 +++++++++++++++++++
>>    1 file changed, 19 insertions(+)
>>
>> diff --git a/src/dircolors.hin b/src/dircolors.hin
>> index 673835201..6dc5a2d74 100644
>> --- a/src/dircolors.hin
>> +++ b/src/dircolors.hin
>> @@ -211,5 +211,24 @@ EXEC 01;32
>>    .spx 00;36
>>    .xspf 00;36
>>    
>> +# backup files
>> +*~ 00;90
>> +*# 00;90
>> +.bak 00;90
>> +.old 00;90
>> +.orig 00;90
>> +.part 00;90
>> +.rej 00;90
>> +.swp 00;90
>> +.tmp 00;90
>> +.dpkg-dist 00;90
>> +.dpkg-old 00;90
>> +.ucf-dist 00;90
>> +.ucf-new 00;90
>> +.ucf-old 00;90
>> +.rpmnew 00;90
>> +.rpmorig 00;90
>> +.rpmsave 00;90
>> +
>>    # Subsequent TERM or COLORTERM entries, can be used to add / override
>>    # config specific to those matching environment variables.
> 
> I quite like that actually.
> The default coloring works well on both dark and light backgrounds.
> Coloring this class of files does seem useful.
> 
> +1 from me anyway.

I've pushed this after some perf testing showing it had negligible perf impact.
https://git.sv.gnu.org/cgit/coreutils.git/commit/?id=52aeae2c3

Here is a small example of the output:
https://www.pixelbeat.org/patches/coreutils/ls-backup-colors.html

cheers,
Pádraig

[Message part 3 (message/rfc822, inline)]
From: Ville Skyttä <ville.skytta <at> iki.fi>
To: bug-coreutils <at> gnu.org
Subject: [PATCH] dircolors: colorize backup files with bright black
Date: Tue, 22 Mar 2022 20:24:27 +0200
Useful as it makes them stand out less than non-backup files.
---
 src/dircolors.hin | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/src/dircolors.hin b/src/dircolors.hin
index 673835201..6dc5a2d74 100644
--- a/src/dircolors.hin
+++ b/src/dircolors.hin
@@ -211,5 +211,24 @@ EXEC 01;32
 .spx 00;36
 .xspf 00;36
 
+# backup files
+*~ 00;90
+*# 00;90
+.bak 00;90
+.old 00;90
+.orig 00;90
+.part 00;90
+.rej 00;90
+.swp 00;90
+.tmp 00;90
+.dpkg-dist 00;90
+.dpkg-old 00;90
+.ucf-dist 00;90
+.ucf-new 00;90
+.ucf-old 00;90
+.rpmnew 00;90
+.rpmorig 00;90
+.rpmsave 00;90
+
 # Subsequent TERM or COLORTERM entries, can be used to add / override
 # config specific to those matching environment variables.
-- 
2.25.1




This bug report was last modified 3 years and 107 days ago.

Previous Next


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