This is an automated email from the ASF dual-hosted git repository.

sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-release.git


The following commit(s) were added to refs/heads/main by this push:
     new c4355b6  Fix a problem with the rsync validation logic
c4355b6 is described below

commit c4355b6f7d16a7e11f6d6d8ddf57ecc2bf2f4538
Author: Sean B. Palmer <[email protected]>
AuthorDate: Tue Jun 17 17:16:59 2025 +0100

    Fix a problem with the rsync validation logic
---
 atr/ssh.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/atr/ssh.py b/atr/ssh.py
index 61ede01..8be2bec 100644
--- a/atr/ssh.py
+++ b/atr/ssh.py
@@ -222,11 +222,10 @@ def _step_02_command_simple_validate(argv: list[str]) -> 
tuple[str | None, int,
 
     # Validate the options argument strictly
     options = argv[option_index]
-    if ("e." in options) and (options.split("e.", 1)[0] != "-vlogDtpr"):
-        # The options after -e. are compatibility flags and can be ignored
-        return "The options argument (after --sender) must be 
'-vlogDtpre.[compatibility flags]'", -1, True
-    elif options != "-vlogDtpr":
-        return "The options argument (after --sender) must be '-vlogDtpr'", 
-1, True
+    if "e." in options:
+        options = options.split("e.", 1)[0]
+    if options != "-vlogDtpr":
+        return "The options argument (after --sender) must be 
'-vlogDtpr[.e<FLAGS>]'", -1, True
 
     ####################################################
     ### Calls _step_03_validate_rsync_args_structure ###


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to