GNU bug report logs - #40325
make backup files for "-i" option invisible

Previous Next

Full log

Message #8 received at 40325 <at> debbugs.gnu.org (full text, mbox):

From: Assaf Gordon <assafgordon <at> gmail.com>
To: nokangaroo <aon.913677709 <at> aon.at>, 40325 <at> debbugs.gnu.org
Subject: Re: bug#40325: make backup files for "-i" option invisible
Date: Mon, 30 Mar 2020 23:20:48 -0600

[Message part 1 (text/plain, inline)]

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 (text/x-patch, attachment)]

Previous Next

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