| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Ian Lance Taylor <ian(at)airs(dot)com> |
| Cc: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Patch application |
| Date: | 2001-03-19 21:08:11 |
| Message-ID: | 25956.985036091@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-jdbc pgsql-odbc |
Ian Lance Taylor <ian(at)airs(dot)com> writes:
> In projects like gcc and the GNU binutils, we use a MAINTAINERS file.
> Some people have blanket write privileges. Some people have write
> priviliges to certain areas of the code. Anybody else needs a patch
> to be approved before they can check it in. Patches which are
> ``obviously correct'' are always OK.
Would you enlarge on what that fourth sentence means in practice?
Seems like the sticky issue here is what constitutes "approval".
We already have a policy that changes originating from non-committers
are supposed to be reviewed before they get applied, but what Bruce
is worried about is the quality of the review process.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ian Lance Taylor | 2001-03-19 21:34:14 | Re: Patch application |
| Previous Message | Bruce Momjian | 2001-03-19 21:04:33 | Re: Patch application |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ian Lance Taylor | 2001-03-19 21:34:14 | Re: Patch application |
| Previous Message | Bruce Momjian | 2001-03-19 21:04:33 | Re: Patch application |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ian Lance Taylor | 2001-03-19 21:34:14 | Re: Patch application |
| Previous Message | Bruce Momjian | 2001-03-19 21:04:33 | Re: Patch application |