Hi,
On 2022-09-16 16:22:35 -0400, Tom Lane wrote:
> Andres Freund <and...@anarazel.de> writes:
> > On 2022-09-16 09:14:20 +1200, Thomas Munro wrote:
> >> On Thu, Sep 15, 2022 at 2:26 PM Andres Freund <and...@anarazel.de> wrote:
> >>> - noticed that libpgport.a had and needed a dependency on errcodes.h -
> >>> that
> >>> seemed wrong. The dependency is due to src/port/*p{read,write}v?.c
> >>> including
> >>> postgres.h - which seems wrong. So I added a patch changing them to
> >>> include
> >>> c.h.
>
> >> Oops. +1
>
> > Looks like this has been the case since
> > 0d56acfbaa799553c0c6ea350fd6e68d81025994 in 14. Any opinions on whether we
> > should backpatch the "fix"?
>
> +1, those files have no business including all of postgres.h
Done.
I've been wondering whether we should protect against this kind of issue on
the buildsystem level. Whenever building frontend code, add something like
-DBUILDING_FRONTEND, and error out if postgres.h is included without going
through postgres_fe.h.
Greetings,
Andres Freund
