<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 5 March 2015 at 11:46, Antony Antony <span dir="ltr"><<a href="mailto:antony@phenome.org" target="_blank">antony@phenome.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Wed, Mar 04, 2015 at 05:10:37PM -0500, Andrew Cagney wrote:<br>
> I think it is time to throw the -Werror switch. So how about sometime<br>
> on/after 11th March (that's a week, everything on the internet takes a<br>
> week)?<br>
<br>
</span>as long as there is one place to turn it off locally, it sounds good me.<br>
<span class=""><br>
><br>
> If for some reason you don't want -Werror then add:<br>
><br>
> WERROR_CFLAGS=<br>
><br>
> to your Makefile.inc.local. If for some reason you don't want the actual<br>
> warnings then also add:<br>
><br>
> WARNING_CFLAGS=<br>
><br>
> (both of these work now)<br>
<br>
</span>does WARNING_CFLAGS= also work in mk/<a href="http://local.mk" target="_blank">local.mk</a>? what is the heirarchy of make files?<br>
What is prefered way mk/<a href="http://local.mk" target="_blank">local.mk</a> or Makefile.inc.local ? I know we are trying to be backword compitable. If I don't know what is the prefered way I could stuck in old universe.<br>
<span class=""><br></span></blockquote><div><br></div><div>Put any overrides in Makefile.inc.local. If you feel the desire to change anything mk/*.mk directly, seek professional advice (or report a bug) :-)<br><br></div><div>mk/<a href="http://local.mk">local.mk</a> is a hack, per:<br># Currently both mk/<a href="http://dirs.mk">dirs.mk</a> and Makefile.inc need to see the local<br># definitions in Makefile.inc.local but Makefile.inc can't (yet)<br># assume <a href="http://dirs.mk">dirs.mk</a> has been included. This wrapper prevents multiple<br># includes.<br><br><br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
> The old WERROR variable may continue to work for a while (cleaning up<br>
> CFLAGS in general will eventually result in its loss).<br>
<br>
</span>Along these lines I have a request. I want to turn off "-O2" from one place, for programs, libs<br>
I beleive in the past I had to edit multiple Makefiles. Disabling -O2 is useful in combination with -g.<br></blockquote><div><br></div><div>I could add DEBUG_CFLAGS = -g and O_CFLAGS = -O2. Unfortunately, per an earlier post, we've no consistency with what flags get passed to CC so as you've noticed it doesn't always work.<br><br></div><div>Somewhere in my queue.<br></div><div><br><br><br></div></div></div></div>