emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#34225: closed (27.0.50; Add passwd backups to etc-


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#34225: closed (27.0.50; Add passwd backups to etc-passwd-generic-mode)
Date: Mon, 28 Jan 2019 20:22:02 +0000

Your message dated Mon, 28 Jan 2019 22:19:39 +0200
with message-id <address@hidden>
and subject line Re: bug#34225: 27.0.50; Add passwd backups to 
etc-passwd-generic-mode
has caused the debbugs.gnu.org bug report #34225,
regarding 27.0.50; Add passwd backups to etc-passwd-generic-mode
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
34225: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=34225
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 27.0.50; Add passwd backups to etc-passwd-generic-mode Date: Sun, 27 Jan 2019 22:34:03 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (x86_64-pc-linux-gnu)
Tags: patch

I don't know why backups /etc/passwd- use the suffix ‘-’ instead of ‘~’
but comparing these files with their backups I noticed the difference
in their font-lock highlighting, thus this patch:

diff --git a/lisp/generic-x.el b/lisp/generic-x.el
index 014379fba9..dd61402015 100644
--- a/lisp/generic-x.el
+++ b/lisp/generic-x.el
@@ -1490,7 +1490,7 @@ etc-passwd-generic-mode
      '("^\\([^:]+\\):\\([^:]*\\):\\([0-9]+\\):\\(.*\\)$"
        (1 font-lock-type-face)
        (4 font-lock-variable-name-face))))
-  '("/etc/passwd\\'" "/etc/group\\'")
+  '("/etc/passwd-?\\'" "/etc/group-?\\'" "/etc/shadow-?\\'")
   (list
    (function
     (lambda ()

--- End Message ---
--- Begin Message --- Subject: Re: bug#34225: 27.0.50; Add passwd backups to etc-passwd-generic-mode Date: Mon, 28 Jan 2019 22:19:39 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (x86_64-pc-linux-gnu)
>>> I don't know why backups /etc/passwd- use the suffix '-' instead of '~'
>>
>> It does?  I seem to be unable to reproduce this:
>>
>>   (make-backup-file-name "/etc/passwd")
>>     => "/etc/passwd~"
>>
>> What am I missing?
>
> These are non-Emacs backups. See eg "man 5 shadow".
>
>     /etc/shadow-
>     Backup file for /etc/shadow.
>
>     Note that this file is used by the tools of the shadow toolsuite,
>     but not by all user and password management tools.

Yes, and "man 5 passwd" shows the same:

       /etc/passwd-
           Backup file for /etc/passwd.

So I installed the patch to master.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]