From owner-svn-src-all@freebsd.org Tue Jun 2 03:41:22 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 B689732867A; Tue, 2 Jun 2020 03:41:22 +0000 (UTC) (envelope-from rpokala@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 49bdBB4RlZz4dqh; Tue, 2 Jun 2020 03:41:22 +0000 (UTC) (envelope-from rpokala@freebsd.org) Received: from [192.168.1.10] (c-98-207-126-143.hsd1.ca.comcast.net [98.207.126.143]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) (Authenticated sender: rpokala) by smtp.freebsd.org (Postfix) with ESMTPSA id C7D47167BF; Tue, 2 Jun 2020 03:41:21 +0000 (UTC) (envelope-from rpokala@freebsd.org) User-Agent: Microsoft-MacOutlook/16.37.20051002 Date: Mon, 01 Jun 2020 20:41:18 -0700 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... From: Ravi Pokala To: CC: Ian Lepore , Dimitry Andric , , , Message-ID: Thread-Topic: 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... References: <202006020247.0522lkCD015032@gndrsh.dnsmgr.net> In-Reply-To: <202006020247.0522lkCD015032@gndrsh.dnsmgr.net> Mime-version: 1.0 Content-type: text/plain; charset="UTF-8" Content-transfer-encoding: quoted-printable 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: Tue, 02 Jun 2020 03:41:22 -0000 > > I generally feel the opposite -- I do a destressing amount of work on a= n 80x24 serial console :-/ -- but I agree that "> 3 lines" is a reasonable c= ompromise. >=20 > I think we have had a communications breakdown? To me it sounds as we bo= th agree that needlessly making Makefiles vertically long is not wanted, but= acceptable when folding these very long lists onto wide lines creates a dif= ferent problem. That's exactly what I said: 'but I agree that "> 3 lines" is a reasonable c= ompromise.' :-) -Ravi (rpokala@) =EF=BB=BF-----Original Message----- From: "Rodney W. Grimes" Reply-To: Date: 2020-06-01, Monday at 19:47 To: Ravi Pokala Cc: , Ian Lepore , Dimitry Andric , , , Subject: Re: svn commit: r361677 - in head/usr.bin/svn: . lib lib/libapr li= b/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... > -----Original Message----- > From: on behalf of "Rodney W. Grim= es" > Reply-To: > Date: 2020-06-01, Monday at 09:40 > To: Ian Lepore > Cc: Dimitry Andric , , <= svn-src-all@freebsd.org>, > Subject: Re: svn commit: r361677 - in head/usr.bin/svn: . lib lib/lib= apr lib/libapr_util lib/libserf lib/libsvn_client lib/libsvn_delta lib/libsv= n_diff lib/libsvn_fs lib/libsvn_fs_fs lib/libsvn_fs_util lib/libs... >=20 > > 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 > > >=20 > > > Log: > > > Change Makefiles under usr.bin/svn to make them easier to > > > incrementally > > > update. No functional change intended. > > > =20 > > > MFC after: 2 weeks > > >=20 > >=20 > > 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. Other= wise the > > requirement to sort the names alphabetically pretty much mandat= es that > > many lines of the file will change just to insert one or two ne= w files, > > and that makes it all but impossible to figure out from a diff = what > > actually changed. >=20 > 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. >=20 > FWIW, I'm partial to the 'FOO+=3Dbar' syntax, rather than continuation = lines. It's too easy to forget to add the continuation when appending or ins= erting. That is indeed a better technique. >=20 > 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. > =20 > If we did it everywhere it would mean lots of scrolling when > working on rather simple makefiles. >=20 > I generally feel the opposite -- I do a destressing amount of work on= an 80x24 serial console :-/ -- but I agree that "> 3 lines" is a reasonable= compromise. I think we have had a communications breakdown? To me it sounds as we both agree that needlessly making Makefiles vertically long is not wanted, but acceptable when folding these very long lists onto wide lines creates a different problem. > -Ravi (rpokala@) >=20 > > -- Ian > >=20 > > >=20 > > [...]=20 > > > -SUBDIR=3D lib .WAIT \ > > > - svn svnadmin svnbench svndumpfilter svnfsfs svnlook svnserv= e \ > > > - svnsync svnversion svnmucc svnrdump > > > +SUBDIR=3D lib \ > > > + .WAIT \ > > > + svn \ > > > + svnadmin \ > > > + svnbench \ > > > + svndumpfilter \ > > > + svnfsfs \ > > > + svnlook \ > > > + svnserve \ > > > + svnsync \ > > > + svnversion \ > > > + svnmucc \ > > > + svnrdump > > > SUBDIR_PARALLEL=3D > > > =20 > >=20 > >=20 >=20 > --=20 > Rod Grimes rgrime= s@freebsd.org >=20 >=20 >=20 --=20 Rod Grimes rgrimes@free= bsd.org