Re: [PATCH] Remove unused #include's in src/backend/commands/* - Mailing list pgsql-hackers

From Andres Freund
Subject Re: [PATCH] Remove unused #include's in src/backend/commands/*
Date
Msg-id ln2ogaopaucmgpcztfb4xukpv3bcqspuoowa5urvtmpbz53uhh@6jrzttvn46bd
Whole thread Raw
In response to Re: [PATCH] Remove unused #include's in src/backend/commands/*  (Aleksander Alekseev <aleksander@tigerdata.com>)
Responses Re: [PATCH] Remove unused #include's in src/backend/commands/*
List pgsql-hackers
Hi,

On 2025-10-09 16:26:20 +0300, Aleksander Alekseev wrote:
> > > Apparently touching collationcmds.c, restricted_token.c and
> > > fe-auth-oauth.c was a mistake. Here is the corrected patch v4.
> >
> > Clearly this approach is not bulletproof enough to actually yield something
> > reliable enough to actually be applied.
> 
> I'm open to suggestions. I only have three machines to test my patches
> (Linux, MacOS and Windows, and I've just discovered that the master
> branch doesn't compile on Windows 11 at the moment).

Manually analyze each of the removed includes by hand. Fix all the issues that
lead to iwyu giving you wrong suggestions. Test it with CI in your own repo.

Or just don't do it.

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: Re: [PATCH] Remove unused #include's in src/backend/commands/*
Next
From: vignesh C
Date:
Subject: Re: Invalid pointer access in logical decoding after error