added support for travis CI #39
Conversation
adrpar
force-pushed
the
travis.adrpar
branch
20 times, most recently
from
f3db9a6 to
9427a2b
Compare
adrpar
force-pushed
the
travis.adrpar
branch
3 times, most recently
from
1d72b72 to
c6067a5
Compare
adrpar
force-pushed
the
travis.adrpar
branch
2 times, most recently
from
3d8eb98 to
3e8c9d5
Compare
adrpar
force-pushed
the
travis.adrpar
branch
11 times, most recently
from
78e935e to
cb41771
Compare
adrpar
force-pushed
the
travis.adrpar
branch
4 times, most recently
from
3cd21f9 to
952dd7e
Compare
adrpar
force-pushed
the
travis.adrpar
branch
4 times, most recently
from
8dc10a0 to
c7dfa17
Compare
|
Added all tests as mentioned in issue #25. Fsanitize on gcc is unfortunately failing, so there must be a memory issue some where (I have clions static analyzer throwing various errors, I guess it has to be one of these). Further, gcc release built only works with O2 optimization flag, most likely due to memory corruption issue that also lets fsanitize fail. But for the start, that's where we are. |
| - sudo apt install libgdal-dev gdal-bin | ||
|
|
||
| script: | ||
| - if [ $TEST_TASK == 'test' ]; |
There was a problem hiding this comment.
I'd prefer to see it in a Makefile
There was a problem hiding this comment.
I agree. We should put all the different targets and test execution into the cmake file. And this reminds me, we should also add an installation target, so that 'make install' will work.
This will not replace the switch case statement though, since this is the only way to have completely different build methods in Travis matrix (or I was not able to find a better solution).
I will create a new issue for the additional cmake targets.
No description provided.