Re: tests fail on windows with default git settings

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Dave Page <dpage(at)pgadmin(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: tests fail on windows with default git settings
Date: 2024-07-11 11:29:30
Message-ID: 2c341c19-3a53-4d15-92f8-7aef38d44332@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 2024-07-11 Th 4:59 AM, Nazir Bilal Yavuz wrote:
> Hi,
>
> On Wed, 10 Jul 2024 at 17:04, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>>
>> On 2024-07-10 We 9:25 AM, Tom Lane wrote:
>>> Dave Page <dpage(at)pgadmin(dot)org> writes:
>>>> On Wed, 10 Jul 2024 at 12:12, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>>>>> As I was looking at this I wondered if there might be anywhere else that
>>>>> needed adjustment. One thing that occurred to me was that that maybe we
>>>>> should replace the use of "-w" in pg_regress.c with this rather less
>>>>> dangerous flag, so instead of ignoring any white space difference we would
>>>>> only ignore line end differences. The use of "-w" apparently dates back to
>>>>> 2009.
>>>> That seems like a good improvement to me.
>>> +1
>>>
>>>
>>
>> OK, done.
> It looks like Postgres CI did not like this change. 'Windows - Server
> 2019, VS 2019 - Meson & ninja' [1] task started to fail after this
> commit, there is one extra space at the end of line in regress test's
> output.
>
> [1] https://cirrus-ci.com/task/6753781205958656
>

Oh, that's annoying. Will investigate. Thanks for the heads up.

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Rafia Sabih 2024-07-11 12:07:27 Re: Things I don't like about \du's "Attributes" column
Previous Message Zhijie Hou (Fujitsu) 2024-07-11 11:22:03 RE: speed up a logical replica setup