From owner-freebsd-ports@FreeBSD.ORG Thu Jun 26 15:55:31 2008 Return-Path: Delivered-To: ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4B50E1065673 for ; Thu, 26 Jun 2008 15:55:31 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.60.158]) by mx1.freebsd.org (Postfix) with ESMTP id 207188FC18 for ; Thu, 26 Jun 2008 15:55:30 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: by syn.atarininja.org (Postfix, from userid 1001) id C7DC65C2E; Thu, 26 Jun 2008 11:56:16 -0400 (EDT) Date: Thu, 26 Jun 2008 11:56:16 -0400 From: Wesley Shields To: Jo Rhett Message-ID: <20080626155616.GI12581@atarininja.org> References: <83EB55D5-11A0-44C9-A469-A5C5BD6D972C@netconsonance.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <83EB55D5-11A0-44C9-A469-A5C5BD6D972C@netconsonance.com> User-Agent: Mutt/1.5.18 (2008-05-17) Cc: ports@freebsd.org Subject: Re: split the quagga ports to remove unstable patches from quagga port 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, 26 Jun 2008 15:55:31 -0000 On Thu, Jun 26, 2008 at 08:38:00AM -0700, Jo Rhett wrote: > The current maintainer has indicated that he is holding up the quagga > 0.99.10 port until an unknown time when the TCP MD5 checksum patches > are again working in the tree. I don't think that this is the right > thing to do, as the TCP MD5 checksums are not necessary for the vast > majority of installations, and DO NOT work with a GENERIC kernel. > > I'd like to request that the port be split into two ports -- one with > a stock quagga installation, and the other with the MD5 checksum > patches, due to the instability and constant work on said patches > making the port unstable. (and being unavailable right now holds up > access to a major bugfix line) You already have the option for TCPMD5 being on or off (off by default). The solution in my mind is not two separate ports, but what is used now: OPTIONS. If an option is broken why not mark it as such until it can be addressed? If the patches become a workable (yet still experimental) option the BROKEN line can be removed. The port can still keep using newer versions while the patches are being developed. -- WXS Index: Makefile =================================================================== RCS file: /u/ncvs/ports/net/quagga/Makefile,v retrieving revision 1.70 diff -u -u -r1.70 Makefile --- Makefile 25 Apr 2008 15:42:13 -0000 1.70 +++ Makefile 26 Jun 2008 15:47:23 -0000 @@ -118,6 +118,7 @@ .endif .if defined(WITH_TCPMD5) +BROKEN= MD5 patches need updating. EXTRA_PATCHES+=${PATCHDIR}/extra-tcpmd5-patch-bgpd-bgp_network.c ${PATCHDIR}/extra-tcpmd5-patch-bgpd-bgp_vty.c ${PATCHDIR}/extra-tcpmd5-patch-bgpd-bgpd.c ${PATCHDIR}/extra-tcpmd5-patch-bgpd-bgpd.h ${PATCHDIR}/extra-tcpmd5-patch-lib-sockopt.c ${PATCHDIR}/extra-tcpmd5-patch-lib-sockopt.h ${PATCHDIR}/extra-tcpmd5-patch-vtysh-extract.pl.in CFLAGS+= -DQUAGGA_TCP_MD5SIG .endif