]> code.delx.au - gnu-emacs/blobdiff - admin/admin.el
Merge from origin/emacs-24
[gnu-emacs] / admin / admin.el
index 9c0015fc413712e8f38ba6538e278d1fc302bac0..18ea4a40ccc37309912c06116d0c336356fe19b5 100644 (file)
@@ -1,6 +1,6 @@
 ;;; admin.el --- utilities for Emacs administration
 
-;; Copyright (C) 2001-2013 Free Software Foundation, Inc.
+;; Copyright (C) 2001-2015 Free Software Foundation, Inc.
 
 ;; This file is part of GNU Emacs.
 
@@ -21,7 +21,7 @@
 
 ;; add-release-logs    Add ``Version X released'' change log entries.
 ;; set-version         Change Emacs version number in source tree.
-;; set-copyright        Change emacs short copyright string (eg as
+;; set-copyright        Change Emacs short copyright string (eg as
 ;;                      printed by --version) in source tree.
 
 ;;; Code:
@@ -46,7 +46,7 @@ Optional argument DATE is the release date, default today."
                                           (funcall add-log-time-format))))))
   (setq root (expand-file-name root))
   (unless (file-exists-p (expand-file-name "src/emacs.c" root))
-    (error "%s doesn't seem to be the root of an Emacs source tree" root))
+    (user-error "%s doesn't seem to be the root of an Emacs source tree" root))
   (require 'add-log)
   (or date (setq date (let ((add-log-time-zone-rule t))
                        (funcall add-log-time-format))))
@@ -62,18 +62,31 @@ Optional argument DATE is the release date, default today."
       (insert entry))))
 
 (defun set-version-in-file (root file version rx)
+  "Subroutine of `set-version' and `set-copyright'."
   (find-file (expand-file-name file root))
   (goto-char (point-min))
-  (unless (re-search-forward rx nil t)
-    (error "Version not found in %s" file))
-  (replace-match (format "%s" version) nil nil nil 1))
+  (setq version (format "%s" version))
+  (unless (re-search-forward rx nil :noerror)
+    (user-error "Version not found in %s" file))
+  (if (not (equal version (match-string 1)))
+      (replace-match version nil nil nil 1)
+    (kill-buffer)
+    (message "No need to update `%s'" file)))
 
 (defun set-version (root version)
   "Set Emacs version to VERSION in relevant files under ROOT.
 Root must be the root of an Emacs source tree."
-  (interactive "DEmacs root directory: \nsVersion number: ")
+  (interactive (list
+               (read-directory-name "Emacs root directory: " source-directory)
+               (read-string "Version number: "
+                            (replace-regexp-in-string "\\.[0-9]+\\'" ""
+                                                      emacs-version))))
   (unless (file-exists-p (expand-file-name "src/emacs.c" root))
-    (error "%s doesn't seem to be the root of an Emacs source tree" root))
+    (user-error "%s doesn't seem to be the root of an Emacs source tree" root))
+  (message "Setting version numbers...")
+  ;; There's also a "version 3" (standing for GPLv3) at the end of
+  ;; `README', but since `set-version-in-file' only replaces the first
+  ;; occurrence, it won't be replaced.
   (set-version-in-file root "README" version
                       (rx (and "version" (1+ space)
                                (submatch (1+ (in "0-9."))))))
@@ -81,80 +94,31 @@ Root must be the root of an Emacs source tree."
                       (rx (and "AC_INIT" (1+ (not (in ?,)))
                                 ?, (0+ space)
                                 (submatch (1+ (in "0-9."))))))
-  (set-version-in-file root "doc/emacs/emacsver.texi" version
-                      (rx (and "EMACSVER" (1+ space)
-                               (submatch (1+ (in "0-9."))))))
-  (set-version-in-file root "doc/man/emacs.1" version
-                      (rx (and ".TH EMACS" (1+ not-newline)
-                                "GNU Emacs" (1+ space)
-                               (submatch (1+ (in "0-9."))))))
+  ;; No longer used, broken in multiple ways, updating version seems pointless.
   (set-version-in-file root "nt/config.nt" version
                       (rx (and bol "#" (0+ blank) "define" (1+ blank)
                                "VERSION" (1+ blank) "\""
                                (submatch (1+ (in "0-9."))))))
+  ;; TODO: msdos could easily extract the version number from
+  ;; configure.ac with sed, rather than duplicating the information.
   (set-version-in-file root "msdos/sed2v2.inp" version
                       (rx (and bol "/^#undef " (1+ not-newline)
                                "define VERSION" (1+ space) "\""
                                (submatch (1+ (in "0-9."))))))
