Date: Sat, 4 Mar 2017 13:21:11 -0800 From: Bryan Drewery <bdrewery@FreeBSD.org> To: rgrimes@FreeBSD.org Cc: Ngie Cooper <ngie@FreeBSD.org>, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@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... Message-ID: <04cd5016-2777-08a6-9279-c0e83e8d3eff@FreeBSD.org> In-Reply-To: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net> References: <201703042104.v24L4X4x078873@pdx.rh.CN85.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --DuADjswumbVdKsFlVdccFp6fMC3xTWMmI Content-Type: multipart/mixed; boundary="gsViOshM24cCSi9U1nBAk7mLeCdgjrDVI"; protected-headers="v1" From: Bryan Drewery <bdrewery@FreeBSD.org> To: rgrimes@FreeBSD.org Cc: Ngie Cooper <ngie@FreeBSD.org>, 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) <yaneurabeya@gmai= l.com> wrote: >>> >>> >>>> On Mar 4, 2017, at 12:42, Ngie Cooper (yaneurabeya) <yaneurabeya@gma= il.com> 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--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?04cd5016-2777-08a6-9279-c0e83e8d3eff>