Re: [PATCH] test/ssl: rework the sslfiles Makefile target
Jacob Champion <pchampion@vmware.com>
From: Jacob Champion <pchampion@vmware.com>
To: "michael@paquier.xyz" <michael@paquier.xyz>
Cc: "daniel@yesql.se" <daniel@yesql.se>, "andrew@dunslane.net" <andrew@dunslane.net>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>, "tgl@sss.pgh.pa.us" <tgl@sss.pgh.pa.us>
Date: 2021-09-03T23:21:59Z
Lists: pgsql-hackers
On Fri, 2021-09-03 at 09:46 +0900, Michael Paquier wrote: > Nice. This is neat. The split helps a lot to understand how you've > changed things from the original implementation. As a whole, this > looks rather committable to me. Great! > One small-ish comment that I have is about all the .config files we > have at the root of src/test/ssl/, bloating the whole. I think that > it would be a bit cleaner to put all of them in a different > sub-directory, say just config/ or conf/. That sounds reasonable. I won't be able to get to it before the holiday weekend, but I can put up a patch sometime next week. Thanks, --Jacob
Commits
-
Refactor the sslfiles Makefile target for ease of use
- b4c4a00eada3 15.0 landed
-
Remove unused regression test certificate server-ss
- 152c2e0ae1a8 15.0 landed