From owner-freebsd-ports@FreeBSD.ORG Thu Apr 27 06:45:13 2006 Return-Path: X-Original-To: ports@freebsd.org Delivered-To: freebsd-ports@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5665B16A401 for ; Thu, 27 Apr 2006 06:45:13 +0000 (UTC) (envelope-from infofarmer@gmail.com) Received: from nz-out-0102.google.com (nz-out-0102.google.com [64.233.162.196]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8A33043D45 for ; Thu, 27 Apr 2006 06:45:12 +0000 (GMT) (envelope-from infofarmer@gmail.com) Received: by nz-out-0102.google.com with SMTP id x3so2032196nzd for ; Wed, 26 Apr 2006 23:45:12 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=Ae6uqjIZRQp+oxxVpghxgmdgfo0AzUbcKuz8ccr75aNAB4J48GdBb33oPV5HxnRr/YlZAR3eUyMWaH2lR0aHt4wvfO7J5zt1eK9o8pZfTmptBHVqARNT/4K9YB3mqMAgR+ntIvQGanoMfsWdg/3Y7udVI0H5noEEwgIzPQMhKcE= Received: by 10.37.13.11 with SMTP id q11mr451757nzi; Wed, 26 Apr 2006 23:45:12 -0700 (PDT) Received: by 10.37.22.74 with HTTP; Wed, 26 Apr 2006 23:45:12 -0700 (PDT) Message-ID: Date: Thu, 27 Apr 2006 10:45:12 +0400 From: "Andrew Pantyukhin" To: "Doug Barton" , "Matthias Andree" , "FreeBSD Ports" In-Reply-To: <20060426215845.GA6337@merlin.emma.line.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline References: <20060426192442.6E3B654E@k7.mavetju> <20060426201101.GA1433@merlin.emma.line.org> <444FE894.70008@FreeBSD.org> <20060426215845.GA6337@merlin.emma.line.org> Cc: Subject: Re: ports/96379: PLEASE DO NOT SEND OPENVPN 2.0.7 REMINDERS! 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: Thu, 27 Apr 2006 06:45:13 -0000 On 4/27/06, Matthias Andree wrote: > On Wed, 26 Apr 2006, Doug Barton wrote: > > > Matthias Andree wrote: > > > Please close ports/96379. > > > > > > I am not accepting patches to update the FreeBSD/DragonflyBSD port > > > security/openvpn to 2.0.7. > > > > You may wish to consider whether the effort to continue telling people = you > > will not update the port will be lesser, or greater, than the effort > > required to update the port. > > My decision (call it ruling if you like) is that I will not ask > thousands of users to update just because a handful of fools can't get > past the version number to the changelog. Most of the requests had > actually been forwarded by committers to ask my feedback... Chill out, man :-) This handful of men are the ones we should praise, not scorn. In fact, they invested their time to learn the basics of making/updating a port, they took the trouble to notice 2.0.7 is out, to try it on their boxes, make a patch and go through a not-so-transparent process of submitting a PR. And what do they expect in return? Well, if they do a great job and the patch gets into the tree they'll get a heart-warming "Committed, thanks!" If not, if they make mistakes, at least we should try not to discourage them. They get no money, fame or glory. Should we really bash them for not being attentive enough? Back to 2.0.6-2.0.7. Below is a fragment of the difference between the versions. As far as I can tell without studying the rest of the code, this does not only fix windows versions. Granted, *nix systems might have get away with a wrong typedef, but it doesn't mean the old code is safe. Only the most important things go to changelogs. What we see here tells us that further investigation might be needed to know the effect of the mistake, even if USE_64_BIT_COUNTERS is defined by default. It doesn't mean that you're wrong about holding the horses and staying with 2.0.6. Saving users the time of download and recompilation is a nice decision. I only ask you to keep an eye out beyond changelogs. We should all go to the source when in doubt. BTW, thanks for a great port (this and many other ones) and for doing such a great job maintaining them all through the years. We use openvpn at several locations and we really appreciate the speed at which your provide updates. Thanks! /* - * Statistics counters. + * Statistics counters and associated printf formats. */ -typedef unsigned long long int counter_type; +#ifdef USE_64_BIT_COUNTERS + typedef unsigned long long int counter_type; +# ifdef WIN32 +# define counter_format "%I64u" +# else +# define counter_format "%llu" +# endif +#else + typedef unsigned int counter_type; +# define counter_format "%u" +#endif