[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: cp preserves mode with --no-preserve=mode
From: |
Pádraig Brady |
Subject: |
Re: cp preserves mode with --no-preserve=mode |
Date: |
Tue, 07 Aug 2012 10:47:45 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:6.0) Gecko/20110816 Thunderbird/6.0 |
On 08/06/2012 12:29 PM, Ondrej Oprala wrote:
> Hi, I'd like to propose a fix for a bug mentioned in the redhat bugzilla:
> https://bugzilla.redhat.com/show_bug.cgi?id=755508
> The patch adds another boolean instead of a proposed tristate, thus not
> upsetting any existing code.
> Also, if the behaviour proposed in the bugzilla entry when --no-preserve=mode
> option
> is specified is correct, I think the last test in cp/link-preserve should be
> changed.
> Shouldn't the --no-preserve=mode always create permissions of 0666 & ~umask?
> The test
> seems to assume that a 0777 & ~umask mode should be set.
I agree with the behavior and approach of the patch.
The NEWS item might be best in the "change in behavior" section?
Also this line should use logical rather than bitwise operators.
x->explicit_no_preserve = ~on_off;
cheers,
Pádraig.
- cp preserves mode with --no-preserve=mode, Ondrej Oprala, 2012/08/06
- Re: cp preserves mode with --no-preserve=mode, Jim Meyering, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode,
Pádraig Brady <=
- Re: cp preserves mode with --no-preserve=mode, Jim Meyering, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode, Ondrej Oprala, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode, Bernhard Voelker, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode, Jim Meyering, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode, Ondrej Oprala, 2012/08/07
- Re: cp preserves mode with --no-preserve=mode, Bernhard Voelker, 2012/08/07