From owner-freebsd-toolchain@FreeBSD.ORG Tue Nov 13 08:35:12 2012 Return-Path: Delivered-To: toolchain@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 8252B824 for ; Tue, 13 Nov 2012 08:35:12 +0000 (UTC) (envelope-from erik@cederstrand.dk) Received: from csmtp2.one.com (csmtp2.one.com [91.198.169.22]) by mx1.freebsd.org (Postfix) with ESMTP id 37C968FC0C for ; Tue, 13 Nov 2012 08:35:11 +0000 (UTC) Received: from [192.168.1.45] (unknown [176.222.238.90]) by csmtp2.one.com (Postfix) with ESMTPA id 4186A3019224; Tue, 13 Nov 2012 08:35:05 +0000 (UTC) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 6.2 \(1499\)) Subject: Re: arflags cleanup From: Erik Cederstrand In-Reply-To: <0693B91A-6320-4CE9-A6FA-821B290BB74C@bsdimp.com> Date: Tue, 13 Nov 2012 09:35:06 +0100 Content-Transfer-Encoding: quoted-printable Message-Id: <553C482F-9A67-4E60-9DBE-D5D6CA548D67@cederstrand.dk> References: <3D0FBCC4-743C-45B0-82E0-4C82501F7E60@cederstrand.dk> <0693B91A-6320-4CE9-A6FA-821B290BB74C@bsdimp.com> To: Warner Losh X-Mailer: Apple Mail (2.1499) Cc: 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: Tue, 13 Nov 2012 08:35:12 -0000 Warner, Den 13/11/2012 kl. 03.26 skrev Warner Losh : >>=20 >> If there are no other objections, I'll try to both patches into the = tree (I have no commit bit myself). >=20 > You haven't answered my objections, so not yet. I haven't had a = chance to see if this message actually answers my concerns though (my = life has been busy). As I understood your objections, you were worried about the negative = performance impact of loosing 'u', and the extra verbosity of loosing = 'c'. The first patch attempts to clean up ar flags, so makefiles respect the = global ARFLAGS if one is defined, without changing the default ARFLAGS = in share/mk/sys.mk. The second patch changes the default from 'rl' to = 'cru'. If this does not address your concerns, could you elaborate a bit more? Thanks, Erik=