-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Travis support #29
Travis support #29
Conversation
- Installing requirements with install script - Add missing include(checkIncludeFile) cmake command - Configuration matrix to compile the project with different options
Thanks very much for doing this, Luis. I hope you've had a nice weekend. This hasn't solved the issue with libintl on MacOSX. However, I can see you're been changing stuff concerning linking exiv2 with other libraries. I've never investigated the .po files Thomas (our German Contributor), knows about them. For sure we initialise libintl from exiv2.cpp. I'm not sure the .po files are used by the library. If they are not, libintl shouldn't be linked to libexiv2. I'll email Thomas and ask him about this. |
Whoops, I spoke too soon. I've pulled down a fresh copy of the code from GitHub and it builds. Thanks for fixing this. And I'm delighted to see that CMake build is publishing the correct headers in /usr/local/include/exiv2. |
You are welcome :) Right I did not check that yet, but now I can check it out even at home using travis. The problem is that we are assuming the existence of Intl:
But I will experiment this week with that. It should be easily fixable ;) |
This seems totally splendid. Is there a Travis node actually building the code? |
Yes, but only on my fork: I left you some messages on slack explaining the alternatives we have to enable the support in the main repo |
Adds travis support for building the project on Linux and Mac OSX with different configurations. It actually helped me to find some issues in the CMake code :)