On Tue Dec 12, 2017 at 02:18:18AM +0100, Jeremie Courreges-Anglas wrote:
> On Sat, Dec 09 2017, Florian Obser <flor...@openbsd.org> wrote:
> > jca pointed out that I send an update for an older version (4.0.5)
> > before, so please disregard that. This jumps directly to the 4.1.x
> > series.
> >
> > - our local patch to json11.cpp is no longer needed, upstream
> > picked it up
> > - two more local patches are needed, mplexer.hh is already fixed by
> > upstream and nproxy.cc is reported so they should go away with
> > the next update
> > - jca pointed out that the master side does https now, so switched it,
> > same for the home page
> > - otto@ pointed out that we install the man pages for the tools but
> > don't build tools. This is enabled now as per his request.
> > - enabling the ldap module introduces a hard dependency on kerberos,
> > (configure aborts) since I have no experience with kerberos,
> > don't use ldap and never heard of anyone using ldap I chose to
> > drop ldap support. It is hard enough to get OKs - let alone
> > tests for powerdns, so I will focus on the things *I* need.
> > If you need other things, please step up.
> >
> > I assume the ldap removal needs some black magic somewhere in quirks?
> > Cluebats?
>
> You'd need to add "powerdns-ldap" to $obsolete_reason in
> devel/quirks/files/Quirks.pm. None of the existing reasons (see @msg)
> really match here, though.
>
> > This runs in production and works for me[tm]
> > Tests, OKs?
>
> Looks good ports-wise except that ''make port-lib-depends-check'' says
> that WANTLIB-main is outdated, some libs aren't used any more. Do you
> see the same?
Same here. Please see below.
>
> > diff --git Makefile Makefile
> > index c96c9a9b108..1a323cde121 100644
> > --- Makefile
> > +++ Makefile
> > @@ -5,25 +5,19 @@ BROKEN-hppa = undefined reference to
> > '__sync_fetch_and_add_4'
> > COMMENT-main= modular authoritative nameserver (with database
> > support)
> > COMMENT-mysql= MySQL database access module for PowerDNS
> > COMMENT-pgsql= PGSQL database access module for PowerDNS
> > -COMMENT-ldap= LDAP module for PowerDNS
> >
> > -V= 4.0.4
> > +V= 4.1.0
> > DISTNAME= pdns-${V}
> > EXTRACT_SUFX = .tar.bz2
> > PKGNAME= powerdns-${V}
> > PKGNAME-main= powerdns-${V}
> > PKGNAME-mysql= powerdns-mysql-${V}
> > PKGNAME-pgsql= powerdns-pgsql-${V}
> > -PKGNAME-ldap= powerdns-ldap-${V}
> > CATEGORIES= net
> > -HOMEPAGE= http://www.powerdns.com/
> > +HOMEPAGE= https://www.powerdns.com/
> > MAINTAINER= Florian Obser <florian+po...@narrans.de>
> > -REVISION-ldap= 0
> > -REVISION-main= 0
> > -REVISION-mysql= 0
> > -REVISION-pgsql= 0
> >
> > -MULTI_PACKAGES= -main -mysql -pgsql -ldap
> > +MULTI_PACKAGES= -main -mysql -pgsql
> >
> > # GPLv2
> > PERMIT_PACKAGE_CDROM= Yes
> > @@ -37,7 +31,7 @@ WANTLIB-main += ${WANTLIB}
Please zap "#WANTLIB += ${MODGCC4_CPPWANTLIB}" Somewhere around here
> > WANTLIB-main += boost_program_options-mt boost_serialization-mt
> > WANTLIB-main += bz2 c crypto gmp sqlite3 z
boost_serialization-mt bz2 gmp can be removed.
> >
> > -MASTER_SITES= http://downloads.powerdns.com/releases/
> > +MASTER_SITES= https://downloads.powerdns.com/releases/
> >
> > NO_TEST= Yes
> >
> > @@ -46,7 +40,7 @@ LIB_DEPENDS-main= ${LIB_DEPENDS} \
> > devel/boost \
> > devel/gmp
... and bye bye devel/gmp
