| Lists: | pgsql-patches |
|---|
| From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
|---|---|
| To: | Patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Install pg_regress by default |
| Date: | 2005-11-21 20:55:00 |
| Message-ID: | 20051121205500.GC31671@surnet.cl |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
This patch allows pg_regress to be installed by default. This was
proposed awhile back but never done, any objections if I commit this?
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
| Attachment | Content-Type | Size |
|---|---|---|
| regress-inst.patch | text/plain | 1.0 KB |
| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
| Cc: | Patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-21 21:17:09 |
| Message-ID: | 4239.1132607829@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> This patch allows pg_regress to be installed by default. This was
> proposed awhile back but never done, any objections if I commit this?
I've forgotten the details, but it seems like there was something else
that this depended on to be of any use? What are you envisioning people
doing with it?
BTW, I think you need an uninstall target too.
regards, tom lane
| From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Patches <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-21 21:37:52 |
| Message-ID: | 20051121213752.GA1305@surnet.cl |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Tom Lane wrote:
> Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> > This patch allows pg_regress to be installed by default. This was
> > proposed awhile back but never done, any objections if I commit this?
>
> I've forgotten the details, but it seems like there was something else
> that this depended on to be of any use? What are you envisioning people
> doing with it?
Actually it works, as you discovered some time ago, using PGXS. I use
it to test PL/php. (Yes, I'm hacking on that ugly thing.) I think
other module authors would also have some sort of regression testing;
having this in place eases things a bit.
I guess people whose packages need to support systems without PGXS
already have some testing harness in place, so it will be less useful to
them.
> BTW, I think you need an uninstall target too.
Ok.
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
| From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
|---|---|
| To: | pgsql-patches(at)postgresql(dot)org |
| Cc: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-26 23:24:01 |
| Message-ID: | 200511270024.02318.peter_e@gmx.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Alvaro Herrera wrote:
> This patch allows pg_regress to be installed by default. This was
> proposed awhile back but never done, any objections if I commit this?
If you do, there should also be documentation, such as a reference page,
about how to use it.
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
| From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
|---|---|
| To: | pgsql-patches(at)postgresql(dot)org |
| Cc: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-26 23:26:12 |
| Message-ID: | 200511270026.13151.peter_e@gmx.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Alvaro Herrera wrote:
> This patch allows pg_regress to be installed by default. This was
> proposed awhile back but never done, any objections if I commit this?
Scripts should be installed using INSTALL_SCRIPT (not _PROGRAM).
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
| From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
|---|---|
| To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
| Cc: | pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-28 16:56:05 |
| Message-ID: | 20051128165605.GC18576@surnet.cl |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Peter Eisentraut wrote:
> Alvaro Herrera wrote:
> > This patch allows pg_regress to be installed by default. This was
> > proposed awhile back but never done, any objections if I commit this?
>
> If you do, there should also be documentation, such as a reference page,
> about how to use it.
Applied with corrections, except for this. I'll do that later today.
Thanks,
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
| From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
|---|---|
| To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
| Cc: | pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: Install pg_regress by default |
| Date: | 2005-11-29 01:48:29 |
| Message-ID: | 20051129014828.GA28151@surnet.cl |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-patches |
Peter Eisentraut wrote:
> Alvaro Herrera wrote:
> > This patch allows pg_regress to be installed by default. This was
> > proposed awhile back but never done, any objections if I commit this?
>
> If you do, there should also be documentation, such as a reference page,
> about how to use it.
Ok, documented in the xfunc.sgml page, where the rest of PGXS is
documented. Feel free to editorialize ...
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support