From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Magnus Hagander <mha(at)sollentuna(dot)net> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: [HACKERS] make check error on -HEAD |
Date: | 2004-11-01 19:21:40 |
Message-ID: | 200411011921.iA1JLen18932@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Patch applied. Thanks.
---------------------------------------------------------------------------
Magnus Hagander wrote:
> >>>I have just seen this failure on Windows - it appears to
> >have nothing to
> >>>do there with finding an out of date libpq.
> >>
> >>Actually, *any* failure in pg_regress's first attempt to invoke psql
> >>yields this message. I tweaked the script yesterday to not suppress
> >>stderr; do you have that update yet?
> >
> >Yes, but on Windows it doesn't give anything on stderr, it gives me a
> >nasty popup with no useful info I can see, and wants to send
> >info off to
> >microsoft.
>
> That's just the lovely way windows handles a "segfault". Gotta be really
> interesting for MS to catch all those dumps...
>
> Anyway. Oops. Seems I ran my regression tests with the old psql, and
> just managed to update the backend, when I tested that patch. Turns out
> there are codepaths where we'd access the Critical Section before it was
> initialized. Attached patch breaks the initializeation off to a separate
> part and adds that one to a much earlier position in the program.
>
> I've verified with Andrew that this fixes his problem.
>
> //Magnus
>
Content-Description: psql_win32lock.patch
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faqs/FAQ.html
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Marek Lewczuk | 2004-11-01 20:36:37 | Re: [PATCHES] Where is the link to cygwin? |
Previous Message | Magnus Hagander | 2004-11-01 19:06:55 | Re: [HACKERS] make check error on -HEAD |