git-cvsserver: handle CVS 'noop' command.
authorStefan Karpinski <stefan.karpinski@gmail.com>
Fri, 30 Jan 2009 01:12:27 +0000 (17:12 -0800)
committerJunio C Hamano <gitster@pobox.com>
Sat, 31 Jan 2009 05:06:27 +0000 (21:06 -0800)
The CVS protocol documentation, found at

  http://www.wandisco.com/techpubs/cvs-protocol.pdf

states the following about the 'noop' command:

  Response expected: yes. This request is a null command
  in the sense that it doesn't do anything, but merely
  (as with any other requests expecting a response) sends
  back any responses pertaining to pending errors, pending
  Notified responses, etc.

In accordance with this, the correct way to handle the 'noop'
command, when issued by a client, is to call req_EMPTY.

The 'noop' command is called by some CVS clients, notably
TortoiseCVS, thus making it desirable for git-cvsserver to
respond to the command rather than choking on it as unknown.

Signed-off-by: Stefan Karpinski <stefan.karpinski@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-cvsserver.perl

index fef7faf33947a87ded0e8a2d9fd6f1af6047d951..277ee4e47729ecfec887c1c2ec16d4f6f51eee43 100755 (executable)
@@ -76,6 +76,7 @@ my $methods = {
     'history'         => \&req_CATCHALL,
     'watchers'        => \&req_EMPTY,
     'editors'         => \&req_EMPTY,
+    'noop'            => \&req_EMPTY,
     'annotate'        => \&req_annotate,
     'Global_option'   => \&req_Globaloption,
     #'annotate'        => \&req_CATCHALL,