RCS

[Ville Skyttä]

On Wednesday 03 October 2007, Adrian Aichner wrote:

> Current CVS has a strange looking section of
> ;;;###autoload(defun vc-load-vc-hooks ()
> ;;;###autoload  (interactive)
> ;;;###autoload  (require 'vc-hooks)
> ;;;###autoload  (vc-find-file-hook))
>
> ;;;###autoload(and (featurep 'menubar)
> ;;;###autoload     (featurep 'xemacs)
> ;;;###autoload     (not (or (featurep 'vc-hooks) (featurep 'infodock)))
> ;;;###autoload     current-menubar
> ;;;###autoload     (car (find-menu-item current-menubar '("Tools")))
> ;;;###autoload     (add-submenu '("Tools")
> ;;;###autoload		  '("Version Control"
> ;;;###autoload		   ["Load VC" vc-load-vc-hooks t])
> ;;;###autoload	"Compare"))
>
> Ville, could you please comment.

Pre-1.13 versions have a big warning above those:

   ;; This NEEDS to go AFTER 'vc-hooks, package load only (XEmacs)

To me that looked like a strange way to make something "package load only" - 
the approach I took in the above "strange looking section" was to make them 
autoload-only which I thought for practical purposes was the same as package 
load only.  But in retrospect, that may not have been a too good assumption.

In any case, I'm not sure how vc-load-vc-hooks can currently be not defined, 
it certainly is here.  If someone knows what this stuff is about, and how to 
fix it (if something's broken), please go right ahead.