[ovs-dev] [PATCH] INSTALL: Mention --enable-Werror.

Ben Pfaff blp at nicira.com
Wed Dec 18 05:45:18 UTC 2013


On Tue, Dec 17, 2013 at 05:29:44PM -0800, Jarno Rajahalme wrote:
> 
> On Dec 17, 2013, at 5:01 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > I think that some developers haven't noticed this, but it's useful, so
> > mention it.
> > 
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > INSTALL |    5 +++++
> > 1 file changed, 5 insertions(+)
> > 
> > diff --git a/INSTALL b/INSTALL
> > index c8b1aa7..27c0e5f 100644
> > --- a/INSTALL
> > +++ b/INSTALL
> > @@ -198,6 +198,11 @@ Prerequisites section, follow the procedure below to build.
> > 
> >       % ./configure --with-linux=/path/to/linux KARCH=mips
> > 
> > +   If you plan to do much Open vSwitch development, you might want to
> > +   add --enable-Werror, which adds the -Werror option to the compiler
> > +   command line, turning warnings into errors.  That makes it
> > +   impossible to miss warnings generated the build.
> > +
> 
> ?by the build."
> 
> Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>

Thanks.  Applied with that change.



More information about the dev mailing list