From owner-cvs-ports@FreeBSD.ORG Sun Sep 25 07:20:08 2011 Return-Path: Delivered-To: cvs-ports@FreeBSD.org Received: from mx2.freebsd.org (mx2.freebsd.org [IPv6:2001:4f8:fff6::35]) by hub.freebsd.org (Postfix) with ESMTP id 9ADDB1065672; Sun, 25 Sep 2011 07:20:08 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from 172-17-198-245.globalsuite.net (hub.freebsd.org [IPv6:2001:4f8:fff6::36]) by mx2.freebsd.org (Postfix) with ESMTP id 9AB3F152396; Sun, 25 Sep 2011 07:20:07 +0000 (UTC) Message-ID: <4E7ED627.2070902@FreeBSD.org> Date: Sun, 25 Sep 2011 00:20:07 -0700 From: Doug Barton Organization: http://SupersetSolutions.com/ User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:6.0.2) Gecko/20110912 Thunderbird/6.0.2 MIME-Version: 1.0 To: Cheng-Lung Sung References: <201107060026.p660Qjxr034952@repoman.freebsd.org> <4E27C1BB.7040901@FreeBSD.org> In-Reply-To: <4E27C1BB.7040901@FreeBSD.org> X-Enigmail-Version: undefined OpenPGP: id=1A1ABC84 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: cvs-ports@FreeBSD.org, Grzegorz Blach , cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/www/trac Makefile ports/www/trac/files tracd.in X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 25 Sep 2011 07:20:08 -0000 maintainer, have you had a chance to look at these changes yet? On 07/20/2011 23:05, Doug Barton wrote: > I realize that you copied this script, but it has issues. :) I know > I've said it before, but this is why I'm so pedantic about reviewing the > rc.d scripts, bad examples are copied, seemingly with more frequency > than good ones. :-/ > > One of the issues with this script is that it's pretty convoluted. > Therefore I _think_ I got everything working the way it did before, but > it needs testing. Particularly the _trac_env stuff. > > The single most important change is tracd_args -> tracd_flags. I don't > know why this _args keeps slipping in, but it's not standard, and > shouldn't be used. The advantage of _flags is that it's handled > transparently by rc.subr. > > Beyond that, I've removed the default empty variable assignments, they > are not necessary or desirable, especially when you have such good > documentation in the headers. > > There are a variety of other simplifications, and I moved the > environment settings into a start_precmd. > > Like I said, this needs testing. If you have any questions, feel free to > follow up to freebsd-rc@. > > > Doug > > > On 07/05/2011 17:26, Cheng-Lung Sung wrote: >> clsung 2011-07-06 00:26:45 UTC >> >> FreeBSD ports repository >> >> Modified files: >> www/trac Makefile >> Added files: >> www/trac/files tracd.in >> Log: >> - Since trac can work as standalone http server, I copied rc script from old trac-multirepos port. >> - also fixed portlint warning about CONFLICTS line. >> >> PR: ports/155295 http://www.FreeBSD.org/cgi/query-pr.cgi?pr=155295 >> Submitted by: "Grzegorz Blach" >> >> Revision Changes Path >> 1.54 +7 -3 ports/www/trac/Makefile >> 1.1 +61 -0 ports/www/trac/files/tracd.in (new) >> >> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/www/trac/Makefile.diff?&r1=1.53&r2=1.54&f=h >> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/www/trac/files/tracd.in >> > > > -- Nothin' ever doesn't change, but nothin' changes much. -- OK Go Breadth of IT experience, and depth of knowledge in the DNS. Yours for the right price. :) http://SupersetSolutions.com/