<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 19, 2022, 3:00 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On 20 Aug 2022, at 3:39 am, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> wrote:<br>
> <br>
> Hi<br>
> <br>
> Stephen tripped across this and I'm reporting it because it is going to <br>
> take some discussion to resolve.<br>
> <br>
> Stephen is using the .pc files to create a configuration file<br>
> for the FACE Conformance Test Suite. This is now basically working<br>
> for multiple BSPs. The amd64 includes -Werror in its BSP compiler<br>
> options and is the only BSP to do so.<br>
<br>
Looks like a bug to me. It should not be in the standard CFLAGS returned by pkgconfig.</blockquote></div></div><div dir="auto"><br></div><div dir="auto">Is there code.on our side filtering flags that missed this? I grep'ed for Werror and Wall but didn't find anything.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br>
<br>
> (1) Do we want a BSP to include -Werror? amd64 being the only one<br>
> and use having a history of chasing warnings after gcc upgrades.<br>
<br>
Yes remove.</blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ok but is this the only way to remove a warning from the PC file?</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br>
<br>
Chris<br>
<br>
> (2) I though warning flags were filtered out on the install. I'm not finding<br>
> the code to do it but shouldn't this be filtered out like the other warning flags?<br>
> <br>
> Any other thoughts?<br>
> <br>
> --joel<br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
<br>
</blockquote></div></div></div>