Re: Add necessary package list to ldap TAP's README
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Postgres hackers <pgsql-hackers@postgresql.org>, Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Date: 2018-06-20T04:09:21Z
Lists: pgsql-hackers
On Wed, Jun 20, 2018 at 12:59:04PM +0900, Michael Paquier wrote: > On Tue, Jun 19, 2018 at 06:59:36PM -0400, Tom Lane wrote: > > Bruce Momjian <bruce@momjian.us> writes: > >> On Tue, Jun 19, 2018 at 06:45:54PM -0400, Tom Lane wrote: > >>> I was a bit disturbed that you'd push information that was "just guessed", > >>> so I went and tried to run the ldap tests on a couple different platforms. > > > >> I assumed they were guessed at and at least the URLs checked out. > >> Should we remove the patch? > > Yes, I was a bit surprised to see this patch pushed without checks about > what I guessed. I am not a user of those other platforms. It was in README that only developer read, so I thought even a guess would better than nothing. Maybe I was wrong. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + As you are, so once was I. As I am, so you will be. + + Ancient Roman grave inscription +
Commits
-
Improve requirements documentation for ldap test suite.
- 425b4c082c82 11.0 landed