Re: win32 version info

From: "Magnus Hagander" <mha(at)sollentuna(dot)net>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "PostgreSQL-patches" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: win32 version info
Date: 2004-07-26 18:51:22
Message-ID: 6BCB9D8A16AC4241919521715F4D8BCE4569AF@algol.sollentuna.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

>> This patch along with the attached files (.tar.gz unpacks in
>src/) adds
>> VERSIONINFO to all binaries under win32.
>
>Ya know, this is the sort of invasive junk that I was afraid people
>would try to force on us for the Windows port :-(.

I fail to see how this is so invasive. And I definitly don't think it's
junk. But that's just me, I suppose (along with windows users, but "we"
clearly don't care much about them).

>Can't you localize this into a single build rule somewhere? And surely
>we do not need to maintain a boilerplate .rc file for every executable.

There is a field in the RC file that is "file description". It's pretty
much different for every file. (There is also a "product description" in
there which is the same).

As for putting it in a single build rule - I don't know if that can be
done. I thought it over and didn't come up with a way, but there may be
other who are better at writing Makefiles... I guess a script could be
written that generates the file, but the per-binary information has to
be somewhere. Not sure how that method would be cleaner.

>> Finally, the patch adds an elephant icon to psql.exe (so
>it'll go nicely
>> in the start menu for example).
>
>You need a lot better reason than that to convince me to put a binary
>file into CVS.

Well, if someone can come up with a way to build an icon and link it
into the binary without any binary files, I'm all for it. We can of
course do without it, but it does make the win32 look very
unproffessional.

I probably don't understand what the potential problems are with binary
files in cvs, because I don't quite see the issue...

//Magnus

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Sean Chittenden 2004-07-26 19:25:18 pg_ctl -o option dumps core when processing postmaster arguments...
Previous Message Magnus Hagander 2004-07-26 18:39:25 Re: win32 version info