Re: [PATCH v7 2/2] Add Travis CI support
From: Lars Schneider <hidden>
Date: 2016-06-15 23:07:18
On 24 Nov 2015, at 21:40, Jeff King [off-list ref] wrote: On Mon, Nov 23, 2015 at 09:25:08AM +0100, larsxschneider@gmail.com wrote:quoted
From: Lars Schneider <redacted> The tests are currently executed on "Ubuntu 12.04 LTS Server Edition 64 bit" and on "OS X Mavericks" using gcc and clang. Perforce and Git-LFS are installed and therefore available for the respective tests.Thanks, I find this one a little easier to digest. I'm iffy on the NO_GETTEXT change from patch 1. I had hoped we could just build out of the box everywhere, but I think the "do we have libintl" decision is a hard one. Most people _do_ have it and want it, but it sounds like the Travis environment does not. So maybe it is a place where it is worth doing the tweak inside travis.yml and leaving the stock build alone.
OK, I'll try to fix it in the next roll. Thanks, Lars