From owner-svn-src-all@freebsd.org Mon Jun 1 16:40:27 2020 Return-Path: Delivered-To: svn-src-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 2BDDF33BE19; Mon, 1 Jun 2020 16:40:27 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (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 49bLWZ4dkZz4Dsd; Mon, 1 Jun 2020 16:40:26 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (localhost [127.0.0.1]) by gndrsh.dnsmgr.net (8.13.3/8.13.3) with ESMTP id 051GeOnA013241; Mon, 1 Jun 2020 09:40:24 -0700 (PDT) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.13.3/8.13.3/Submit) id 051GeOaf013240; Mon, 1 Jun 2020 09:40:24 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <202006011640.051GeOaf013240@gndrsh.dnsmgr.net> Subject: Re: svn commit: r361677 - in head/usr.bin/svn: . lib lib/libapr lib/libapr_util lib/libserf lib/libsvn_client lib/libsvn_delta lib/libsvn_diff lib/libsvn_fs lib/libsvn_fs_fs lib/libsvn_fs_util lib/libs... In-Reply-To: <304bb13cb6e80791f8924a573355a3643d614184.camel@freebsd.org> To: Ian Lepore Date: Mon, 1 Jun 2020 09:40:24 -0700 (PDT) CC: Dimitry Andric , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 49bLWZ4dkZz4Dsd X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:13868, ipnet:69.59.192.0/19, country:US] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.33 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Jun 2020 16:40:27 -0000 > On Sun, 2020-05-31 at 22:04 +0000, Dimitry Andric wrote: > > Author: dim > > Date: Sun May 31 22:04:51 2020 > > New Revision: 361677 > > URL: https://svnweb.freebsd.org/changeset/base/361677 > > > > Log: > > Change Makefiles under usr.bin/svn to make them easier to > > incrementally > > update. No functional change intended. > > > > MFC after: 2 weeks > > > > I wish we could get style.Makefile(9) updated to mandate this 1-per- > line style when listing sources, dirs, etc, when the number of items is > greater than N, where N is something like 3-6 filenames. Otherwise the > requirement to sort the names alphabetically pretty much mandates that > many lines of the file will change just to insert one or two new files, > and that makes it all but impossible to figure out from a diff what > actually changed. I like this idea, though rather than 3-6 filenames I propose it to be anything longer than 3 lines, which is kinda about when the pain point should start. See the immediate SUBDIR below, it is 11 items on 2ish/3 lines, and any change would worst case be a 3 line diff. This probably covers a large portion of the tree. I particularly do not like massive amounts of vertical white space which this would create, but lacking an automated tool this is probably a reasonable compromise. If we did it everywhere it would mean lots of scrolling when working on rather simple makefiles. > -- Ian > > > > [...] > > -SUBDIR= lib .WAIT \ > > - svn svnadmin svnbench svndumpfilter svnfsfs svnlook svnserve \ > > - svnsync svnversion svnmucc svnrdump > > +SUBDIR= lib \ > > + .WAIT \ > > + svn \ > > + svnadmin \ > > + svnbench \ > > + svndumpfilter \ > > + svnfsfs \ > > + svnlook \ > > + svnserve \ > > + svnsync \ > > + svnversion \ > > + svnmucc \ > > + svnrdump > > SUBDIR_PARALLEL= > > > > -- Rod Grimes rgrimes@freebsd.org