Re upgrade paths / defaults (i.e. whether to have -minimal as a low-dep 
option and include zstd etc otherwise, or whether to avoid them by default 
and have -full as the version with run deps) - I think it really depends on 
what would be considered acceptable by whoever it was that complained last 
time - I don't remember who that was.

-- 
  Sent from a phone, apologies for poor formatting.

On 8 July 2024 22:56:41 Jeremie Courreges-Anglas <jca@wxcvbn.org> wrote:

> On Mon, Jul 08, 2024 at 12:26:52PM -0700, Bryan Vyhmeister wrote:
>> This patch to switch rsync to being built by default with iconv, lz4,
>> and zstd and then also build a -minimal flavor which removes all of
>> those has been floating around on ports@ for several weeks. Any further
>> comments?
>
> FWIW this version looks fine to me but there's a question still open
> regarding the upgrade paths.  Will keep you posted.
>
>> I suspect switching to zstd compression (which would be the
>> default once this patch is in) would help the load on mirrors in the
>> future as well.
>
> Yep
>
> --
> jca