]> code.delx.au - gnu-emacs/blobdiff - lisp/vc-hooks.el
* files.el (auto-save-file-name-transforms): Put remote files in
[gnu-emacs] / lisp / vc-hooks.el
index dd19ac4a0d92cca369e123207cd81f40fb34b3c5..8b2eecf4b2633da95fcff8779d4adcd1b729626e 100644 (file)
@@ -1,9 +1,11 @@
 ;;; vc-hooks.el --- resident support for version-control
 
-;; Copyright (C) 1992, 1993, 1994 Free Software Foundation, Inc.
+;; Copyright (C) 1992,93,94,95,96,98,99,2000  Free Software Foundation, Inc.
 
-;; Author: Eric S. Raymond <esr@snark.thyrsus.com>
-;; Version: 5.3
+;; Author:     FSF (see vc.el for full credits)
+;; Maintainer: Andre Spiegel <spiegel@gnu.org>
+
+;; $Id: vc-hooks.el,v 1.134 2001/09/24 16:36:14 monnier Exp $
 
 ;; This file is part of GNU Emacs.
 
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
-;; See the commentary of vc.el.
+;; This is the always-loaded portion of VC.  It takes care of
+;; VC-related activities that are done when you visit a file, so that
+;; vc.el itself is loaded only when you use a VC command.  See the
+;; commentary of vc.el.
 
 ;;; Code:
 
