Re: DragonFly-2.2.0.34.g50a5f DragonFly_RELEASE_2_2 bin/cpdup cpdup.c

[YONETANI Tomokazu <qhwt+dfly@xxxxxxxxxx>]

On Mon, Apr 27, 2009 at 08:52:14PM -0700, Matthew Dillon wrote:
> 
> :As I replied to Matt's commit(74fa57e3), this patch alone makes cpdup
> :complain about failure of chown() (which is OK), and return non-zero
> :code(which is NOT OK), if you run it as a non-root user, and you're
> :trying to mirror a tree you don't own.  It also breaks buildworld
> :as a non-root user, but apparently another commit(17e9c4cc) has fixed
> :the issue.
> 
>     After testing it does appear that my ownership commit did fix the
>     issue, and it should be in the 2.2.1 that I rolled.
> 
>     The 2.2.1 that I rolled does not have the ownership check on directories
>     MFC, but it does have the allow-chown-to-fail-if-not-root fix.  With
>     the allow-chown-to-fail fix in it should work with or without the
>     ownership check on directories.

Ah, I'm sorry, it seems I overlooked that 17e9c4cc(2.3.0) and
85eeefce(2.2.0) are the same ones.

Cheers.