From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Sigh ... |
Date: | 2008-05-02 19:50:44 |
Message-ID: | 481B7094.3060709@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers pgsql-patches |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Tom Lane wrote:
>>
>>> Well, maybe the right answer is to take a step back and figure out what
>>> pg_config.h.win32's excuse for living is at all.
>>>
>
>
>> Oops, the file is only used by MSVC/BCC, not by MinGW. Sorry for the
>> mistake about that.
>>
>
> Well, I'm still wondering why it is an input for MSVC. If we have a
> configure substitute for MSVC, why isn't it working off pg_config.h.in?
>
> The original idea of pg_config.h.win32 was to support the pre-8.0
> method for building native libpq.dll. I'd like to see us obsolete that
> method altogether and get rid of pg_config.h.win32.
>
That makes sense - I'll look at that after I have cleaned up the current
stuff.
>
>> However, all the values are hardcoded, so nothing in it should relate to
>> settings that come from configure, I believe. These should be dealt with
>> in src/tools/msvc/Solution.pm (mostly in GenerateFiles() ).
>>
>
> FYI, I'm about to commit changes moving XLOG_BLCKSZ and XLOG_SEG_SIZE
> into the domain of configurable stuff, too.
>
>
>
So I gathered. I'll pick those up as part of the fixes I'm currently coding.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-05-02 19:52:37 | pgsql: Extend yesterday's patch making BLCKSZ and RELSEG_SIZE |
Previous Message | Tom Lane | 2008-05-02 19:44:13 | Re: [COMMITTERS] pgsql: Sigh ... |
From | Date | Subject | |
---|---|---|---|
Next Message | Chris Browne | 2008-05-02 19:58:01 | Re: Protection from SQL injection |
Previous Message | Tom Lane | 2008-05-02 19:44:13 | Re: [COMMITTERS] pgsql: Sigh ... |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-05-02 19:57:28 | Re: configure option for XLOG_BLCKSZ |
Previous Message | Tom Lane | 2008-05-02 19:44:13 | Re: [COMMITTERS] pgsql: Sigh ... |