The branch, master has been updated
       via  323c42d5 Improve how the env restricts negotiated strings
       via  d1fdf9ff Avoid -U if --atimes is disabled.
       via  e93f40d8 Apple needs a leading underscore.
       via  4df1b1d4 Makefile.in: Use srcdir for installing rsync-ssl
       via  1af58f6b Improve the info about compression.
      from  a8fc8fc2 Preparing for release of 3.2.1

https://git.samba.org/?p=rsync.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 323c42d51e2a7d94dd641d7be5de89a2704d441c
Author: Wayne Davison <wa...@opencoder.net>
Date:   Tue Jun 23 17:19:58 2020 -0700

    Improve how the env restricts negotiated strings
    
    - The env on the server side now affects the negotiated strings
      that are sent to the client.
    - A too-old remote rsync gets a default negotiated string value
      so that an env restriction now handles old clients the same way
      as new ones.

commit d1fdf9ff8dad4f64259d78f8bc912aed782848d9
Author: Wayne Davison <wa...@opencoder.net>
Date:   Tue Jun 23 15:53:47 2020 -0700

    Avoid -U if --atimes is disabled.

commit e93f40d8b4068a0170ec0d8a6d6ab74850ba358f
Author: Wayne Davison <wa...@opencoder.net>
Date:   Tue Jun 23 15:47:27 2020 -0700

    Apple needs a leading underscore.

commit 4df1b1d4fe213d4409d5fad962f831837a4c7e23
Author: Hiroshi Takekawa <sian...@gmail.com>
Date:   Tue Jun 23 19:17:53 2020 +0900

    Makefile.in: Use srcdir for installing rsync-ssl
    
    When building out of source tree, we can't find rsync-ssl in the current
    directory and installation fails.  Fix it by using the srcdir variable for 
the
    path to rsync-ssl.
    
    Signed-off-by: Hiroshi Takekawa <sian...@gmail.com>

commit 1af58f6b77154a3b3d40e40647220c4d54ee6cdc
Author: Wayne Davison <wa...@opencoder.net>
Date:   Mon Jun 22 20:37:52 2020 -0700

    Improve the info about compression.

-----------------------------------------------------------------------

Summary of changes:
 Makefile.in           |  2 +-
 checksum.c            |  2 +-
 compat.c              | 35 +++++++++++++++++-----------
 lib/md5-asm-x86_64.S  |  4 ++++
 rsync.1.md            | 64 +++++++++++++++++++++++++--------------------------
 testsuite/daemon.test | 13 ++++++-----
 6 files changed, 66 insertions(+), 54 deletions(-)


Changeset truncated at 500 lines:

diff --git a/Makefile.in b/Makefile.in
index 6ca7d9eb..e13e241b 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -77,7 +77,7 @@ all: Makefile rsync$(EXEEXT) stunnel-rsyncd.conf @MAKE_MAN@
 install: all
        -${MKDIR_P} ${DESTDIR}${bindir}
        ${INSTALLCMD} ${INSTALL_STRIP} -m 755 rsync$(EXEEXT) ${DESTDIR}${bindir}
-       ${INSTALLCMD} -m 755 rsync-ssl ${DESTDIR}${bindir}
+       ${INSTALLCMD} -m 755 $(srcdir)/rsync-ssl ${DESTDIR}${bindir}
        -${MKDIR_P} ${DESTDIR}${mandir}/man1
        -${MKDIR_P} ${DESTDIR}${mandir}/man5
        if test -f rsync.1; then ${INSTALLMAN} -m 644 rsync.1 
${DESTDIR}${mandir}/man1; fi
diff --git a/checksum.c b/checksum.c
index 87e83658..7c915173 100644
--- a/checksum.c
+++ b/checksum.c
@@ -85,7 +85,7 @@ static const char *checksum_name(int num)
 {
        struct name_num_item *nni = get_nni_by_num(&valid_checksums, num);
 
-       return nni ? nni->name : num < CSUM_MD4 ? "MD4" : "UNKNOWN";
+       return nni ? nni->name : num < CSUM_MD4 ? "md4" : "UNKNOWN";
 }
 
 void parse_checksum_choice(int final_call)
