From owner-freebsd-ports@FreeBSD.ORG Fri May 25 19:25:49 2012 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 87448106564A; Fri, 25 May 2012 19:25:49 +0000 (UTC) (envelope-from Joerg.Pulz@frm2.tum.de) Received: from mailhost.frm2.tum.de (mailhost.frm2.tum.de [129.187.179.12]) by mx1.freebsd.org (Postfix) with ESMTP id D99B38FC08; Fri, 25 May 2012 19:25:48 +0000 (UTC) Received: from mailhost.frm2.tum.de (localhost [127.0.0.1]) by mailhost.frm2.tum.de (8.14.4/8.14.4) with ESMTP id q4PJPl0K076643; Fri, 25 May 2012 21:25:47 +0200 (CEST) (envelope-from Joerg.Pulz@frm2.tum.de) X-Virus-Scanned: at mailhost.frm2.tum.de Received: from [10.155.6.10] ([10.155.6.10]) (authenticated bits=0) by mailhost.frm2.tum.de (8.14.4/8.14.4) with ESMTP id q4PJPijU076638 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Fri, 25 May 2012 21:25:44 +0200 (CEST) (envelope-from Joerg.Pulz@frm2.tum.de) Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=us-ascii From: Joerg Pulz In-Reply-To: <20120525183913.GC24924@atarininja.org> Date: Fri, 25 May 2012 21:25:43 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: <8AFE1D09-F9D3-49F0-9155-24E6C1EF2031@frm2.tum.de> References: <20120522125710.GB18115@atarininja.org> <20120522211434.GA5483@atarininja.org> <20120525003844.GA24924@atarininja.org> <20120525165617.GB24924@atarininja.org> <20120525183913.GC24924@atarininja.org> To: Wesley Shields X-Mailer: Apple Mail (2.1084) X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.6 (mailhost.frm2.tum.de [129.187.179.12]); Fri, 25 May 2012 21:25:44 +0200 (CEST) Cc: freebsd-ports@freebsd.org, eadler@freebsd.org, Robert Simmons , bug-followup@freebsd.org Subject: Re: ports/168214: 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: Fri, 25 May 2012 19:25:49 -0000 On 25.05.2012, at 20:39, Wesley Shields wrote: > On Fri, May 25, 2012 at 01:20:46PM -0400, Robert Simmons wrote: >> On Fri, May 25, 2012 at 12:56 PM, Wesley Shields = wrote: >>> On Fri, May 25, 2012 at 12:21:54PM -0400, Robert Simmons wrote: >>>> On Thu, May 24, 2012 at 8:38 PM, Wesley Shields = wrote: >>>>> On Tue, May 22, 2012 at 06:29:20PM -0400, Robert Simmons wrote: >>>>>> On Tue, May 22, 2012 at 5:14 PM, Wesley Shields = wrote: >>>>>>> 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. >>>>>>>>=20 >>>>>>>> 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. >>>>>>>=20 >>>>>>> I'm not opposed to removing the change to the default, but it = does cause >>>>>>> another problem. See below. >>>>>>>=20 >>>>>>>> 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? >>>>>>>=20 >>>>>>> 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. >>>>>>=20 >>>>>> I may have been too hasty. ?I've thought of a situation where one >>>>>> would want to build the port with no backend at all. ?If one = wanted to >>>>>> use the tools in the port to administrate a remote install of = Heimdal, >>>>>> they may want to build it without a backend. >>>>>>=20 >>>>>> My initial thoughts were only for installing the port as a = Heimdal >>>>>> server, and with the --with-berkeley-db=3Dno problem fixed it = does not >>>>>> wrongly find the version of BDB in the base OS. ?With this fix, = the >>>>>> port can function with no backends selected. ?It just won't be = able to >>>>>> function in a server capacity. >>>>>>=20 >>>>>> I am also not an expert in Heimdal, I just installed it from = source >>>>>> via its own instructions and compared that with what the FreeBSD = port >>>>>> was doing. ?I'd wait for the maintainer to make changes to the = default >>>>>> behavior for the above reason. >>>>>=20 >>>>> This all sounds perfectly reasonable to me. :) >>>>>=20 >>>>> If I'm understanding you correctly the patch[1] in ports/168214 is = the >>>>> correct one to commit. The only change I would make is not bumping >>>>> PORTREVISION since the option is off by default. Sounds like the = only >>>>> thing left to do is wait for maintainer comment on the PR and = commit >>>>> accordingly. >>>>=20 >>>> Sounds good. ?One question: what do you mean by PORTREVISION being = off >>>> by default? >>>=20 >>> There is no need to bump PORTREVISION because the option which you = are >>> changing is off by default so there's no need to force a rebuild of = it >>> on the package cluster since your changes are going to have no = effect >>> there. >>>=20 >>> For those that have the option to on, it hasn't built properly for = them >>> yet so bumping is going to have no effect either. >>=20 >> I understand what you're saying. However, my change would actually >> change the package cluster. Because those packages were built with >> "--without-berkeley-db" rather than "--with-berkeley-db=3Dno" the old >> packages were built with broken BDB support by accident. By fixing >> this, the default package is actually going to be different than the >> one built before this change. I would recommend bumping PORTREVISION >> because of this. >=20 > That makes sense. Thanks for the clarification. I will be awaiting > maintainer approval or timeout then. Hi, please go ahead and commit and close ports/168214 using the last version = of the patch and please bump PORTREVISION. Sorry for this, i didn't noticed the configure arg change for building = without berkeley-db. Kind regards and thanks Joerg=