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

  1. Improve requirements documentation for ldap test suite.