search

mas-bandwidth / yojimbo

A network library for client/server games written in C++
BSD 3-Clause "New" or "Revised" License
2.46k stars 239 forks source link

fix build for gcc 6 and up #104

Closed jarrettchisholm closed 5 years ago

gafferongames commented 5 years ago

Looks like it's not building on the old GCC.

test.cpp: In function ‘void test_reliable_fragment_overflow_bug()’: test.cpp:2176:18: warning: unused variable ‘deltaTime’ [-Wunused-variable] const double deltaTime = 0.1; ^ Linking test g++: error: unrecognized command line option ‘-no-pie’ make[1]: [bin/test] Error 1 make: [test] Error 2

gafferongames commented 5 years ago

Can't accept until this fix passes our tests. Please fix

jarrettchisholm commented 5 years ago

Sorry, I'm not really sure how to fix this. Any chance we can bump the minimum GCC version to 6?

gafferongames commented 5 years ago

Not sure. Let's check with everybody else... I'd prefer that we continue to run on older GCCs if possible... what linux distribution is required to get GCC 6 for Debian, Ubuntu, etc?

jarrettchisholm commented 5 years ago

I believe Ubuntu 16.10 is the first version of Ubuntu to have GCC 6. Having said that, most GCC versions can be added to Ubuntu with Toolchain test builds PPA.

I believe Debian 9 (stretch) is the first version of Debian to have GCC 6.