semlanik / qtprotobuf

Protobuf generator and bindings for Qt framework
https://semlanik.github.io/qtprotobuf
MIT License
169 stars 38 forks source link

Build errors #274

Closed RBQt closed 2 years ago

RBQt commented 2 years ago

Describe the bug Build failure with gcc 11, boringssl GNU CXX Flag 'Werror' fails on multiple issues, workaround by takin 'Werror' out of the GNU CXX Flag list in qtprotobuf/3rdparty/grpc/third_party/boringssl/CMakeLists.txt:

In file included from qtprotobuf/3rdparty/googletest/googletest/src/gtest-all.cc:42: qtprotobuf/3rdparty/googletest/googletest/src/gtest-death-test.cc: In function ‘bool testing::internal::StackGrowsDown()’: qtprotobuf/3rdparty/googletest/googletest/src/gtest-death-test.cc:1301:24: error: ‘dummy’ may be used uninitialized [-Werror=maybe-uninitialized] 1301 | StackLowerThanAddress(&dummy, &result);

A more suistanable fix would be to propagate CXX_FLAGS with "add_compile_options" from tprotobuf/CMakeLists.txt. Would be nice to be able to request the compilation of the protobuf generator trough a cache variable to bypass any issue with the 3rd parties.

RBQt commented 2 years ago

move to feature request