diff --git a/compat.c b/compat.c
index 9ec39ea0..7a97cfc3 100644
--- a/compat.c
+++ b/compat.c
@@ -390,20 +390,12 @@ static void send_negotiate_str(int f_out, struct 
name_num_obj *nno, const char *
 {
        char tmpbuf[MAX_NSTR_STRLEN];
        const char *list_str = getenv(env_name);
-       int len, fail_if_empty = list_str && strstr(list_str, "FAIL");
+       int len;
 
-       if (!do_negotiated_strings) {
-               if (!am_server && fail_if_empty) {
-                       rprintf(FERROR, "Remote rsync is too old for %s 
negotiation\n", nno->type);
-                       exit_cleanup(RERR_UNSUPPORTED);
-               }
-               return;
-       }
-
-       if (list_str && *list_str && (!am_server || local_server)) {
+       if (list_str && *list_str) {
                init_nno_saw(nno, 0);
                len = parse_nni_str(nno, list_str, tmpbuf, MAX_NSTR_STRLEN);
-               if (fail_if_empty && !len)
+               if (!len)
                        len = strlcpy(tmpbuf, "FAIL", MAX_NSTR_STRLEN);
                list_str = tmpbuf;
        } else
@@ -423,7 +415,7 @@ static void send_negotiate_str(int f_out, struct 
name_num_obj *nno, const char *
                /* A local server doesn't bother to send/recv the strings, it 
just constructs
                 * and parses the same string on both sides. */
                recv_negotiate_str(-1, nno, tmpbuf, len);
-       } else {
+       } else if (do_negotiated_strings) {
                /* Each side sends their list of valid names to the other side 
and then both sides
                 * pick the first name in the client's list that is also in the 
server's list. */
                write_vstring(f_out, tmpbuf, len);
@@ -442,13 +434,28 @@ static void negotiate_the_strings(int f_in, int f_out)
 
        if (valid_checksums.saw) {
                char tmpbuf[MAX_NSTR_STRLEN];
-               recv_negotiate_str(f_in, &valid_checksums, tmpbuf, -1);
+               int len;
+               if (do_negotiated_strings)
+                       len = -1;
+               else
+                       len = strlcpy(tmpbuf, protocol_version >= 30 ? "md5" : 
"md4", MAX_NSTR_STRLEN);
+               recv_negotiate_str(f_in, &valid_checksums, tmpbuf, len);
        }
 
        if (valid_compressions.saw) {
                char tmpbuf[MAX_NSTR_STRLEN];
-               recv_negotiate_str(f_in, &valid_compressions, tmpbuf, -1);
+               int len;
+               if (do_negotiated_strings)
+                       len = -1;
+               else
+                       len = strlcpy(tmpbuf, "zlib", MAX_NSTR_STRLEN);
+               recv_negotiate_str(f_in, &valid_compressions, tmpbuf, len);
        }
+
+       /* If the other side is too old to negotiate, the above steps just made 
sure that
+        * the env didn't disallow the old algorithm. Mark things as 
non-negotiated. */
+       if (!do_negotiated_strings)
+               valid_checksums.negotiated_name = 
valid_compressions.negotiated_name = NULL;
 }
 
 void setup_protocol(int f_out,int f_in)
diff --git a/lib/md5-asm-x86_64.S b/lib/md5-asm-x86_64.S
index 814aac59..383f193a 100644
--- a/lib/md5-asm-x86_64.S
+++ b/lib/md5-asm-x86_64.S
@@ -29,6 +29,10 @@
 
 #if !defined USE_OPENSSL && CSUM_CHUNK == 64
 
+#ifdef __APPLE__
+#define md5_process_asm _md5_process_asm
+#endif
+
 .text
 .align 16
 
diff --git a/rsync.1.md b/rsync.1.md
index 9d9dfbae..ebd59dca 100644
--- a/rsync.1.md
+++ b/rsync.1.md
@@ -1490,7 +1490,8 @@ your home directory (remove the '=' for that).
     a negotiation between the client and the server as follows:
 
     If both the client and the server are at least version 3.2.0, they will
-    exchange a list of checksum names and choose the first one in the list that
+    exchange a list of checksum names and choose the first one in the client's
+    list that
     they have in common.  This typically means that they will choose xxh64 if
     they both support it and fall back to MD5.  If one side of the transfer is
     not new enough to support this checksum negotiation, then a value is chosen
@@ -1503,8 +1504,8 @@ your home directory (remove the '=' for that).
     client exits with an error.  This method does not allow you to specify the
     transfer checksum separately from the pre-transfer checksum, and it ignores
     "auto" and all unknown checksum names.  If the remote rsync is not new
-    enough to handle a checksum negotiation list, the list is silently ignored
-    unless it contains the string "FAIL".
+    enough to handle a checksum negotiation list, its list is assumed to
+    consist of a single "md5" or "md4" item based on the protocol version.
 
     The use of the `--checksum-choice` option overrides this environment list.
 
@@ -2260,24 +2261,27 @@ your home directory (remove the '=' for that).
     destination machine, which reduces the amount of data being transmitted --
     something that is useful over a slow connection.
 
-    The "zlib" compression method typically achieves better compression ratios
-    than can be achieved by using a compressing remote shell or a compressing
-    transport because it takes advantage of the implicit information in the
-    matching data blocks that are not explicitly sent over the connection.
-    This matching-data compression comes at a cost of CPU, though, and can be
-    disabled by using the "zlibx" compresson method instead.  This can be
-    selected by repeating the `-z` option or specifying
-    `--compress-choice=zlibx`, but it only works if both sides of the transfer
-    are at least version 3.1.1.
+    Rsync supports multiple compression methods and will choose one for you
+    unless you force the choice using the `--compress-choice` option.
 
-    Note that if you see an error about an option named `--old-compress` or
-    `--new-compress`, this is rsync trying to send the `--compress-choice=zlib`
-    or `--compress-choice=zlibx` option in a backward-compatible manner that
-    more rsync versions understand.  This error indicates that the older rsync
-    version will not allow you to force the compression type.
+    Run `rsync -V` to see the compress list compiled into your version.
+
+    When both sides of the transfer are at least 3.2.0, rsync chooses the first
+    algorithm in the client's list of choices that is also in the server's list
+    of choices.  Your default order can be customized by setting the 
environment
+    variable RSYNC_COMPRESS_LIST to a space-separated list of acceptable
+    compression names.  If no common compress choice is found, the client exits
+    with an error.  If the remote rsync is too old to support checksum 
negotiation,
+    its list is assumed to be "zlib".
+
+    There are some older rsync versions that were configured to reject a `-z`
+    option and require the use of `-zz` because their compression library was
+    not compatible with the default zlib compression method.  You can usually
+    ignore this weirdness unless the rsync server complains and tells you to
+    specify `-zz`.
 
-    See the `--skip-compress` option for the default list of file suffixes that
-    will not be compressed.
+    See also the `--skip-compress` option for the default list of file suffixes
+    that will trasnferred with no (or minimal) compression.
 
 0.  `--compress-choice=STR`, `--zc=STR`
 
@@ -2294,24 +2298,20 @@ your home directory (remove the '=' for that).
 
     Run `rsync -V` to see the compress list compiled into your version.
 
-    The "zlibx" algorithm is given preference over "zlib" if both sides of the
-    transfer are at least version 3.2.0, otherwise it will choose "zlib" unless
-    you override it via something like `-zz`.  These 2 algorithms are the stame
-    except that "zlibx" does not try to include matched data that was not
-    transferred in the compression computations.
+    Note that if you see an error about an option named `--old-compress` or
+    `--new-compress`, this is rsync trying to send the `--compress-choice=zlib`
+    or `--compress-choice=zlibx` option in a backward-compatible manner that
+    more rsync versions understand.  This error indicates that the older rsync
+    version on the server will not allow you to force the compression type.
+
+    Note that the "zlibx" compression algorithm is just the "zlib" algorithm
+    with matched data excluded from the compression stream (to try to make it
+    more compatible with an external zlib implementation).
 
     If "none" is specified, that is equivalent to using `--no-compress`.
 
     This option implies `--compress` unless "none" was specified.
 
-    You can also override the compression negotiation using the
-    RSYNC_COMPRESS_LIST environment variable by setting it to a space-separated
-    list of compression names that you consider acceptable.  If no common
-    compress choice is found, the client exits with an error.  It ignores
-    "auto" and all unknown compression names.  If the remote rsync is not new
-    enough to handle a compression negotiation list, the list is silently
-    ignored unless it contains the string "FAIL".
-
 0.  `--compress-level=NUM`, `--zl=NUM`
 
     Explicitly set the compression level to use (see `--compress`, `-z`)
diff --git a/testsuite/daemon.test b/testsuite/daemon.test
index a8acd04e..a736ee83 100644
--- a/testsuite/daemon.test
+++ b/testsuite/daemon.test
@@ -91,12 +91,13 @@ drwxr-xr-x         DIR ####/##/## ##:##:## foo
 EOT
 diff $diffopt "$chkfile" "$outfile" || test_fail "test 3 failed"
 
-$RSYNC -rU localhost::test-from/f* \
-    | sed "$FILE_REPL" | sed "$DIR_REPL" | sed "$LS_REPL" \
-    | tee "$outfile"
-cat <<EOT >"$chkfile"
+if $RSYNC --version | grep "[, ] atimes" >/dev/null; then
+    $RSYNC -rU localhost::test-from/f* \
+       | sed "$FILE_REPL" | sed "$DIR_REPL" | sed "$LS_REPL" \
+       | tee "$outfile"
+    cat <<EOT >"$chkfile"
 drwxr-xr-x         DIR ####/##/## ##:##:##                     foo
 -rw-r--r--           4 ####/##/## ##:##:## ####/##/## ##:##:## foo/one
 EOT
-diff $diffopt "$chkfile" "$outfile" || test_fail "test 3 failed"
-
+    diff $diffopt "$chkfile" "$outfile" || test_fail "test 3 failed"
+fi


-- 
The rsync repository.

_______________________________________________
rsync-cvs mailing list
rsync-cvs@lists.samba.org
https://lists.samba.org/mailman/listinfo/rsync-cvs

Reply via email to