From owner-svn-src-all@FreeBSD.ORG Tue Dec 28 15:43:51 2010 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4174D106564A; Tue, 28 Dec 2010 15:43:51 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from mail.zoral.com.ua (mx0.zoral.com.ua [91.193.166.200]) by mx1.freebsd.org (Postfix) with ESMTP id B1BFF8FC08; Tue, 28 Dec 2010 15:43:50 +0000 (UTC) Received: from deviant.kiev.zoral.com.ua (root@deviant.kiev.zoral.com.ua [10.1.1.148]) by mail.zoral.com.ua (8.14.2/8.14.2) with ESMTP id oBSFhkj8010080 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Tue, 28 Dec 2010 17:43:46 +0200 (EET) (envelope-from kostikbel@gmail.com) Received: from deviant.kiev.zoral.com.ua (kostik@localhost [127.0.0.1]) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4) with ESMTP id oBSFhkRG031443; Tue, 28 Dec 2010 17:43:46 +0200 (EET) (envelope-from kostikbel@gmail.com) Received: (from kostik@localhost) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4/Submit) id oBSFhkFL031442; Tue, 28 Dec 2010 17:43:46 +0200 (EET) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: deviant.kiev.zoral.com.ua: kostik set sender to kostikbel@gmail.com using -f Date: Tue, 28 Dec 2010 17:43:46 +0200 From: Kostik Belousov To: John Baldwin Message-ID: <20101228154346.GT90883@deviant.kiev.zoral.com.ua> References: <201012250842.oBP8gchA016800@svn.freebsd.org> <20101228132942.GY23098@acme.spoerlein.net> <20101228133247.GR90883@deviant.kiev.zoral.com.ua> <201012281009.41028.jhb@freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="iskw6J4cuOvZ6IVF" Content-Disposition: inline In-Reply-To: <201012281009.41028.jhb@freebsd.org> User-Agent: Mutt/1.4.2.3i X-Virus-Scanned: clamav-milter 0.95.2 at skuns.kiev.zoral.com.ua X-Virus-Status: Clean X-Spam-Status: No, score=-3.4 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00, DNS_FROM_OPENWHOIS autolearn=no version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on skuns.kiev.zoral.com.ua Cc: svn-src-head@freebsd.org, Ulrich Sp??rlein , src-committers@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r216694 - head/libexec/rtld-elf X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Dec 2010 15:43:51 -0000 --iskw6J4cuOvZ6IVF Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Dec 28, 2010 at 10:09:40AM -0500, John Baldwin wrote: > On Tuesday, December 28, 2010 8:32:47 am Kostik Belousov wrote: > > On Tue, Dec 28, 2010 at 02:29:43PM +0100, Ulrich Sp??rlein wrote: > > > On Sat, 25.12.2010 at 08:42:38 +0000, Konstantin Belousov wrote: > > > > Author: kib > > > > Date: Sat Dec 25 08:42:38 2010 > > > > New Revision: 216694 > > > > URL: http://svn.freebsd.org/changeset/base/216694 > > > >=20 > > > > Log: > > > > Add a hook to pass debug flags to the build of rtld when doing ma= ke in > > > > the rtld directory. > > > > =20 > > > > Reviewed by: kan > > >=20 > > > Please revert this, $(VAR) is against style, and passing DEBUG_FLAGS = is > > > the canonical way to achieve what you wanted, eg. make DEBUG_FLAGS=3D= -g is > > > working just fine for me. > > DEBUG_FLAGS=3D-g might be fine for you, but it does not do what rtld de= bugging > > needs. Look at the -DDEBUG. > >=20 > > If you want ${} instead of $(), feel free to change. >=20 > Err, why can't you use 'make DEBUG_FLAGS=3D-DDEBUG' or > 'make DEBUG_FLAGS=3D"-g -DDEBUG"'? >=20 > If you are just doing 'make DEBUG=3D-DDEBUG' with this change, then I agr= ee > with Ulrich, DEBUG_FLAGS is already suitable for this purpose. >=20 I previously had a version that used RTLD_DEBUG, that I set from make.conf. Now I do make DEBUG_FLAGS=3D"-g" DEBUG=3D"-DDEBUG". The patch allows to force the compilation of dbg() without affecting global DEBUG_FLAGS setting= s. All big patches to rtld have -DDEBUG addition, and I tired trying to not forget removing them before the commit. Anyway, if somebody wants to revert the whole commit, feel free, I will return to local patching. --iskw6J4cuOvZ6IVF Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iEYEARECAAYFAk0aBbIACgkQC3+MBN1Mb4iJiwCgyxhO3ER2aOca2oPOIKEaIVPd WksAn1uGYQYnDgzcB/TWN8a289KbwAnP =8uXH -----END PGP SIGNATURE----- --iskw6J4cuOvZ6IVF--