[Emacs-diffs] trunk r114877: * lisp/Makefile.in (autoloads): Do not use abs_lisp.

[Glenn Morris]

------------------------------------------------------------
revno: 114877
revision-id: address@hidden
parent: address@hidden
committer: Glenn Morris <address@hidden>
branch nick: trunk
timestamp: Wed 2013-10-30 16:27:35 -0400
message:
  * lisp/Makefile.in (autoloads): Do not use abs_lisp.
  
  This should stop recent weirdness, eg:
  http://lists.gnu.org/archive/html/emacs-diffs/2013-10/msg00381.html
  
  * lisp/emacs-lisp/autoload.el: Comment.
modified:
  lisp/ChangeLog                 changelog-20091113204419-o5vbwnq5f7feedwu-1432
  lisp/Makefile.in               
makefile.in-20091113204419-o5vbwnq5f7feedwu-1831
  lisp/emacs-lisp/autoload.el    autoload.el-20091113204419-o5vbwnq5f7feedwu-397

=== modified file 'lisp/ChangeLog'
--- a/lisp/ChangeLog    2013-10-30 20:03:07 +0000
+++ b/lisp/ChangeLog    2013-10-30 20:27:35 +0000
@@ -1,5 +1,7 @@
 2013-10-30  Glenn Morris  <address@hidden>
 
+       * Makefile.in (autoloads): Do not use abs_lisp.
+
        * emacs-lisp/autoload.el (autoload-generate-file-autoloads):
        `newline' does not respect `standard-output', so use `princ'.
 

=== modified file 'lisp/Makefile.in'
--- a/lisp/Makefile.in  2013-10-30 17:17:12 +0000
+++ b/lisp/Makefile.in  2013-10-30 20:27:35 +0000
@@ -178,13 +178,15 @@
        $(emacs) -l finder --eval '(setq generated-finder-keywords-file 
(unmsys--file-name "$(abs_lisp)/finder-inf.el"))' -f 
finder-compile-keywords-make-dist $$wins
 
 # The chmod +w is to handle env var CVSREAD=1.
+# Use expand-file-name rather than $abs_lisp so that Emacs does not
+# get confused when it compares file-names for equality.
 autoloads: $(LOADDEFS) doit
        cd $(lisp) && chmod +w $(AUTOGEN_VCS)
        cd $(lisp) && $(setwins_almost); \
        echo Directories: $$wins; \
        $(emacs) -l autoload \
            --eval '(setq autoload-builtin-package-versions t)' \
-           --eval '(setq generated-autoload-file (unmsys--file-name 
"$(abs_lisp)/loaddefs.el"))' \
+           --eval '(setq generated-autoload-file (expand-file-name 
(unmsys--file-name "$(lisp)/loaddefs.el")))' \
            -f batch-update-autoloads $$wins
 
 # This is required by the bootstrap-emacs target in ../src/Makefile, so

=== modified file 'lisp/emacs-lisp/autoload.el'
--- a/lisp/emacs-lisp/autoload.el       2013-10-30 20:03:07 +0000
+++ b/lisp/emacs-lisp/autoload.el       2013-10-30 20:27:35 +0000
@@ -528,6 +528,9 @@
                (if (stringp generated-autoload-load-name)
                    generated-autoload-load-name
                  (autoload-file-load-name absfile)))
+          ;; FIXME? Comparing file-names for equality with just equal
+          ;; is fragile, eg if one has an automounter prefix and one
+          ;; does not, but both refer to the same physical file.
           (when (and outfile
                      (not
                      (if (memq system-type '(ms-dos windows-nt))