thiagoralves / OpenPLC_v3

OpenPLC Runtime version 3
GNU General Public License v3.0
1.1k stars 442 forks source link

Fix the build of glue_generator #111

Closed innir closed 4 years ago

innir commented 4 years ago

Without this fix I get a linker error, as 'md5.c' is treated as a C++ file. Compiling it with a C compiler is fine as md5.h defines 'extern "C"' anyway.

@garretfick: Does this work for you?

garretfick commented 4 years ago

This works (I've built and tested it locally) and has my consent.

@innir What environment are you building on? I haven't seen this issue before and it would be very good to ensure that we include that in the test matrix so that this cannot come back.

innir commented 4 years ago

@garretfick I'm building on Debian testing (codename: Bullseye)

$ cmake --version
cmake version 3.13.4

$ g++ --version
g++ (Debian 9.2.1-19) 9.2.1 20191109

$ ld --version
GNU ld (GNU Binutils for Debian) 2.33.1