diff options
author | Martin Pool <mbp@samba.org> | 2000-10-31 01:05:42 +0000 |
---|---|---|
committer | Martin Pool <mbp@samba.org> | 2000-10-31 01:05:42 +0000 |
commit | bc363ea983a5b5db44c71bf0cb8310fddb740dc8 (patch) | |
tree | 0bffc8fc624f104845ce7628fcd348a92b3284d4 | |
parent | 84f69dad19e869eee500eb57665a9c8f5807efe2 (diff) | |
download | rsync-bc363ea983a5b5db44c71bf0cb8310fddb740dc8.tar.gz |
Display a warning about pointlessly using --rsh with clientserver mode.
-rw-r--r-- | clientserver.c | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/clientserver.c b/clientserver.c index 914d642f..41b8dfa8 100644 --- a/clientserver.c +++ b/clientserver.c @@ -17,7 +17,7 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -/* the socket based protocol for setting up a connection wit rsyncd */ +/* the socket based protocol for setting up a connection with rsyncd */ #include "rsync.h" @@ -28,6 +28,11 @@ extern int rsync_port; char *auth_user; int sanitize_paths = 0; + +/* + * Run a client connected to an rsyncd. The alternative to this + * function for remote-shell connections is do_cmd. + */ int start_socket_client(char *host, char *path, int argc, char *argv[]) { int fd, i; @@ -38,12 +43,25 @@ int start_socket_client(char *host, char *path, int argc, char *argv[]) extern int remote_version; extern int am_sender; extern struct in_addr socket_address; + extern char *shell_cmd; if (argc == 0 && !am_sender) { extern int list_only; list_only = 1; } + /* This is just a friendliness enhancement: if the connection + * is to an rsyncd then there is no point specifying the -e option. + * Note that this is only set if the -e was explicitly specified, + * not if the environment variable just happens to be set. + * See http://lists.samba.org/pipermail/rsync/2000-September/002744.html + */ + if (shell_cmd) { + rprintf(FERROR, "WARNING: --rsh or -e option ignored when " + "connecting to rsync daemon\n"); + /* continue */ + } + if (*path == '/') { rprintf(FERROR,"ERROR: The remote path must start with a module name\n"); return -1; |