Re: Importing pg_bsd_indent into our source tree

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Importing pg_bsd_indent into our source tree
Date: 2023-02-10 00:08:41
Message-ID: 20230210000841.4chvtmr4gkrfsotb@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2023-02-09 18:19:06 -0500, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On 2023-02-09 17:19:22 -0500, Tom Lane wrote:
> >> Hmmm .. a shell script would be fine by me, but it won't help in
> >> testing a Windows build. Maybe we need to make it a Perl script?
>
> > At least for casual testing a shell script actually mostly works, due to git
> > it's easy enough to have a sh.exe around... Not something I'd necessarily want
> > to make a hard dependency, but for something like this it might suffice. Of
> > course perl would be more dependable...
>
> Yeah, also less question about whether it works on Windows.
> I'll see about moving that into Perl. It's short enough.

Cool.

> > I also attached yours so that
> > cfbot could continue to work, if you registered this.
>
> I thought about registering it, but that won't teach us anything unless
> we make it built-by-default, which was not my intention. I guess we
> could temporarily include it in the build.

The meson patch I sent did build it by default, that's why I saw the windows
failure and the freebsd warnings. If we don't want that, we'd need to add
build_by_default: false

I'm fine either way. It's barely noticeable compared to the rest of postgres.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2023-02-10 00:20:37 Re: Importing pg_bsd_indent into our source tree
Previous Message Tatsuo Ishii 2023-02-10 00:07:01 Re: make_ctags: use -I option to ignore pg_node_attr macro