From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Leo Liu Newsgroups: gmane.emacs.bugs Subject: bug#12357: 24.2; list-load-path-shadow should ignore .dir-locals.el Date: Wed, 05 Mar 2014 08:54:15 +0800 Message-ID: References: <87harcsi6n.fsf@bredband.net> <87ipbor4jg.fsf@bredband.net> <871ui1lrtk.fsf@bredband.net> <87y50yqmwd.fsf@gnu.org> <8761nwqmd5.fsf@bredband.net> <87r46kgrco.fsf@gnu.org> <87eh2hr41n.fsf@bredband.net> Reply-To: 12357@debbugs.gnu.org NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1393980913 30334 80.91.229.3 (5 Mar 2014 00:55:13 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 5 Mar 2014 00:55:13 +0000 (UTC) Cc: 12357@debbugs.gnu.org To: Johan Claesson Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Mar 05 01:55:21 2014 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1WL07E-0001wh-PZ for geb-bug-gnu-emacs@m.gmane.org; Wed, 05 Mar 2014 01:55:21 +0100 Original-Received: from localhost ([::1]:49530 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WL07E-00040h-9Z for geb-bug-gnu-emacs@m.gmane.org; Tue, 04 Mar 2014 19:55:20 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:56631) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WL074-0003yx-Rk for bug-gnu-emacs@gnu.org; Tue, 04 Mar 2014 19:55:18 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WL06x-00055J-6q for bug-gnu-emacs@gnu.org; Tue, 04 Mar 2014 19:55:10 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:49731) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WL06x-00054m-2l for bug-gnu-emacs@gnu.org; Tue, 04 Mar 2014 19:55:03 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1WL06v-0007XH-V2 for bug-gnu-emacs@gnu.org; Tue, 04 Mar 2014 19:55:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Leo Liu Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 05 Mar 2014 00:55:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 12357 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 12357-submit@debbugs.gnu.org id=B12357.139398087228921 (code B ref 12357); Wed, 05 Mar 2014 00:55:01 +0000 Original-Received: (at 12357) by debbugs.gnu.org; 5 Mar 2014 00:54:32 +0000 Original-Received: from localhost ([127.0.0.1]:50913 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1WL06R-0007WO-H7 for submit@debbugs.gnu.org; Tue, 04 Mar 2014 19:54:32 -0500 Original-Received: from mail-pb0-f54.google.com ([209.85.160.54]:49696) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1WL06P-0007WG-Fj for 12357@debbugs.gnu.org; Tue, 04 Mar 2014 19:54:29 -0500 Original-Received: by mail-pb0-f54.google.com with SMTP id ma3so312143pbc.13 for <12357@debbugs.gnu.org>; Tue, 04 Mar 2014 16:54:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:references:face:reply-to:date:in-reply-to :message-id:user-agent:mime-version:content-type; bh=cauehRlmuLluvE3gYJ3shx0h0YzF5RnOwAipdVpuMbY=; b=qs18Sq8ZcwAc5zSZeugWjX1yvGUzULr8OqvV8xlVM0uEFAuEZsDmCvrsgfvtnLEwIV Bj1MxDfUeEMHj9ecmqYxteIrVFGuFChrQM0N0r4MMwImUK4D58DJd6BxvgBmIK7yO2Pb T7v3URfR4vpBUvE51FWTasvaeFDnMUQEYIqovVFO2hQ5YHru28xpmvNIOwsa1pBzBIJu RTx5XPGcUKhOFM+1UEhXtZcHGIWgeO3taxIvQ7dp0DVhW78zktYbWObznI0Z3cW3fmx4 tHTQGZH6bMNsQ593yoPT9go40ezGcSSgj9qEJaBmumTYqC0a/uxN83x0Rlqd6MN9tr5m uhdg== X-Received: by 10.66.185.103 with SMTP id fb7mr2927538pac.125.1393980868650; Tue, 04 Mar 2014 16:54:28 -0800 (PST) Original-Received: from Zeuss-MacBook.local ([123.119.86.190]) by mx.google.com with ESMTPSA id db3sm1520900pbb.10.2014.03.04.16.54.24 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 04 Mar 2014 16:54:28 -0800 (PST) Face: iVBORw0KGgoAAAANSUhEUgAAACgAAAAoBAMAAAB+0KVeAAAAElBMVEUAAAAAAP+LRRP0pGC+ vr7///+7mT1iAAAAAWJLR0QAiAUdSAAAAAlwSFlzAAALEwAACxMBAJqcGAAAAAd0SU1FB9cBBwMO DhglKe4AAAEsSURBVCjPbZNBboQwDEV/Cd4X9QJRThApmn0XYW+Jyf2v0m+HhqDBgiAe9rcTG7QH w/1Vn2Ar8gBb/ocywSN3qK9T3z4eFDB4eApocBpeBs1RSykoJd8gQcm8pGmHXFso3ajnmsqV0TnY DQkOfXUfN5NwaI7AWTVOyEhcu1aHmdWItHddUVUcUgUBCkitu8V6ditHVOVdqzl2EQ1ZVGTbdK0V 7cqn8vWzoU5Q/bF9Y/Y0cRU1xwkys5dJ+Dt6pBDWifcNQml8Gh2JVmPSoQzo7en0grswkxrUGYJ7 0hSxxAGr7ZMwYcHIzprpi7TENEE1xtiYxixRlCfPBsUUrwHD7uGIwATrbnODJcVrPpVn3hxiGloe m/S+z3CtuzUSMo83N4DPH+F0evwR3P4A2k+75838OKQAAAAASUVORK5CYII= In-Reply-To: <87eh2hr41n.fsf@bredband.net> (Johan Claesson's message of "Tue, 04 Mar 2014 23:29:24 +0100") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (OS X 10.9.2) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:86559 Archived-At: On 2014-03-05 06:29 +0800, Johan Claesson wrote: > * emacs-lisp/shadows.el (load-path-shadows-find): Do not report > .dir-locals.el files. > (list-dir-locals-shadows): New command that reports shadows of > .dir-locals.el in a way that make sense for that type of file. > (dir-locals-shadows-find, dir-locals-shadows-find-1) > (dir-locals-class-name, dir-locals-shadows-display): Auxiliary > functions for list-dir-locals-shadows. I am not too sure what problems are solved by the change. In the case of list-load-path-shadows, it should print anything when in doubt since false positives can be helpful. So it is not entirely wrong to report .dir-locals.el. Use .dir-locals.el in a child directory to shadow parent directory isn't uncommon practice. In future we might even make it capable of inheriting settings in parent directory. So I am not even sure the use case of listing .dir-locals files (defaulting to the load-path)?? Leo