i guess my patch was too short sightet, as i think this only works when
pushing from linux to macos
when pushing from macos to linux, won't we also need to transfer the
option to the remote ?
https://github.com/RsyncProject/rsync/commit/224afb815e2dc1543b53fdb92228632298c0ca24
unfortunately, i cannot yet test this, i need to setup build-env on
macos first , when i have free
system around wherei can do that
regards
Roland
Am 01.06.26 um 11:42 schrieb RolandK via rsync:
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
