From owner-freebsd-arch@FreeBSD.ORG Tue Mar 30 12:38:36 2010 Return-Path: Delivered-To: freebsd-arch@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BE35A1065672; Tue, 30 Mar 2010 12:38:36 +0000 (UTC) (envelope-from alexander@leidinger.net) Received: from mail.ebusiness-leidinger.de (mail.ebusiness-leidinger.de [217.11.53.44]) by mx1.freebsd.org (Postfix) with ESMTP id 5FCAE8FC1D; Tue, 30 Mar 2010 12:38:36 +0000 (UTC) Received: from outgoing.leidinger.net (pD9E2CD4C.dip.t-dialin.net [217.226.205.76]) by mail.ebusiness-leidinger.de (Postfix) with ESMTPSA id A4188844410; Tue, 30 Mar 2010 14:38:27 +0200 (CEST) Received: from webmail.leidinger.net (webmail.leidinger.net [192.168.1.102]) by outgoing.leidinger.net (Postfix) with ESMTP id 9BC86508C; Tue, 30 Mar 2010 14:38:24 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=Leidinger.net; s=outgoing-alex; t=1269952704; bh=w/3TSSBD5jnn3mn65llzAC6HNQyjoKfKS1EeoQMbbqs=; h=Message-ID:Date:From:To:Cc:Subject:References:In-Reply-To: MIME-Version:Content-Type:Content-Transfer-Encoding; b=hEtwnXDWxKsoc6imNG+NHUevbXW6najoCQq0OslOaWfrEPDsnPl0A0qHcPIJrOJV2 UKBPhx1TESRtTcRXrl4OX8DyrXC/n0p5B6FiMzoSVPNHpPgufsyfbAF0Av07+oJJIY 4Q/Tcn4mIy6NadcS5mjt9UIM5k/fKKCwx25oR4veu38kLQPEVV7aTLfPrGr08E5y+G j4bd/b5Dfrwm0PpmlJYlHmfMDOzocaxPU9MH1nKiMrJHRaPfbWq3Jcdo16nsZTwV+1 6JrG3tOrHiRYTEBNAZ0hsQBfkEuED/iXf+AbFPMs1x9YUl9i1cFHRrDTYK/+7Mudql K14QjQytYTmFw== Received: (from www@localhost) by webmail.leidinger.net (8.14.3/8.13.8/Submit) id o2UCcOtN010711; Tue, 30 Mar 2010 14:38:24 +0200 (CEST) (envelope-from Alexander@Leidinger.net) Received: from pslux.cec.eu.int (pslux.cec.eu.int [158.169.9.14]) by webmail.leidinger.net (Horde Framework) with HTTP; Tue, 30 Mar 2010 14:38:24 +0200 Message-ID: <20100330143824.252520tulyv7bxes@webmail.leidinger.net> Date: Tue, 30 Mar 2010 14:38:24 +0200 From: Alexander Leidinger To: gary.jennejohn@freenet.de References: <20100326101524.15695bisy2324t8g@webmail.leidinger.net> <20100326.093717.857133809997909007.imp@bsdimp.com> <20100326171241.16524bklcroedou8@webmail.leidinger.net> <20100326.102344.364422370204095721.imp@bsdimp.com> <20100327101846.5b4d6b18@ernst.jennejohn.org> In-Reply-To: <20100327101846.5b4d6b18@ernst.jennejohn.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; DelSp="Yes"; format="flowed" Content-Disposition: inline Content-Transfer-Encoding: 7bit User-Agent: Dynamic Internet Messaging Program (DIMP) H3 (1.1.4) X-EBL-MailScanner-Information: Please contact the ISP for more information X-EBL-MailScanner-ID: A4188844410.C98DE X-EBL-MailScanner: Found to be clean X-EBL-MailScanner-SpamCheck: not spam, spamhaus-ZEN, SpamAssassin (not cached, score=-1.44, required 6, autolearn=disabled, ALL_TRUSTED -1.44, DKIM_SIGNED 0.00, DKIM_VERIFIED -0.00) X-EBL-MailScanner-From: alexander@leidinger.net X-EBL-MailScanner-Watermark: 1270557508.26716@P9WXXz1vYO5jNV4f7qymaQ X-EBL-Spam-Status: No Cc: rwatson@freebsd.org, freebsd-arch@freebsd.org Subject: Re: CTF patch for testing/review X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 30 Mar 2010 12:38:36 -0000 Quoting Gary Jennejohn (from Sat, 27 Mar 2010 10:18:46 +0100): > On Fri, 26 Mar 2010 10:23:44 -0600 (MDT) > "M. Warner Losh" wrote: > >> In message: <20100326171241.16524bklcroedou8@webmail.leidinger.net> >> Alexander Leidinger writes: > [entry in build(7)] >> : I will have a look at it next week. As I'm not a native english >> : speaker, you can expect another patch for review (text suggestions for >> : build.9 are welcome, I will take care about the mdoc markup then). >> > > Note, it's build(7), not build(9). At least, I find no build(9) on > my -current system installed two days ago. > >> How does this text look (posted so everybody can kibitz): >> >> WITH_CTF the build process will run the DTrace CTF conversion >> tools on built objects. Please note that this WITH_ >> option is handled differently than all other WITH_ >> options (there's no WITHOUT_CTF, or corresponding > ==> there is - no contractions, please >> MK_CTF in the build system). >> > > Otherwise it looks OK to me. http://www.leidinger.net/test/ctf4.diff Changes to previous one: - added @ to hide the CTF stuff - man-page (to test: "groff -mdoc -Tascii /path/to/build.7 | less") Someone with mdoc-fu should have a look if I got it right, the others please have a look if the place in the man-page pleases you. The changed commit log would be: ---snip--- WITH_CTF can now be specified in src.conf (not recommended, there are some problems with static executables), make.conf (would also affect ports which do not use GNU make and do not override the compile targets) or in the kernel config (via "makeoptions WITH_CTF=yes"). Additional (related) changes: - propagate WITH_CTF to module builds - do not add -g to the linker flags, it's a noop there anyway (at least according to the man page of ld) - do not add -g to CFLAGS unconditionally we need to have a look if it is really needed (IMO not) or if there is a way to add it only when WITH_CTF is used Note: ctfconvert / ctfmerge lines will not appear in the build output, to protect the innocent (those which do not build with WITH_CTF would see the shell-test and may think WITH_CTF is used). Reviewed by: imp, jhb, scottl (earlier version) Discussed on: arch@ ---snip--- What about Friday morning (GMT+2) for a commit date (if nobody complains): not long enough for review or is it OK? Bye, Alexander. -- The clearest way into the Universe is through a forest wilderness. -- John Muir http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137