Обсуждение: Re: [BUGS] BUG #5394: invalid declspec for PG_MODULE_MAGIC
"Vladimir Barzionov" <snego.barsik@gmail.com> wrote: > Same problem was already discussed for example here > http://dbaspot.com/forums/postgresql/393683-re-general-custom-c-function-palloc-broken.html > > Looks like the simplest way for correcting the issue is declaring additional > macro (something like PGMODULEEXPORT) Sure, I agree it is a longstanding bug in PostgreSQL. Developers who use MSVC (not mingw) always encounter the bug; machines in the buildfarm can build Windows binaries just because they have non-standard equipments. A patch attached. The name of "PGMODULEEXPORT" might be arguable. Regards, --- Takahiro Itagaki NTT Open Source Software Center
Вложения
On Mon, Mar 29, 2010 at 11:47 AM, Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp> wrote: > > "Vladimir Barzionov" <snego.barsik@gmail.com> wrote: > >> Same problem was already discussed for example here >> http://dbaspot.com/forums/postgresql/393683-re-general-custom-c-function-palloc-broken.html >> >> Looks like the simplest way for correcting the issue is declaring additional >> macro (something like PGMODULEEXPORT) > > Sure, I agree it is a longstanding bug in PostgreSQL. Developers who use > MSVC (not mingw) always encounter the bug; machines in the buildfarm can > build Windows binaries just because they have non-standard equipments. > > A patch attached. The name of "PGMODULEEXPORT" might be arguable. I agree with this in principle, but won't this break every single add-on module out there that supports Win32? -- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/
Magnus Hagander <magnus@hagander.net> writes: > On Mon, Mar 29, 2010 at 11:47 AM, Takahiro Itagaki > <itagaki.takahiro@oss.ntt.co.jp> wrote: >> A patch attached. The name of "PGMODULEEXPORT" might be arguable. > I agree with this in principle, but won't this break every single > add-on module out there that supports Win32? The patch didn't touch the contrib modules, so why would it break third-party sources? regards, tom lane
On Tue, Apr 6, 2010 at 21:55, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Magnus Hagander <magnus@hagander.net> writes: >> On Mon, Mar 29, 2010 at 11:47 AM, Takahiro Itagaki >> <itagaki.takahiro@oss.ntt.co.jp> wrote: >>> A patch attached. The name of "PGMODULEEXPORT" might be arguable. > >> I agree with this in principle, but won't this break every single >> add-on module out there that supports Win32? > > The patch didn't touch the contrib modules, so why would it break > third-party sources? Oh, d'oh. I was clearly too tired when reading that thing. I was thinking we required changes in the contrib modules/third party modules, but they don't actually use DLLEXPORT, do they? In that case, that argument clearly falls, and I'm in favour of the patch :-) Just make sure you test it on both current and semi-old versions of mingw, they tend to not always act the same way. Or just get it in and the buildfarm can figure that part out for us. -- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/