+  ;; No longer used, broken in multiple ways, updating version seems pointless.
   (set-version-in-file root "nt/makefile.w32-in" version
                       (rx (and "VERSION" (0+ space) "=" (0+ space)
                                (submatch (1+ (in "0-9."))))))
-  ;; nt/emacs.rc also contains the version number, but in an awkward
-  ;; format. It must contain four components, separated by commas, and
-  ;; in two places those commas are followed by space, in two other
-  ;; places they are not.
-  (let* ((version-components (append (split-string version "\\.")
-                                   '("0" "0")))
-        (comma-version
-         (concat (car version-components) ","
-                 (cadr version-components) ","
-                 (cadr (cdr version-components)) ","
-                 (cadr (cdr (cdr version-components)))))
-        (comma-space-version
-         (concat (car version-components) ", "
-                 (cadr version-components) ", "
-                 (cadr (cdr version-components)) ", "
-                 (cadr (cdr (cdr version-components))))))
-    (set-version-in-file root "nt/emacs.rc" comma-version
-                        (rx (and "FILEVERSION" (1+ space)
-                                 (submatch (1+ (in "0-9,"))))))
-    (set-version-in-file root "nt/emacs.rc" comma-version
-                        (rx (and "PRODUCTVERSION" (1+ space)
-                                 (submatch (1+ (in "0-9,"))))))
-    (set-version-in-file root "nt/emacs.rc" comma-space-version
-                        (rx (and "\"FileVersion\"" (0+ space) ?, (0+ space)
-                                 ?\" (submatch (1+ (in "0-9, "))) "\\0\"")))
-    (set-version-in-file root "nt/emacs.rc" comma-space-version
-                        (rx (and "\"ProductVersion\"" (0+ space) ?,
-                                 (0+ space) ?\" (submatch (1+ (in "0-9, ")))
-                                 "\\0\"")))
-    ;; Likewise for emacsclient.rc
-    (set-version-in-file root "nt/emacsclient.rc" comma-version
-                        (rx (and "FILEVERSION" (1+ space)
-                                 (submatch (1+ (in "0-9,"))))))
-    (set-version-in-file root "nt/emacsclient.rc" comma-version
-                        (rx (and "PRODUCTVERSION" (1+ space)
-                                 (submatch (1+ (in "0-9,"))))))
-    (set-version-in-file root "nt/emacsclient.rc" comma-space-version
-                        (rx (and "\"FileVersion\"" (0+ space) ?, (0+ space)
-                                 ?\" (submatch (1+ (in "0-9, "))) "\\0\"")))
-    (set-version-in-file root "nt/emacsclient.rc" comma-space-version
-                        (rx (and "\"ProductVersion\"" (0+ space) ?,
-                                 (0+ space) ?\" (submatch (1+ (in "0-9, ")))
-                                 "\\0\"")))
-    ;; Major version only.
-    (when (string-match "\\([0-9]\\{2,\\}\\)" version)
-      (setq version (match-string 1 version))
-      (set-version-in-file root "src/msdos.c" version
-                          (rx (and "Vwindow_system_version" (1+ not-newline)
-                                   ?\( (submatch (1+ (in "0-9"))) ?\))))
-      (set-version-in-file root "etc/refcards/ru-refcard.tex" version
-                          "\\\\newcommand{\\\\versionemacs}\\[0\\]\
-{\\([0-9]\\{2,\\}\\)}.+%.+version of Emacs")
-      (set-version-in-file root "etc/refcards/emacsver.tex" version
-                          "\\\\def\\\\versionemacs\
-{\\([0-9]\\{2,\\}\\)}.+%.+version of Emacs"))))
-
+  ;; Major version only.
+  (when (string-match "\\([0-9]\\{2,\\}\\)" version)
+    (setq version (match-string 1 version))
+    (set-version-in-file root "src/msdos.c" version
+                        (rx (and "Vwindow_system_version" (1+ not-newline)
+                                 ?\( (submatch (1+ (in "0-9"))) ?\))))
+    (set-version-in-file root "etc/refcards/ru-refcard.tex" version
+                        "\\\\newcommand{\\\\versionemacs}\\[0\\]\
+{\\([0-9]\\{2,\\}\\)}.+%.+version of Emacs"))
+  (message "Setting version numbers...done"))
 
 ;; Note this makes some assumptions about form of short copyright.
 (defun set-copyright (root copyright)
@@ -167,7 +131,8 @@ Root must be the root of an Emacs source tree."
                  (format "Copyright (C) %s Free Software Foundation, Inc."
                          (format-time-string "%Y")))))
   (unless (file-exists-p (expand-file-name "src/emacs.c" root))
-    (error "%s doesn't seem to be the root of an Emacs source tree" root))
+    (user-error "%s doesn't seem to be the root of an Emacs source tree" root))
+  (message "Setting copyrights...")
   (set-version-in-file root "configure.ac" copyright
                       (rx (and bol "copyright" (0+ (not (in ?\")))
                                ?\" (submatch (1+ (not (in ?\")))) ?\")))
@@ -187,30 +152,36 @@ Root must be the root of an Emacs source tree."
     (set-version-in-file root "etc/refcards/ru-refcard.tex" copyright
                         "\\\\newcommand{\\\\cyear}\\[0\\]\
 {\\([0-9]\\{4\\}\\)}.+%.+copyright year")
-    (set-version-in-file root "etc/refcards/emacsver.tex" copyright
+    (set-version-in-file root "etc/refcards/emacsver.tex.in" copyright
                         "\\\\def\\\\year\
-{\\([0-9]\\{4\\}\\)}.+%.+copyright year")))
+{\\([0-9]\\{4\\}\\)}.+%.+copyright year"))
+  (message "Setting copyrights...done"))
 
 ;;; Various bits of magic for generating the web manuals
 
 (defun manual-misc-manuals (root)
-  "Return doc/misc manuals as list of strings."
-  ;; Like `make -C doc/misc echo-info', but works if unconfigured.
+  "Return doc/misc manuals as list of strings.
+ROOT should be the root of an Emacs source tree."
+  ;; Similar to `make -C doc/misc echo-info', but works if unconfigured,
+  ;; and for INFO_TARGETS rather than INFO_INSTALL.
   (with-temp-buffer
     (insert-file-contents (expand-file-name "doc/misc/Makefile.in" root))
-    (search-forward "INFO_TARGETS = ")
-    (let ((start (point))
-         res)
+    ;; Should really use expanded value of INFO_TARGETS.
+    (search-forward "INFO_COMMON = ")
+    (let ((start (point)))
       (end-of-line)
       (while (and (looking-back "\\\\")
                  (zerop (forward-line 1)))
        (end-of-line))
-      (split-string (replace-regexp-in-string
-                    "\\(\\\\\\|\\.info\\)" ""
-                    (buffer-substring start (point)))))))
+      (append (split-string (replace-regexp-in-string
+                            "\\(\\\\\\|\\.info\\)" ""
+                            (buffer-substring start (point))))
+             '("efaq-w32")))))
 
+;; TODO report the progress
 (defun make-manuals (root &optional type)
   "Generate the web manuals for the Emacs webpage.
+ROOT should be the root of an Emacs source tree.
 Interactively with a prefix argument, prompt for TYPE.
 Optional argument TYPE is type of output (nil means all)."
   (interactive (let ((root (read-directory-name "Emacs root directory: "
@@ -287,9 +258,8 @@ Optional argument TYPE is type of output (nil means all)."
 
 (defun manual-misc-html (name root html-node-dir html-mono-dir)
   ;; Hack to deal with the cases where .texi creates a different .info.
-  ;; Blech.  TODO Why not just rename the .texi files?
+  ;; Blech.  TODO Why not just rename the .texi (or .info) files?
   (let* ((texiname (cond ((equal name "ccmode") "cc-mode")
-                        ((equal name "efaq") "faq")
                         (t name)))
         (texi (expand-file-name (format "doc/misc/%s.texi" texiname) root)))
     (manual-html-node texi (expand-file-name name html-node-dir))
@@ -318,6 +288,7 @@ the @import directive."
     (manual-html-fix-node-div)
     (goto-char (point-max))
     (re-search-backward "</body>[\n \t]*</html>")
+    ;; Close the div id="content" that fix-index-1 added.
     (insert "</div>\n\n")
     (save-buffer)))
 
@@ -327,7 +298,7 @@ This function also edits the HTML files so that they validate as
 HTML 4.01 Transitional, and pulls in the gnu.org stylesheet using
 the @import directive."
   (unless (file-exists-p texi-file)
-    (error "Manual file %s not found" texi-file))
+    (user-error "Manual file %s not found" texi-file))
   (make-directory dir t)
   (call-process "makeinfo" nil nil nil
                "-D" "WWW_GNU_ORG"
@@ -358,13 +329,14 @@ the @import directive."
                (manual-html-fix-index-2)
                (if copyright-text
                    (insert copyright-text))
+               ;; Close the div id="content" that fix-index-1 added.
                (insert "\n</div>\n"))
            ;; For normal nodes, give the header div a blue bg.
-           (manual-html-fix-node-div))
+           (manual-html-fix-node-div t))
          (save-buffer))))))
 
 (defun manual-pdf (texi-file dest)
-  "Run texi2pdf on TEXI-FILE, emitting pdf output to DEST."
+  "Run texi2pdf on TEXI-FILE, emitting PDF output to DEST."
   (make-directory (or (file-name-directory dest) ".") t)
   (let ((default-directory (file-name-directory texi-file)))
     (call-process "texi2pdf" nil nil nil
@@ -376,6 +348,7 @@ the @import directive."
   (make-directory (or (file-name-directory dest) ".") t)
   (let ((dvi-dest (concat (file-name-sans-extension dest) ".dvi"))
        (default-directory (file-name-directory texi-file)))
+    ;; FIXME: Use `texi2dvi --ps'?  --xfq
     (call-process "texi2dvi" nil nil nil
                  "-I" "../emacs" "-I" "../misc"
                  texi-file "-o" dvi-dest)
@@ -385,119 +358,298 @@ the @import directive."
 
 (defun manual-html-fix-headers ()
   "Fix up HTML headers for the Emacs manual in the current buffer."
-  (let (opoint)
-    (insert manual-doctype-string)
+  (let ((texi5 (search-forward "<!DOCTYPE" nil t))
+       opoint)
+    ;; Texinfo 5 supplies a DOCTYPE.
+    (or texi5
+       (insert manual-doctype-string))
     (search-forward "<head>\n")
     (insert manual-meta-string)
     (search-forward "<meta")
     (setq opoint (match-beginning 0))
-    (re-search-forward "<!--")
+    (unless texi5
+      (search-forward "<!--")
+      (goto-char (match-beginning 0))
+      (delete-region opoint (point))
+      (search-forward "<meta http-equiv=\"Content-Style")
+      (setq opoint (match-beginning 0)))
+    (search-forward "</head>")
     (goto-char (match-beginning 0))
     (delete-region opoint (point))
     (insert manual-style-string)
-    (search-forward "<meta http-equiv=\"Content-Style")
-    (setq opoint (match-beginning 0))
-    (search-forward "</head>")
-    (delete-region opoint (match-beginning 0))))
+    ;; Remove Texinfo 5 hard-coding bgcolor, text, link, vlink, alink.
+    (when (re-search-forward "<body lang=\"[^\"]+\"" nil t)
+      (setq opoint (point))
+      (search-forward ">")
+      (if (> (point) (1+ opoint))
+         (delete-region opoint (1- (point))))
+      (search-backward "</head"))))
 
-(defun manual-html-fix-node-div ()
+;; Texinfo 5 changed these from class = "node" to "header", yay.
+(defun manual-html-fix-node-div (&optional split)
   "Fix up HTML \"node\" divs in the current buffer."
-  (let (opoint div-end)
-    (while (search-forward "<div class=\"node\">" nil t)
-      (replace-match
-       "<div class=\"node\" style=\"background-color:#DDDDFF\">"
-       t t)
+  (let (opoint div-end type)
+    (while (re-search-forward "<div class=\"\\(node\\|header\\)\"\\(>\\)" nil t)
+      (setq type (match-string 1))
+      ;; NB it is this that makes the bg of non-header cells in the
+      ;; index tables be blue.  Is that intended?
+      ;; Also, if you don't remove the <hr>, the color of the first
+      ;; row in the table will be wrong.
+      ;; This all seems rather odd to me...
+      (replace-match " style=\"background-color:#DDDDFF\">" t t nil 2)
       (setq opoint (point))
-      (re-search-forward "</div>")
-      (setq div-end (match-beginning 0))
-      (goto-char opoint)
-      (if (search-forward "<hr>" div-end 'move)
-         (replace-match "" t t)))))
+      (when (or split (equal type "node"))
+       ;; In Texinfo 4, the <hr> (and anchor) comes after the <div>.
+       (re-search-forward "</div>")
+       (setq div-end (if (equal type "node")
+                         (match-beginning 0)
+                       (line-end-position 2)))
+       (goto-char opoint)
+       (if (search-forward "<hr>" div-end 'move)
+               (replace-match "" t t)
+         (if split (forward-line -1))))
+      ;; In Texinfo 5, the <hr> (and anchor) comes before the <div> (?).
+      ;; Except in split output, where it comes on the line after
+      ;; the <div>.  But only sometimes.  I have no clue what the
+      ;; logic of where it goes is.
+      (when (equal type "header")
+       (goto-char opoint)
+       (when (re-search-backward "^<hr>$" (line-beginning-position -3) t)
+         (replace-match "")
+         (goto-char opoint))))))
+
 
 (defun manual-html-fix-index-1 ()
+  "Remove the h1 header, and the short and long contents lists.
+Also start a \"content\" div."
   (let (opoint)
-    (re-search-forward "<body>\n")
+    (re-search-forward "<body.*>\n")
     (setq opoint (match-end 0))
-    (search-forward "<h2 class=\"")
+    ;; FIXME?  Fragile if a Texinfo 5 document does not use @top.
+    (or (re-search-forward "<h1 class=\"top\"" nil t) ; Texinfo 5
+       (search-forward "<h2 class=\""))
     (goto-char (match-beginning 0))
     (delete-region opoint (point))
+    ;; NB caller must close this div.
     (insert "<div id=\"content\" class=\"inner\">\n\n")))
 
 (defun manual-html-fix-index-2 (&optional table-workaround)
-  "Replace the index list in the current buffer with a HTML table."
-  (let (done open-td tag desc)
-    ;; Convert the list that Makeinfo made into a table.
-    (or (search-forward "<ul class=\"menu\">" nil t)
-       (search-forward "<ul>"))
-    (replace-match "<table style=\"float:left\" width=\"100%\">")
-    (forward-line 1)
-    (while (not done)
-      (cond
-       ((or (looking-at "<li>\\(<a.+</a>\\):[ \t]+\\(.*\\)$")
-           (looking-at "<li>\\(<a.+</a>\\)$"))
-       (setq tag (match-string 1))
-       (setq desc (match-string 2))
-       (replace-match "" t t)
-       (when open-td
-         (save-excursion
-           (forward-char -1)
-           (skip-chars-backward " ")
-           (delete-region (point) (line-end-position))
-           (insert "</td>\n  </tr>")))
-       (insert "  <tr>\n    ")
-       (if table-workaround
-           ;; This works around a Firefox bug in the mono file.
-           (insert "<td bgcolor=\"white\">")
-         (insert "<td>"))
-       (insert tag "</td>\n    <td>" (or desc ""))
-       (setq open-td t))
-       ((eq (char-after) ?\n)
-       (delete-char 1)
-       ;; Negate the following `forward-line'.
-       (forward-line -1))
-       ((looking-at "<!-- ")
-       (search-forward "-->"))
-       ((looking-at "<p>[- ]*The Detailed Node Listing[- \n]*")
-       (replace-match "  </td></tr></table>\n
+  "Replace the index list in the current buffer with a HTML table.
+Leave point after the table."
+  (if (re-search-forward "<table class=\"menu\"\\(.*\\)>" nil t)
+      ;; Texinfo 5 already uses a table.  Tweak it a bit.
+      (let (opoint done)
+       (replace-match " style=\"float:left\" width=\"100%\"" nil t nil 1)
+       (forward-line 1)
+       (while (not done)
+         (cond ((re-search-forward "<tr><td.*&bull; \\(<a.*</a>\\)\
+:</td><td>&nbsp;&nbsp;</td><td[^>]*>\\(.*\\)" (line-end-position) t)
+                (replace-match (format "<tr><td%s>\\1</td>\n<td>\\2"
+                                       (if table-workaround
+                                           " bgcolor=\"white\"" "")))
+                (search-forward "</td></tr>")
+                (forward-line 1))
+               ((looking-at "<tr><th.*<pre class=\"menu-comment\">\n")
+                (replace-match "<tr><th colspan=\"2\" align=\"left\" \
+style=\"text-align:left\">")
+                (search-forward "</pre></th></tr>")
+                (replace-match "</th></tr>\n"))
+               ;; Not all manuals have the detailed menu.
+               ;; If it is there, split it into a separate table.
+               ((re-search-forward "<tr>.*The Detailed Node Listing *"
+                                   (line-end-position) t)
+                (setq opoint (match-beginning 0))
+                (while (and (looking-at " *&mdash;")
+                            (zerop (forward-line 1))))
+                (delete-region opoint (point))
+                (insert "</table>\n\n\
+<h2>Detailed Node Listing</h2>\n\n<p>")
+                ;; FIXME Fragile!
+                ;; The Emacs and Elisp manual have some text at the
+                ;; start of the detailed menu that is not part of the menu.
+                ;; Other manuals do not.
+                (if (re-search-forward "in one step:" (line-end-position 3) t)
+                    (forward-line 1))
+                (insert "</p>\n")
+                (search-forward "</pre></th></tr>")
+                (delete-region (match-beginning 0) (match-end 0))
+                (forward-line -1)
+                (or (looking-at "^$") (error "Parse error 1"))
+                (forward-line -1)
+                (if (looking-at "^$") (error "Parse error 2"))
+                (forward-line -1)
+                (or (looking-at "^$") (error "Parse error 3"))
+                (forward-line 1)
+                (insert "<table class=\"menu\" style=\"float:left\" width=\"100%\">\n\
+<tr><th colspan=\"2\" align=\"left\" style=\"text-align:left\">\n")
+                (forward-line 1)
+                (insert "</th></tr>")
+                (forward-line 1))
+               ((looking-at ".*</table")
+                (forward-line 1)
+                (setq done t)))))
+    (let (done open-td tag desc)
+      ;; Convert the list that Makeinfo made into a table.
+      (or (search-forward "<ul class=\"menu\">" nil t)
+         ;; FIXME?  The following search seems dangerously lax.
+         (search-forward "<ul>"))
+      (replace-match "<table style=\"float:left\" width=\"100%\">")
+      (forward-line 1)
+      (while (not done)
+       (cond
+        ((or (looking-at "<li>\\(<a.+</a>\\):[ \t]+\\(.*\\)$")
+             (looking-at "<li>\\(<a.+</a>\\)$"))
+         (setq tag (match-string 1))
+         (setq desc (match-string 2))
+         (replace-match "" t t)
+         (when open-td
+           (save-excursion
+             (forward-char -1)
+             (skip-chars-backward " ")
+             (delete-region (point) (line-end-position))
+             (insert "</td>\n  </tr>")))
+         (insert "  <tr>\n    ")
+         (if table-workaround
+             ;; This works around a Firefox bug in the mono file.
+             (insert "<td bgcolor=\"white\">")
+           (insert "<td>"))
+         (insert tag "</td>\n    <td>" (or desc ""))
+         (setq open-td t))
+        ((eq (char-after) ?\n)
+         (delete-char 1)
+         ;; Negate the following `forward-line'.
+         (forward-line -1))
+        ((looking-at "<!-- ")
+         (search-forward "-->"))
+        ((looking-at "<p>[- ]*The Detailed Node Listing[- \n]*")
+         (replace-match "  </td></tr></table>\n
 <h3>Detailed Node Listing</h3>\n\n" t t)
-       (search-forward "<p>")
-       (search-forward "<p>" nil t)
-       (goto-char (match-beginning 0))
-       (skip-chars-backward "\n ")
-       (setq open-td nil)
-       (insert "</p>\n\n<table  style=\"float:left\" width=\"100%\">"))
-       ((looking-at "</li></ul>")
-       (replace-match "" t t))
-       ((looking-at "<p>")
-       (replace-match "" t t)
-       (when open-td
-         (insert "  </td></tr>")
-         (setq open-td nil))
-       (insert "  <tr>
+         (search-forward "<p>")
+         ;; FIXME Fragile!
+         ;; The Emacs and Elisp manual have some text at the
+         ;; start of the detailed menu that is not part of the menu.
+         ;; Other manuals do not.
+         (if (looking-at "Here are some other nodes")
+             (search-forward "<p>"))
+         (goto-char (match-beginning 0))
+         (skip-chars-backward "\n ")
+         (setq open-td nil)
+         (insert "</p>\n\n<table  style=\"float:left\" width=\"100%\">"))
+        ((looking-at "</li></ul>")
+         (replace-match "" t t))
+        ((looking-at "<p>")
+         (replace-match "" t t)
+         (when open-td
+           (insert "  </td></tr>")
+           (setq open-td nil))
+         (insert "  <tr>
     <th colspan=\"2\" align=\"left\" style=\"text-align:left\">")
-       (if (re-search-forward "</p>[ \t\n]*<ul class=\"menu\">" nil t)
-           (replace-match "  </th></tr>")))
-       ((looking-at "[ \t]*</ul>[ \t]*$")
-       (replace-match
-        (if open-td
-            "  </td></tr>\n</table>"
-          "</table>") t t)
-       (setq done t))
-       (t
-       (if (eobp)
-           (error "Parse error in %s"
-                  (file-name-nondirectory buffer-file-name)))
-       (unless open-td
-         (setq done t))))
-      (forward-line 1))))
+         (if (re-search-forward "</p>[ \t\n]*<ul class=\"menu\">" nil t)
+             (replace-match "  </th></tr>")))
+        ((looking-at "[ \t]*</ul>[ \t]*$")
+         (replace-match
+          (if open-td
+              "  </td></tr>\n</table>"
+            "</table>") t t)
+         (setq done t))
+        (t
+         (if (eobp)
+             (error "Parse error in %s"
+                    (file-name-nondirectory buffer-file-name)))
+         (unless open-td
+           (setq done t))))
+       (forward-line 1)))))
+
+\f
+(defconst make-manuals-dist-output-variables
+  `(("@\\(top_\\)?srcdir@" . ".")      ; top_srcdir is wrong, but not used
+    ("^\\(\\(?:texinfo\\|buildinfo\\|emacs\\)dir *=\\).*" . "\\1 .")
+    ("^\\(clean:.*\\)" . "\\1 infoclean")
+    ("@MAKEINFO@" . "makeinfo")
+    ("@MKDIR_P@" . "mkdir -p")
+    ("@INFO_EXT@" . ".info")
+    ("@INFO_OPTS@" . "")
+    ("@SHELL@" . "/bin/bash")
+    ("@prefix@" . "/usr/local")
+    ("@datarootdir@" . "${prefix}/share")
+    ("@datadir@" . "${datarootdir}")
+    ("@PACKAGE_TARNAME@" . "emacs")
+    ("@docdir@" . "${datarootdir}/doc/${PACKAGE_TARNAME}")
+    ("@\\(dvi\\|html\\|pdf\\|ps\\)dir@" . "${docdir}")
+    ("@GZIP_PROG@" . "gzip")
+    ("@INSTALL@" . "install -c")
+    ("@INSTALL_DATA@" . "${INSTALL} -m 644")
+    ("@configure_input@" . ""))
+  "Alist of (REGEXP . REPLACEMENT) pairs for `make-manuals-dist'.")
+
+(defun make-manuals-dist--1 (root type)
+  "Subroutine of `make-manuals-dist'."
+  (let* ((dest (expand-file-name "manual" root))
+        (default-directory (progn (make-directory dest t)
+                                  (file-name-as-directory dest)))
+        (version (with-temp-buffer
+                   (insert-file-contents "../doc/emacs/emacsver.texi")
+                   (re-search-forward "@set EMACSVER \\([0-9.]+\\)")
+                   (match-string 1)))
+        (stem (format "emacs-%s-%s" (if (equal type "emacs") "manual" type)
+                      version))
+        (tarfile (format "%s.tar" stem)))
+    (message "Doing %s..." type)
+    (if (file-directory-p stem)
+       (delete-directory stem t))
+    (make-directory stem)
+    (copy-file "../doc/misc/texinfo.tex" stem)
+    (or (equal type "emacs") (copy-file "../doc/emacs/emacsver.texi" stem))
+    (dolist (file (directory-files (format "../doc/%s" type) t))
+      (if (or (string-match-p "\\(\\.texi\\'\\|/ChangeLog\\|/README\\'\\)" file)
+             (and (equal type "lispintro")
+                  (string-match-p "\\.\\(eps\\|pdf\\)\\'" file)))
+         (copy-file file stem)))
+    (with-temp-buffer
+      (let ((outvars make-manuals-dist-output-variables))
+       (push `("@version@" . ,version) outvars)
+       (insert-file-contents (format "../doc/%s/Makefile.in" type))
+       (dolist (cons outvars)
+         (while (re-search-forward (car cons) nil t)
+           (replace-match (cdr cons) t))
+         (goto-char (point-min))))
+      (let (ats)
+       (while (re-search-forward "@[a-zA-Z_]+@" nil t)
+         (setq ats t)
+         (message "Unexpanded: %s" (match-string 0)))
+       (if ats (error "Unexpanded configure variables in Makefile?")))
+      (write-region nil nil (expand-file-name (format "%s/Makefile" stem))
+                   nil 'silent))
+    (call-process "tar" nil nil nil "-cf" tarfile stem)
+    (delete-directory stem t)
+    (message "...created %s" tarfile)))
+
+;; Does anyone actually use these tarfiles?
+(defun make-manuals-dist (root &optional type)
+  "Make the standalone manual source tarfiles for the Emacs webpage.
+ROOT should be the root of an Emacs source tree.
+Interactively with a prefix argument, prompt for TYPE.
+Optional argument TYPE is type of output (nil means all)."
+  (interactive (let ((root (read-directory-name "Emacs root directory: "
+                                               source-directory nil t)))
+                (list root
+                      (if current-prefix-arg
+                          (completing-read
+                           "Type: "
+                           '("emacs" "lispref" "lispintro" "misc"))))))
+  (unless (file-exists-p (expand-file-name "src/emacs.c" root))
+    (user-error "%s doesn't seem to be the root of an Emacs source tree" root))
+  (dolist (m '("emacs" "lispref" "lispintro" "misc"))
+    (if (member type (list nil m))
+       (make-manuals-dist--1 root m))))
 
 \f
-;; Stuff to check new defcustoms got :version tags.
+;; Stuff to check new `defcustom's got :version tags.
 ;; Adapted from check-declare.el.
 
 (defun cusver-find-files (root &optional old)
-  "Find .el files beneath directory ROOT that contain defcustoms.
-If optional OLD is non-nil, also include defvars."
+  "Find .el files beneath directory ROOT that contain `defcustom's.
+If optional OLD is non-nil, also include `defvar's."
   (process-lines find-program root
                 "-name" "*.el"
                 "-exec" grep-program
@@ -509,14 +661,14 @@ If optional OLD is non-nil, also include defvars."
 
 (defvar cusver-new-version (format "%s.%s" emacs-major-version
                                   (1+ emacs-minor-version))
-  "Version number that new defcustoms should have.")
+  "Version number that new `defcustom's should have.")
 
 (defun cusver-scan (file &optional old)
   "Scan FILE for `defcustom' calls.
 Return a list with elements of the form (VAR . VER),
 This means that FILE contains a defcustom for variable VAR, with
 a :version tag having value VER (may be nil).
-If optional argument OLD is non-nil, also scan for defvars."
+If optional argument OLD is non-nil, also scan for `defvar's."
   (let ((m (format "Scanning %s..." file))
        (re (format "^[ \t]*\\((def%s\\)[ \t\n]"
                    (if old "\\(custom\\|var\\)" "\\(custom\\|group\\)")))
@@ -525,13 +677,19 @@ If optional argument OLD is non-nil, also scan for defvars."
     (with-temp-buffer
       (insert-file-contents file)
       ;; FIXME we could theoretically be inside a string.
-      (while (re-search-forward re nil t)
+      (while (re-search-forward re nil :noerror)
         (goto-char (match-beginning 1))
         (if (and (setq form (ignore-errors (read (current-buffer))))
                 (setq var (car-safe (cdr-safe form)))
                 ;; Exclude macros, eg (defcustom ,varname ...).
                 (symbolp var))
            (progn
+             ;; FIXME It should be cus-test-apropos that does this.
+             (and (not old)
+                  (equal "custom" (match-string 2))
+                  (not (memq :type form))
+                  (display-warning 'custom
+                                   (format "Missing type in: `%s'" form)))
              (setq ver (car (cdr-safe (memq :version form))))
              (if (equal "group" (match-string 2))
                  ;; Group :version could be old.
@@ -567,7 +725,7 @@ If optional argument OLD is non-nil, also scan for defvars."
 (define-button-type 'cusver-xref 'action #'cusver-goto-xref)
 
 (defun cusver-goto-xref (button)
-  "Jump to a lisp file for the BUTTON at point."
+  "Jump to a Lisp file for the BUTTON at point."
   (let ((file (button-get button 'file))
        (var (button-get button 'var)))
     (if (not (file-readable-p file))
@@ -583,34 +741,36 @@ If optional argument OLD is non-nil, also scan for defvars."
 ;; TODO Check cus-start if something moved from C to Lisp.
 ;; TODO Handle renamed things with aliases to the old names.
 (defun cusver-check (newdir olddir version)
-  "Check that defcustoms have :version tags where needed.
-NEWDIR is the current lisp/ directory, OLDDIR is that from the previous
-release.  A defcustom that is only in NEWDIR should have a :version
-tag.  We exclude cases where a defvar exists in OLDDIR, since
-just converting a defvar to a defcustom does not require a :version bump.
+  "Check that `defcustom's have :version tags where needed.
+NEWDIR is the current lisp/ directory, OLDDIR is that from the
+previous release, VERSION is the new version number.  A
+`defcustom' that is only in NEWDIR should have a :version tag.
+We exclude cases where a `defvar' exists in OLDDIR, since 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 (list (read-directory-name "New Lisp directory: ")
-                    (read-directory-name "Old Lisp directory: ")
+  (interactive (list (read-directory-name "New Lisp directory: " nil nil t)
+                    (read-directory-name "Old Lisp directory: " nil nil t)
                     (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))
+      (user-error "Directory `%s' not found" newdir))
   (or (file-directory-p (setq olddir (expand-file-name olddir)))
-      (error "Directory `%s' not found" olddir))
+      (user-error "Directory `%s' not found" olddir))
   (setq cusver-new-version version)
-  (let* ((newfiles (progn (message "Finding new files with defcustoms...")
+  (let* ((newfiles (progn (message "Finding new files with `defcustom's...")
                          (cusver-find-files newdir)))
-        (oldfiles (progn (message "Finding old files with defcustoms...")
+        (oldfiles (progn (message "Finding old files with `defcustom's...")
                          (cusver-find-files olddir t)))
-        (newcus (progn (message "Reading new defcustoms...")
+        (newcus (progn (message "Reading new `defcustom's...")
                        (mapcar
                         (lambda (file)
                           (cons file (cusver-scan file))) newfiles)))
         oldcus result thisfile file)
-    (message "Reading old defcustoms...")
+    (message "Reading old `defcustom's...")
     (dolist (file oldfiles)
       (setq oldcus (append oldcus (cusver-scan file t))))
     (setq oldcus (append oldcus (cusver-scan-cus-start
@@ -635,7 +795,7 @@ changes (in a non-trivial way).  This function does not check for that."
        (message "No missing :version tags")
       (pop-to-buffer "*cusver*")
       (erase-buffer)
-      (insert "These defcustoms might be missing :version tags:\n\n")
+      (insert "These `defcustom's might be missing :version tags:\n\n")
       (dolist (elem result)
        (let* ((str (file-relative-name (car elem) newdir))
               (strlen (length str)))