Re: pgsql: Fix headerscheck failure in replication/worker_internal.h - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: pgsql: Fix headerscheck failure in replication/worker_internal.h
Date
Msg-id 20211116171430.GI26257@tamriel.snowman.net
Whole thread Raw
In response to Re: pgsql: Fix headerscheck failure in replication/worker_internal.h  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Responses Re: pgsql: Fix headerscheck failure in replication/worker_internal.h
List pgsql-hackers
Greetings,

* Alvaro Herrera (alvherre@alvh.no-ip.org) wrote:
> On 2021-Nov-16, Alvaro Herrera wrote:
>
> > Fix headerscheck failure in replication/worker_internal.h
>
> The other failure is in src/include/libpq/be-gssapi-common.h:
>
> In file included from /tmp/headerscheck.a6f0um/test.c:2:
> ./src/include/libpq/be-gssapi-common.h:20:10: fatal error: gssapi/gssapi.h: No existe el fichero o el directorio
>    20 | #include <gssapi/gssapi.h>
>       |          ^~~~~~~~~~~~~~~~~
> compilation terminated.
>
> One possible solution for this one is to add an exclusion in
> headerscheck (and cpluspluscheck?); the other possible solution seems to
> be to wrap the whole contents of the file in "#ifdef ENABLE_GSS".  I
> think the latter is roughly the approach used for OpenSSL inclusions.

Short answer is yes, inclusion of be-gssapi-common.h is typically
wrapped in a #ifdef, see src/backend/libpq/auth.c

Thanks,

Stephen

Attachment

pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: pgsql: Fix headerscheck failure in replication/worker_internal.h
Next
From: Alvaro Herrera
Date:
Subject: Re: pgsql: Fix headerscheck failure in replication/worker_internal.h