From owner-freebsd-ports@FreeBSD.ORG Tue May 22 21:14:35 2012 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id CC41C106566B; Tue, 22 May 2012 21:14:35 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.49.45]) by mx1.freebsd.org (Postfix) with ESMTP id 846218FC0C; Tue, 22 May 2012 21:14:35 +0000 (UTC) Received: by syn.atarininja.org (Postfix, from userid 1001) id 7DDC05C45; Tue, 22 May 2012 17:14:34 -0400 (EDT) Date: Tue, 22 May 2012 17:14:34 -0400 From: Wesley Shields To: Robert Simmons Message-ID: <20120522211434.GA5483@atarininja.org> References: <4FAD492C.3060807@gmx.de> <9880B4F2567E41A582B6B9DA178AECFD@black> <20120522125710.GB18115@atarininja.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="NzB8fVQJ5HfG6fxh" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Cc: eadler@FreeBSD.org, freebsd-ports@freebsd.org Subject: Re: Heimdal 1.5.2 problem X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 May 2012 21:14:35 -0000 --NzB8fVQJ5HfG6fxh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, May 22, 2012 at 03:08:31PM -0400, Robert Simmons wrote: > On Tue, May 22, 2012 at 8:57 AM, Wesley Shields wrote: > > As the person who committed this update I will take responsibility for > > seeing this through. Would you mind opening a PR with this patch and CC > > both myself and the maintainer so it can be properly tracked. I will > > work with both of you to make sure it is addressed. > > I got some good feedback about the patch. I was missing a "\". Also, > it was noted that I shouldn't make changes to the default settings in > this patch since it is meant to correct a problem. I removed the > change to default. I'm not opposed to removing the change to the default, but it does cause another problem. See below. > Perhaps the different default is not the best solution. Maybe there > should be a message that at least one backend is needed for the port > to function, but none have been selected by default? If a backend is required the port should refuse to build if no backend is selected. This is pretty easy to do, just check for at least one of the backends. I have no idea if multiple backends can be supported so you may or may not want to also check for that. If it does require a backend then the port should default to one of them. If we don't pick one as a default then we get no package with the changes you are suggesting above (the IGNORE line I put in place will always happen on the package cluster). I'm attaching an updated version of your patch to this email that flips BDB on by default and does the check to make sure at least one backend is selected. If you feel it's sufficent we can get it in the PR so it is properly tracked. > I have attached the updated patch, and I've opened a PR here: > http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/168214 Thank you for opening a PR. All too often things can fall through the mailing list cracks, and if it's in a PR we can at least have a record of it. I see eadler@ has grabbed your PR. As I said earlier, I'm willing to step up and commit this (pending maintainer approval or timeout). -- WXS --NzB8fVQJ5HfG6fxh Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="heimdal.diff" Index: Makefile =================================================================== RCS file: /home/ncvs/ports/security/heimdal/Makefile,v retrieving revision 1.94 diff -u -r1.94 Makefile --- Makefile 20 May 2012 00:08:19 -0000 1.94 +++ Makefile 22 May 2012 20:45:56 -0000 @@ -7,13 +7,12 @@ PORTNAME= heimdal PORTVERSION= 1.5.2 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= security ipv6 MASTER_SITES= http://www.h5l.org/dist/src/ \ http://ftp.pdc.kth.se/pub/heimdal/src/ \ ftp://ftp.pdc.kth.se/pub/heimdal/src/ \ - ftp://ftp.sunet.se/pub/unix/admin/mirror-pdc/heimdal/src/ \ - ftp://ftp.ayamura.org/pub/heimdal/ + ftp://ftp.sunet.se/pub/unix/admin/mirror-pdc/heimdal/src/ MAINTAINER= Joerg.Pulz@frm2.tum.de COMMENT= A popular BSD-licensed implementation of Kerberos 5 @@ -22,7 +21,7 @@ OPTIONS= IPV6 "Enable IPV6 support" on \ KCM "Enable Kerberos Credentials Manager" on \ - BDB "Enable BerkeleyDB KDC backend support" off \ + BDB "Enable BerkeleyDB KDC backend support" on \ SQLITE "Enable SQLite KDC backend support" off \ LDAP "Enable OpenLDAP KDC backend support" off \ PKINIT "Enable PK-INIT support" on \ @@ -48,6 +47,10 @@ .include +.if !defined(WITH_BDB) && !defined(WITH_SQLITE) && !defined(WITH_LDAP) +IGNORE= Need a backend. +.endif + .if ${ARCH} == "amd64" CFLAGS+= -fPIC .endif @@ -77,10 +80,10 @@ CFLAGS+= -I${BDB_INCLUDE_DIR} CPPFLAGS+= -I${BDB_INCLUDE_DIR} LDFLAGS+= -L${BDB_LIB_DIR} -CONFIGURE_ARGS+= --with-berkeley-db=${LOCALBASE} -# --with-berkeley-db-include=${BDB_INCLUDE_DIR} +CONFIGURE_ARGS+= --with-berkeley-db=${LOCALBASE} \ + --with-berkeley-db-include=${BDB_INCLUDE_DIR} .else -CONFIGURE_ARGS+= --without-berkeley-db +CONFIGURE_ARGS+= --with-berkeley-db=no .endif .if defined(WITH_SQLITE) --NzB8fVQJ5HfG6fxh--