]> code.delx.au - gnu-emacs/blobdiff - lisp/emacs-lisp/shadow.el
Add a provide statement.
[gnu-emacs] / lisp / emacs-lisp / shadow.el
index 73650de88c892727edf576e2f594cb2c85505e0e..eeaaa7ffc0aa9a66dbc2a2ded17a22d026584a22 100644 (file)
@@ -1,4 +1,4 @@
-;;; shadow.el --- Locate Emacs Lisp file shadowings.
+;;; shadow.el --- locate Emacs Lisp file shadowings
 
 ;; Copyright (C) 1995 Free Software Foundation, Inc.
 
 
 ;;; Code:
 \f
+(defgroup lisp-shadow nil
+  "Locate Emacs Lisp file shadowings."
+  :prefix "shadows-"
+  :group 'lisp)
+
+(defcustom shadows-compare-text-p nil
+  "*If non-nil, then shadowing files are reported only if their text differs.
+This is slower, but filters out some innocuous shadowing."
+  :type 'boolean
+  :group 'lisp-shadow)
+
 (defun find-emacs-lisp-shadows (&optional path)
   "Return a list of Emacs Lisp files that create shadows.
 This function does the work for `list-load-path-shadows'.
@@ -63,7 +74,7 @@ the file in position 2i+1.  Emacs Lisp file suffixes \(.el and .elc\)
 are stripped from the file names in the list.
 
 See the documentation for `list-load-path-shadows' for further information."
-  
+
   (or path (setq path load-path))
 
   (let (true-names                     ; List of dirs considered.
@@ -75,10 +86,10 @@ See the documentation for `list-load-path-shadows' for further information."
        files-seen-this-dir             ; Files seen so far in this dir.
        file)                           ; The current file.
 
-  
+
     (while path
 
-      (setq dir (file-truename (or (car path) ".")))
+      (setq dir (directory-file-name (file-truename (or (car path) "."))))
       (if (member dir true-names)
          ;; We have already considered this PATH redundant directory.
          ;; Show the redundancy if we are interactiver, unless the PATH
@@ -89,15 +100,15 @@ See the documentation for `list-load-path-shadows' for further information."
              (and (car path)
                   (not (string= (car path) "."))
                   (message "Ignoring redundant directory %s" (car path))))
-       
+
        (setq true-names (append true-names (list dir)))
-       (setq dir (or (car path) "."))
+       (setq dir (directory-file-name (or (car path) ".")))
        (setq curr-files (if (file-accessible-directory-p dir)
-                               (directory-files dir nil ".\\.elc?$" t)))
+                            (directory-files dir nil ".\\.elc?$" t)))
        (and curr-files
             (not noninteractive)
             (message "Checking %d files in %s..." (length curr-files) dir))
-       
+
        (setq files-seen-this-dir nil)
 
        (while curr-files
@@ -106,21 +117,28 @@ See the documentation for `list-load-path-shadows' for further information."
          (setq file (substring
                      file 0 (if (string= (substring file -1) "c") -4 -3)))
 
-         ;; 'file' now contains the current file name, with no suffix.
-         (if (member file files-seen-this-dir)
-             nil
-                   
+         ;; FILE now contains the current file name, with no suffix.
+         (unless (or (member file files-seen-this-dir)
+                     ;; Ignore these files.
+                     (member file '("subdirs")))
            ;; File has not been seen yet in this directory.
            ;; This test prevents us declaring that XXX.el shadows
            ;; XXX.elc (or vice-versa) when they are in the same directory.
            (setq files-seen-this-dir (cons file files-seen-this-dir))
-             
+
            (if (setq orig-dir (assoc file files))
                ;; This file was seen before, we have a shadowing.
-               (setq shadows
-                     (append shadows
-                             (list (concat (cdr orig-dir) "/" file)
-                                   (concat dir "/" file))))
+               ;; Report it unless the files are identical.
+               (let ((base1 (concat (cdr orig-dir) "/" file))
+                     (base2 (concat dir "/" file)))
+                 (if (not (and shadows-compare-text-p
+                               (shadow-same-file-or-nonexistent
+                                (concat base1 ".el") (concat base2 ".el"))
+                               ;; This is a bit strict, but safe.
+                               (shadow-same-file-or-nonexistent
+                                (concat base1 ".elc") (concat base2 ".elc"))))
+                     (setq shadows
+                           (append shadows (list base1 base2)))))
 
              ;; Not seen before, add it to the list of seen files.
              (setq files (cons (cons file dir) files))))
@@ -131,10 +149,22 @@ See the documentation for `list-load-path-shadows' for further information."
     ;; Return the list of shadowings.
     shadows))
 
+;; Return true if neither file exists, or if both exist and have identical
+;; contents.
+(defun shadow-same-file-or-nonexistent (f1 f2)
+  (let ((exists1 (file-exists-p f1))
+       (exists2 (file-exists-p f2)))
+    (or (and (not exists1) (not exists2))
+       (and exists1 exists2
+            (or (equal (file-truename f1) (file-truename f2))
+                ;; As a quick test, avoiding spawning a process, compare file
+                ;; sizes.
+                (and (= (nth 7 (file-attributes f1))
+                        (nth 7 (file-attributes f2)))
+                     (eq 0 (call-process "cmp" nil nil nil "-s" f1 f2))))))))
 \f
 ;;;###autoload
 (defun list-load-path-shadows ()
-  
   "Display a list of Emacs Lisp files that shadow other files.
 
 This function lists potential load-path problems.  Directories in the
@@ -172,32 +202,60 @@ considered to shadow a later file XXX.el, and vice-versa.
 When run interactively, the shadowings \(if any\) are displayed in a
 buffer called `*Shadows*'.  Shadowings are located by calling the
 \(non-interactive\) companion function, `find-emacs-lisp-shadows'."
-  
+
   (interactive)
-  (let* ((shadows (find-emacs-lisp-shadows))
-        (n (/ (length shadows) 2))
-        (msg (format "%s Emacs Lisp load-path shadowing%s found"
-                     (if (zerop n) "No" (concat "\n" (number-to-string n)))
-                     (if (= n 1) " was" "s were"))))
-    (if (interactive-p)
-       (save-excursion
-         ;; We are interactive.
-         ;; Create the *Shadows* buffer and display shadowings there.
-         (let ((output-buffer (get-buffer-create "*Shadows*")))
-           (display-buffer output-buffer)
-           (set-buffer output-buffer)
-           (erase-buffer)
-           (while shadows
-             (insert (format "%s hides %s\n" (car shadows)
-                             (car (cdr shadows))))
-             (setq shadows (cdr (cdr shadows))))
-           (insert msg "\n")))
-      ;; We are non-interactive, print shadows via message.
-      (while shadows
-       (message "%s hides %s" (car shadows) (car (cdr shadows)))
-       (setq shadows (cdr (cdr shadows))))
-      (message "%s" msg))))
+  (let* ((path (copy-sequence load-path))
+       (tem path)
+       toplevs)
+    ;; If we can find simple.el in two places,
+    (while tem
+      (if (file-exists-p (expand-file-name "simple.el" (car tem)))
+         (setq toplevs (cons (car tem) toplevs)))
+      (setq tem (cdr tem)))
+    (if (> (length toplevs) 1)
+       ;; Cut off our copy of load-path right before
+       ;; the last directory which has simple.el in it.
+       ;; This avoids loads of duplications between the source dir
+       ;; and the dir where these files were copied by installation.
+       (let ((break (car toplevs)))
+         (setq tem path)
+         (while tem
+           (if (eq (nth 1 tem) break)
+               (progn
+                 (setcdr tem nil)
+                 (setq tem nil)))
+           (setq tem (cdr tem)))))
+
+    (let* ((shadows (find-emacs-lisp-shadows path))
+          (n (/ (length shadows) 2))
+          (msg (format "%s Emacs Lisp load-path shadowing%s found"
+                       (if (zerop n) "No" (concat "\n" (number-to-string n)))
+                       (if (= n 1) " was" "s were"))))
+      (if (interactive-p)
+         (save-excursion
+           ;; We are interactive.
+           ;; Create the *Shadows* buffer and display shadowings there.
+           (let ((output-buffer (get-buffer-create "*Shadows*")))
+             (display-buffer output-buffer)
+             (set-buffer output-buffer)
+             (erase-buffer)
+             (while shadows
+               (insert (format "%s hides %s\n" (car shadows)
+                               (car (cdr shadows))))
+               (setq shadows (cdr (cdr shadows))))
+             (insert msg "\n")))
+       ;; We are non-interactive, print shadows via message.
+       (when shadows
+         (message "This site has duplicate Lisp libraries with the same name.
+If a locally-installed Lisp library overrides a library in the Emacs release,
+that can cause trouble, and you should probably remove the locally-installed
+version unless you know what you are doing.\n")
+         (while shadows
+           (message "%s hides %s" (car shadows) (car (cdr shadows)))
+           (setq shadows (cdr (cdr shadows))))
+         (message "%s" msg))))))
 
 (provide 'shadow)
 
+;;; arch-tag: 0480e8a7-62ed-4a12-a9f6-f44ded9b0830
 ;;; shadow.el ends here