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

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: 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-04-24 07:27:35
Message-ID: CABUevEyDe4FWupst7iB1E8p3720AGj0Ejpxcfy1KA-a9iQBO-w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Thu, Apr 24, 2014 at 7:54 AM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com>wrote:

> Hi all,
>
> A colleague of mine pointed out that the file version is missing in a
> couple of dll and exe files when building on windows using the
> community scripts in src/tools/msvc. After having a closer look, I
> noticed that a *lot* of files are missing the shot:
> - all the exe/dll in contrib/
> - dll of PL languages (perl, python, tcl, pgsql)
> - libpqwalreceiver, snowball
> - ecpg stuff
> - regression and isolation test stuff
> - conversion_procs thingies
> Having a version number associated to a build is important for all
> companies creating builds of Postgres on Windows, so it would be good
> to have the patch attached applied and back-patched.
>
>
At least some fo that is intentional - things that are considered
"internal" were not given a version resource intentionally. E.g. the
conversion_procs is very intentional. The EXE/DLL in contrib should
definitely have them though.

it also seems like the wrong way to go about it - for all the other files,
it's added by rule (when PGFILEDESC is specified in the Makefile). Which
currently appears to be the *only* way it's added, so are you saying this
just doesn't work? Or does it work for some of them?

I think the proper solutioni s to add PGFILEDESC entries to the Makefile's,
and if that one doesn't actually work then fix the build system to work :)
(Sorry, don't have a win32 build environment around to test it right now)

Then it will be consistent between mingw and msvc, which your patch isn't,
I believe?

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Dave Page 2014-04-24 07:50:01 Re: Missing file versions for a bunch of dll/exe files in Windows builds
Previous Message tcoq 2014-04-24 06:37:34 LOG: incomplete message from client