From owner-svn-ports-all@freebsd.org Thu Nov 15 22:06:16 2018 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 07D1E110BB21; Thu, 15 Nov 2018 22:06:16 +0000 (UTC) (envelope-from joneum@FreeBSD.org) Received: from toco-domains.de (mail.toco-domains.de [176.9.39.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 402F481958; Thu, 15 Nov 2018 22:06:15 +0000 (UTC) (envelope-from joneum@FreeBSD.org) Received: from phantomias.home.jochen-neumeister.de (p5B0FD568.dip0.t-ipconnect.de [91.15.213.104]) by toco-domains.de (Postfix) with ESMTPA id 94BF21351307; Thu, 15 Nov 2018 23:06:07 +0100 (CET) Subject: Re: svn commit: r485048 - in head/databases: . pgpool-II-40 To: Tobias Kortkamp , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org References: <201811152106.wAFL6RwE098338@repo.freebsd.org> <1542319393.56339.1578537352.30511AF8@webmail.messagingengine.com> From: Jochen Neumeister Message-ID: <2cd73458-5a29-bb9b-4b1f-2f91dc196950@FreeBSD.org> Date: Thu, 15 Nov 2018 23:06:06 +0100 MIME-Version: 1.0 In-Reply-To: <1542319393.56339.1578537352.30511AF8@webmail.messagingengine.com> Content-Language: de-DE X-Rspamd-Queue-Id: 402F481958 X-Spamd-Result: default: False [-103.47 / 40.00]; ARC_NA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; ALLOW_DOMAIN_WHITELIST(-100.00)[FreeBSD.org]; RECEIVED_SPAMHAUS_PBL(0.00)[104.213.15.91.zen.spamhaus.org : 127.0.0.10]; FROM_HAS_DN(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; TO_DN_SOME(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MIME_GOOD(-0.10)[multipart/alternative,text/plain]; DMARC_NA(0.00)[FreeBSD.org]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; R_SPF_SOFTFAIL(0.00)[~all]; URI_COUNT_ODD(1.00)[7]; IP_SCORE(-1.36)[ipnet: 176.9.0.0/16(-3.96), asn: 24940(-2.85), country: DE(-0.01)]; MX_GOOD(-0.01)[cached: mx1.FreeBSD.org]; NEURAL_HAM_SHORT(-1.00)[-0.998,0]; RCVD_IN_DNSWL_NONE(0.00)[170.39.9.176.list.dnswl.org : 127.0.10.0]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; FROM_EQ_ENVFROM(0.00)[]; R_DKIM_NA(0.00)[]; RCVD_TLS_LAST(0.00)[]; ASN(0.00)[asn:24940, ipnet:176.9.0.0/16, country:DE]; MID_RHS_MATCH_FROM(0.00)[]; RCVD_COUNT_TWO(0.00)[2] X-Rspamd-Server: mx1.freebsd.org Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Nov 2018 22:06:16 -0000 On 15.11.18 23:03, Tobias Kortkamp wrote: > On Thu, Nov 15, 2018, at 22:06, Jochen Neumeister wrote: >> Author: joneum >> Date: Thu Nov 15 21:06:26 2018 >> New Revision: 485048 >> URL: https://svnweb.freebsd.org/changeset/ports/485048 >> >> Log: >> New Port: databases/pgpool-II-40 >> >> pgpool is a connection pool server for PostgreSQL. pgpool runs between >> PostgreSQL's clients(front ends) and servers(back ends). A PostgreSQL >> client can >> connect to pgpool as if it were a standard PostgreSQL server. >> >> pgpool caches the connection to PostgreSQL server to reduce the overhead to >> establish the connection to it. >> >> WWW: http://pgpool.net/ >> >> PR: 232957 >> Reported by: Franco Ricci >> Approved by: tz (implicit) >> Sponsored by: Netzkommune GmbH >> >> Added: >> head/databases/pgpool-II-40/ >> - copied from r485040, head/databases/pgpool-II-37/ >> Modified: >> head/databases/Makefile >> head/databases/pgpool-II-40/Makefile >> head/databases/pgpool-II-40/distinfo >> head/databases/pgpool-II-40/pkg-plist >> >> Modified: head/databases/Makefile >> ============================================================================== >> --- head/databases/Makefile Thu Nov 15 20:56:56 2018 (r485047) >> +++ head/databases/Makefile Thu Nov 15 21:06:26 2018 (r485048) >> @@ -612,6 +612,7 @@ >> SUBDIR += pgpool-II-35 >> SUBDIR += pgpool-II-36 >> SUBDIR += pgpool-II-37 >> + SUBDIR += pgpool-II-40 >> SUBDIR += pgreplay >> SUBDIR += pgroonga >> SUBDIR += pgrouting >> >> Modified: head/databases/pgpool-II-40/Makefile >> ============================================================================== >> --- head/databases/pgpool-II-37/Makefile Thu Nov 15 19:08:47 2018 (r485040) >> +++ head/databases/pgpool-II-40/Makefile Thu Nov 15 21:06:26 2018 (r485048) >> @@ -1,10 +1,10 @@ >> # $FreeBSD$ >> >> PORTNAME= pgpool-II >> -PORTVERSION= 3.7.5 >> +PORTVERSION= 4.0.1 >> CATEGORIES= databases >> MASTER_SITES= http://www.pgpool.net/mediawiki/images/ >> -PKGNAMESUFFIX= 37 >> +PKGNAMESUFFIX= 40 >> >> MAINTAINER= tz@FreeBSD.org >> COMMENT= Connection pool server for PostgreSQL >> @@ -12,10 +12,10 @@ COMMENT= Connection pool server for PostgreSQL >> LICENSE= MIT >> LICENSE_FILE= ${WRKSRC}/COPYING >> >> -BROKEN_SSL= openssl111 >> -BROKEN_SSL_REASON_openssl111= library 'ssl' is required for OpenSSL >> +BROKEN_SSL= openssl-devel >> +BROKEN_SSL_REASON_openssl-devel= library 'ssl' is required for OpenSSL > This part should be reverted. It was ok before and openssl-devel > was removed, so BROKEN_SSL=openssl-devel has no effect. That's true. That was still so from the PR and was overlooked by me. Thank you > >> -PORTSCOUT= limit:^3.7.[0-9]* >> +PORTSCOUT= limit:^4.0.[0-9]* >> >> USES= gmake libtool pgsql:9.6+ >> GNU_CONFIGURE= yes >> @@ -26,11 +26,27 @@ CONFLICTS= pgpool-II-[0-9]* >> >> PORTDOCS= * >> >> -OPTIONS_DEFINE= DOCS SSL >> +PAM_DESC= Build with PAM support >> +SSL_DESC= Build with OpenSSL support >> +MEMCACHED_DESC = Use memcached for in memory query cache >> + >> +OPTIONS_DEFINE= DOCS SSL PAM MEMCACHED >> OPTIONS_DEFAULT= SSL >> >> SSL_CONFIGURE_WITH= openssl >> SSL_USES= ssl >> + >> +.include >> + >> +.if ${PORT_OPTIONS:MPAM} >> +#CONFIGURE_ARGS+= --with-pam >> +SSL_CONFIGURE_WITH= pam >> +.endif > Hmm, was this supposed to be PAM_CONFIGURE_WITH ? > > +.if ${PORT_OPTIONS:MMEMCACHED} > +CONFIGURE_ARGS+=--with-memcached=${LOCALBASE}/include > +LIB_DEPENDS+= libmemcached.so:databases/libmemcached > +.endif > > Why not use options helpers here too? > Thanks for this opportunity. I'll test right away.