From owner-svn-src-all@freebsd.org Sat Mar 4 21:21:33 2017 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9ED2CCF566F; Sat, 4 Mar 2017 21:21:33 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 6AF451500; Sat, 4 Mar 2017 21:21:33 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id 955DC1A88; Sat, 4 Mar 2017 21:21:32 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 1DE5133401; Sat, 4 Mar 2017 21:21:31 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id kt3EP_sk_2BK; Sat, 4 Mar 2017 21:21:24 +0000 (UTC) Subject: Re: svn commit: r314654 - in head/cddl: lib/drti lib/libavl lib/libctf lib/libdtrace lib/libnvpair lib/libumem lib/libuutil lib/libzfs lib/libzfs_core lib/libzpool sbin/zfs sbin/zpool usr.bin/ctfconver... DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com DC2E3333F9 To: rgrimes@FreeBSD.org References: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net> Cc: Ngie Cooper , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Organization: FreeBSD Message-ID: <04cd5016-2777-08a6-9279-c0e83e8d3eff@FreeBSD.org> Date: Sat, 4 Mar 2017 13:21:11 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="DuADjswumbVdKsFlVdccFp6fMC3xTWMmI" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.23 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: Sat, 04 Mar 2017 21:21:33 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --DuADjswumbVdKsFlVdccFp6fMC3xTWMmI Content-Type: multipart/mixed; boundary="gsViOshM24cCSi9U1nBAk7mLeCdgjrDVI"; protected-headers="v1" From: Bryan Drewery To: rgrimes@FreeBSD.org Cc: Ngie Cooper , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org Message-ID: <04cd5016-2777-08a6-9279-c0e83e8d3eff@FreeBSD.org> Subject: Re: svn commit: r314654 - in head/cddl: lib/drti lib/libavl lib/libctf lib/libdtrace lib/libnvpair lib/libumem lib/libuutil lib/libzfs lib/libzfs_core lib/libzpool sbin/zfs sbin/zpool usr.bin/ctfconver... References: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net> In-Reply-To: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net> --gsViOshM24cCSi9U1nBAk7mLeCdgjrDVI Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 3/4/2017 1:04 PM, Rodney W. Grimes wrote: >> >>> On Mar 4, 2017, at 12:46, Ngie Cooper (yaneurabeya) wrote: >>> >>> >>>> On Mar 4, 2017, at 12:42, Ngie Cooper (yaneurabeya) wrote: >>> >>> ... >>>> I care about this for the reasons brewery posted, but I also care be= cause it slows down my terminal output and it bloats my disk with typescr= ipt logs that contain unnecessary information. >>> >>> And it?s not just my personal laptop. It bloats log files in CI servi= ces, slows down grep?ing logfiles, increases data retention requirements,= etc. >>> >>> We (at $work) do thousands of builds a week. Saving a few MB here and= there can go a long way over the course of weeks, months, etc. >> >> Sidenote: it really peeves me how viewing a log in Jenkins will crash = my browser when it tries to render the text. Maybe now it will work a bit= better. >=20 > That is a very bad reason, if this disk space saving and not having > to see all the ../.. scroll by on your terminal as make buildworld > runs is your motivation for this I would say we need to stop > now and have a more reasonable and serious discussion about why > this is actually a good idea. >=20 > Please go fix the bug in your browser and hold off on these > changes (in either direction) until a proper discussion, possibly > real tests to see if there is any savings in cycles worthy > of this sized change. >=20 >> -Ngie >=20 Your reason is "a very bad reason" being "my gutt is telling me this change is Bad(tm)." vs the objective list of reasons SRCTOP is better than '../..'. (quotes from you). Changing to SRCTOP is not about speed or saving grand or trivial amounts of space. It's about making Makefiles simpler to write and understand and making the output simpler to read and use. I don't think it is worth the effort to discuss every little thing like this. There has been a discussion among people who maintain the build and all agreed with it. I was telling my wife just last night that if I had put up every little change I did for review/discussion in the build over last year that I likely would be delayed by years due to bikeshedding trivial aspects of them, and would have just given up. I still have a lot left to do as well. Among all of the changes, the only real dissent I've had in the end is from one person saying that some of the work slows down the kernel build by a few seconds when building on a small system over a slow NFS link. That's a valid complaint though vs this thread which is starting to feel like a bikeshed. --=20 Regards, Bryan Drewery --gsViOshM24cCSi9U1nBAk7mLeCdgjrDVI-- --DuADjswumbVdKsFlVdccFp6fMC3xTWMmI Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJYuy/HAAoJEDXXcbtuRpfPUdgIAIE1Z0pnrOsFMkPM5T8bLtYI VfIeADWSTG8vtgneNTD9beEJzT5rJdZPUDqOXRYY/Xxa56X3mUlApH59oVGlLYBa WzeNGoeHbhmfMQpguLnytk1ivQ3rSNUyHL54NcoPbErdGYGbg/rJ0uY1q6uDygAC XG7iI8MnF3DrMkXZLJPybljpGBmx5x9d5SihaLloZFuMUROW4VVVLC+LGLbRkUVU 87Z8wrXMMXC7fec81bJVh7KT22yGPIIVe2xmfswvjGQxAPXZyQpHkY9Hguf6dJoY vSvv50DLKMr3n+E5a89e/X6r8bzVPIeFJK2kPUZmvFnVAgfc/RAF6xFiEzcWlvs= =1xe7 -----END PGP SIGNATURE----- --DuADjswumbVdKsFlVdccFp6fMC3xTWMmI--