<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 31, 2021 at 9:56 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Should we set the cstd version at the top-level?<br></blockquote><div><br></div><div>That's not what was done for C++ when I looked for examples. I suppose this</div><div>lets the C++ version vary by tool or library. And we did import some third-party</div><div>code.</div><div><br></div><div>--joel</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Tue, Mar 30, 2021 at 3:16 PM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
><br>
> CentOS 7 has gcc 4.8 which defaults to C90 and this results in warnings<br>
> for code that is valid C99 but not C90.<br>
> ---<br>
>  misc/wscript | 3 ++-<br>
>  1 file changed, 2 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/misc/wscript b/misc/wscript<br>
> index 521eddf..21e7f75 100644<br>
> --- a/misc/wscript<br>
> +++ b/misc/wscript<br>
> @@ -51,7 +51,8 @@ def build(bld):<br>
>      #<br>
>      conf['warningflags'] = ['-Wall', '-Wextra', '-pedantic']<br>
>      conf['optflags'] = bld.env.C_OPTS<br>
> -    conf['cflags'] = ['-pipe', '-g'] + conf['optflags']<br>
> +    cstd = '-std=c99'<br>
> +    conf['cflags'] = [cstd] + ['-pipe', '-g'] + conf['optflags']<br>
>      conf['linkflags'] = ['-g']<br>
><br>
>      #<br>
> --<br>
> 1.8.3.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>