From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: do we want to gitignore regression-test-failure files? |
Date: | 2010-09-26 20:19:11 |
Message-ID: | AANLkTi=5rPXBzkUUSeOUFredXgggj+qm7ikinMmVpoRu@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Sep 26, 2010 at 3:45 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> Seems like it's probably a good idea, but I wanted to double-check
>> that no one has a different thought.
>
> -1. If the lack of an ignore causes a problem for you, it indicates
> that you're trying to commit code that fails the regression tests.
> Is it really a good idea to let that happen without any manual cleanup?
I think it just means that the regression tests have failed at some
point since the last time you cleaned out your tree. Those files
don't get removed on a successful make check, do they?
The reason I assumed we'd want to ignore these is because they're
automatically generated files - unlike *.rej files, which are never
going to end up in your tree as a result of make anything. It doesn't
actually matter that much to me in practice, except that I fear
creating a complex and indecipherable rule about what to ignore vs.
not.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company
From | Date | Subject | |
---|---|---|---|
Next Message | Marios Vodas | 2010-09-26 20:20:42 | forming tuple as an attribute inside another tuple in c function |
Previous Message | Tom Lane | 2010-09-26 19:50:06 | Re: Patch: Extend NOT NULL representation to pg_constraint |