Skip to content

Commit

Permalink
DRi#2764 Travis OSX: skip OSX for PR's (#2074)
Browse files Browse the repository at this point in the history
Skips running OSX for pull requests for now, since the Travis queue is
too long. We'll rely on the push to master to catch OSX issues and the
dev will have to revert or fix quickly at that point.

Re-writes the Travis build config from a matrix defined by os, compiler,
and env with exclusions to an explicit list of jobs, in order to use
conditional jobs. This is also clearer and easier to read.
  • Loading branch information
shawndenbow authored and derekbruening committed Jan 21, 2018
1 parent c1c866c commit 6f7d5f2
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions .travis.yml
@@ -1,5 +1,5 @@
# **********************************************************
# Copyright (c) 2016-2017 Google, Inc. All rights reserved.
# Copyright (c) 2016-2018 Google, Inc. All rights reserved.
# **********************************************************

# Dr. Memory: the memory debugger
Expand Down Expand Up @@ -41,20 +41,20 @@ language:
- c
- cpp

os:
- linux
- osx

# We request 2 runs, one with clang and one with gcc:
compiler:
- clang
- gcc

matrix:
exclude:
# gcc on Travis claims to not be CMAKE_COMPILER_IS_GNUCC so we disable:
- os: osx
# We use a jobs include approach rather than an os, compiler, env matrix
# with exludes so we can use conditional builds (plus it's clearer this way).
jobs:
include:
- os: linux
compiler: gcc
- os: linux
compiler: clang
- os: osx
# gcc on Travis claims to not be CMAKE_COMPILER_IS_GNUCC so we only run clang.
compiler: clang
# XXX: DRi#2764: Travis OSX resources are over-subscribed and it can take
# hours to get an OSX machine, so we skip running PR's for now.
if: type = push

# For C/C++ there is no default install, so we set "install", not "before_install".
install:
Expand Down

0 comments on commit 6f7d5f2

Please sign in to comment.