Horton

Re: Missing include <openssl/x509.h> in be-secure-openssl.c?

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: Daniel Gustafsson <daniel@yesql.se>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, buildfarm@sraoss.co.jp
Date: 2021-11-01T22:33:46Z
Lists: pgsql-hackers 
Michael Paquier <michael@paquier.xyz> writes:
> I would follow the practice of upstream to include both if were me
> to be consistent, but I'm also fine if you just add x509v3.h to
> be-secure-openssl.c.

Another thing that's potentially relevant here is that
be-secure-openssl.c and fe-secure-openssl.c are including
ssl-related headers in different orders.  Theoretically
that shouldn't make any difference, but ...

			regards, tom lane

Commits

  1. contrib/sslinfo needs a fix too to make hamerkop happy.

  2. Second attempt to silence SSL compile failures on hamerkop.

  3. Blind attempt to silence SSL compile failures on hamerkop.