2017.06.09 11:12 "Re: [Tiff] [PATCH] Unix/Linux CI support for GitHub and Travis", by Even Rouault

On vendredi 9 juin 2017 11:54:53 CEST rleigh@codelibre.net wrote:

On 2017-06-08 21:46, Even Rouault wrote:

> >> Together with the Windows patches, this provides pretty decent test

> >> coverage for common platforms and compilers, and should pick up most

> >> regressions in the codebase.
> >
> > All 3 patches committed. Thanks!

Thank you. One problem I noticed is that the build/travis-ci script does not have executable permissions, which is needed to build. I think with CVS that requires a chmod +x of the ,v file in the repository if was not set when initially added.

Arg... damned CVS. I do not have access to the server. I've asked an admin if he can do that (I tried to cvs delete, commit, do chmod +x, cvs add, commit, but that doesn't seem to hve an effect)

I'll defer applying your new patch until this is solved

I've also attached a new patch with a few small improvements and the addition of MacOS X to the test matrix. You can see the results here:

https://github.com/rleigh-codelibre/libtiff/commits/ci-fixes https://travis-ci.org/rleigh-codelibre/libtiff/builds/241129101

https://ci.appveyor.com/project/rleigh-codelibre/libtiff/build/1.0.53

--
Spatialys - Geospatial professional services
http://www.spatialys.com