Regression tests

Started by Magnus Haganderabout 19 years ago7 messagespatches
Jump to latest
#1Magnus Hagander
magnus@hagander.net

Joachim Wieland attempted to post this patch, but it appears to be gone.
I tried a repost, and notivced it got rejected because it was >100kb.
Let me repeat previous objections that it really should be possible to
post a patch >100kb.
That said, here's a gzipped version.

Joachim, once it comes through, feel free to post whatever comments you
had in your original mail.

//Magnus

Attachments:

pg_regression_msvc.3.diff.gzapplication/x-gzip; name=pg_regression_msvc.3.diff.gzDownload
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Magnus Hagander (#1)
Re: Regression tests

Magnus Hagander <magnus@hagander.net> writes:

Joachim Wieland attempted to post this patch, but it appears to be gone.

I trust the applied version will contain neither Windows newlines nor
non-English comments.

regards, tom lane

#3Magnus Hagander
magnus@hagander.net
In reply to: Tom Lane (#2)
Re: Regression tests

Tom Lane wrote:

Magnus Hagander <magnus@hagander.net> writes:

Joachim Wieland attempted to post this patch, but it appears to be gone.

I trust the applied version will contain neither Windows newlines nor
non-English comments.

Certainly would assume so. I haven't actually looked at the patch yet, I
just forwarded it.
(And I may well be responsible for the windows newlines - it may have
happened when I gzipped it on a windows box)

//Magnus

#4Magnus Hagander
magnus@hagander.net
In reply to: Magnus Hagander (#1)
Re: Regression tests

Magnus Hagander wrote:

Joachim Wieland attempted to post this patch, but it appears to be gone.
I tried a repost, and notivced it got rejected because it was >100kb.
Let me repeat previous objections that it really should be possible to
post a patch >100kb.
That said, here's a gzipped version.

Joachim, once it comes through, feel free to post whatever comments you
had in your original mail.

Here's an updated version of this patch. I reversed some parts of where
files went, which made the patch a whole lot smaller and easier to read,
and changed a couple of smaller things.

I have not yet looked at the actual ecpg parts (except noticed that they
are working on msvc at least), just the changes to main regression
tests. But I wanted to send off my new version for others to look at as
well before I do that.

This is still WIP, but comments appreciated.

//Magnus

Attachments:

regress.difftext/plain; name=regress.diffDownload+484-338
ecpg_scheduletext/plain; name=ecpg_scheduleDownload
pg_regress_ecpg.ctext/plain; name=pg_regress_ecpg.cDownload
pg_regress_main.ctext/plain; name=pg_regress_main.cDownload
pg_regress.htext/plain; name=pg_regress.hDownload
#5Magnus Hagander
magnus@hagander.net
In reply to: Magnus Hagander (#4)
Re: Regression tests

On Sat, Jun 09, 2007 at 10:55:55PM +0200, Magnus Hagander wrote:

Magnus Hagander wrote:

Joachim Wieland attempted to post this patch, but it appears to be gone.
I tried a repost, and notivced it got rejected because it was >100kb.
Let me repeat previous objections that it really should be possible to
post a patch >100kb.
That said, here's a gzipped version.

Joachim, once it comes through, feel free to post whatever comments you
had in your original mail.

Here's an updated version of this patch. I reversed some parts of where
files went, which made the patch a whole lot smaller and easier to read,
and changed a couple of smaller things.

I have not yet looked at the actual ecpg parts (except noticed that they
are working on msvc at least), just the changes to main regression
tests. But I wanted to send off my new version for others to look at as
well before I do that.

This is still WIP, but comments appreciated.

I have applied a fairly well reworked version of this. The big thing is
that I moved the building of the pgc code out of the regression test driver
and into the build system using msbuild.

I also did a couple of minor fixes to the threading tests and such.

//Magnus

#6Michael Meskes
meskes@postgresql.org
In reply to: Magnus Hagander (#5)
Re: Regression tests

On Tue, Jun 12, 2007 at 01:16:32PM +0200, Magnus Hagander wrote:

I have applied a fairly well reworked version of this. The big thing is
that I moved the building of the pgc code out of the regression test driver
and into the build system using msbuild.

I also did a couple of minor fixes to the threading tests and such.

Thanks.

It seems you missed one file. make checktcp says:

pg_regress: could not open file "./ecpg_schedule_tcp" for reading

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes@jabber.org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

#7Magnus Hagander
magnus@hagander.net
In reply to: Michael Meskes (#6)
Re: Regression tests

On Tue, Jun 12, 2007 at 02:25:24PM +0200, Michael Meskes wrote:

On Tue, Jun 12, 2007 at 01:16:32PM +0200, Magnus Hagander wrote:

I have applied a fairly well reworked version of this. The big thing is
that I moved the building of the pgc code out of the regression test driver
and into the build system using msbuild.

I also did a couple of minor fixes to the threading tests and such.

Thanks.

It seems you missed one file. make checktcp says:

pg_regress: could not open file "./ecpg_schedule_tcp" for reading

Oops, search/replace error. Fixed.

//Magnus