Skip to content

Add new compilers #689

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Jun 11, 2019
Merged

Add new compilers #689

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
81327b9
Move linux distro selection to global scope
Jun 10, 2019
a730646
Add more C++ compilers to build matrix
Jun 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 54 additions & 14 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
dist: xenial

sudo: false

matrix:
Expand All @@ -7,21 +9,31 @@ matrix:
language: java
jdk: openjdk8
os: linux
dist: xenial
Copy link
Contributor

@mjpt777 mjpt777 Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this changed to default? The default for Travis is 14.04 when most people are on 16.04 or later.

Copy link
Contributor

@tmontgomery tmontgomery Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct me if I am wrong.... Xenial is 16.04 LTS. Isn't it the latest LTS? Bionic is 18.04 LTS. Not sure it's image in Travis is that stable yet, though.

Copy link
Contributor

@mjpt777 mjpt777 Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tmontgomery You are correct. However Travis default is Trusty. https://docs.travis-ci.com/user/reference/overview/

Copy link
Contributor

@tmontgomery tmontgomery Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trusty being 5 years old.... shouldn't it be compiling against a newer version? I'm just making the argument as I see it. I don't know if the PR author had any other reason or reasons. Maybe @rigtorp can weigh in.

Copy link
Contributor

@mjpt777 mjpt777 Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed it to be Xenial to be more recent and now it is reverted.

Copy link
Contributor

@tmontgomery tmontgomery Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't xenial now the default at the top? Making it unnecessary to specify it on each one?

Copy link
Contributor

@mjpt777 mjpt777 Jun 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed that. Sorry.

script: ./gradlew --scan

- name: "OpenJDK 11"
language: java
jdk: openjdk11
os: linux
dist: xenial
script: ./gradlew --scan

- env: COMPILER=g++-5
language: cpp
compiler: gcc
os: linux
addons:
apt:
packages:
- g++-5
- libsparsehash-dev
sources:
- ubuntu-toolchain-r-test
script: ./cppbuild/cppbuild

- env: COMPILER=g++-6
language: cpp
compiler: gcc
os: linux
dist: xenial
addons:
apt:
packages:
Expand All @@ -35,7 +47,6 @@ matrix:
language: cpp
compiler: gcc
os: linux
dist: xenial
addons:
apt:
packages:
Expand All @@ -49,7 +60,6 @@ matrix:
language: cpp
compiler: gcc
os: linux
dist: xenial
addons:
apt:
packages:
Expand All @@ -59,54 +69,84 @@ matrix:
- ubuntu-toolchain-r-test
script: ./cppbuild/cppbuild

- env: COMPILER=g++-9
language: cpp
compiler: gcc
os: linux
addons:
apt:
packages:
- g++-9
- libsparsehash-dev
sources:
- ubuntu-toolchain-r-test
script: ./cppbuild/cppbuild

- env: COMPILER=clang++-4.0
language: cpp
compiler: clang
os: linux
addons:
apt:
packages:
- clang-4.0
- libsparsehash-dev
sources:
- llvm-toolchain-xenial-4.0
script: ./cppbuild/cppbuild

- env: COMPILER=clang++-5.0
language: cpp
compiler: clang
os: linux
dist: xenial
addons:
apt:
packages:
- clang-5.0
- libstdc++-7-dev
- libsparsehash-dev
sources:
- ubuntu-toolchain-r-test
- llvm-toolchain-xenial-5.0
script: ./cppbuild/cppbuild

- env: COMPILER=clang++-6.0
language: cpp
compiler: clang
os: linux
dist: xenial
addons:
apt:
packages:
- clang-6.0
- libstdc++-7-dev
- libsparsehash-dev
sources:
- ubuntu-toolchain-r-test
- llvm-toolchain-xenial-6.0
script: ./cppbuild/cppbuild

- env: COMPILER=clang++-7
language: cpp
compiler: clang
os: linux
dist: xenial
addons:
apt:
packages:
- clang-7
- libstdc++-7-dev
- libsparsehash-dev
sources:
- ubuntu-toolchain-r-test
- llvm-toolchain-xenial-7
script: ./cppbuild/cppbuild

- env: COMPILER=clang++-8
language: cpp
compiler: clang
os: linux
addons:
apt:
packages:
- clang-8
- libsparsehash-dev
sources:
- llvm-toolchain-xenial-8
script: ./cppbuild/cppbuild

- env: COMPILER=clang++
language: c++
compiler: clang
Expand Down