Mark D. Baushke wrote:
> Subsequent to that release,
> the patch you mention was applied to the FEATURE branch sources:
> 
> 2005-12-07  Derek Price  <[EMAIL PROTECTED]>
> 
>         * client.c (handle_redirect): Give stack control of string key.
> 
>         * client.c (start_server), root.c (method_names), root.h (CVSmethod):
>         Handle :extssh: as a kindness to Eclipse users.
>         (Suggestion from Joseph P. Skudlarek <[EMAIL PROTECTED]>.)

This patch is incomplete. The same patch as in CVS HEAD [1] is also found
in cvs-1.11.22, and it does not work:

$ cvs-1.11.22 log ...
cvs-1.11.22 log: Unknown method (`extssh') in CVSROOT.
cvs-1.11.22 log: in directory .:
cvs-1.11.22 log: ignoring CVS/Root because it does not contain a valid root.
cvs-1.11.22 [log aborted]: received interrupt signal

There are three ingredients in Skudlarek's patch for 1.11.x [2] and my
adaptation for 1.12.x [3]:

  1) Add extssh to the enumeration.
  2) Recognize :extssh: when it occurs as part of CVS/Root.
  3) Use "ssh" as remote shell, regardless of the CVS_RSH environment
     variable.

Derek's commit contains only #1. #2 and #3 are missing.

Obviously Derek's commit has not been tested. I understand that you don't
want to install Eclipse and learn how to use it, just to verify an
interoperability issue. If you provide me a source tarball, I can test
it for you.

Bruno


[1] http://cvs.savannah.gnu.org/viewvc/cvs/ccvs/src/
[2] http://lists.gnu.org/archive/html/bug-cvs/2005-11/msg00058.html
[3] http://lists.gnu.org/archive/html/bug-cvs/2008-01/msg00007.html



_______________________________________________
Bug-cvs mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/bug-cvs

Reply via email to