From owner-svn-ports-head@FreeBSD.ORG Wed Jul 30 07:46:43 2014 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 4F2EEAC2; Wed, 30 Jul 2014 07:46:43 +0000 (UTC) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 2169C27B7; Wed, 30 Jul 2014 07:46:42 +0000 (UTC) Received: from [192.168.0.21] (unknown [130.255.19.191]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 54AEC43BA8; Wed, 30 Jul 2014 02:46:20 -0500 (CDT) Message-ID: <53D8A2BB.7090704@marino.st> Date: Wed, 30 Jul 2014 09:46:03 +0200 From: John Marino Reply-To: marino@freebsd.org User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Mathieu Arnold , marino@freebsd.org, Max Brazhnikov , Alexey Dokuchaev Subject: Re: svn commit: r363361 - in head/editors/fte: . files References: <201407291646.s6TGkjHH090335@svn.freebsd.org> <41D25BC1-AC62-4280-A342-8A2BDD84B1E0@adamw.org> <20140730070412.GA97692@FreeBSD.org> <3898057.T8DsoXnEEp@mercury.ph.man.ac.uk> <53D89EBF.4080805@marino.st> <2D24420529C9ECAEABB9A791@atuin.in.mat.cc> In-Reply-To: <2D24420529C9ECAEABB9A791@atuin.in.mat.cc> X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org, William Grzybowski , Adam Weinberger X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Jul 2014 07:46:43 -0000 On 7/30/2014 09:37, Mathieu Arnold wrote: > +--On 30 juillet 2014 09:29:03 +0200 John Marino > wrote: > | On 7/30/2014 09:25, Max Brazhnikov wrote: > |> On Wed, 30 Jul 2014 07:04:13 +0000 Alexey Dokuchaev wrote: > |>> On Tue, Jul 29, 2014 at 01:10:45PM -0400, Adam Weinberger wrote: > |>>> I'm about to rename all patches that contain ::. Can we please stop > |>>> producing patches with :: as a path separator, and use __ instead? > |>> > |>> Can we also stop using double underscore as it makes patch names so ugly > |>> and unreadable that I have to spend twice as much time working on ports > |>> that uses this stupid convention? > |> > |> It comes from the makepatch target. > | > | Makepatch name convention is no good. I was complaining about this (to > | bapt) about the lack of a patch name convention and he said to propose > | one. I've been meaning to do just that, but it also means fixing tools > | like this. > > makepatch name convention is the convention, I find it pretty simple, and > easy to remember, it does "s|/|__|" and that's all. Whatever the > convention, there's always someone who will not like it, but it doesn't > really matter, it's a convention so that everyone knows how to name patch > files the same way. I disagree, there is no convention. Whoever created makepatch attempted to create a defacto convention that most people don't like (as evidence that patches not created by makepatch don't use it but rather the more conventional* single underscore. I don't automatically recognize the author of makepatch the authority. * As seen in other repos such as pkgsrc Take a vote. I'd bet the majority of people do not like "__" not just "someone".