Re: Missing file versions for a bunch of dll/exe files in Windows builds

From: Noah Misch <noah(at)leadboat(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: MauMau <maumau307(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-bugs <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: Missing file versions for a bunch of dll/exe files in Windows builds
Date: 2014-07-11 13:34:39
Message-ID: 20140711133439.GB1947575@tornado.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Fri, Jul 11, 2014 at 01:21:57PM +0900, Michael Paquier wrote:
> On Fri, Jul 11, 2014 at 12:02 AM, Noah Misch <noah(at)leadboat(dot)com> wrote:
> > Second, the MinGW build uses few or none of the new PGFILEDESC entries; you
> > need to mention $(WIN32RES) as in (most) existing PGFILEDESC-using makefiles.
> I see, that's specified in src/makefiles/Makefile.win32. For the
> portion of conpversion_procs, updating directly proc.mk seems to be
> enough, so done this way. Also, an entry in conversion_procs was not
> updated according to the PGFILEDESC you introduced.
>
> Btw, looking more at those patches, I found a limitation with
> contrib/spi for MinGW builds: in order to pass WIN32RES to OBJS, some
> of the contrib defined as MODULES need to be defined as MODULE_big to
> accept a custom list of OBJS, but MODULE_big is not able to accept a
> list of shared libraries. This limitation causes all the dll generated
> in contrib/spi to not be versioned with MinGW. MSVC works fine though.

Please fix the MODULES case, perhaps by linking $(WIN32RES) into each library
implicitly.

--
Noah Misch
EnterpriseDB http://www.enterprisedb.com

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2014-07-11 14:55:58 Re: Compile fails on AIX 6.1
Previous Message Mitu Verma 2014-07-11 08:03:03 Re: BUG #10888: application is getting hanged in the poll() function of libpq.so.