GNU bug report logs - #6731
23.2; remote .dir-locals.el

Previous Next

Package: emacs;

Reported by: Leo <sdl.web <at> gmail.com>

Date: Mon, 26 Jul 2010 14:30:03 UTC

Severity: normal

Tags: patch

Merged with 1933

Found in version 23.2

Fixed in version 24.2

Done: Michael Albinus <michael.albinus <at> gmx.de>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 6731 in the body.
You can then email your comments to 6731 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to owner <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org:
bug#6731; Package emacs. (Mon, 26 Jul 2010 14:30:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Leo <sdl.web <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Mon, 26 Jul 2010 14:30:03 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.2; remote .dir-locals.el
Date: Mon, 26 Jul 2010 15:29:42 +0100
It looks like hack-dir-local-variables ignores .dir-locals.el for remote
files. I think if a project depends on .dir-locals.el it will not be
remotely editable without loading .dir-locals.el. I wonder if this could
be improved. Thanks.


In GNU Emacs 23.2.11 (x86_64-apple-darwin10.4.0, Carbon Version 1.6.0 AppKit 1038.32)
 of 2010-07-23 on Victoria.local
Windowing system distributor `Apple Inc.', version 10.6.4




Information forwarded to owner <at> debbugs.gnu.org, bug-gnu-emacs <at> gnu.org:
bug#6731; Package emacs. (Mon, 02 Aug 2010 14:37:03 GMT) Full text and rfc822 format available.

Message #8 received at 6731 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Leo <sdl.web <at> gmail.com>
Cc: tramp-devel <at> gnu.org, 6731 <at> debbugs.gnu.org
Subject: Re: Can tramp load .dir-locals.el in a project?
Date: Mon, 02 Aug 2010 16:36:27 +0200
Leo <sdl.web <at> gmail.com> writes:

> On 2010-05-20 17:04 +0100, Michael Albinus wrote:
>> | If you put a file with a special name `.dir-locals.el'(1) in a
>> | directory, Emacs will read it when it visits any file in that directory
>> | or any of its subdirectories, and apply the settings it specifies to
>> | the file's buffer.  Emacs searches for `.dir-locals.el' starting in the
>> | directory of the visited file, and moving up the directory tree.  (To
>> | avoid slowdown, this search is skipped for remote files.)
>>
>> I wouldn't like to overrule this on my own. If you believe it is
>> important, you might ask on the emacs-devel mailing list for this. Maybe
>> one could enable this by a custom variable.
>
> I have reported this as a bug.¹ I think it is important enough for the
> .dir-locals.el to be loaded, otherwise, users end up editing files in a
> way non-conforming to the requirement of a project and editing is the
> most important reason to use tramp for the comfort of Emacs.
>
> Would be interesting to hear your thought and input. Thanks.

Loading .dir-locals.el is a non-trivial operation, because several
directories must be traversed for existence check of such a file,
starting at the current directory, and navigating up to "/". This might
be time-consuming.

I suspect, this is the reason it has been disabled for remote
directories.

One could improve the checks by adding a new file name handler for
searching and loading of .dir-locals.el. This would allow to apply all
checks at once on the remote side, with a better performance.

A similar file name handler was introduced for `vc-registered', which
was improved by a special implementation for remote files.

> Kind regards,
> Leo

Best regards, Michael.




Forcibly Merged 1933 6731. Request was from Glenn Morris <rgm <at> gnu.org> to control <at> debbugs.gnu.org. (Thu, 24 Feb 2011 01:46:02 GMT) Full text and rfc822 format available.

Reply sent to Michael Albinus <michael.albinus <at> gmx.de>:
You have taken responsibility. (Wed, 06 Jun 2012 14:07:02 GMT) Full text and rfc822 format available.

Notification sent to Leo <sdl.web <at> gmail.com>:
bug acknowledged by developer. (Wed, 06 Jun 2012 14:07:03 GMT) Full text and rfc822 format available.

Message #15 received at 6731-done <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Ian Eure <ieure <at> mac.com>, Leo <sdl.web <at> gmail.com>
Cc: 6731-done <at> debbugs.gnu.org, 1933-done <at> debbugs.gnu.org
Subject: Re: bug#1933: 23.0.60;
	[PATCH] dir-local variables don't work on remote files
Date: Wed, 06 Jun 2012 16:04:17 +0200
Version: 24.2

Ian Eure <ieure <at> mac.com> writes:

Hi Ian and Leo,

> I guess for speed reasons, dir-local-variables don’t work when opening
> remote fiales. Which is unfortunate for me, since I need them. Here’s
> a patch against HEA which implements a simple switch for it, defaulted
> to the current behavior.

It took a while, but finally I have applied the patch, modified. The
customer option is called `enable-remote-dir-locals'. When set to `t',
remote .dir-locals files are evaluated as well. And I've fixed an error
of the patch in the logic of the condition.

I hope it is still useful for you.

Best regards, Michael.




Reply sent to Michael Albinus <michael.albinus <at> gmx.de>:
You have taken responsibility. (Wed, 06 Jun 2012 14:07:03 GMT) Full text and rfc822 format available.

Notification sent to Ian Eure <ieure <at> mac.com>:
bug acknowledged by developer. (Wed, 06 Jun 2012 14:07:03 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 05 Jul 2012 11:24:03 GMT) Full text and rfc822 format available.

This bug report was last modified 12 years and 355 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.