Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Tom Lane
Michael Paquier writes: > On Tue, Jun 19, 2018 at 06:59:36PM -0400, Tom Lane wrote: >> No, but it needs some adjustments. I'll go fix what I checked. > c992dca2 did not change anything about ldap packages. Are the ones > listed for RHEL and FreeBSD correct? No, that patch was more-or-less-unre

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Bruce Momjian
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 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", > >

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Michael Paquier
On Wed, Jun 20, 2018 at 11:06:52AM +1200, Thomas Munro wrote: > I think he used Brew paths? > > https://www.postgresql.org/message-id/0a7646b1-f6f6-1364-f0e3-814aed302db4%402ndquadrant.com Yes, stuff in /usr/local/ involves homebrew. -- Michael signature.asc Description: PGP signature

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Michael Paquier
On Tue, Jun 19, 2018 at 06:59:36PM -0400, Tom Lane wrote: > Bruce Momjian 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 platform

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Thomas Munro
On Wed, Jun 20, 2018 at 10:45 AM, Tom Lane wrote: > FreeBSD 12: you need openldap-client (NOT openldap24-client) to build > with --with-ldap at all. Then also openldap-server to run this test. Well, it's "pkg install openldap-client" but it's "net/openldap24-client" if you're building ports from

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Tom Lane
Bruce Momjian 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 chec

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Bruce Momjian
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 th

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Tom Lane
Bruce Momjian writes: > On Mon, May 21, 2018 at 10:34:25AM +0900, Michael Paquier wrote: >> The kerberos test suite mentions the package list to use on a set of >> distributions, which is very useful. However we don't do the same for >> ldap. Something like the attached would be adapted then to

Re: Add necessary package list to ldap TAP's README

2018-06-19 Thread Bruce Momjian
On Mon, May 21, 2018 at 10:34:25AM +0900, Michael Paquier wrote: > Hi all, > > The kerberos test suite mentions the package list to use on a set of > distributions, which is very useful. However we don't do the same for > ldap. Something like the attached would be adapted then to help setting >