bug#29961: [PATCH] mv: document the missing atomicity of 'mv -n'

2018-01-03 Thread Jim Meyering
On Wed, Jan 3, 2018 at 8:27 AM, Kamil Dudka wrote: > On Wednesday, January 3, 2018 4:08:51 PM CET Pádraig Brady wrote: >> Eep, Seems like we should use RENAME_NOREPLACE in this case, >> rather than document the caveat? Good catch/fix. Thanks to both of you.

bug#29961: [PATCH] mv: document the missing atomicity of 'mv -n'

2018-01-03 Thread Kamil Dudka
On Wednesday, January 3, 2018 4:08:51 PM CET Pádraig Brady wrote: > Eep, Seems like we should use RENAME_NOREPLACE in this case, > rather than document the caveat? Thanks for the suggestion! I will give it a try... Kamil > This is already used in shred. > > cheers, > Pádraig

bug#29961: [PATCH] mv: document the missing atomicity of 'mv -n'

2018-01-03 Thread Pádraig Brady
On 03/01/18 15:01, Kamil Dudka wrote: > * src/mv.c (usage): Add note about the missing atomicity of 'mv -n'. > * doc/coreutils.texi (mv invocation): Likewise. > --- > doc/coreutils.texi | 3 +++ > src/mv.c | 2 ++ > 2 files changed, 5 insertions(+) > > diff --git a/doc/coreutils.texi b/

bug#29961: [PATCH] mv: document the missing atomicity of 'mv -n'

2018-01-03 Thread Kamil Dudka
* src/mv.c (usage): Add note about the missing atomicity of 'mv -n'. * doc/coreutils.texi (mv invocation): Likewise. --- doc/coreutils.texi | 3 +++ src/mv.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/doc/coreutils.texi b/doc/coreutils.texi index 3fa083085..e7ca6a737 100644 -