From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: updated emacs configuration |
Date: | 2013-06-23 20:37:40 |
Message-ID: | 9051.1372019860@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> The idea is a very good one in principle, but my short experiment with
> the provided .dir-locals.el didn't give me BSD style brace indentation.
> It works if we can do those "unsafe" things, but then we surely don't
> want to have a user prompted to accept the settings each time. If
> .dir-locals.el won't do what we need on its own, it seems to me hardly
> worth having.
I'm un-thrilled with this as well, though for a slightly different
reason: we have a policy that the PG sources should be tool agnostic,
and in fact removed file-local emacs settings awhile back because of
that. Even though I use emacs, I would much rather keep such settings
in my personal library. (TBH, I keep enable-local-variables turned off,
and would thus get no benefit from the proposed file anyhow.)
Another thing I'm not too happy about is that the proposed patch
summarily discards the information we had about how to work with older
emacs versions. I'm not sure it will actually break anybody's setup,
but that would depend on whether .dir-locals.el was added before or
after the last round of rewriting of c-mode.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-06-23 20:43:23 | Re: changeset generation v5-01 - Patches & git tree |
Previous Message | Andrew Dunstan | 2013-06-23 20:11:13 | Re: updated emacs configuration |