From owner-svn-ports-all@freebsd.org Thu Mar 12 04:22:36 2020 Return-Path: Delivered-To: svn-ports-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 47AA52746BE; Thu, 12 Mar 2020 04:22:36 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 48dFzb5MN7z3D6J; Thu, 12 Mar 2020 04:22:35 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id 72B88ABCE; Thu, 12 Mar 2020 04:22:35 +0000 (UTC) Date: Thu, 12 Mar 2020 04:22:35 +0000 From: Alexey Dokuchaev To: Baptiste Daroussin Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r528226 - in head/net-p2p: . udpt udpt/files Message-ID: <20200312042235.GA66660@FreeBSD.org> References: <202003111051.02BApxf7054140@repo.freebsd.org> <20200311180936.dbjgbynjmbxstukk@ivaldir.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200311180936.dbjgbynjmbxstukk@ivaldir.net> X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.29 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: Thu, 12 Mar 2020 04:22:36 -0000 On Wed, Mar 11, 2020 at 07:09:36PM +0100, Baptiste Daroussin wrote: > On Wed, Mar 11, 2020 at 10:51:59AM +0000, Alexey Dokuchaev wrote: > > New Revision: 528226 > > URL: https://svnweb.freebsd.org/changeset/ports/528226 > > > > Log: > > UDPT is a lightweight UDP torrent tracker for Unix-like systems. > > ... Please don't overquote. > > +# Alternative to DOS2UNIX_FILES that preserves original timestamps, > > +# to be used together with `-l' switch added to PATCH_ARGS below. > > +fixpatches: > > + @${FIND} ${FILESDIR} -name patch-* | ${XARGS} ${SED} -i '' \ > > + -E 's,[[:cntrl:]]*$$,,' > > +PATCH_ARGS+= -l > > Seriously? why not improving the framework instead??? Admittedly, I'm not quite sure that it's the framework that should be fixed rather than SVN rules laxed to allow mixed line endings. While I hate CRLF as much as anyone else, technically there's nothing wrong with patches that patch DOS-style code generated on Unix, thus having LF-ended header and CRLF or even mixed body. I'd say that people use DOS2UNIX in this case mainly to circumvent SVN commit hooks. That said, it still can be handled within the framework, but the most straightforward way as I see it would require PATCH_ARGS+=-l and I am not sure if that's acceptable. There are probably other ways I don't see, of course, but then again, why not just fine-tune SVN? ./danfe