From xemacs-m  Thu Sep 11 02:21:53 1997
Received: from kurims.kurims.kyoto-u.ac.jp (kurims.kurims.kyoto-u.ac.jp [130.54.16.1])
	by xemacs.org (8.8.5/8.8.5) with ESMTP id CAA05785
	for <xemacs-beta@xemacs.org>; Thu, 11 Sep 1997 02:21:45 -0500 (CDT)
Received: from orion.kurims.kyoto-u.ac.jp (orion.kurims.kyoto-u.ac.jp [130.54.16.5]) by kurims.kurims.kyoto-u.ac.jp (8.8.5/3.4W2) with SMTP id QAA16367; Thu, 11 Sep 1997 16:21:00 +0900 (JST)
Received: (from petersen@localhost) by orion.kurims.kyoto-u.ac.jp (SMI-8.6/3.5Wbeta) id QAA18809; Thu, 11 Sep 1997 16:21:02 +0900
To: Rick Campbell <rickc@lehman.com>
Cc: Jan Vroonhof <vroonhof@math.ethz.ch>, xemacs-beta@xemacs.org
Subject: Re: another [patch] to "vc.el" in xemacs-20.3beta19
References: <9709092137.AA09609@cfdevx1.lehman.com>
X-Emacs: 20.3 "Kyiv" XEmacs  Lucid (beta19) with mule
Mime-Version: 1.0 (generated by SEMI MIME-Edit 0.86 "Naka-Tsurugi")
Content-Type: text/plain; charset=US-ASCII
From: Jens-Ulrik Holger Petersen <petersen@kurims.kyoto-u.ac.jp>
Date: 11 Sep 1997 16:21:01 +0900
In-Reply-To: Rick Campbell's message of "Tue, 09 Sep 1997 17:35:59 -0400"
Message-ID: <lb7mconwo2.fsf@orion.kurims.kyoto-u.ac.jp>
Lines: 60
X-Mailer: Gnus v5.4.65 + SEMI patch (r2.1)/XEmacs 20.3(beta19) - "Kyiv"

>>>>> "Rick" == Rick Campbell <rickc@lehman.com> writes:

    Rick>     Jens-Ulrik Holger Petersen <petersen@kurims.kyoto-u.ac.jp> writes:
    
    >> 1997-09-09  Jens-Ulrik Holger Petersen  <petersen@kurims.kyoto-u.ac.jp>
    >> 
    >> * packages/vc.el (vc-register): Make the default initial
    >> version be `vc-default-init-version', a new variable
    >> defaulting to "0.1".
    
    Jan>     What about: If vc-default-init-version is non-nil this is
    Jan>     taken as the default initial version. If
    Jan>     vc-default-init-version is nil then the default version
    Jan>     of the version control system. Make
    Jan>     vc-default-init-version default to nil.
    
    Jan>     Sounds much cleaner to me than forcing you choice of
    Jan>     initial version in a backwards compatible way.

    Rick> I second this.  I think it's fine to have this variable, but
    Rick> a seasoned RCS user trying out VC for the first time should
    Rick> not be suprised, i. e. the default behavior should not
    Rick> arbitrarily change something like the initial version
    Rick> number.

Ok, fair point.  My thinking was that the RCS default of "1.1" is so
stupid that making the xemacs default be "0.1" was ok, but maybe some
people really do start from version "1.1"!  Perhaps other vc programs
having more meaningful default initial version numbers?


Anyway, since the consensus seems to be for `nil', here is a patch to
do that.

Jens


1997-09-11  Jens-Ulrik Holger Petersen  <petersen@kurims.kyoto-u.ac.jp>

	* packages/vc.el (vc-default-init-version): Make the default value 
	be `nil'.  Improve the docstring.

diff -u lisp/packages/vc.el~ lisp/packages/vc.el
--- lisp/packages/vc.el~	Thu Sep 11 15:57:19 1997
+++ lisp/packages/vc.el		Thu Sep 11 15:57:19 1997
@@ -123,9 +123,11 @@
 (defvar vc-directory-exclusion-list '("SCCS" "RCS" "CVS")
   "*A list of directory names ignored by functions that recursively 
 walk file trees.")
-(defvar vc-default-init-version "0.1"
-  "*A string giving the default version number when a new file is registered.
-Can be overridden by giving a prefix argument to `vc-register'.")
+(defvar vc-default-init-version nil
+  "*A string giving the default version number for the function `vc-register'.
+If `nil' (default), the choice of initial version is left to the
+version control program.  Can be overridden by giving a prefix
+argument to `vc-register'.")
 
 (defconst vc-maximum-comment-ring-size 32
   "Maximum number of saved comments in the comment ring.")

