On Tue, May  6, 2014 at 05:05:00PM -0400, Tom Lane wrote:
> Bruce Momjian <br...@momjian.us> writes:
> > On Tue, May  6, 2014 at 03:54:24PM -0400, Tom Lane wrote:
> >> But having said that, if this didn't work then there's something broken
> >> about the make rules for the ecpg tests.  I'm a bit suspicious of commit
> >> 69e9768e7b183d4b276d0e067a5a0000689580eb.
> 
> I looked into this, and find that the cause of the problem is that
> pgindent touched src/interfaces/ecpg/include/sqlca.h, which is copied
> verbatim into preprocessed files by the ecpg preprocessor, so the expected
> files had to change in tandem.  This amounts to a dependency, but the make
> rules don't know about it.  Should they?  That particular file changes so
> seldom that it'd hardly be worth worrying about, but I'm not sure which
> other files can get copied similarly.
> 
> > What _is_ odd is that I had to change these files after the pgindent run
> > in head, but _not_ in the back branches when I removed the tabs from
> > comments.  I assume there is something new in 9.4 about they way they
> > are built.
> 
> I'm confused by this statement.  Your tab-adjustment commits in the back
> branches also touched both sqlca.h and the ecpg expected files.

They probably did in the back branches as I hit _all_ C files.  I wonder
if pgindent somehow skipped some of them.

Ah, found it.  There is an excludes pattern file list I had forgotten
about;  it has:

        /s_lock\.h$
        /ecpg/test/expected/
        /snowball/libstemmer/
        /ecpg/include/(sqlda|sqltypes)\.h$
        /ecpg/include/preproc/struct\.h$
        /pl/plperl/ppport\.h$
        
I am thinking I should back out the tab/comment changes in those files
in the back branches, though I would then need to adjust the ecpg
regression tests.  In practice, these files are rarely patched, so it
might be fine to just leave them alone.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + Everyone has their own god. +


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to