Re: [PATCH] rsync add zstd,lz4,iconv options as default and -minimal flavor

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