On 2014/10/27 18:59, Giovanni Bechis wrote:
>  DISTNAME=            courier-authlib-0.66.1
> +REVISION=            1
>  PKGNAME-main=                ${DISTNAME}
>  PKGNAME-ldap=                ${DISTNAME:S/lib-/lib-ldap-/}
>  PKGNAME-mysql=               ${DISTNAME:S/lib-/lib-mysql-/}
>  PKGNAME-pgsql=               ${DISTNAME:S/lib-/lib-pgsql-/}
>  PKGNAME-userdb=              ${DISTNAME:S/lib-/lib-userdb-/}
>  
> -REVISION=            0
> -REVISION-mysql=              1
> +REVISION-mysql=              2
> +REVISION-ldap=               0
> +REVISION-pgsql=              0
> +REVISION-userdb=     0

Could you put all of the REVISION variables together please? It will
reduce the risk of accidents if people bump things later..
Actually, it might be simpler to just have a single REVISION=2 line
instead of a bunch of different ones for all the subpackages..you're
bumping all of them anyway so there's no disadvantage to doing that.

> -CONFIGURE_ENV=               LIBS="-L${LOCALBASE}/lib" \
> +CONFIGURE_ENV=               LIBS="-L${LOCALBASE}/lib -lpthread" \

Is it enough to just add -lpthread to authdaemond rather than to
everything? Like this ..

Index: patches/patch-Makefile_in
===================================================================
RCS file: /cvs/ports/mail/courier-authlib/patches/patch-Makefile_in,v
retrieving revision 1.4
diff -u -p -r1.4 patch-Makefile_in
--- patches/patch-Makefile_in   11 Apr 2014 09:35:04 -0000      1.4
+++ patches/patch-Makefile_in   27 Oct 2014 19:20:13 -0000
@@ -1,6 +1,6 @@
 $OpenBSD: patch-Makefile_in,v 1.4 2014/04/11 09:35:04 giovanni Exp $
---- Makefile.in.orig   Mon Nov 11 19:58:27 2013
-+++ Makefile.in        Sun Jan 12 17:31:43 2014
+--- Makefile.in.orig   Mon Nov 11 18:58:27 2013
++++ Makefile.in        Mon Oct 27 19:19:57 2014
 @@ -654,7 +654,7 @@ CLEANFILES = authldaprc.h authmysqlrc.h authsqliterc.h
  
  DISTCLEANFILES = dbobj.config README_authlib.html
@@ -46,3 +46,12 @@ $OpenBSD: patch-Makefile_in,v 1.4 2014/0
  include_HEADERS = courier_auth_config.h courierauth.h courierauthsasl.h \
        courierauthsaslclient.h courierauthdebug.h courierauthstaticlist.h
  
+@@ -822,7 +822,7 @@ authdaemondprog_LDADD = $(LIBLTDL) libcourierauthcommo
+       libs/libhmac/libhmac.la libs/md5/libmd5.la libs/sha1/libsha1.la \
+       libs/rfc822/libencode.la libs/numlib/libnumlib.la
+ 
+-authdaemondprog_LDFLAGS = -export-dynamic $(modules:%=-dlopen %) @NETLIBS@
++authdaemondprog_LDFLAGS = -export-dynamic $(modules:%=-dlopen %) -lpthread 
@NETLIBS@
+ authdaemontest_SOURCES = authdaemontest.c
+ authdaemontest_DEPENDENCIES = libcourierauthcommon.la
+ authdaemontest_LDADD = libcourierauthcommon.la @NETLIBS@

Reply via email to