Re: pgsql: Sigh ...

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)postgresql(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Sigh ...
Date: 2008-05-02 07:57:43
Message-ID: 481AC977.8030609@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers pgsql-patches

Tom Lane wrote:
> Log Message:
> -----------
> Sigh ... pg_config.h.win32 needs to define BLCKSZ and RELSEG_SIZE now.
>

Tom,

this reminds me a bit of Talleyrand's reply to the beggar who said to
him "Monseigneur, il faut que je vive," ... "Je n'en vois pas la
nécessité.".

This fix is surely wrong for several reasons:
. the configure changes only broke MSVC builds, not all Windows builds
(see narwhal, for example), but this change applies to both.
. fixing a change that adds a configure option by hardcoding it in
pg_config.h.win32 is simply the wrong fix - the right fix is to add the
equivalent logic to src/tools/Solution.pm.

I don't mind if you ask someone (realistically that will usually be
Magnus or me) to unbreak MSVC builds due to a configure change, because
you are not set up to test it yourself. But I do mind the wrong solution
being applied just to unbreak the buildfarm.

I make corrective surgery in the morning.

cheers

andrew

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2008-05-02 09:27:51 pgsql: One-letter backslash commands now need a space before the first
Previous Message Tom Lane 2008-05-02 03:41:46 pgsql: Sigh ...

Browse pgsql-hackers by date

  From Date Subject
Next Message Zdenek Kotala 2008-05-02 08:05:33 Re: Regression test fails when BLCKSZ is 1kB
Previous Message tomas 2008-05-02 05:51:47 Re: GDB X Postgresql 8 - Failed to read a valid object file image from memory

Browse pgsql-patches by date

  From Date Subject
Next Message Heikki Linnakangas 2008-05-02 11:55:45 Re: plpgsql CASE statement - last version
Previous Message Joshua D. Drake 2008-05-02 07:04:41 Re: configure option for XLOG_BLCKSZ