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
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 |