Subject: Re: bug#61386: [PATCH] cp,mv,install: Disable sparse copy on macOS

From: Pádraig Brady <[email-masked]>
Date: Fri, 10 Feb 2023 22:02:54 +0000

On 10/02/2023 21:50, Paul Eggert wrote:
> On 2/10/23 13:35, George Valkov wrote:
> 
>> Since the source and it’s clone have separate metadata,
>> it should be possible to change it on the clone, to comply with standards.
> 
> Attached is a hacky patch to do that. It also uses the new CLONE_ACL
> flag. The old code apparently mishandled ACLs; the new code tries to do
> a better job.

Yes this is looking much better.
I'm not fully convinced that we should also update the times,
but not against it either.

> This whole area is a mess, but the patch should improve correctness on
> macOS, while also being efficient in the usual case.

Yes it's a pity there is no fd (data) only interface for this,
rather than dealing with names (metadata) too.

thanks,
Pádraig.