CTF patch for testing/review

Alexander Leidinger Alexander at Leidinger.net
Fri Mar 26 09:15:35 UTC 2010

Quoting "M. Warner Losh" <imp at bsdimp.com> (from Thu, 25 Mar 2010  
07:37:36 -0600 (MDT)):

> In general, I think this is OK.  I'd like to see the revised patch
> before it is committed.

And here it is:

My commit log would be something like this:
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

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 yo 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: don't worry when you see ctfconvert lines appearing in your build,
they are protected with a shell conditional and are not run as long as
you don't have WITH_CTF defined.

Reviewed by:    imp, jhb, scottl (earlier version)
Discussed on:   arch@

If nobody comes up with problems (or wants more time for the review):  
I would like to commit this next week.


We are each only one drop in a great
ocean -- but some of the drops sparkle!

http://www.Leidinger.net    Alexander @ Leidinger.net: PGP ID = B0063FE7
http://www.FreeBSD.org       netchild @ FreeBSD.org  : PGP ID = 72077137

More information about the freebsd-arch mailing list