Re: [HACKERS] use of pager on Windows psql

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] use of pager on Windows psql
Date: 2008-05-18 14:06:17
Message-ID: 483037D9.1030900@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian wrote:
>
>
>> This broke the buildfarm and finally explains the following kluge which
>> has been puzzling me for four years:
>>
>> /*
>> * for some reason MinGW (and MSVC) outputs an extra newline, so
>> this
>> * suppresses it
>> */
>> #ifndef WIN32
>> fputc('\n', fout);
>> #endif
>>
>> I have removed the kluge (and yes, I tested it).
>>
>
> Oh, that kluge. Why did the isatty() addition fix this? Was the pager
> being used on Win32 for the regression tests and somehow eating a line
> or something?
>

It apparently produced an extra line which we had compensated for with
the kluge (without really understanding why we had to).

Anyway, all is good now, as the buildfarm shows.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Chernow 2008-05-18 14:33:24 Re: libpq object hooks (libpq events)
Previous Message Bruce Momjian 2008-05-18 13:49:45 Re: [HACKERS] use of pager on Windows psql

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Chernow 2008-05-18 14:33:24 Re: libpq object hooks (libpq events)
Previous Message Bruce Momjian 2008-05-18 13:49:45 Re: [HACKERS] use of pager on Windows psql