From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
Cc: | Florents Tselai <florents(dot)tselai(at)gmail(dot)com>, Euler Taveira <euler(at)eulerto(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, jian he <jian(dot)universality(at)gmail(dot)com>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: split func.sgml to separated individual sgml files |
Date: | 2025-10-02 12:27:10 |
Message-ID: | 416f21b0-96a4-447d-a122-4b7d2f3daa3e@dunslane.net |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2025-10-02 Th 2:58 AM, Nazir Bilal Yavuz wrote:
> Hi,
>
> On Wed, 1 Oct 2025 at 23:02, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>>
>> On 2025-10-01 We 8:27 AM, Nazir Bilal Yavuz wrote:
>>> Hi,
>>>
>>> On Wed, 1 Oct 2025 at 15:09, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> wrote:
>>>> On Tue, 30 Sept 2025 at 22:48, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>>>>> Hi Bilal,
>>>>>
>>>>> This got preempted slightly by Tom's commit 170a8a3f460, but I think
>>>>> it's worth doing. I tried to simplify it some. See attached. There
>>>>> doesn't seem to me to be any point in using a different set of files for
>>>>> the tab tests and the NBSP tests. If we use the same set of files we can
>>>>> improve the efficiency easily by opening them only once. Here we just
>>>>> look for all the sgml files and all the xsl files and process them all.
>>>>>
>>>>> WDYT?
>>>> It looks good to me. I made 2 changes to your patch:
>>>>
>>>> 1- Declaration of $line_no is lost, I re-added it.
>>>> 2- s/.cirrus.tasks,yml/.cirrus.tasks.yml/ in the commit message.
>>> Two more minor changes that I missed in the v2:
>>>
>>> 1- I added $line_no and removed $_ from the tab check's warning
>>> message. I think it is better this way, otherwise if the line only
>>> contains tab character; $_ will print an empty looking line.
>>> 2- s/Tabsand/Tabs and/
>>>
>> OK, thanks, looks good. How do we go about doing what Tom wants (i.e.
>> running the tests by default) under meson. I think in the Makefile we
>> could just add it to the html target.
> I might be misunderstanding, but these syntax checks already run by
> default under meson build with this patch. Would we just need to add
> this test to the HTML target in the Makefile?
>
Oh, ok, I missed that about meson. I will adjust the Makefile.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2025-10-02 12:33:09 | Re: [PATCH] Add tests for Bitmapset |
Previous Message | Tatsuo Ishii | 2025-10-02 12:15:50 | Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options |