On Mon, 07/25 10:48, Peter Maydell wrote:
> On 25 July 2016 at 10:37, Fam Zheng <f...@redhat.com> wrote:
> > Cool, so we do have that version, it's the -Werror=missing-braces option we 
> > are
> > missing.  Are there any other recommended gcc options I can add to
> > tests/docker/ (the criteria is simply "if this stops you from processing a
> > pull, it should be tested" :)?
> 
> My pull tests just use the default configure options here. In
> particular -Wmissing-braces is part of -Wall, and -Werror is
> enabled by default for Linux builds from git.

Okay, there isn't a .git in the docker checkout, so -Werror is turned off by
configure. Looks like I should add --enable-werror to the configure command.

Fam

Reply via email to