| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org |
| Cc: | Robert Haas <rhaas(at)postgresql(dot)org>, pgsql-committers(at)postgresql(dot)org |
| Subject: | Re: pgsql: Remove outdated comments from the regression test files. |
| Date: | 2010-11-27 19:46:56 |
| Message-ID: | 29697.1290887216@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> Robert Haas wrote:
>> Remove outdated comments from the regression test files.
>>
>> Since 2004, int2 and int4 operators do detect overflow; this was fixed by
>> commit 4171bb869f234281a13bb862d3b1e577bf336242.
>>
>> Extracted from a larger patch by Andres Freund.
> I noticed with this commit that we are referencing pre-git-conversion
> git branches, basically adding a dependency on git to our commit
> messages. I don't see a problem with this, but did we ever reference
> CVS details in CVS commits? I don't remember any.
I've usually preferred to use a date, eg, "my patch of 2009-10-07",
when referring to previous patches in commit messages. I think people
have occasionally mentioned CVS revision IDs, but the folly of that
should now be obvious. I agree that reference to a git hash is way
way way too fragile and git-centric.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2010-11-27 19:49:37 | Re: pgsql: Remove outdated comments from the regression test files. |
| Previous Message | Bruce Momjian | 2010-11-27 18:45:13 | Re: pgsql: Remove outdated comments from the regression test files. |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2010-11-27 19:49:37 | Re: pgsql: Remove outdated comments from the regression test files. |
| Previous Message | Jeff Janes | 2010-11-27 19:44:59 | Re: contrib: auth_delay module |