<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Here is the patch with the ld option check as well. <br>
<br>
I had to make a few changes to Ron's original patch. One of the
problems was that CFLAGS was being set as a := in the Makefile and was
overwriting the += that was in Makefile.settings. So I added
DISTRO_CFLAGS and DISTRO_LFLAGS for the two additonal flags. The
reason for two was that the build-id option is only used during linking
and generated warnings if used on a compile line and was not being
pulled in for the linker lines. These defines were added to the
appropriate places in the Makefile via the Config.lb file. <br>
<br>
<br>
<br>
<pre class="moz-signature" cols="72">/*********************
Marc Karasek
MTS
Sun Microsystems
<a class="moz-txt-link-freetext" href="mailto:marc.karasek@sun.com">mailto:marc.karasek@sun.com</a>
ph:770.360.6415
*********************/
</pre>
<br>
<br>
ron minnich wrote:
<blockquote
cite="mid:13426df10801031043o3ea5df06r62f51e7c63fabaaa@mail.gmail.com"
type="cite">
<pre wrap="">On Jan 3, 2008 10:33 AM, Marc Karasek <a class="moz-txt-link-rfc2396E" href="mailto:Marc.Karasek@sun.com"><Marc.Karasek@sun.com></a> wrote:
</pre>
<blockquote type="cite">
<pre wrap="">I could add the check to this patch (using awk to check ld) to add the
-Wl option to the EXTRA_CFLAGS.
</pre>
</blockquote>
<pre wrap=""><!---->
cool! I'll take it and add it.
ron
</pre>
</blockquote>
</body>
</html>