From owner-svn-src-projects@FreeBSD.ORG Wed Jan 28 19:51:29 2015 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 179F283D for ; Wed, 28 Jan 2015 19:51:29 +0000 (UTC) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id E9D2B35D for ; Wed, 28 Jan 2015 19:51:28 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.9/8.14.9) with ESMTP id t0SJpSHU052810 for ; Wed, 28 Jan 2015 19:51:28 GMT (envelope-from bdrewery@freefall.freebsd.org) Received: (from bdrewery@localhost) by freefall.freebsd.org (8.14.9/8.14.9/Submit) id t0SJpSEO052805 for svn-src-projects@freebsd.org; Wed, 28 Jan 2015 19:51:28 GMT (envelope-from bdrewery) Received: (qmail 14166 invoked from network); 28 Jan 2015 13:51:24 -0600 Received: from unknown (HELO ?10.10.1.139?) (freebsd@shatow.net@10.10.1.139) by sweb.xzibition.com with ESMTPA; 28 Jan 2015 13:51:24 -0600 Message-ID: <54C93DBD.6020609@FreeBSD.org> Date: Wed, 28 Jan 2015 13:51:25 -0600 From: Bryan Drewery Organization: FreeBSD User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 Subject: Re: svn commit: r277803 - projects/clang360-import/lib/clang/include References: <201501271925.t0RJPem3010417@svn.freebsd.org> <20150127191134.4fe3a17f@kan> <20150128091457.1b0ea3a7@kan> In-Reply-To: <20150128091457.1b0ea3a7@kan> OpenPGP: id=6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="fe9R1IesFKse5vRnb0Uj6wEKKC0NCF3xH" Cc: Benjamin Kaduk , svn-src-projects@freebsd.org, "src-committers@freebsd.org" , Dimitry Andric , Alexander Kabaev X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Jan 2015 19:51:29 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --fe9R1IesFKse5vRnb0Uj6wEKKC0NCF3xH Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 1/28/2015 8:14 AM, Alexander Kabaev wrote: > On Wed, 28 Jan 2015 08:42:48 +0100 > Dimitry Andric wrote: >=20 >> On 28 Jan 2015, at 01:11, Alexander Kabaev wrote: >> On Tue, 27 Jan 2015 14:34:21 -0500 >>> >>> Benjamin Kaduk wrote: >>> >>>> On Tue, Jan 27, 2015 at 2:25 PM, Dimitry Andric >>>> wrote: >>>> >>>>> Author: dim >>>>> Date: Tue Jan 27 19:25:39 2015 >>>>> New Revision: 277803 >>>>> URL: https://svnweb.freebsd.org/changeset/base/277803 >>>>> >>>>> Log: >>>>> Change the path to clang's private headers. Upstream has always >>>>> stored these in $LIBDIR/clang/$VERSION/include, instead of our >>>>> previous custom location in /usr/include/clang/$VERSION. This >>>>> allows us to drop yet another FreeBSD-specific patch. >>>>> >>>>> Modified: >>>>> projects/clang360-import/lib/clang/include/Makefile >>>>> >>>>> Modified: projects/clang360-import/lib/clang/include/Makefile >>>>> >>> >>> I think spreading .h files all over the tree is actually a >>> regression. >> >> I'm not sure what the problem is with storing a compiler's >> internal-only headers in the location where upstream expects them to >> be? Note that gcc does something similar; for example with the gcc49 >> port, it stores all its internal headers under: >> >> /usr/local/lib/gcc49/gcc/i386-portbld-freebsd11.0/4.9.3/include >> >> While I do agree that this is not a pretty-looking path, upstream has >> chosen it to be like this, and there are most likely good reasons for >> it. As for clang, I just want to get rid of as many "FreeBSD is a >> special snowflake" patches as I can. >> >> -Dimitry >> >=20 > Nobody _expects_ them to be there, for they are internal and not > directly addressable by anyone. We can tweak locations as we see fit > with no user visible consequences. What you destroy by this is the nice= > property we had to date - all of the base headers could be searched by > simple grep on /usr/include and no obscure directories need to be > remembered. Also note that ports can do whatever they want as long they= > do not stomp on the word, my concern is about base only. >=20 I've often grepped internal headers in /usr/include to try to understand some compiler errors better. This was mostly with C++ template errors. I did not know GCC was storing headers in /usr/lib/.../include. Having to include /usr/lib/.../include is a little annoying, but it is even more annoying to have FreeBSD be different than other platforms, assuming those other systems are adopting the upstream location as well. --=20 Regards, Bryan Drewery --fe9R1IesFKse5vRnb0Uj6wEKKC0NCF3xH 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 iQEcBAEBAgAGBQJUyT29AAoJEDXXcbtuRpfPTecH/RbT/oPVCgNDVB9yUGCfTfX/ cs85pFvCq+AFrMHq46I8ATEx7yySR8jBsCnq55XesKttXNN+qQoZD7Bk7qayA2AG C0ta9QRxXP/9lzJNHAoIPyGC4TAsVicg2SdpJv0vmSoKsYgV6Dgck8NqEkTMepzJ Gn87CD1SXBNaUNPyjpYAwlPmTUmi4JGzTgjQrCFDk/hyEAaELk5UqUVqpxgImyZy wAG+mtoPU2Of0OXuUbrLHoe9McZ3AqiX/x5wCI9G1qh7olrFIMVD7dLMQw+3aEZf LAoyYgbpdiX4mebJIDLdU8fXo2R6u2r7aXi/bbsKa+fRjxIcqfj6Fzzn0dN4S3w= =asA4 -----END PGP SIGNATURE----- --fe9R1IesFKse5vRnb0Uj6wEKKC0NCF3xH--