On Thu, Dec 1, 2016 at 1:23 PM Timo Rothenpieler <t...@rothenpieler.org> wrote:
> Travis can only run scheduled builds daily, weekly or monthly. So we run them daily, and use a bit of logic in the .travis.yml to > cancel out early on 3 days per week. > Nice! Didn't know Travis CI could do this. > --- > .travis.yml | 32 +++++++------------------------- > 1 file changed, 7 insertions(+), 25 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index e541ee1..abc264a 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -1,26 +1,8 @@ > -language: c > -sudo: false > -os: > - - linux > - - osx > -addons: > - apt: > - packages: > - - yasm > - - diffutils > -compiler: > - - clang > - - gcc > -cache: > - directories: > - - ffmpeg-samples > -before_install: > - - if [ "$TRAVIS_OS_NAME" == "osx" ]; then brew update --all; fi > -install: > - - if [ "$TRAVIS_OS_NAME" == "osx" ]; then brew install yasm; fi > +sudo: required > +services: > + - docker > script: > - - mkdir -p ffmpeg-samples > - - ./configure --samples=ffmpeg-samples --cc=$CC > - - make -j 8 > - - make fate-rsync > - - make check -j 8 > +- DOW="$(date "+%u")" > +- for d in 2 4 6; do [[ "$d" == "$DOW" ]] && exit 0; done > +- docker pull ffmpeg/coverity > +- docker run --env COV_EMAIL --env COV_TOKEN ffmpeg/coverity > A few nits: indent the array, just as you did for `services`; the official Travis CI-Coverity bridge uses COVERITY_SCAN_NOTIFICATION_EMAIL and COVERITY_SCAN_TOKEN, so for consistency you might want to change that. Another thing is that currently https://github.com/BtbN/FFmpeg-Coverity (the source of "ffmpeg/coverity" image) belongs to your GitHub account. Maybe we should think of transferring that to github.com/FFmpeg? I also have a few comments on your current build scripts, but we can change those once this patch is in. Timothy _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
