search

itemisCREATE / statecharts

YAKINDU Statechart Tools (http://www.statecharts.org)
Eclipse Public License 1.0
176 stars 86 forks source link

Migrate to newer googletest verion #2903

Closed andreasmuelder closed 4 years ago

andreasmuelder commented 4 years ago

We have to update the googletest framework since GCC > 7.1 throws a warning with the current tests: 1.8.1 + 1.10.0 do not compile for C++:

Compilation failed (exit status 1): /home/travis/build/Yakindu/statecharts/googletest-release-1.8.1/googlemock/build/libgmock_main.a(gmock_main.cc.o): In function main': gmock_main.cc:(.text+0x31): undefined reference totesting::InitGoogleMock(int*, char**)' collect2: error: ld returned 1 exit status at org.yakindu.sct.generator.cpp.test.OperationsTest.setUp(OperationsTest.java:33)

rherrmannr commented 4 years ago

There was a bug report: https://github.com/google/googletest/issues/1094 But seems like there is no fix.

rherrmannr commented 4 years ago

Actually, we don't use anything from gmock. We can simply remove the include.