-(defvar vc-master-templates
-  '(("%sRCS/%s,v" . RCS) ("%s%s,v" . RCS) ("%sRCS/%s" . RCS)
-    ("%sSCCS/s.%s" . SCCS) ("%ss.%s". SCCS))
-  "*Where to look for version-control master files.
-The first pair corresponding to a given back end is used as a template
-when creating new masters.")
-
-(defvar vc-make-backup-files nil
+(eval-when-compile
+  (require 'cl))
+
+;; Customization Variables (the rest is in vc.el)
+
+(defvar vc-ignore-vc-files nil "Obsolete -- use `vc-handled-backends'.")
+(defvar vc-master-templates () "Obsolete -- use vc-BACKEND-master-templates.")
+(defvar vc-header-alist () "Obsolete -- use vc-BACKEND-header.")
+
+(defcustom vc-handled-backends '(RCS CVS SCCS)
+  "*List of version control backends for which VC will be used.
+Entries in this list will be tried in order to determine whether a
+file is under that sort of version control.
+Removing an entry from the list prevents VC from being activated
+when visiting a file managed by that backend.
+An empty list disables VC altogether."
+  :type '(repeat symbol)
+  :version "21.1"
+  :group 'vc)
+
+(defcustom vc-path
+  (if (file-directory-p "/usr/sccs")
+      '("/usr/sccs")
+    nil)
+  "*List of extra directories to search for version control commands."
+  :type '(repeat directory)
+  :group 'vc)
+
+(defcustom vc-make-backup-files nil
   "*If non-nil, backups of registered files are made as with other files.
-If nil (the default), files covered by version control don't get backups.")
-
-(defvar vc-rcs-status t
-  "*If non-nil, revision and locks on RCS working file displayed in modeline.
-Otherwise, not displayed.")
+If nil (the default), files covered by version control don't get backups."
+  :type 'boolean
+  :group 'vc)
+
+(defcustom vc-follow-symlinks 'ask
+  "*What to do if visiting a symbolic link to a file under version control.
+Editing such a file through the link bypasses the version control system,
+which is dangerous and probably not what you want.
+
+If this variable is t, VC follows the link and visits the real file,
+telling you about it in the echo area.  If it is `ask', VC asks for
+confirmation whether it should follow the link.  If nil, the link is
+visited and a warning displayed."
+  :type '(choice (const :tag "Ask for confirmation" ask)
+                (const :tag "Visit link and warn" nil)
+                (const :tag "Follow link" t))
+  :group 'vc)
+
+(defcustom vc-display-status t
+  "*If non-nil, display revision number and lock status in modeline.
+Otherwise, not displayed."
+  :type 'boolean
+  :group 'vc)
+
+
+(defcustom vc-consult-headers t
+  "*If non-nil, identify work files by searching for version headers."
+  :type 'boolean
+  :group 'vc)
+
+(defcustom vc-keep-workfiles t
+  "*If non-nil, don't delete working files after registering changes.
+If the back-end is CVS, workfiles are always kept, regardless of the
+value of this flag."
+  :type 'boolean
+  :group 'vc)
+
+(defcustom vc-mistrust-permissions nil
+  "*If non-nil, don't assume permissions/ownership track version-control status.
+If nil, do rely on the permissions.
+See also variable `vc-consult-headers'."
+  :type 'boolean
+  :group 'vc)
+
+(defun vc-mistrust-permissions (file)
+  "Internal access function to variable `vc-mistrust-permissions' for FILE."
+  (or (eq vc-mistrust-permissions 't)
+      (and vc-mistrust-permissions
+          (funcall vc-mistrust-permissions
+                   (vc-backend-subdirectory-name file)))))
 
 ;; Tell Emacs about this new kind of minor mode
-(if (not (assoc 'vc-mode minor-mode-alist))
-    (setq minor-mode-alist (cons '(vc-mode vc-mode)
-                                minor-mode-alist)))
+(add-to-list 'minor-mode-alist '(vc-mode vc-mode))
 
 (make-variable-buffer-local 'vc-mode)
 (put 'vc-mode 'permanent-local t)
 
-;; We need a notion of per-file properties because the version
-;; control state of a file is expensive to derive --- we don't
-;; want to recompute it even on every find.
+(defun vc-mode (&optional arg)
+  ;; Dummy function for C-h m
+  "Version Control minor mode.
+This minor mode is automatically activated whenever you visit a file under
+control of one of the revision control systems in `vc-handled-backends'.
+VC commands are globally reachable under the prefix `\\[vc-prefix-map]':
+\\{vc-prefix-map}")
 
 (defmacro vc-error-occurred (&rest body)
-  (list 'condition-case nil (cons 'progn (append body '(nil))) '(error t)))
+  `(condition-case nil (progn ,@body nil) (error t)))
 
-(defvar vc-file-prop-obarray [0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0]
+;; We need a notion of per-file properties because the version
+;; control state of a file is expensive to derive --- we compute
+;; them when the file is initially found, keep them up to date
+;; during any subsequent VC operations, and forget them when
+;; the buffer is killed.
+
+(defvar vc-file-prop-obarray (make-vector 17 0)
   "Obarray for per-file properties.")
 
+(defvar vc-touched-properties nil)
+
 (defun vc-file-setprop (file property value)
-  ;; set per-file property
+  "Set per-file VC PROPERTY for FILE to VALUE."
+  (if (and vc-touched-properties
+          (not (memq property vc-touched-properties)))
+      (setq vc-touched-properties (append (list property)
+                                         vc-touched-properties)))
   (put (intern file vc-file-prop-obarray) property value))
 
 (defun vc-file-getprop (file property)
-  ;; get per-file property
+  "Get per-file VC PROPERTY for FILE."
   (get (intern file vc-file-prop-obarray) property))
 
-;;; actual version-control code starts here
+(defun vc-file-clearprops (file)
+  "Clear all VC properties of FILE."
+  (setplist (intern file vc-file-prop-obarray) nil))
+
+\f
+;; We keep properties on each symbol naming a backend as follows:
+;;  * `vc-functions': an alist mapping vc-FUNCTION to vc-BACKEND-FUNCTION.
+
+(defun vc-make-backend-sym (backend sym)
+  "Return BACKEND-specific version of VC symbol SYM."
+  (intern (concat "vc-" (downcase (symbol-name backend))
+                 "-" (symbol-name sym))))
+
+(defun vc-find-backend-function (backend fun)
+  "Return BACKEND-specific implementation of FUN.
+If there is no such implementation, return the default implementation; 
+if that doesn't exist either, return nil."
+  (let ((f (vc-make-backend-sym backend fun)))
+    (if (fboundp f) f
+      ;; Load vc-BACKEND.el if needed.
+      (require (intern (concat "vc-" (downcase (symbol-name backend)))))
+      (if (fboundp f) f
+       (let ((def (vc-make-backend-sym 'default fun)))
+         (if (fboundp def) (cons def backend) nil))))))
+
+(defun vc-call-backend (backend function-name &rest args)
+  "Call for BACKEND the implementation of FUNCTION-NAME with the given ARGS.
+Calls
+
+    (apply 'vc-BACKEND-FUN ARGS)
+
+if vc-BACKEND-FUN exists (after trying to find it in vc-BACKEND.el)
+and else calls
+
+    (apply 'vc-default-FUN BACKEND ARGS)
+
+It is usually called via the `vc-call' macro."
+  (let ((f (cdr (assoc function-name (get backend 'vc-functions)))))
+    (unless f
+      (setq f (vc-find-backend-function backend function-name))
+      (put backend 'vc-functions (cons (cons function-name f)
+                                      (get backend 'vc-functions))))
+    (if (consp f)
+       (apply (car f) (cdr f) args)
+      (apply f args))))
+
+(defmacro vc-call (fun file &rest args)
+  ;; BEWARE!! `file' is evaluated twice!!
+  `(vc-call-backend (vc-backend ,file) ',fun ,file ,@args))
+
+\f
+(defsubst vc-parse-buffer (pattern i)
+  "Find PATTERN in the current buffer and return its Ith submatch."
+  (goto-char (point-min))
+  (if (re-search-forward pattern nil t)
+      (match-string i)))
+
+(defun vc-insert-file (file &optional limit blocksize)
+  "Insert the contents of FILE into the current buffer.
+
+Optional argument LIMIT is a regexp.  If present, the file is inserted
+in chunks of size BLOCKSIZE (default 8 kByte), until the first
+occurrence of LIMIT is found.  The function returns non-nil if FILE 
+exists and its contents were successfully inserted."
+  (erase-buffer)
+  (when (file-exists-p file)
+    (if (not limit)
+        (insert-file-contents file)
+      (if (not blocksize) (setq blocksize 8192))
+      (let ((filepos 0))
+        (while
+           (and (< 0 (cadr (insert-file-contents
+                            file nil filepos (incf filepos blocksize))))
+                (progn (beginning-of-line)
+                       (not (re-search-forward limit nil 'move)))))))
+    (set-buffer-modified-p nil)
+    t))
+
+;; Access functions to file properties
+;; (Properties should be _set_ using vc-file-setprop, but
+;; _retrieved_ only through these functions, which decide
+;; if the property is already known or not. A property should
+;; only be retrieved by vc-file-getprop if there is no
+;; access function.)
+
+;; properties indicating the backend being used for FILE
 
 (defun vc-registered (file)
-  (let (handler handlers)
+  "Return non-nil if FILE is registered in a version control system.
+
+This function performs the check each time it is called.  To rely
+on the result of a previous call, use `vc-backend' instead.  If the
+file was previously registered under a certain backend, then that
+backend is tried first."
+  (let (handler)
     (if (boundp 'file-name-handler-alist)
-       (setq handler (find-file-name-handler file 'vc-registered)))
+       (setq handler (find-file-name-handler file 'vc-registered)))
     (if handler
-       (funcall handler 'vc-registered file)
-      ;; Search for a master corresponding to the given file
-      (let ((dirname (or (file-name-directory file) ""))
-           (basename (file-name-nondirectory file)))
-       (catch 'found
+        ;; handler should set vc-backend and return t if registered
+       (funcall handler 'vc-registered file)
+      ;; There is no file name handler.
+      ;; Try vc-BACKEND-registered for each handled BACKEND.
+      (catch 'found
+       (let ((backend (vc-file-getprop file 'vc-backend)))
          (mapcar
-          (function (lambda (s)
-             (let ((trial (format (car s) dirname basename)))
-               (if (and (file-exists-p trial)
-                        ;; Make sure the file we found with name
-                        ;; TRIAL is not the source file itself.
-                        ;; That can happen with RCS-style names
-                        ;; if the file name is truncated
-                        ;; (e.g. to 14 chars).  See if either
-                        ;; directory or attributes differ.
-                        (or (not (string= dirname
-                                          (file-name-directory trial)))
-                            (not (equal
-                                  (file-attributes file)
-                                  (file-attributes trial)))))
-                   (throw 'found (cons trial (cdr s)))))))
-          vc-master-templates)
-         nil)))))
+          (lambda (b)
+            (and (vc-call-backend b 'registered file)
+                 (vc-file-setprop file 'vc-backend b)
+                 (throw 'found t)))
+          (if (or (not backend) (eq backend 'none)) 
+              vc-handled-backends
+            (cons backend vc-handled-backends))))
+        ;; File is not registered.
+        (vc-file-setprop file 'vc-backend 'none)
+        nil))))
+
+(defun vc-backend (file)
+  "Return the version control type of FILE, nil if it is not registered."
+  ;; `file' can be nil in several places (typically due to the use of
+  ;; code like (vc-backend (buffer-file-name))).
+  (when (stringp file)
+    (let ((property (vc-file-getprop file 'vc-backend)))
+      ;; Note that internally, Emacs remembers unregistered
+      ;; files by setting the property to `none'.
+      (cond ((eq property 'none) nil)
+           (property)
+           ;; vc-registered sets the vc-backend property
+           (t (if (vc-registered file)
+                  (vc-file-getprop file 'vc-backend)
+                nil))))))
+
+(defun vc-backend-subdirectory-name (file)
+  "Return where the master and lock FILEs for the current directory are kept."
+  (symbol-name (vc-backend file)))
 
 (defun vc-name (file)
-  "Return the master name of a file, nil if it is not registered."
+  "Return the master name of FILE.
+If the file is not registered, or the master name is not known, return nil."
+  ;; TODO: This should ultimately become obsolete, at least up here
+  ;; in vc-hooks.
   (or (vc-file-getprop file 'vc-name)
-      (let ((name-and-type (vc-registered file)))
-       (if name-and-type
-           (progn
-             (vc-file-setprop file 'vc-backend (cdr name-and-type))
-             (vc-file-setprop file 'vc-name (car name-and-type)))))))
-
-(defun vc-backend-deduce (file)
-  "Return the version-control type of a file, nil if it is not registered."
-  (and file
-       (or (vc-file-getprop file 'vc-backend)
-          (let ((name-and-type (vc-registered file)))
-            (if name-and-type
-                (progn
-                  (vc-file-setprop file 'vc-name (car name-and-type))
-                  (vc-file-setprop file 'vc-backend (cdr name-and-type))))))))
-
-(defun vc-toggle-read-only ()
+      ;; force computation of the property by calling
+      ;; vc-BACKEND-registered explicitly
+      (if (and (vc-backend file)
+              (vc-call-backend (vc-backend file) 'registered file))
+         (vc-file-getprop file 'vc-name))))
+
+(defun vc-checkout-model (file)
+  "Indicate how FILE is checked out.
+
+Possible values:
+
+  'implicit   File is always writeable, and checked out `implicitly'
+              when the user saves the first changes to the file.
+
+  'locking    File is read-only if up-to-date; user must type
+              \\[vc-toggle-read-only] before editing.  Strict locking
+              is assumed.
+
+  'announce   File is read-only if up-to-date; user must type
+              \\[vc-toggle-read-only] before editing.  But other users
+              may be editing at the same time."
+  (or (vc-file-getprop file 'vc-checkout-model)
+      (vc-file-setprop file 'vc-checkout-model
+                       (vc-call checkout-model file))))
+
+(defun vc-user-login-name (&optional uid)
+  "Return the name under which the user is logged in, as a string.
+\(With optional argument UID, return the name of that user.)
+This function does the same as function `user-login-name', but unlike
+that, it never returns nil.  If a UID cannot be resolved, that
+UID is returned as a string."
+  (or (user-login-name uid)
+      (number-to-string (or uid (user-uid)))))
+
+(defun vc-state (file)
+  "Return the version control state of FILE.
+
+The value returned is one of:
+
+  'up-to-date        The working file is unmodified with respect to the
+                     latest version on the current branch, and not locked.
+
+  'edited            The working file has been edited by the user.  If
+                     locking is used for the file, this state means that
+                     the current version is locked by the calling user.
+
+  USER               The current version of the working file is locked by
+                     some other USER (a string).
+            
+  'needs-patch       The file has not been edited by the user, but there is
+                     a more recent version on the current branch stored
+                     in the master file.
+
+  'needs-merge       The file has been edited by the user, and there is also
+                     a more recent version on the current branch stored in
+                     the master file.  This state can only occur if locking
+                     is not used for the file.
+
+  'unlocked-changes  The current version of the working file is not locked,
+                     but the working file has been changed with respect
+                     to that version.  This state can only occur for files
+                     with locking; it represents an erroneous condition that
+                     should be resolved by the user (vc-next-action will
+                     prompt the user to do it)."
+  (or (vc-file-getprop file 'vc-state)
+      (vc-file-setprop file 'vc-state
+                      (vc-call state-heuristic file))))
+
+(defsubst vc-up-to-date-p (file)
+  "Convenience function that checks whether `vc-state' of FILE is `up-to-date'."
+  (eq (vc-state file) 'up-to-date))
+
+(defun vc-default-state-heuristic (backend file)
+  "Default implementation of vc-state-heuristic.
+It simply calls the real state computation function `vc-BACKEND-state'
+and does not employ any heuristic at all."
+   (vc-call-backend backend 'state file))
+
+(defun vc-workfile-version (file)
+  "Return version level of the current workfile FILE."
+  (or (vc-file-getprop file 'vc-workfile-version)
+      (vc-file-setprop file 'vc-workfile-version
+                       (vc-call workfile-version file))))
+
+;;; actual version-control code starts here
+
+(defun vc-default-registered (backend file)
+  "Check if FILE is registered in BACKEND using vc-BACKEND-master-templates."
+  (let ((sym (vc-make-backend-sym backend 'master-templates)))
+    (unless (get backend 'vc-templates-grabbed)
+      (put backend 'vc-templates-grabbed t)
+      (set sym (append (delq nil
+                            (mapcar
+                             (lambda (template)
+                               (and (consp template)
+                                    (eq (cdr template) backend)
+                                    (car template)))
+                             vc-master-templates))
+                      (symbol-value sym))))
+    (let ((result (vc-check-master-templates file (symbol-value sym))))
+      (if (stringp result)
+         (vc-file-setprop file 'vc-name result)
+       nil))))                         ; Not registered
+
+(defun vc-possible-master (s dirname basename)
+  (cond
+   ((stringp s) (format s dirname basename))
+   ((functionp s)
+    ;; The template is a function to invoke.  If the
+    ;; function returns non-nil, that means it has found a
+    ;; master.  For backward compatibility, we also handle
+    ;; the case that the function throws a 'found atom
+    ;; and a pair (cons MASTER-FILE BACKEND).
+    (let ((result (catch 'found (funcall s dirname basename))))
+      (if (consp result) (car result) result)))))
+
+(defun vc-check-master-templates (file templates)
+  "Return non-nil if there is a master corresponding to FILE,
+according to any of the elements in TEMPLATES.
+
+TEMPLATES is a list of strings or functions.  If an element is a
+string, it must be a control string as required by `format', with two
+string placeholders, such as \"%sRCS/%s,v\".  The directory part of
+FILE is substituted for the first placeholder, the basename of FILE
+for the second.  If a file with the resulting name exists, it is taken
+as the master of FILE, and returned.
+
+If an element of TEMPLATES is a function, it is called with the
+directory part and the basename of FILE as arguments.  It should
+return non-nil if it finds a master; that value is then returned by
+this function."
+  (let ((dirname (or (file-name-directory file) ""))
+        (basename (file-name-nondirectory file)))
+    (catch 'found
+      (mapcar
+       (lambda (s)
+        (let ((trial (vc-possible-master s dirname basename)))
+          (if (and trial (file-exists-p trial)
+                   ;; Make sure the file we found with name
+                   ;; TRIAL is not the source file itself.
+                   ;; That can happen with RCS-style names if
+                   ;; the file name is truncated (e.g. to 14
+                   ;; chars).  See if either directory or
+                   ;; attributes differ.
+                   (or (not (string= dirname
+                                     (file-name-directory trial)))
+                       (not (equal (file-attributes file)
+                                   (file-attributes trial)))))
+              (throw 'found trial))))
+       templates))))
+
+(defun vc-toggle-read-only (&optional verbose)
   "Change read-only status of current buffer, perhaps via version control.
 If the buffer is visiting a file registered with version control,
 then check the file in or out.  Otherwise, just change the read-only flag
-of the buffer."
-  (interactive)
-  (if (vc-backend-deduce (buffer-file-name))
-      (vc-next-action nil)
+of the buffer.
+With prefix argument, ask for version number to check in or check out.
+Check-out of a specified version number does not lock the file;
+to do that, use this command a second time with no argument."
+  (interactive "P")
+  (if (or (and (boundp 'vc-dired-mode) vc-dired-mode)
+         ;; use boundp because vc.el might not be loaded
+         (vc-backend (buffer-file-name)))
+      (vc-next-action verbose)
     (toggle-read-only)))
 (define-key global-map "\C-x\C-q" 'vc-toggle-read-only)
 
-(defun vc-mode-line (file &optional label)
+(defun vc-default-make-version-backups-p (backend file)
+  "Return non-nil if unmodified repository versions should 
+be backed up locally.  The default is to switch off this feature."
+  nil)
+
+(defun vc-version-backup-file-name (file &optional rev manual regexp)
+  "Return a backup file name for REV or the current version of FILE.
+If MANUAL is non-nil it means that a name for backups created by
+the user should be returned; if REGEXP is non-nil that means to return
+a regexp for matching all such backup files, regardless of the version."
+  (if regexp
+      (concat (regexp-quote (file-name-nondirectory file))
+              "\\.~[0-9.]+" (unless manual "\\.") "~")
+    (expand-file-name (concat (file-name-nondirectory file) 
+                              ".~" (or rev (vc-workfile-version file)) 
+                              (unless manual ".") "~")
+                      (file-name-directory file))))
+
+(defun vc-delete-automatic-version-backups (file)
+  "Delete all existing automatic version backups for FILE."
+  (condition-case nil
+      (mapcar
+       'delete-file
+       (directory-files (or (file-name-directory file) default-directory) t
+                       (vc-version-backup-file-name file nil nil t)))
+    ;; Don't fail when the directory doesn't exist.
+    (file-error nil)))
+
+(defun vc-make-version-backup (file)
+  "Make a backup copy of FILE, which is assumed in sync with the repository.
+Before doing that, check if there are any old backups and get rid of them."
+  (unless (and (fboundp 'msdos-long-file-names)
+               (not (msdos-long-file-names)))
+    (vc-delete-automatic-version-backups file)
+    (copy-file file (vc-version-backup-file-name file)
+               nil 'keep-date)))
+
+(defun vc-before-save ()
+  "Function to be called by `basic-save-buffer' (in files.el)."
+  ;; If the file on disk is still in sync with the repository,
+  ;; and version backups should be made, copy the file to
+  ;; another name.  This enables local diffs and local reverting.
+  (let ((file (buffer-file-name)))
+    (and (vc-backend file)
+        (vc-up-to-date-p file)
+        (eq (vc-checkout-model file) 'implicit)
+        (vc-call make-version-backups-p file)
+         (vc-make-version-backup file))))
+
+(defun vc-after-save ()
+  "Function to be called by `basic-save-buffer' (in files.el)."
+  ;; If the file in the current buffer is under version control,
+  ;; up-to-date, and locking is not used for the file, set
+  ;; the state to 'edited and redisplay the mode line.
+  (let ((file (buffer-file-name)))
+    (and (vc-backend file)
+        (or (and (equal (vc-file-getprop file 'vc-checkout-time)
+                        (nth 5 (file-attributes file)))
+                 ;; File has been saved in the same second in which
+                 ;; it was checked out.  Clear the checkout-time
+                 ;; to avoid confusion.
+                 (vc-file-setprop file 'vc-checkout-time nil))
+            t)
+         (vc-up-to-date-p file)
+         (eq (vc-checkout-model file) 'implicit)
+         (vc-file-setprop file 'vc-state 'edited)
+        (vc-mode-line file)
+        (if (featurep 'vc)
+            ;; If VC is not loaded, then there can't be
+            ;; any VC Dired buffer to synchronize.
+            (vc-dired-resynch-file file)))))
+
+(defun vc-mode-line (file)
   "Set `vc-mode' to display type of version control for FILE.
 The value is set in the current buffer, which should be the buffer
 visiting FILE."
-  (interactive (list buffer-file-name nil))
-  (if file
-      (let ((vc-type (vc-backend-deduce file)))
-       (setq vc-mode
-             (and vc-type
-                  (concat " " (or label (symbol-name vc-type))
-                          (if (and vc-rcs-status (eq vc-type 'RCS))
-                              (vc-rcs-status file)))))
-       ;; Even root shouldn't modify a registered file without locking it first.
-       (and vc-type
-            (not buffer-read-only)
-            (zerop (user-uid))
-            (require 'vc)
-            (not (string-equal (user-login-name) (vc-locking-user file)))
-            (setq buffer-read-only t))
-       (and (null vc-type)
-            (file-symlink-p file)
-            (let ((link-type (vc-backend-deduce (file-symlink-p file))))
-              (if link-type
-                  (message "Warning: symbolic link to %s-controlled source file"
-                           link-type))))
-       (force-mode-line-update)
-       ;;(set-buffer-modified-p (buffer-modified-p))  ;;use this if Emacs 18
-       vc-type)))
-
-(defun vc-rcs-status (file)
-  ;; Return string for placement in modeline by `vc-mode-line'.
-  ;; If FILE is not registered under RCS, return nil.
-  ;; If FILE is registered but not locked, return " REV" if there is a head
-  ;; revision and " @@" otherwise.
-  ;; If FILE is locked then return all locks in a string of the
-  ;; form " LOCKER1:REV1 LOCKER2:REV2 ...", where "LOCKERi:" is empty if you
-  ;; are the locker, and otherwise is the name of the locker followed by ":".
-
-  ;; Algorithm: 
-
-  ;; 1. Check for master file corresponding to FILE being visited.
-  ;; 
-  ;; 2. Insert the first few characters of the master file into a work
-  ;; buffer.
-  ;;  
-  ;; 3. Search work buffer for "locks...;" phrase; if not found, then
-  ;; keep inserting more characters until the phrase is found.
-  ;; 
-  ;; 4. Extract the locks, and remove control characters
-  ;; separating them, like newlines; the string " user1:revision1
-  ;; user2:revision2 ..." is returned.
-
-  ;; Limitations:
-
-  ;; The output doesn't show which version you are actually looking at.
-  ;; The modeline can get quite cluttered when there are multiple locks.
-  ;; The head revision is probably not what you want if you've used `rcs -b'.
-
-  (let ((master (vc-name file))
-       found)
-
-    ;; If master file exists, then parse its contents, otherwise we return the 
-    ;; nil value of this if form.
-    (if master
-        (save-excursion
-
-          ;; Create work buffer.
-          (set-buffer (get-buffer-create " *vc-rcs-status*"))
-          (setq buffer-read-only nil
-                default-directory (file-name-directory master))
-          (erase-buffer)
-
-          ;; Check if we have enough of the header.
-         ;; If not, then keep including more.
-          (while
-             (not (or found
-                      (let ((s (buffer-size)))
-                        (goto-char (1+ s))
-                        (zerop (car (cdr (insert-file-contents
-                                          master nil s (+ s 8192))))))))
-           (beginning-of-line)
-           (setq found (re-search-forward "^locks\\([^;]*\\);" nil t)))
-
-          (if found
-             ;; Clean control characters and self-locks from text.
-             (let* ((lock-pattern
-                     (concat "[ \b\t\n\v\f\r]+\\("
-                             (regexp-quote (user-login-name))
-                             ":\\)?"))
-                    (locks
-                     (save-restriction
-                       (narrow-to-region (match-beginning 1) (match-end 1))
-                       (goto-char (point-min))
-                       (while (re-search-forward lock-pattern nil t)
-                         (replace-match (if (eobp) "" ":") t t))
-                       (buffer-string)))
-                    (status
-                     (if (not (string-equal locks ""))
-                         locks
-                       (goto-char (point-min))
-                       (if (looking-at "head[ \b\t\n\v\f\r]+\\([.0-9]+\\)")
-                           (concat "-" (buffer-substring (match-beginning 1)
-                                                         (match-end 1)))
-                         " @@"))))
-               ;; Clean work buffer.
-               (erase-buffer)
-               (set-buffer-modified-p nil)
-               status))))))
-
-;;; install a call to the above as a find-file hook
+  (interactive (list buffer-file-name))
+  (unless (not (vc-backend file))
+    (setq vc-mode (concat " " (if vc-display-status
+                                 (vc-call mode-line-string file)
+                               (symbol-name (vc-backend file)))))
+    ;; If the file is locked by some other user, make
+    ;; the buffer read-only.  Like this, even root
+    ;; cannot modify a file that someone else has locked.
+    (and (equal file (buffer-file-name))
+         (stringp (vc-state file))
+        (setq buffer-read-only t))
+    ;; If the user is root, and the file is not owner-writable,
+    ;; then pretend that we can't write it
+    ;; even though we can (because root can write anything).
+    ;; This way, even root cannot modify a file that isn't locked.
+    (and (equal file (buffer-file-name))
+        (not buffer-read-only)
+        (zerop (user-real-uid))
+        (zerop (logand (file-modes (buffer-file-name)) 128))
+        (setq buffer-read-only t)))
+  (force-mode-line-update)
+  (vc-backend file))
+
+(defun vc-default-mode-line-string (backend file)
+  "Return string for placement in modeline by `vc-mode-line' for FILE.
+Format:
+
+  \"BACKEND-REV\"        if the file is up-to-date
+  \"BACKEND:REV\"        if the file is edited (or locked by the calling user)
+  \"BACKEND:LOCKER:REV\" if the file is locked by somebody else
+
+This function assumes that the file is registered."
+  (setq backend (symbol-name backend))
+  (let ((state   (vc-state file))
+       (rev     (vc-workfile-version file)))
+    (cond ((or (eq state 'up-to-date)
+              (eq state 'needs-patch))
+          (concat backend "-" rev))
+          ((stringp state)
+          (concat backend ":" state ":" rev))
+          (t
+           ;; Not just for the 'edited state, but also a fallback
+           ;; for all other states.  Think about different symbols
+           ;; for 'needs-patch and 'needs-merge.
+           (concat backend ":" rev)))))
+
+(defun vc-follow-link ()
+  "If current buffer visits a symbolic link, visit the real file.
+If the real file is already visited in another buffer, make that buffer
+current, and kill the buffer that visits the link."
+  (let* ((truename (abbreviate-file-name (file-chase-links buffer-file-name)))
+         (true-buffer (find-buffer-visiting truename))
+        (this-buffer (current-buffer)))
+    (if (eq true-buffer this-buffer)
+       (progn
+         (kill-buffer this-buffer)
+         ;; In principle, we could do something like set-visited-file-name.
+         ;; However, it can't be exactly the same as set-visited-file-name.
+         ;; I'm not going to work out the details right now. -- rms.
+         (set-buffer (find-file-noselect truename)))
+      (set-buffer true-buffer)
+      (kill-buffer this-buffer))))
+
 (defun vc-find-file-hook ()
+  "Function for `find-file-hooks' activating VC mode if appropriate."
   ;; Recompute whether file is version controlled,
   ;; if user has killed the buffer and revisited.
-  (if buffer-file-name
-      (vc-file-setprop buffer-file-name 'vc-backend nil))
-  (if (and (vc-mode-line buffer-file-name) (not vc-make-backup-files))
-      (progn
-       ;; Use this variable, not make-backup-files,
-       ;; because this is for things that depend on the file name.
-       (make-local-variable 'backup-inhibited)
-       (setq backup-inhibited t))))
+  (when buffer-file-name
+    (vc-file-clearprops buffer-file-name)
+    (cond
+     ((vc-backend buffer-file-name)
+      (vc-mode-line buffer-file-name)
+      (cond ((not vc-make-backup-files)
+            ;; Use this variable, not make-backup-files,
+            ;; because this is for things that depend on the file name.
+            (make-local-variable 'backup-inhibited)
+            (setq backup-inhibited t))))
+     ((let* ((link (file-symlink-p buffer-file-name))
+            (link-type (and link (vc-backend (file-chase-links link)))))
+       (if link-type
+            (cond ((eq vc-follow-symlinks nil)
+                   (message
+        "Warning: symbolic link to %s-controlled source file" link-type))
+                  ((or (not (eq vc-follow-symlinks 'ask))
+                      ;; If we already visited this file by following
+                      ;; the link, don't ask again if we try to visit
+                      ;; it again.  GUD does that, and repeated questions
+                      ;; are painful.
+                      (get-file-buffer
+                       (abbreviate-file-name
+                         (file-chase-links buffer-file-name))))
+                      
+                  (vc-follow-link)
+                  (message "Followed link to %s" buffer-file-name)
+                  (vc-find-file-hook))
+                  (t
+                   (if (yes-or-no-p (format
+        "Symbolic link to %s-controlled source file; follow link? " link-type))
+                       (progn (vc-follow-link)
+                              (message "Followed link to %s" buffer-file-name)
+                              (vc-find-file-hook))
+                     (message
+        "Warning: editing through the link bypasses version control")
+                     )))))))))
 
 (add-hook 'find-file-hooks 'vc-find-file-hook)
 
-;;; more hooks, this time for file-not-found
+;; more hooks, this time for file-not-found
 (defun vc-file-not-found-hook ()
-  "When file is not found, try to check it out from RCS or SCCS.
-Returns t if checkout was successful, nil otherwise."
-  (if (vc-backend-deduce buffer-file-name)
-      (progn
-       (require 'vc)
-       (not (vc-error-occurred (vc-checkout buffer-file-name))))))
+  "When file is not found, try to check it out from version control.
+Returns t if checkout was successful, nil otherwise.
+Used in `find-file-not-found-hooks'."
+  ;; When a file does not exist, ignore cached info about it
+  ;; from a previous visit.
+  (vc-file-clearprops buffer-file-name)
+  (if (and (vc-backend buffer-file-name)
+          (yes-or-no-p
+           (format "File %s was lost; check out from version control? "
+                   (file-name-nondirectory buffer-file-name))))
+    (save-excursion
+      (require 'vc)
+      (setq default-directory (file-name-directory buffer-file-name))
+      (not (vc-error-occurred (vc-checkout buffer-file-name))))))
 
 (add-hook 'find-file-not-found-hooks 'vc-file-not-found-hook)
 
-;;; Now arrange for bindings and autoloading of the main package.
-;;; Bindings for this have to go in the global map, as we'll often
-;;; want to call them from random buffers.
-
-(setq vc-prefix-map (lookup-key global-map "\C-xv"))
-(if (not (keymapp vc-prefix-map))
-    (progn
-      (setq vc-prefix-map (make-sparse-keymap))
-      (define-key global-map "\C-xv" vc-prefix-map)
-      (define-key vc-prefix-map "a" 'vc-update-change-log)
-      (define-key vc-prefix-map "c" 'vc-cancel-version)
-      (define-key vc-prefix-map "d" 'vc-directory)
-      (define-key vc-prefix-map "h" 'vc-insert-headers)
-      (define-key vc-prefix-map "i" 'vc-register)
-      (define-key vc-prefix-map "l" 'vc-print-log)
-      (define-key vc-prefix-map "r" 'vc-retrieve-snapshot)
-      (define-key vc-prefix-map "s" 'vc-create-snapshot)
-      (define-key vc-prefix-map "u" 'vc-revert-buffer)
-      (define-key vc-prefix-map "v" 'vc-next-action)
-      (define-key vc-prefix-map "=" 'vc-diff)
-      (define-key vc-prefix-map "~" 'vc-version-other-window)
-      ))
+(defun vc-kill-buffer-hook ()
+  "Discard VC info about a file when we kill its buffer."
+  (if (buffer-file-name)
+      (vc-file-clearprops (buffer-file-name))))
+
+;; ??? DL: why is this not done?
+;;;(add-hook 'kill-buffer-hook 'vc-kill-buffer-hook)
+
+;; Now arrange for (autoloaded) bindings of the main package.
+;; Bindings for this have to go in the global map, as we'll often
+;; want to call them from random buffers.
+
+;; Autoloading works fine, but it prevents shortcuts from appearing
+;; in the menu because they don't exist yet when the menu is built.
+;; (autoload 'vc-prefix-map "vc" nil nil 'keymap)
+(defvar vc-prefix-map
+  (let ((map (make-sparse-keymap)))
+    (define-key map "a" 'vc-update-change-log)
+    (define-key map "b" 'vc-switch-backend)
+    (define-key map "c" 'vc-cancel-version)
+    (define-key map "d" 'vc-directory)
+    (define-key map "g" 'vc-annotate)
+    (define-key map "h" 'vc-insert-headers)
+    (define-key map "i" 'vc-register)
+    (define-key map "l" 'vc-print-log)
+    (define-key map "m" 'vc-merge)
+    (define-key map "r" 'vc-retrieve-snapshot)
+    (define-key map "s" 'vc-create-snapshot)
+    (define-key map "u" 'vc-revert-buffer)
+    (define-key map "v" 'vc-next-action)
+    (define-key map "=" 'vc-diff)
+    (define-key map "~" 'vc-version-other-window)
+    map))
+(fset 'vc-prefix-map vc-prefix-map)
+(define-key global-map "\C-xv" 'vc-prefix-map)
+
+(if (not (boundp 'vc-menu-map))
+    ;; Don't do the menu bindings if menu-bar.el wasn't loaded to defvar
+    ;; vc-menu-map.
+    ()
+  ;;(define-key vc-menu-map [show-files]
+  ;;  '("Show Files under VC" . (vc-directory t)))
+  (define-key vc-menu-map [vc-retrieve-snapshot]
+    '("Retrieve Snapshot" . vc-retrieve-snapshot))
+  (define-key vc-menu-map [vc-create-snapshot]
+    '("Create Snapshot" . vc-create-snapshot))
+  (define-key vc-menu-map [vc-directory] '("VC Directory Listing" . vc-directory))
+  (define-key vc-menu-map [separator1] '("----"))
+  (define-key vc-menu-map [vc-annotate] '("Annotate" . vc-annotate))
+  (define-key vc-menu-map [vc-rename-file] '("Rename File" . vc-rename-file))
+  (define-key vc-menu-map [vc-version-other-window]
+    '("Show Other Version" . vc-version-other-window))
+  (define-key vc-menu-map [vc-diff] '("Compare with Last Version" . vc-diff))
+  (define-key vc-menu-map [vc-update-change-log]
+    '("Update ChangeLog" . vc-update-change-log))
+  (define-key vc-menu-map [vc-print-log] '("Show History" . vc-print-log))
+  (define-key vc-menu-map [separator2] '("----"))
+  (define-key vc-menu-map [undo] '("Undo Last Check-In" . vc-cancel-version))
+  (define-key vc-menu-map [vc-revert-buffer]
+    '("Revert to Last Version" . vc-revert-buffer))
+  (define-key vc-menu-map [vc-insert-header]
+    '("Insert Header" . vc-insert-headers))
+  (define-key vc-menu-map [vc-next-action] '("Check In/Out" . vc-next-action))
+  (define-key vc-menu-map [vc-register] '("Register" . vc-register)))
+
+;; These are not correct and it's not currently clear how doing it
+;; better (with more complicated expressions) might slow things down
+;; on older systems.
+
+;;(put 'vc-rename-file 'menu-enable 'vc-mode)
+;;(put 'vc-annotate 'menu-enable '(eq (vc-buffer-backend) 'CVS))
+;;(put 'vc-version-other-window 'menu-enable 'vc-mode)
+;;(put 'vc-diff 'menu-enable 'vc-mode)
+;;(put 'vc-update-change-log 'menu-enable
+;;     '(member (vc-buffer-backend) '(RCS CVS)))
+;;(put 'vc-print-log 'menu-enable 'vc-mode)
+;;(put 'vc-cancel-version 'menu-enable 'vc-mode)
+;;(put 'vc-revert-buffer 'menu-enable 'vc-mode)
+;;(put 'vc-insert-headers 'menu-enable 'vc-mode)
+;;(put 'vc-next-action 'menu-enable 'vc-mode)
+;;(put 'vc-register 'menu-enable '(and buffer-file-name (not vc-mode)))
 
 (provide 'vc-hooks)