Re: PATCH: add --no-symlink-perms option for ignoring symlink permissions on transfer
Andrew Tridgell via rsync <[email protected]> Mon, 1 Jun 2026 08:24:28 +1000
| Newsgroups | gmane.network.rsync.general |
|---|---|
| Message-ID | <CAAbv5Gb2pxcGoDeqgOnR26n0NRdwfBvptvx8AJCfyFO2HBXHkA@mail.gmail.com> |
Hi Roland, This sounds like it is worth fixing. I have a mac mini here I could test against. 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 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). On your patch, I'd prefer --omit-link-perms, similar to --omit-link-times. Maybe we should make this the default for macos -> linux transfers? Cheers, Tridge -- 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