Hi,

Am 01.06.26 um 00:24 schrieb Andrew Tridgell:
Hi Roland,
This sounds like it is worth fixing. I have a mac mini here I could
test against.

nice, thanks for positive feedback.

A PR would be welcome, but please see if you can come up with a
testsuite (may be a bit tricky as this is an inter-platform issue, we
may need to spin up a linux VM within the macos CI runner).

i don't yet have a clue, how automated testing in rsync works. think i
will need a while to get into this.

I have no problem with you using AI to develop patches. All changes
will also be reviewed by humans. I've using claude a lot, but I also
cross-check all changes with codex and check things by hand as well. I
do also require test suites for all new features (and I've been
greatly expanding the test suites for existing features).

sounds like a good approach !

On your patch, I'd prefer --omit-link-perms, similar to
--omit-link-times.

ok, sounds reasonable. i will commit a fix for this.

Maybe we should make this the default for macos ->
linux transfers?
Cheers, Tridge

ok, but how can this be done ?

regards
Roland
-- 
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

Reply via email to