Re: updated emacs configuration - Mailing list pgsql-hackers

From Tom Lane
Subject Re: updated emacs configuration
Date
Msg-id 28740.1402325554@sss.pgh.pa.us
Whole thread Raw
In response to Re: updated emacs configuration  (Noah Misch <noah@leadboat.com>)
Responses Re: updated emacs configuration  (Noah Misch <noah@leadboat.com>)
List pgsql-hackers
Noah Misch <noah@leadboat.com> writes:
> On Wed, Aug 07, 2013 at 07:57:53AM -0400, Peter Eisentraut wrote:
>> Did anyone have any outstanding concerns about this latest version?  I
>> thought it looked ready to commit.

> After upgrading to GNU Emacs 23.4.1 from a version predating directory-local
> variables, I saw switch/case indentation go on the fritz.  My hooks were
> issuing (c-set-style "postgresql"), but ".dir-locals.el" set it back to plain
> "bsd" style.  The most-reasonable fix I found was to locally add c-file-style
> to ignored-local-variables.

It seems pretty ugly for this patch to be overriding our own
.dir-locals.el, especially in a way with as many potential side effects
as that.  Why can't we fix .dir-locals.el to play nice with this?
Is there a way for it to check for whether postgresql style is defined?
        regards, tom lane



pgsql-hackers by date:

Previous
From: Linos
Date:
Subject: Re: performance regression in 9.2/9.3
Next
From: Merlin Moncure
Date:
Subject: Re: performance regression in 9.2/9.3