Would this patch be useful...

Wayne Davison wayned at samba.org
Fri May 14 22:15:21 GMT 2004

On Tue, May 11, 2004 at 11:41:02AM -0400, John Taylor wrote:
> I am considering a patch that would exclude compressing a given list
> of file extensions when the -z option is used.

Well, since the daemon code has this, it makes sense to at least
consider adding this to the normal remote-shell code.  I'm not sure how
much this option speeds things up -- have you done any measuring?

I'd also not want to let someone who is grabbing files from a server
specify a shorter list than the one that the server specified, so we'd
need to make sure that the user's list appended to the server's list
when pulling.

Though the docs don't mention this, the daemon mode only appears to use
the "dont compress" value when the daemon is sending files -- it doesn't
appear to do anything when it is receiving files.  This is because the
list is not shared across the socket, and only the sender cares about
this value.

> I want to receive input on how to implement the user-interface.  I was
> thinking about two command line options, --dont-compress and
> --dont-compress-from.

The easiest way would be to add only the --dont-compress="foo bar ..."
option since that would allow the value to be sent to the sending side
via the existing option mechanism.  Adding a --dont-compress-from option
would mean that the protocol would have to be extended to transfer this
data in a manner similar to the current exclude list, which requires a
protocol bump.  If you go that route, I'd suggest making the options
work just like the exclude options so that each --dont-compress option
would only specify a single item (and it would be easiest to switch the
internal dont-compress code over to using the exclude-list code so that
you don't need to re-invent a bunch of identical functions).

So, I'm not convinced yet that this is worth it, but it might be.  If
you'd care to submit a patch, I'd at least put it in the patches dir,
and certainly consider it for the release version.  What do others


More information about the rsync mailing list