Re: pgsql: Add key management system

Bruce Momjian <bruce@momjian.us> writes:
> On Fri, Dec 25, 2020 at 02:37:06PM -0500, Tom Lane wrote:
>> pg_attribute_noreturn() seems like a good idea, but we're also going to
>> need dummy return statements in the callers, to satisfy compilers that
>> don't understand that.

> Yes, done. I tested it with a non-OpenSSL configure run now and it
> worked.  Thanks for the report.

Now that it compiles cleanly, what about test cases?  It looks
to me like you broke src/test/Makefile:

@@ -30,7 +30,7 @@ endif
 endif
 ifeq ($(with_openssl),yes)
 ifneq (,$(filter ssl,$(PG_TEST_EXTRA)))
-SUBDIRS += ssl
+SUBDIRS += ssl crypto
 endif
 endif

because there is no such subdirectory.  Assuming that that's a
case of forgetting to "git add" the whole subdirectory, I still
don't much care for this implementation: the user should be able
to decide which subdirectories get run.  Maybe more like

 ifeq ($(with_openssl),yes)
 ifneq (,$(filter ssl,$(PG_TEST_EXTRA)))
 SUBDIRS += ssl
 endif
+ifneq (,$(filter crypto,$(PG_TEST_EXTRA)))
+SUBDIRS += crypto
+endif
 endif

            regards, tom lane