search

amagnasco / xwpe

Upload of an abandoned ncurses-based programming environment
http://www.identicalsoftware.com/xwpe/
GNU General Public License v2.0
29 stars 6 forks source link

Using check library across platforms: static vs dynamic #70

Closed gbonnema closed 7 years ago

gbonnema commented 7 years ago

When working on a clean up and more or less rewriting parts of the search algorithm, I found it necessary to create test cases first. And I did, and it worked ..... on my machine.

Come time to do the pull request, Travis CI refuses the commits because in Ubuntu the check library is only staticly available, not dynamically.

I have posted a question in Stackoverflow and I am awaiting response.

Until then I will leave the branch as it is as I can not merge it, until I get some kind of a resolution.

gbonnema commented 7 years ago

This problem was a nuisance. By accident I found what appears to be the solution. But I am not confident. The solution turns out to be the order of libraries in test/Makefile.am where libxwpe.so should be named before libxwpe_test.so.

As I said, I am not confident. I do not like it when I do not fully understand what solved the problem or even what the problem was! Firstly, it worked on GCC-7 (my machine) and on CLANG (travis). Secondly, why would the order of linking libraries matter?

I solved it, and I will close this. Still not happy.