[Cvs-cvs] ccvs/src ChangeLog client.c [cvs1-11-x-branch]

[Derek Robert Price]

CVSROOT:        /cvsroot/cvs
Module name:    ccvs
Branch:         cvs1-11-x-branch
Changes by:     Derek Robert Price <address@hidden>     06/05/26 04:21:52

Modified files:
        src            : ChangeLog client.c 

Log message:
        * client.c (start_rsh_server): Default rsh client to RSH_DFLT.  Remove
        verbose comment attempting to justify the previous default.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/cvs/ccvs/src/ChangeLog.diff?only_with_tag=cvs1-11-x-branch&tr1=1.2336.2.452&tr2=1.2336.2.453&r1=text&r2=text
http://cvs.savannah.gnu.org/viewcvs/cvs/ccvs/src/client.c.diff?only_with_tag=cvs1-11-x-branch&tr1=1.318.4.39&tr2=1.318.4.40&r1=text&r2=text

Patches:
Index: ccvs/src/ChangeLog
diff -u ccvs/src/ChangeLog:1.2336.2.452 ccvs/src/ChangeLog:1.2336.2.453
--- ccvs/src/ChangeLog:1.2336.2.452     Thu May 25 15:09:08 2006
+++ ccvs/src/ChangeLog  Fri May 26 04:21:51 2006
@@ -1,6 +1,7 @@
 2006-05-25  Derek Price  <address@hidden>
 
-       * client.c (start_rsh_server): Default rsh client to RSH_DFLT.
+       * client.c (start_rsh_server): Default rsh client to RSH_DFLT.  Remove
+       verbose comment attempting to justify the previous default.
 
 2006-05-24  Larry Jones  <address@hidden>
 
Index: ccvs/src/client.c
diff -u ccvs/src/client.c:1.318.4.39 ccvs/src/client.c:1.318.4.40
--- ccvs/src/client.c:1.318.4.39        Thu May 25 15:09:09 2006
+++ ccvs/src/client.c   Fri May 26 04:21:51 2006
@@ -4727,26 +4727,6 @@
     char *rsh_argv[10];
 
     if (!cvs_rsh)
-       /* People sometimes suggest or assume that this should default
-          to "remsh" on systems like HPUX in which that is the
-          system-supplied name for the rsh program.  However, that
-          causes various problems (keep in mind that systems such as
-          HPUX might have non-system-supplied versions of "rsh", like
-          a Kerberized one, which one might want to use).  If we
-          based the name on what is found in the PATH of the person
-          who runs configure, that would make it harder to
-          consistently produce the same result in the face of
-          different people producing binary distributions.  If we
-          based it on "remsh" always being the default for HPUX
-          (e.g. based on uname), that might be slightly better but
-          would require us to keep track of what the defaults are for
-          each system type, and probably would cope poorly if the
-          existence of remsh or rsh varies from OS version to OS
-          version.  Therefore, it seems best to have the default
-          remain "rsh", and tell HPUX users to specify remsh, for
-          example in CVS_RSH or other such mechanisms to be devised,
-          if that is what they want (the manual already tells them
-          that).  */
        cvs_rsh = RSH_DFLT;
     if (!cvs_server)
        cvs_server = "cvs";