Re: compiling pg_bsd_indent

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: compiling pg_bsd_indent
Date: 2025-10-09 15:57:39
Message-ID: aOfbc6FT-u542h8J@momjian.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Oct 9, 2025 at 11:49:31AM -0400, Andres Freund wrote:
> Hi,
>
> On 2025-10-09 11:43:57 -0400, Robert Haas wrote:
> > While $SUBJECT today -- using the sources from
> > https://git.postgresql.org/git/pg_bsd_indent.git -- I ran into some
> > problems. I got errors due to the replacement of
> > pg_attribute_noreturn() with pg_return(), and warnings due to variable
> > definitions with no previous extern declaration. Here's a quick and
> > dirty patch.
>
> I thought we basically only used the in-tree pg_bsd_indent these days?

The last commit in that branch says:

commit 681a370 (HEAD -> master, origin/master, origin/HEAD)
Author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Date: Sun Apr 23 10:38:15 2023 -0400

Mark this git repo as obsolete.

Add a README note explaining that we've pulled pg_bsd_indent
into the main Postgres source tree. This copy will not be
maintained anymore.

Might be nice if we could just remove it. It was created by Tom Lane in
2017.

The in-tree ./src/tools/pg_bsd_indent/ doesn't give me any warnings on
gcc 14.2.0.

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com

Do not let urgent matters crowd out time for investment in the future.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2025-10-09 15:57:43 Re: compiling pg_bsd_indent
Previous Message Robert Haas 2025-10-09 15:53:29 Re: compiling pg_bsd_indent