[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
0001-sed-make-backup-files-for-i-option-invisible.patch
Description: Text Data