[libprotobuf WARNING google/protobuf/compiler/parser.cc:491] No syntax
specified for the proto file. Please use 'syntax = "proto2";' or
'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2
syntax.)
The release notes say a future release will turn this warning into an
error.
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
clang was spewing warnings about the unrecognized -fno-default-inline. (Oddly,
it warns only with -c, not when compiling directly to an executable.) For
completeness we also check -pipe, even though clang is OK with that one.
It should be fine to omit either flag. gcc -fno-default-inline drops the
implicit 'inline' annotation on functions defined inside a class scope, but
'inline' is only a hint anyway. -fno-default-inline does not change linkage.
-pipe is merely a compile speed optimization.
With automake 1.11, this gives us Linux kernel–style silent output
that makes warnings more obvious:
GEN userinput.pb.cc
CXX userinput.pb.o
AR libmoshprotos.a
CXXLD mosh-client
(Use ‘make V=1’ for the traditional verbose output.)
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
This turns off -Werror by default except in the Debian package.
-Werror is inherently unportable and breaks things on OS X. While it
can cause problems in Debian too, this is at least a little better.
It also turns off -pedantic in src/protobufs, because apparently
protoc output doesn’t compile with -pedantic on all platforms.
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
John Hood