[Emacs-diffs] master 6213ce5 5/5: Merge from origin/emacs-26

[Glenn Morris]

branch: master
commit 6213ce554410213d5ffd3acb09ea2ebde8402281
Merge: 224fc14 76040d1
Author: Glenn Morris <address@hidden>
Commit: Glenn Morris <address@hidden>

    Merge from origin/emacs-26
    
    76040d1 (origin/emacs-26) Handle case-insensitive filenames for load-...
    0c9b050 ; * test/file-organization.org: Fix typo.
---
 lisp/emacs-lisp/shadow.el                       | 10 ++++-
 test/file-organization.org                      |  2 +-
 test/lisp/emacs-lisp/shadow-resources/p1/foo.el |  1 +
 test/lisp/emacs-lisp/shadow-resources/p2/FOO.el |  1 +
 test/lisp/emacs-lisp/shadow-tests.el            | 49 +++++++++++++++++++++++++
 5 files changed, 60 insertions(+), 3 deletions(-)

diff --git a/lisp/emacs-lisp/shadow.el b/lisp/emacs-lisp/shadow.el
index 88a494f..1788f0d 100644
--- a/lisp/emacs-lisp/shadow.el
+++ b/lisp/emacs-lisp/shadow.el
@@ -78,6 +78,7 @@ See the documentation for `list-load-path-shadows' for 
further information."
        shadows                         ; List of shadowings, to be returned.
        files                           ; File names ever seen, with dirs.
        dir                             ; The dir being currently scanned.
+        dir-case-insensitive            ; `file-name-case-insentive-p' for dir.
        curr-files                      ; This dir's Emacs Lisp files.
        orig-dir                        ; Where the file was first seen.
        files-seen-this-dir             ; Files seen so far in this dir.
@@ -104,6 +105,9 @@ See the documentation for `list-load-path-shadows' for 
further information."
             (message "Checking %d files in %s..." (length curr-files) dir))
 
        (setq files-seen-this-dir nil)
+        ;; We assume that case sensitivity of a directory applies to
+        ;; its files.
+        (setq dir-case-insensitive (file-name-case-insensitive-p dir))
 
        (dolist (file curr-files)
 
@@ -123,10 +127,12 @@ See the documentation for `list-load-path-shadows' for 
further information."
            ;; 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))
+            (if (setq orig-dir (assoc file files
+                                      (when dir-case-insensitive
+                                        (lambda (f1 f2) (eq (compare-strings 
f1 nil nil f2 nil nil t) t)))))
                ;; This file was seen before, we have a shadowing.
                ;; Report it unless the files are identical.
-               (let ((base1 (concat (cdr orig-dir) "/" file))
+                (let ((base1 (concat (cdr orig-dir) "/" (car orig-dir)))
                      (base2 (concat dir "/" file)))
                  (if (not (and load-path-shadows-compare-text
                                (load-path-shadows-same-file-or-nonexistent
diff --git a/test/file-organization.org b/test/file-organization.org
index 6c93c28..34bd0b9 100644
--- a/test/file-organization.org
+++ b/test/file-organization.org
@@ -55,5 +55,5 @@ located in the same directory as the feature.  Hence, the 
lisp file
 directory called ~test/lisp/progmodes/flymake-resources~.
 
 No guidance is given for the organization of resource files inside the
-~-resource~ directory; files can be organized at the author's
+~-resources~ directory; files can be organized at the author's
 discretion.
diff --git a/test/lisp/emacs-lisp/shadow-resources/p1/foo.el 
b/test/lisp/emacs-lisp/shadow-resources/p1/foo.el
new file mode 100644
index 0000000..465038b
--- /dev/null
+++ b/test/lisp/emacs-lisp/shadow-resources/p1/foo.el
@@ -0,0 +1 @@
+;;; This file intentionally left blank.
diff --git a/test/lisp/emacs-lisp/shadow-resources/p2/FOO.el 
b/test/lisp/emacs-lisp/shadow-resources/p2/FOO.el
new file mode 100644
index 0000000..465038b
--- /dev/null
+++ b/test/lisp/emacs-lisp/shadow-resources/p2/FOO.el
@@ -0,0 +1 @@
+;;; This file intentionally left blank.
diff --git a/test/lisp/emacs-lisp/shadow-tests.el 
b/test/lisp/emacs-lisp/shadow-tests.el
new file mode 100644
index 0000000..9d4969f
--- /dev/null
+++ b/test/lisp/emacs-lisp/shadow-tests.el
@@ -0,0 +1,49 @@
+;;; shadow-tests.el --- Test suite for shadow.  -*- lexical-binding: t -*-
+
+;; Copyright (C) 2018 Free Software Foundation, Inc.
+
+;; This file is part of GNU Emacs.
+
+;; GNU Emacs is free software: you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+
+;; GNU Emacs is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with GNU Emacs.  If not, see <https://www.gnu.org/licenses/>.
+
+;;; Code:
+
+(require 'ert)
+(require 'shadow)
+(eval-when-compile (require 'cl-lib))
+
+(defconst shadow-tests-data-directory
+  (expand-file-name "lisp/emacs-lisp/shadow-resources"
+                    (or (getenv "EMACS_TEST_DIRECTORY")
+                        (expand-file-name "../../.."
+                                          (or load-file-name
+                                              buffer-file-name))))
+  "Directory for shadow test files.")
+
+(ert-deftest shadow-case-insensitive ()
+  "Test shadowing for case insensitive filenames."
+  ;; Override `file-name-case-insentive-p' so we test the same thing
+  ;; regardless of what file system we're running on.
+  (cl-letf (((symbol-function 'file-name-case-insensitive-p) (lambda (_f) t)))
+    (should (equal (list (expand-file-name "p1/foo" 
shadow-tests-data-directory)
+                         (expand-file-name "p2/FOO" 
shadow-tests-data-directory))
+                   (load-path-shadows-find
+                    (list (expand-file-name "p1/" shadow-tests-data-directory)
+                          (expand-file-name "p2/" 
shadow-tests-data-directory))))))
+  (cl-letf (((symbol-function 'file-name-case-insensitive-p) (lambda (_f) 
nil)))
+    (should-not (load-path-shadows-find
+                 (list (expand-file-name "p1/" shadow-tests-data-directory)
+                       (expand-file-name "p2/" 
shadow-tests-data-directory))))))
+
+;;; shadow-tests.el ends here.