bug-sed
[Top][All Lists]
Advanced

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

bug#40325: make backup files for "-i" option invisible


From: Assaf Gordon
Subject: bug#40325: make backup files for "-i" option invisible
Date: Mon, 30 Mar 2020 23:20:48 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0

Hello,

On 2020-03-28 10:00 a.m., nokangaroo wrote:
The visible backup files create problems with file managers, e. g. they show up
in MATE, and I have to reload the folder to make them disappear. The following
patch creates invisible files:

Nice idea, and doesn't seem to cause any regressions.
If there are no objects (Jim?), I'll push this soon in your name.
The patch is small enough that copyright assignment is not necessary.

BTW why not use /tmp instead of '.'?

When using "-i/--in-place", the new content is first written
to the temporary file, and then a rename(2) sys-call is used to replace
the input file with the new file.

When the two files are in the same directory,
such rename(2) sys-call is atomic and quick.

On many systems the "/tmp" directory is mounted
on a different file system (or in memory),
and we'd have to use a non-atomic and slower "copy+delete" instead of
a rename(2).

regards,
 - assaf




Attachment: 0001-sed-make-backup-files-for-i-option-invisible.patch
Description: Text Data


reply via email to

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