|
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).
[Prev in Thread] | Current Thread | [Next in Thread] |