search

USC-NSL / Computational-Agroecology

6 stars 1 forks source link

Use Cmake and also still support makefile #65

Closed tszxxx closed 5 years ago

ralphchung commented 5 years ago

Why do test.cc's need #define guards?

tszxxx commented 5 years ago

Why do test.cc's need #define guards? because if we don't do that, there are so many main function there for cmake to make

ralphchung commented 5 years ago

Why couldn't they be separated?

tszxxx commented 5 years ago

Why couldn't they be separated?

you mean that we create separated cmakefiles for each test? I think that's also ok, but now it also works well.

ralphchung commented 5 years ago

Could it be a number of executables instead of one executable? #define guard looks weird in cc files.

ralphchung commented 5 years ago

Also, you should always give instructions to guide others how to build.