Bug 201957 - [patch] net/freeradius3 Correct LDAP option
Summary: [patch] net/freeradius3 Correct LDAP option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Ryan Steinmetz
URL:
Keywords: patch
: 202681 203259 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-07-29 04:05 UTC by Emmanuel Vadot
Modified: 2015-09-24 03:51 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (zi)

Attachments
add WANT_OPENLDAP_SASL (353 bytes, patch)
2015-07-29 04:05 UTC, Emmanuel Vadot 		no flags Details | Diff
LIB_DEPENDS cyrus-sasl2 (391 bytes, patch)
2015-09-19 12:07 UTC, geoffroy desvernay 		dgeo: maintainer-approval?
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Emmanuel Vadot 2015-07-29 04:05:06 UTC 
Created attachment 159353 [details]
add WANT_OPENLDAP_SASL

The LDAP module of freeradius3 wants sasl support.
This patch make it depends on openldap-client-sasl and not openldap-client.

Cheers,
Comment 1 amvandemore 2015-08-26 18:10:50 UTC 
*** Bug 202681 has been marked as a duplicate of this bug. ***
Comment 2 Ryan Steinmetz freebsd_committer freebsd_triage 2015-08-26 22:04:25 UTC 
Can you try your build again with adding the following?

LIB_DEPENDS+=           libsasl2.so:${PORTSDIR}/security/cyrus-sasl2

If this works, I think it will be safer than trying to force OpenLDAP with SASL support.
Comment 3 amvandemore 2015-08-26 22:26:57 UTC 
Yes that works for me.
Comment 4 geoffroy desvernay 2015-09-19 12:06:53 UTC 
Here we don't use OPENLDAP_SASL, an I'd prefer not to be forced to use it (as you have to recompile everything to change that)

Tried here
LIB_DEPENDS+=  libsasl2.so:${PORTSDIR}/security/cyrus-sasl2

It seems to work (for me)
Comment 5 geoffroy desvernay 2015-09-19 12:07:33 UTC 
Created attachment 161188 [details]
LIB_DEPENDS cyrus-sasl2
Comment 6 Ryan Steinmetz freebsd_committer freebsd_triage 2015-09-24 03:48:16 UTC 
*** Bug 203259 has been marked as a duplicate of this bug. ***
Comment 7 commit-hook freebsd_committer freebsd_triage 2015-09-24 03:50:40 UTC 
A commit references this bug:

Author: zi
Date: Thu Sep 24 03:50:06 UTC 2015
New revision: 397675
URL: https://svnweb.freebsd.org/changeset/ports/397675

Log:
  - Resolve build failure in certain cases when LDAP support is enabled

  PR:		201957
  Reported by:	Emmanuel Vadot <manu@bidouilliste.com>

Changes:
  head/net/freeradius3/Makefile
Comment 8 Ryan Steinmetz freebsd_committer freebsd_triage 2015-09-24 03:51:10 UTC 
Fix committed.