From owner-svn-ports-all@FreeBSD.ORG Tue Jun 3 16:11:51 2014 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1033) id AE831DBA; Tue, 3 Jun 2014 16:11:51 +0000 (UTC) Date: Tue, 3 Jun 2014 16:11:51 +0000 From: Alexey Dokuchaev To: "Drewery, Bryan" Subject: Re: svn commit: r354497 - head/sysutils/pear-Cache_Lite Message-ID: <20140603161151.GA90588@FreeBSD.org> References: <201405190713.s4J7DAuF096825@svn.freebsd.org> <20140603012726.GA60685@FreeBSD.org> <538DED66.6060501@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <538DED66.6060501@FreeBSD.org> User-Agent: Mutt/1.5.23 (2014-03-12) Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, Martin Wilke , ports-committers@freebsd.org X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Jun 2014 16:11:51 -0000 On Tue, Jun 03, 2014 at 10:44:38AM -0500, Drewery, Bryan wrote: > On Mon Jun 2 20:27:26 2014, Alexey Dokuchaev wrote: > >I'm afraid this change is wrong: > > > >- pear.php.net itself treats package entry page as a directory: Download, > > Documentation, Bugs, Trackbacks links are implemented as subdirectories; > > Main entry points to "http://pear.php.net/package/HTTP_Request2/" (note > > the trailing slash) > > > >- PH section 3.2: "Include a URL to the official WWW homepage [...] with > > WWW: (followed by single space) so that automated tools will work > > correctly. If the URI is the root of the website or directory, it should > > be terminated with a slash." > > > >These changes present in the next six commits of yours. I think these > >parts should be backed out, and Mr. Rutkowski kindly advised to refresh > >his PH knowledge. > > This is a bit extreme and pedantic. The URL is fine as is. There It is pedantic; however, this is 1) not a newly added file, it was correct before; 2) it contradicts pear.php.net's own output; 3) it contradicts PHB. > was little need to change it, and even less need to revert it > and cause grief to a contributor. Contributor should not have to get upset, he should reread the PHB. > The modern web more and more makes trailing / more incorrect > than correct. Right, but not in this case. > You have absolutely no way to tell if a URL is the "root of a directory" > with so many internal rewrites and dynamic URLs around the web now. Of course; that's why, in every case, manual checking is in order. I did verify it in this particular case before hitting "send" button. Then again: we have tons of submissions that contain hunks that stir the repo for no real use, clutter the diff, harm "svn blame"; yet every time someone who points it out gets cursed. Shouldn't we discourage meaningless parts of the submitted patches get landed in the first place instead? Trust me, I hate to be "that guy", but every time I get a break and sit silently watching, very few actually speak up. Oh boy how do I miss sat@ and his reviews. ./danfe