Improve cusver-check's handling of the version number

* admin/admin.el (cusver-new-version): Set default.
(cusver-check): Improve interactive argument reading.

diff --git a/admin/ChangeLog b/admin/ChangeLog
index 82a0188..7d49218 100644 (file)
--- a/admin/ChangeLog
+++ b/admin/ChangeLog
@@ -1,3 +1,8 @@
+2012-10-07  Glenn Morris  <rgm@gnu.org>
+
+       * admin.el (cusver-new-version): Set default.
+       (cusver-check): Improve interactive argument reading.
+
 2012-10-06  Glenn Morris  <rgm@gnu.org>
 
        * admin.el (cusver-new-version): New variable.

diff --git a/admin/admin.el b/admin/admin.el
index 60a09a1..3e3fbba 100644 (file)
--- a/admin/admin.el
+++ b/admin/admin.el
@@ -442,8 +442,8 @@ If optional OLD is non-nil, also include defvars."
                                     ))
                 "{}" "+"))
 
-; FIXME Calculate default based on running emacs-version.
-(defvar cusver-new-version nil
+(defvar cusver-new-version (format "%s.%s" emacs-major-version
+                                  (1+ emacs-minor-version))
   "Version number that new defcustoms should have.")
 
 ;; TODO do something about renamed variables with aliases to the old name?
@@ -515,7 +515,11 @@ just converting a defvar to a defcustom does not require a :version bump.
 
 Note that a :version tag should also be added if the value of a defcustom
 changes (in a non-trivial way).  This function does not check for that."
-  (interactive "DNew Lisp directory: \nDOld Lisp directory: \nsNew version number: ")
+  (interactive (list (read-directory-name "New Lisp directory: ")
+                    (read-directory-name "Old Lisp directory: ")
+                    (number-to-string
+                     (read-number "New version number: "
+                                  (string-to-number cusver-new-version)))))
   (or (file-directory-p (setq newdir (expand-file-name newdir)))
       (error "Directory `%s' not found" newdir))
   (or (file-directory-p (setq olddir (expand-file-name olddir)))