cp: fix the --no-preserve=mode option
authorOndrej Oprala <ooprala@redhat.com>
Tue, 7 Aug 2012 14:56:52 +0000 (16:56 +0200)
committerJim Meyering <meyering@redhat.com>
Fri, 28 Sep 2012 10:21:02 +0000 (12:21 +0200)
commit24ebca61a3a0f10d6cd2800b188b3c034d1c4755
treedfb6ffe8a3e1447432b9310a93f2785d787bcfb8
parent64be8b40f457f6480f9e79777d98afce40469786
cp: fix the --no-preserve=mode option

The --no-preserve=mode option did not do what its name implies:
it would mistakenly preserve permission mode bits.
* NEWS: Mention the fix.
* TODO: Remove an entry.
* src/copy.c (copy_reg): Add a condition to properly
handle the --no-preserve=mode option for files
(copy_internal): Add a condition to properly handle the
--no-preserve=mode option for directories.
* src/copy.h (struct cp_options): Add a new boolean.
* src/cp.c (cp_option_init,decode_preserve_arg): Set the
new boolean value according to specified options.
* src/install.c (struct cp_options): Initialize the new boolean.
* src/mv.c (struct cp_options): Initialize the new boolean.
* tests/cp/preserve-mode.sh: Add a new test.
* tests/cp/link-preserve.sh (-a --no-preserve=mode): Adjust the
expected perms: now, --no-preserve=mode overrides the --preserve=mode
that is inherent in -a, as it should.
* tests/local.mk: Add the new test to the list.
NEWS
TODO
src/copy.c
src/copy.h
src/cp.c
src/install.c
src/mv.c
tests/cp/link-preserve.sh
tests/cp/preserve-mode.sh [new file with mode: 0755]
tests/local.mk