From owner-svn-src-head@freebsd.org Sat Mar 4 20:13:35 2017 Return-Path: Delivered-To: svn-src-head@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 92E4FCF96EA; Sat, 4 Mar 2017 20:13:35 +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 5201515C3; Sat, 4 Mar 2017 20:13:35 +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 7E1487CEF; Sat, 4 Mar 2017 20:13:34 +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 B843833050; Sat, 4 Mar 2017 20:13:33 +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 vIO_khJk-nbP; Sat, 4 Mar 2017 20:13:26 +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 F0E6C33049 To: rgrimes@freebsd.org, Ngie Cooper References: <201703041520.v24FK9ne077279@pdx.rh.CN85.dnsmgr.net> Cc: 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: Date: Sat, 4 Mar 2017 12:13:28 -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: <201703041520.v24FK9ne077279@pdx.rh.CN85.dnsmgr.net> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Ihov9xFw63k2E5qU2sKmSnDWK5MQqd05A" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 04 Mar 2017 20:13:35 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --Ihov9xFw63k2E5qU2sKmSnDWK5MQqd05A Content-Type: multipart/mixed; boundary="nggwp8opMlslWnoBvLLiSJmhicnkAl1rW"; protected-headers="v1" From: Bryan Drewery To: rgrimes@freebsd.org, Ngie Cooper Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: 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: <201703041520.v24FK9ne077279@pdx.rh.CN85.dnsmgr.net> In-Reply-To: <201703041520.v24FK9ne077279@pdx.rh.CN85.dnsmgr.net> --nggwp8opMlslWnoBvLLiSJmhicnkAl1rW Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 3/4/2017 7:20 AM, Rodney W. Grimes wrote: > [ Charset UTF-8 unsupported, converting... ] >> Author: ngie >> Date: Sat Mar 4 11:30:04 2017 >> New Revision: 314654 >> URL: https://svnweb.freebsd.org/changeset/base/314654 >> >> Log: >> cddl: normalize paths using SRCTOP-relative paths or :H when possibl= e >> =20 >> This simplifies make logic/output >> =20 >> While here, remove bogus CFLAGS which look for headers in cddl/lib/l= ibumem. >> There aren't any source files there (just Makefiles) > Please again, dont intermingle other chnages when doing a tree wide swe= ep. >=20 > IMHO if you write While here, you probably should do that thing in a=20 > seperate commit, unless this is a single thing your touching in a singl= e > commit. >=20 > It is much easier to glue 2 commits togeather than seperate 1 that > changes 2 things. >=20 Agreed. There were also various comment changes in here. This is why git is nice, you can git add -p and stage multiple commits before pushing. git-svn handles this fine. >=20 > It is also still not clear to me that this was adeqautly disccussed as = I > only saw 2 +'s for staying relative and no one +ing to move forward wit= h this. Where is this discussion? The only one I can find is https://reviews.freebsd.org/D9207 which seems to have a consensus of moving to SRCTOP and :H vs '../..'. --=20 Regards, Bryan Drewery --nggwp8opMlslWnoBvLLiSJmhicnkAl1rW-- --Ihov9xFw63k2E5qU2sKmSnDWK5MQqd05A 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 iQEcBAEBAgAGBQJYux/pAAoJEDXXcbtuRpfPyzQH/iUvkWdAtldR8PQy2gCAgYVh 3sTDhw0y4eLg7RoGZkorKG1T44RS43nFTkmJ9T46fl45DvHJ1qS7n1YFhlE9luPN EwmTqTps9+Tmlf/0aIxCRZFM3fdurR0vpzaxetLh7qRX7hE/tW+nTfiK8HBW+k6/ YYiQQ9GEBvwu/Pe6o//1IuqC/Ndr9AQJceXGN0nHxnNSluevSEtQN93WmRMIhQVH cVkeOF1i5I/6hs9pKqkhsBHC98SlbwtV/WL10F115R3BmDa96WdwjRfO1vf1XGUJ JOJ3iv6na1td1VRHFhrJahSxCtDMLEC+c2kYgvzMjd/1lE3zS+1WfhdTqfWt36Y= =PMH0 -----END PGP SIGNATURE----- --Ihov9xFw63k2E5qU2sKmSnDWK5MQqd05A--