Re: Missing include in be-secure-openssl.c? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Missing include in be-secure-openssl.c?
Date
Msg-id 1129759.1635773615@sss.pgh.pa.us
Whole thread Raw
In response to Re: Missing include in be-secure-openssl.c?  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Missing include in be-secure-openssl.c?
List pgsql-hackers
Daniel Gustafsson <daniel@yesql.se> writes:
> It does make sense, but it's a bit worrisome that the indirect inclusion no
> longer works as there is no obvious explanation as to why.

Yeah.  Just to make things even more confusing, hamerkop is not failing
in the back branches.  v14 at least has exactly the same contents of
be-secure-openssl.c, so how's that happening?

>> x509v3.h includes x509.h, so fe-secure-openssl.h would not need an
>> update.  Now could it be a better practice to include both there?

> Judging by OpenSSL, including both is common practice unless the module only
> deals with v3 extensions. Following that lead seems reasonable.

I see that fe-secure-openssl.c includes only x509v3.h, and it builds
successfully on hamerkop.  So I'm now inclined to make be-secure-openssl.c
match that.  But it'd still be a good thing to trace the real cause.

            regards, tom lane



pgsql-hackers by date:

Previous
From: "osumi.takamichi@fujitsu.com"
Date:
Subject: RE: Failed transaction statistics to measure the logical replication progress
Next
From: Daniel Gustafsson
Date:
Subject: Commitfest 2021-11