From owner-freebsd-toolchain@FreeBSD.ORG Thu Jul 18 13:16:18 2013 Return-Path: Delivered-To: freebsd-toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 25C22FCA; Thu, 18 Jul 2013 13:16:18 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from tensor.andric.com (tensor.andric.com [IPv6:2001:7b8:3a7:1:2d0:b7ff:fea0:8c26]) by mx1.freebsd.org (Postfix) with ESMTP id E2E7EDA2; Thu, 18 Jul 2013 13:16:17 +0000 (UTC) Received: from coleburn.avinity.tv (host-229-161-243.77.avinity.tv [77.243.161.229]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by tensor.andric.com (Postfix) with ESMTPSA id BDECA5C5A; Thu, 18 Jul 2013 15:16:13 +0200 (CEST) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\)) Subject: Re: clang looking all over for linux libs [9.2 PRERELEASE] From: Dimitry Andric In-Reply-To: <51E7E41B.3030504@pix.net> Date: Thu, 18 Jul 2013 15:16:12 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: <106B8A43-785E-48BC-83DE-DA0A0FC5604E@FreeBSD.org> References: <51E6FAF5.3080802@pix.net> <20130717213347.GA23999@freebsd.org> <51E7E41B.3030504@pix.net> To: Kurt Lidl X-Mailer: Apple Mail (2.1508) Cc: freebsd-toolchain@freebsd.org X-BeenThere: freebsd-toolchain@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Maintenance of FreeBSD's integrated toolchain List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Jul 2013 13:16:18 -0000 On Jul 18, 2013, at 14:48, Kurt Lidl wrote: > On 7/17/13 5:33 PM, Roman Divacky wrote: >> Yes, this is because the FreeBSD driver toolchain shares the same >> infrastructure with Linux. So we stat ~200 linux dirs :( >>=20 >> This rys.vlakno.cz/~rdivacky/freebsd-driver.patch proof of concept >> patch fixes it by unsharing FreeBSD from Linux but I am not >> convinced it's worth it. >>=20 >> It should also be possible to only stat those libs when -m32/-m64 >> is specified. That would be the preferred solution. >>=20 >> Roman >=20 > Thanks for that patch, but when applied to the 9.2-PRERELEASE > clang tree, a buildworld fails like this: >=20 > building static c library > ranlib libc.a > building shared library libc.so.7 > /usr/bin/ld: this linker was not configured to use sysroots > cc: error: linker command failed with exit code 1 (use -v to see = invocation) > *** [libc.so.7] Error code 1 > 1 error > *** [lib/libc__L] Error code 2 > 1 error > *** [libraries] Error code 2 > 1 error > *** [_libraries] Error code 2 > 1 error > *** [buildworld] Error code 2 > 1 error I'm not sure what goes wrong here, but please take the problem upstream, instead of using local patches. We want less of those, not more. :-) In any case, the probability of such a change making it into 9.2 is very small, since it is in no way critical. Apparently clang has been doing this for years, and you are the first one to notice... -Dimitry