From owner-cvs-ports@FreeBSD.ORG Thu Feb 12 09:56:16 2004 Return-Path: Delivered-To: cvs-ports@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 47CC716A4CE; Thu, 12 Feb 2004 09:56:16 -0800 (PST) Received: from srv01.sparkit.no (srv01.sparkit.no [193.69.116.226]) by mx1.FreeBSD.org (Postfix) with ESMTP id B27D943D1D; Thu, 12 Feb 2004 09:56:15 -0800 (PST) (envelope-from eivind@FreeBSD.org) Received: from ws ([193.69.114.88]) by srv01.sparkit.no (8.12.10/8.12.10) with ESMTP id i1CHu6iA084460; Thu, 12 Feb 2004 18:56:10 +0100 (CET) (envelope-from eivind@FreeBSD.org) Received: from ws (localhost [127.0.0.1]) by ws (8.12.9/8.12.10) with ESMTP id i1CHthRe027937; Thu, 12 Feb 2004 17:55:43 GMT (envelope-from eivind@ws) Received: (from eivind@localhost) by ws (8.12.9/8.12.10/Submit) id i1CHtgPK027913; Thu, 12 Feb 2004 17:55:42 GMT (envelope-from eivind) Date: Thu, 12 Feb 2004 17:54:35 +0000 From: Eivind Eklund To: Trevor Johnson Message-ID: <20040212175219.GE3148@FreeBSD.org> References: <200402110716.i1B7GH9D017803@repoman.freebsd.org> <1076508074.88428.14.camel@pav.hide.vol.cz> <20040211122704.X93022@blues.jpj.net> <20040211165940.Q98525@blues.jpj.net> <20040212031228.E31447@blues.jpj.net> <20040212102842.GC3148@FreeBSD.org> <20040212110623.A77057@blues.jpj.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040212110623.A77057@blues.jpj.net> User-Agent: Mutt/1.5.4i cc: cvs-all@FreeBSD.org cc: ports-committers@FreeBSD.org cc: Pav Lucistnik cc: Joe Marcus Clarke cc: cvs-ports@FreeBSD.org cc: Dag-Erling =?iso-8859-1?Q?Sm=F8rgrav?= Subject: Re: cvs commit: ports/www/firefox Makefile distinfo pkg-descr pkg-message pkg-plist ports/www/firefox/files firebird.rb firefox.rb mkdistfile mozconfig.in patch-Double.cpp patch-build_unix_run-mozilla X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.1 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: Thu, 12 Feb 2004 17:56:16 -0000 Please take the below as an attempt at being constructive and actually keeping the discussion on subject. It is somewhat directly formulated as I don't have time to do much more editing to make it more calm. On Thu, Feb 12, 2004 at 11:13:25AM -0500, Trevor Johnson wrote: > > My opinion is that DES is right anyway. This lowers the consistency of > > the ports tree overall, > > Not really--many other ports do the same thing. The fact that there are ports other places that have not been updated to use REINPLACE_CMD is not a reason for introducing a new one. And I would hope the other ports are moving *away* from using perl also for replacements, not towards it. The set of people using <= 4.6 was larger earlier, and they were less out of date earlier, and thus more reasonable to do hacks for. [Text deleted in an attempt to keep this as much on subject as possible.] > If someone (you?) have ideas about removing the Perl dependency, that > would be a really stunning enhancement to the port. As I said it's > been a huge bugbear. I don't have enough disk space to even extract it available at the moment; I may look at it again later (perhaps even tonight.) However, this is a straw man. The relevant issues (as far as I can tell) are: - The replacement of REINPLACE_CMD with PERL -pi is an efficiency hack. - The efficiency hack increase efficiency on systems of 4.6-vintage and below. It does this at the expense of consistency with our conventions, and potentially efficiency on systems newer than 4.6 (due to larger executable, more shared libs for perl, etc). Whether increasing efficiency for <=4.6 even has value *when seen completely apart from the other issues* has been questioned. - A noticable number of people have said that we value consistency in the ports tree, and that the change should be reverted. This is enough to normally trigger a re-evaluation of what is being done. - There have been no attempt from your side to argue what level of value is present in the perl changes, beyond saying they remove a dependency on <= 4.6. Eivind.