-Werror on amd64 BSP makes it to installed package configuration file

[Joel Sherrill]

On Fri, Aug 19, 2022, 3:00 PM Chris Johns <chrisj at rtems.org> wrote:

> > On 20 Aug 2022, at 3:39 am, Joel Sherrill <joel at rtems.org> wrote:
> >
> > Hi
> >
> > Stephen tripped across this and I'm reporting it because it is going to
> > take some discussion to resolve.
> >
> > Stephen is using the .pc files to create a configuration file
> > for the FACE Conformance Test Suite. This is now basically working
> > for multiple BSPs. The amd64 includes -Werror in its BSP compiler
> > options and is the only BSP to do so.
>
> Looks like a bug to me. It should not be in the standard CFLAGS returned
> by pkgconfig.


Is there code.on our side filtering flags that missed this? I grep'ed for
Werror and Wall but didn't find anything.


>
> > (1) Do we want a BSP to include -Werror? amd64 being the only one
> > and use having a history of chasing warnings after gcc upgrades.
>
> Yes remove.


Ok but is this the only way to remove a warning from the PC file?

>
>
> Chris
>
> > (2) I though warning flags were filtered out on the install. I'm not
> finding
> > the code to do it but shouldn't this be filtered out like the other
> warning flags?
> >
> > Any other thoughts?
> >
> > --joel
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20220819/56d8cdbf/attachment-0001.htm>