[SSSD] [PATCH] Cleanups for library linking

2009-09-09 Thread Sumit Bose
Hi,

I think the LDAP provider should link agains a LDAP library. So far it
only worked, because sssd_be exports all symbols and links against
libldb which links against libldap. 

bye,
Sumit
From 911e1f245e00b95621d1d21e43c688e8973ceb12 Mon Sep 17 00:00:00 2001
From: sbose sb...@sles10.site
Date: Wed, 9 Sep 2009 12:14:07 +0200
Subject: [PATCH] Cleanups for library linking

- remove unused PAM_LIBS from LDAP and Kerberos provider
- add OPENLDAP_LIBS to LDAP provider
---
 server/Makefile.am |3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/server/Makefile.am b/server/Makefile.am
index e7885b3..27ac01d 100644
--- a/server/Makefile.am
+++ b/server/Makefile.am
@@ -374,7 +374,7 @@ libsss_ldap_la_CFLAGS = \
 $(AM_CFLAGS) \
 $(LDAP_CFLAGS)
 libsss_ldap_la_LIBADD = \
-$(PAM_LIBS)
+$(OPENLDAP_LIBS)
 libsss_ldap_la_LDFLAGS = \
 -version-info 1:0:0 \
 -module
@@ -396,7 +396,6 @@ libsss_krb5_la_CFLAGS = \
 $(AM_CFLAGS) \
 $(KRB5_CFLAGS)
 libsss_krb5_la_LIBADD = \
-$(PAM_LIBS) \
 $(KRB5_LIBS)
 libsss_krb5_la_LDFLAGS = \
 -version-info 1:0:0 \
-- 
1.6.2.5

___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel


Re: [SSSD] [PATCH] Cleanups for library linking

2009-09-09 Thread Simo Sorce
On Wed, 2009-09-09 at 14:19 +0200, Sumit Bose wrote:
 Hi,
 
 I think the LDAP provider should link agains a LDAP library. So far it
 only worked, because sssd_be exports all symbols and links against
 libldb which links against libldap. 

Absolutely, full ack!
Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel


Re: [SSSD] [PATCH] Cleanups for library linking

2009-09-09 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 09/09/2009 08:36 AM, Simo Sorce wrote:
 On Wed, 2009-09-09 at 14:19 +0200, Sumit Bose wrote:
 Hi,

 I think the LDAP provider should link agains a LDAP library. So far it
 only worked, because sssd_be exports all symbols and links against
 libldb which links against libldap. 
 
 Absolutely, full ack!
 Simo.
 

Pushed to master.

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkqnqnQACgkQeiVVYja6o6OyuwCfV8BxN3tAF7iTqzbtWmhmpk5P
WaQAniaLdZxW4Nu/8H5sNoUGM06+gTIf
=X718
-END PGP SIGNATURE-
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel