On Tuesday, December 27, 2011 17:37 CET, Piotr Sikora 
<piotr.sik...@frickle.com> wrote: 
 
> Hey Sebastian,
> 
> > So how about this now?
> 
> SHARED_LIBS looks good now, thanks.
> 
> However, I think we should also look into fixing PLIST. I believe that 
> SHARED_LIBS versions should be added to SUBST_VARS automatically by 
> bsd.port.mk, but it seems that doesn't happen and we've got explicit version 
> numbers listed in the PLIST file, would you mind looking into that?
> 
> Other than that it looks good to me.

The last diff I sent, did not had the PLIST changes contained :(

However, I don't think that the SHARED_LIBS should be added to SUBST_VARS, since
how should it decide which version number to take? I.e. libmemcached, hashkit, 
and libmemcachedutil are all three 1.0.
How should it know where to take which of those substitutions. In the 
PFRAG.shared file it knows, since there the 
library name is specified.

so OK?

Sebastian

Index: Makefile
===================================================================
RCS file: /cvs/ports/devel/libmemcached/Makefile,v
retrieving revision 1.10
diff -u -r1.10 Makefile
--- Makefile    11 Jul 2011 14:04:42 -0000      1.10
+++ Makefile    28 Dec 2011 11:52:37 -0000
@@ -2,9 +2,8 @@
 
 COMMENT=               C C++ API for memcached
 
-VERSION=               0.48
+VERSION =              1.0.2
 DISTNAME=              libmemcached-${VERSION}
-REVISION=              0
 CATEGORIES=            devel misc
 
 HOMEPAGE=              http://libmemcached.org/
@@ -13,10 +12,10 @@
 MAINTAINER=            Sebastian Reitenbach <sebas...@openbsd.org>
 
 USE_LIBTOOL=           Yes
-SHARED_LIBS +=  memcached                 0.0 # 0.0
-SHARED_LIBS +=  hashkit                   0.0 # 0.0
+SHARED_LIBS +=  memcached                 1.0 # 8.0
+SHARED_LIBS +=  hashkit                   1.0 # 1.0
 SHARED_LIBS +=  memcachedprotocol         0.0 # 0.0
-SHARED_LIBS +=  memcachedutil             0.0 # 1.0
+SHARED_LIBS +=  memcachedutil             1.0 # 2.0
 
 # BSD
 PERMIT_PACKAGE_CDROM=   Yes
@@ -25,7 +24,8 @@
 PERMIT_DISTFILES_FTP=   Yes
 
 USE_GROFF =    Yes
-WANTLIB += c event m pthread sasl2
+USE_GMAKE =    Yes
+WANTLIB += c m pthread sasl2 stdc++
 LIB_DEPENDS=           security/cyrus-sasl2
 REGRESS_DEPENDS=       misc/memcached
 
Index: distinfo
===================================================================
RCS file: /cvs/ports/devel/libmemcached/distinfo,v
retrieving revision 1.3
diff -u -r1.3 distinfo
--- distinfo    14 Apr 2011 11:17:49 -0000      1.3
+++ distinfo    28 Dec 2011 11:52:37 -0000
@@ -1,5 +1,5 @@
-MD5 (libmemcached-0.48.tar.gz) = 4ArlFJKh+dy7bPulDcVHWA==
-RMD160 (libmemcached-0.48.tar.gz) = h7L+IkDHDPZuPWWI8ZyfEt+qhAE=
-SHA1 (libmemcached-0.48.tar.gz) = 9z0K1Va+QQl95rQmhyhBLxfaNUI=
-SHA256 (libmemcached-0.48.tar.gz) = 
rQDyqGBhy1YfQiCDED+Y1eUAASPDkXfPXhtNv3bwUpY=
-SIZE (libmemcached-0.48.tar.gz) = 763754
+MD5 (libmemcached-1.0.2.tar.gz) = Wh69XeIvbiRHTLn1eUVv1Q==
+RMD160 (libmemcached-1.0.2.tar.gz) = gKGDq2hToMwHy7O811IobBhyIEg=
+SHA1 (libmemcached-1.0.2.tar.gz) = 2ZKEX+cSEOLpMUe6jynfH8oU5xY=
+SHA256 (libmemcached-1.0.2.tar.gz) = 
xcoh27hc5KTekeSu1rPR+KRZmYCqYG7wgS7rUrfQF/I=
+SIZE (libmemcached-1.0.2.tar.gz) = 942891
Index: patches/patch-configure
===================================================================
RCS file: /cvs/ports/devel/libmemcached/patches/patch-configure,v
retrieving revision 1.2
diff -u -r1.2 patch-configure
--- patches/patch-configure     14 Apr 2011 11:17:49 -0000      1.2
+++ patches/patch-configure     28 Dec 2011 11:52:37 -0000
@@ -1,10 +1,11 @@
 $OpenBSD: patch-configure,v 1.2 2011/04/14 11:17:49 sebastia Exp $
 
 get rid of a truckload of warnings when compiling
+allow to detect MSG_NOSIGNAL and friends
 
---- configure.orig     Wed Mar 16 03:40:33 2011
-+++ configure  Fri Apr  1 01:47:32 2011
-@@ -17719,7 +17719,7 @@ fi
+--- configure.orig     Wed Oct 26 17:48:39 2011
++++ configure  Thu Dec 15 11:12:42 2011
+@@ -7820,7 +7820,7 @@ fi
  
  fi
  
@@ -13,7 +14,7 @@
        CXX_WARNINGS="${BASE_WARNINGS} -Woverloaded-virtual -Wnon-virtual-dtor 
-Wctor-dtor-privacy -Wno-long-long ${CXX_WARNINGS_FULL}"
  
        { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether it is safe to 
use -Wmissing-declarations from C++" >&5
-@@ -17868,7 +17868,7 @@ ac_link='$CXX -o conftest$ac_exeext $CXXFLAGS $CPPFLAG
+@@ -7969,7 +7969,7 @@ ac_link='$CXX -o conftest$ac_exeext $CXXFLAGS $CPPFLAG
  ac_compiler_gnu=$ac_cv_cxx_compiler_gnu
  
           save_CXXFLAGS="${CXXFLAGS}"
@@ -22,7 +23,7 @@
           cat confdefs.h - <<_ACEOF >conftest.$ac_ext
  /* end confdefs.h.  */
  
-@@ -17901,9 +17901,9 @@ fi
+@@ -8002,9 +8002,9 @@ fi
  { $as_echo "$as_me:${as_lineno-$LINENO}: result: 
$ac_cv_safe_to_use_Wredundant_decls_" >&5
  $as_echo "$ac_cv_safe_to_use_Wredundant_decls_" >&6; }
        if test "$ac_cv_safe_to_use_Wredundant_decls_" = "yes"; then :
@@ -34,7 +35,7 @@
  fi
  
        { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether it is safe to 
use -Wattributes from C++" >&5
-@@ -18013,7 +18013,7 @@ fi
+@@ -8114,7 +8114,7 @@ fi
  fi
  
  
@@ -43,3 +44,27 @@
              PROTOSKIP_WARNINGS="-Wno-effc++ -Wno-shadow -Wno-missing-braces 
${NO_ATTRIBUTES}"
        NO_WERROR="-Wno-error"
        PERMISSIVE_WARNINGS="-Wno-error -Wno-unused-function -fpermissive"
+@@ -25737,6 +25737,7 @@ ac_compiler_gnu=$ac_cv_c_compiler_gnu
+     cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+ /* end confdefs.h.  */
+ 
++#include <sys/types.h>
+ #include <sys/socket.h>
+ 
+ int
+@@ -25785,6 +25786,7 @@ ac_compiler_gnu=$ac_cv_c_compiler_gnu
+     cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+ /* end confdefs.h.  */
+ 
++#include <sys/types.h>
+ #include <sys/socket.h>
+ 
+ int
+@@ -25833,6 +25835,7 @@ ac_compiler_gnu=$ac_cv_c_compiler_gnu
+     cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+ /* end confdefs.h.  */
+ 
++#include <sys/types.h>
+ #include <sys/socket.h>
+ 
+ int
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/devel/libmemcached/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 PLIST
--- pkg/PLIST   1 Oct 2010 09:32:36 -0000       1.1.1.1
+++ pkg/PLIST   28 Dec 2011 11:52:37 -0000
@@ -6,65 +6,84 @@
 @bin bin/memcp
 @bin bin/memdump
 @bin bin/memerror
+@bin bin/memexist
 @bin bin/memflush
+@bin bin/memparse
+@bin bin/memping
 @bin bin/memrm
 @bin bin/memstat
+@bin bin/memtouch
 include/libhashkit/
-include/libhashkit/algorithm.h
-include/libhashkit/behavior.h
-include/libhashkit/configure.h
-include/libhashkit/digest.h
-include/libhashkit/function.h
+include/libhashkit-1.0/
+include/libhashkit-1.0/algorithm.h
+include/libhashkit-1.0/behavior.h
+include/libhashkit-1.0/configure.h
+include/libhashkit-1.0/digest.h
+include/libhashkit-1.0/function.h
+include/libhashkit-1.0/has.h
+include/libhashkit-1.0/hashkit.h
+include/libhashkit-1.0/hashkit.hpp
+include/libhashkit-1.0/str_algorithm.h
+include/libhashkit-1.0/strerror.h
+include/libhashkit-1.0/types.h
+include/libhashkit-1.0/visibility.h
 include/libhashkit/hashkit.h
-include/libhashkit/strerror.h
-include/libhashkit/types.h
-include/libhashkit/visibility.h
 include/libmemcached/
-include/libmemcached/allocators.h
-include/libmemcached/analyze.h
-include/libmemcached/auto.h
-include/libmemcached/behavior.h
-include/libmemcached/callback.h
-include/libmemcached/configure.h
-include/libmemcached/constants.h
-include/libmemcached/delete.h
-include/libmemcached/dump.h
-include/libmemcached/exception.hpp
-include/libmemcached/fetch.h
-include/libmemcached/flush.h
-include/libmemcached/flush_buffers.h
-include/libmemcached/get.h
-include/libmemcached/hash.h
-include/libmemcached/memcached/
+include/libmemcached-1.0/
+include/libmemcached-1.0/allocators.h
+include/libmemcached-1.0/analyze.h
+include/libmemcached-1.0/auto.h
+include/libmemcached-1.0/basic_string.h
+include/libmemcached-1.0/behavior.h
+include/libmemcached-1.0/callback.h
+include/libmemcached-1.0/configure.h
+include/libmemcached-1.0/constants.h
+include/libmemcached-1.0/delete.h
+include/libmemcached-1.0/dump.h
+include/libmemcached-1.0/error.h
+include/libmemcached-1.0/exception.hpp
+include/libmemcached-1.0/exist.h
+include/libmemcached-1.0/fetch.h
+include/libmemcached-1.0/flush.h
+include/libmemcached-1.0/flush_buffers.h
+include/libmemcached-1.0/get.h
+include/libmemcached-1.0/hash.h
+include/libmemcached-1.0/memcached.h
+include/libmemcached-1.0/memcached.hpp
+include/libmemcached-1.0/options.h
+include/libmemcached-1.0/parse.h
+include/libmemcached-1.0/platform.h
+include/libmemcached-1.0/quit.h
+include/libmemcached-1.0/result.h
+include/libmemcached-1.0/return.h
+include/libmemcached-1.0/sasl.h
+include/libmemcached-1.0/server.h
+include/libmemcached-1.0/server_list.h
+include/libmemcached-1.0/stats.h
+include/libmemcached-1.0/storage.h
+include/libmemcached-1.0/strerror.h
+include/libmemcached-1.0/string.h
+include/libmemcached-1.0/touch.h
+include/libmemcached-1.0/types.h
+include/libmemcached-1.0/verbosity.h
+include/libmemcached-1.0/version.h
+include/libmemcached-1.0/visibility.h
 include/libmemcached/memcached.h
 include/libmemcached/memcached.hpp
-include/libmemcached/memcached/protocol_binary.h
-include/libmemcached/memcached_util.h
-include/libmemcached/parse.h
-include/libmemcached/platform.h
-include/libmemcached/protocol/
-include/libmemcached/protocol/cache.h
-include/libmemcached/protocol/callback.h
-include/libmemcached/protocol_handler.h
-include/libmemcached/quit.h
-include/libmemcached/result.h
-include/libmemcached/sasl.h
-include/libmemcached/server.h
-include/libmemcached/server_list.h
-include/libmemcached/stats.h
-include/libmemcached/storage.h
-include/libmemcached/strerror.h
-include/libmemcached/string.h
-include/libmemcached/types.h
-include/libmemcached/util/
 include/libmemcached/util.h
-include/libmemcached/util/ping.h
-include/libmemcached/util/pool.h
-include/libmemcached/util/version.h
-include/libmemcached/verbosity.h
-include/libmemcached/version.h
-include/libmemcached/visibility.h
-include/libmemcached/watchpoint.h
+include/libmemcachedprotocol-0.0/
+include/libmemcachedprotocol-0.0/binary.h
+include/libmemcachedprotocol-0.0/callback.h
+include/libmemcachedprotocol-0.0/handler.h
+include/libmemcachedprotocol-0.0/vbucket.h
+include/libmemcachedutil-1.0/
+include/libmemcachedutil-1.0/flush.h
+include/libmemcachedutil-1.0/ostream.hpp
+include/libmemcachedutil-1.0/pid.h
+include/libmemcachedutil-1.0/ping.h
+include/libmemcachedutil-1.0/pool.h
+include/libmemcachedutil-1.0/util.h
+include/libmemcachedutil-1.0/version.h
 lib/libhashkit.a
 lib/libhashkit.la
 lib/libmemcached.a
@@ -75,6 +94,7 @@
 lib/libmemcachedutil.la
 lib/pkgconfig/
 lib/pkgconfig/libmemcached.pc
+@man man/man1/memaslap.1
 @man man/man1/memcapable.1
 @man man/man1/memcat.1
 @man man/man1/memcp.1
@@ -98,9 +118,13 @@
 @man man/man3/hashkit_md5.3
 @man man/man3/hashkit_murmur.3
 @man man/man3/hashkit_value.3
+@man man/man3/libhashkit.3
 @man man/man3/libmemcached.3
+@man man/man3/libmemcached_check_configuration.3
+@man man/man3/libmemcached_configuration.3
 @man man/man3/libmemcached_examples.3
 @man man/man3/libmemcachedutil.3
+@man man/man3/memcached.3
 @man man/man3/memcached_add.3
 @man man/man3/memcached_add_by_key.3
 @man man/man3/memcached_analyze.3
@@ -120,11 +144,14 @@
 @man man/man3/memcached_delete_by_key.3
 @man man/man3/memcached_destroy_sasl_auth_data.3
 @man man/man3/memcached_dump.3
+@man man/man3/memcached_exist.3
+@man man/man3/memcached_exist_by_key.3
 @man man/man3/memcached_fetch.3
 @man man/man3/memcached_fetch_execute.3
 @man man/man3/memcached_fetch_result.3
 @man man/man3/memcached_flush_buffers.3
 @man man/man3/memcached_free.3
+@man man/man3/memcached_generate_hash.3
 @man man/man3/memcached_generate_hash_value.3
 @man man/man3/memcached_get.3
 @man man/man3/memcached_get_by_key.3
@@ -133,17 +160,22 @@
 @man man/man3/memcached_get_user_data.3
 @man man/man3/memcached_increment.3
 @man man/man3/memcached_increment_with_initial.3
+@man man/man3/memcached_last_error_message.3
 @man man/man3/memcached_lib_version.3
 @man man/man3/memcached_mget.3
 @man man/man3/memcached_mget_by_key.3
 @man man/man3/memcached_mget_execute.3
 @man man/man3/memcached_mget_execute_by_key.3
+@man man/man3/memcached_pool.3
 @man man/man3/memcached_pool_behavior_get.3
 @man man/man3/memcached_pool_behavior_set.3
 @man man/man3/memcached_pool_create.3
 @man man/man3/memcached_pool_destroy.3
+@man man/man3/memcached_pool_fetch.3
 @man man/man3/memcached_pool_pop.3
 @man man/man3/memcached_pool_push.3
+@man man/man3/memcached_pool_release.3
+@man man/man3/memcached_pool_st.3
 @man man/man3/memcached_prepend.3
 @man man/man3/memcached_prepend_by_key.3
 @man man/man3/memcached_quit.3
@@ -170,5 +202,7 @@
 @man man/man3/memcached_stat_get_value.3
 @man man/man3/memcached_stat_servername.3
 @man man/man3/memcached_strerror.3
+@man man/man3/memcached_touch.3
+@man man/man3/memcached_touch_by_key.3
 @man man/man3/memcached_verbosity.3
 @man man/man3/memcached_version.3
 
 
 

Reply via email to