Re: patch for win32 dynloader

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Magnus Hagander <mha(at)sollentuna(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers-win32(at)postgresql(dot)org, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: patch for win32 dynloader
Date: 2005-08-12 21:23:24
Message-ID: 200508122123.j7CLNON16297@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers-win32 pgsql-patches


Patch applied. Thanks.

Backpatched to 8.0.X.

---------------------------------------------------------------------------

Magnus Hagander wrote:
> > >> 3) I restarted the postmaster both times. I got this error
> > both times.
> > >> :25: ERROR: could not load library "C:/Program
> > >> Files/PostgreSQL/8.0/lib/testtrigfuncs.dll": dynamic load error
> >
> > > Yes. We really need to look at fixing that error message. I had
> > > forgotten it completely :-(
> >
> > > Bruce, you think we can sneak that in after feature freeze? I would
> > > call it a bugfix :-)
> >
> > Me too. That's been on the radar for awhile --- please do
> > send in a patch.
>
> Here we go, that wasn't too hard :-)
>
> Apart from adding the error handling, it does one more thing: it changes
> the errormode when loading the DLLs. Previously if a DLL was broken, or
> referenced other DLLs that couldn't be found, a popup dialog box would
> appear on the screen. Which had to be clicked before the backend could
> continue. This patch also disables the popup error message for DLL
> loads.
>
>
> I think this is something we should consider doing for the entire
> backend - disable those popups, and say we deal with it ourselves. What
> do you other win32 hackers thinnk about this?
>
> In the meantime, this patch fixes the error msgs. Please apply for 8.1
> and please consider a backpatch to 8.0.
>
>
> //Magnus

Content-Description: win32_dynloader_err.patch

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: explain analyze is your friend

--
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

In response to

Browse pgsql-hackers-win32 by date

  From Date Subject
Next Message Solange 2005-08-16 19:05:13 PostgreSQL 8.0.3 and Ipv6
Previous Message Dave Page 2005-08-11 14:24:34 Re: Installing postgres on win XP

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-08-12 21:37:06 Re: Win32 eventlog level fix
Previous Message Bruce Momjian 2005-08-12 21:20:39 Re: [HACKERS] Duplicate object names in GRANT