coreutils
[Top][All Lists]
Advanced

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

Re: [PATCH 3/4] copy: make backup files more reliably


From: Paul Eggert
Subject: Re: [PATCH 3/4] copy: make backup files more reliably
Date: Thu, 3 Aug 2017 04:17:40 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1

Kamil Dudka wrote:

Still not perfect.  Now it produces false positives with --backup=existing:

Yes, I know that, but there's no easy way to change that without introducing a race condition the other way. I thought it better to be conservative and reject the (extremely unusual) case that you mentioned, rather than make the common case significantly less-efficient or (worse) to lose data in some rare cases.

I suppose we could change the diagnostic to say "might destroy source" rather than "would destroy source". That would be more accurate.

As for the test, could it be simplified like this?

Could be. I didn't look at the test in detail (it's now the same as before).



reply via email to

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