From owner-freebsd-ports@FreeBSD.ORG Sat Dec 18 15:57:32 2010 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 4D2F6106566B; Sat, 18 Dec 2010 15:57:32 +0000 (UTC) (envelope-from ohauer@FreeBSD.org) Received: from u18-124.dslaccess.de (unknown [194.231.39.124]) by mx1.freebsd.org (Postfix) with ESMTP id EEB298FC15; Sat, 18 Dec 2010 15:57:31 +0000 (UTC) Received: from [172.20.1.100] (unknown [172.20.1.100]) by u18-124.dslaccess.de (Postfix) with ESMTPSA id B0BFC20576; Sat, 18 Dec 2010 16:57:29 +0100 (CET) Message-ID: <4D0CDA89.1040506@FreeBSD.org> Date: Sat, 18 Dec 2010 17:00:09 +0100 From: Olli Hauer User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7 MIME-Version: 1.0 To: Mark Murray References: In-Reply-To: X-Enigmail-Version: 1.1.1 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: ports@freebsd.org, krion@freebsd.org Subject: Re: Exim port tidyup. X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: ohauer@FreeBSD.org List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 18 Dec 2010 15:57:32 -0000 On 2010-12-18 15:10, Mark Murray wrote: > Hi > > I've done a bit of maintenance on a port that I originally wrote, Exim. > > I redid the patches by using the "make makepatch" target (it makes doing > maintainance easier, as the patch names are regularised and you get to > see diffs easier). > > As part of this, there is a minor change to the way Exim works, which is > a security feature, due to be default in future Exim releases. In the > EDITME file, it is ALT_CONFIG_ROOT_ONLY, and if its OK to commit this > cleanup, then I'd commit that separately. > > Is it ok for me to commit this please? > > http://people.freebsd.org/~markm/exim.tar.gz > > ... and it is straight out of my ports/mail/exim/... tree, with > CVS/... files intact and ready to go, except that I'd do the > ALT_CONFIG_ROOT_ONLY bit separately. > > M Hi Mark, since the old patch-files have a long history I suggest open a change-request PR to repocopy the files and assign the PR to portmgr. If portmgr has done the repocopy you can remove the old patch files and apply your diff (the only significant change I see is in src__EDITME) Things I miss in your diff - bump PORTREVISION? (if more than EDITME change) - remove MD5 from distinfo Isn't it easier to remove the targets o show-options-list o show-variables-list from Makefile.options and use only the code from the file options directly? -- Regards, olli