Re: [HACKERS] use of pager on Windows psql

Andrew Dunstan <[email protected]>
Newsgroups gmane.comp.db.postgresql.devel.patches
Message-ID <[email protected]>

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

-- 
Sent via pgsql-patches mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.