Page MenuHomeFreeBSD

net/rsync: Add back the FLAGS option
Needs ReviewPublic

Authored by des on Thu, Jun 18, 2:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 24, 2:54 AM
Unknown Object (File)
Tue, Jun 23, 2:54 AM
Unknown Object (File)
Tue, Jun 23, 1:37 AM
Unknown Object (File)
Tue, Jun 23, 1:01 AM
Unknown Object (File)
Mon, Jun 22, 12:30 AM
Unknown Object (File)
Sun, Jun 21, 8:47 PM
Unknown Object (File)
Sun, Jun 21, 1:26 PM
Unknown Object (File)
Sat, Jun 20, 10:48 PM
Subscribers
None

Details

Reviewers
rodrigo
Summary

This restores support for copying file flags and fixes a long-standing
peeve of mine by renaming the command-line option to --file-flags,
better matching the naming style of other rsync options. Full backward
compatibility is maintained, including with older servers.

MFH: 2026Q2

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73975
Build 70858: arc lint + arc unit

Event Timeline

des requested review of this revision.Thu, Jun 18, 2:19 PM
des created this revision.
net/rsync/files/extra-patch-file-flags.diff
487

--force switch still exists and has the same effect of --force-delete
so its description in the manpage should be :

--force an alias for --force-delete

Since this patch is larger than a simple porting patch, and given that it is intended to evolve in the near future,
it probably easier to keep it in a separate VCS system to help track evolution and changes.

If you don't mind I can create a separate GH repo for this additional patch, so I don't have to involve yourself
at any rsync update who break things.
BTW, I'm open to other alternatives too.

net/rsync/files/extra-patch-file-flags.diff
479

--fileflags switch was kept in the command line options, which is good to prevent POLA breakage
but we should also keep it in the manpage to avoid users confusion:

--fileflags an alias for --file-flags

Since this patch is larger than a simple porting patch, and given that it is intended to evolve in the near future,
it probably easier to keep it in a separate VCS system to help track evolution and changes.

https://github.com/dag-erling/rsync/tree/des/file-flags

Hi des,

Thanks for the github repo, but unfortunately this doesn't include my remarks.
Aside from that I'll be ready to land your changes in the ports tree, here is what the commit will look like (using your current commit):

https://people.freebsd.org/~rodrigo/rsync-flags.patch

Cheers,

  • rodrigo

Thanks for the github repo, but unfortunately this doesn't include my remarks.

What remarks?

Aside from that I'll be ready to land your changes in the ports tree, here is what the commit will look like (using your current commit):

That's not how we work. You either approve this review and I commit the change myself, or you run git arc patch -c D57645 in your ports tree and push the result.