Skip to content

Commit e7160d5

Browse files
authored
Merge pull request #135 from saxbophone/revert-134-josh/gcc-v5-travis-exeriment
Revert "Enable gcc-5 builds on Travis CI"
2 parents 0cf5b65 + 1c9ab34 commit e7160d5

File tree

3 files changed

+11
-26
lines changed

3 files changed

+11
-26
lines changed

.travis.yml

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,18 @@
11
language: c
2-
# Ubuntu 14.04 Trusty support
3-
sudo: required
4-
dist: trusty
52
# build matrix with both OSes and Compilers
63
os:
74
- linux
85
- osx
96
compiler:
107
- clang
11-
- gcc
8+
# commenting out gcc until installing GCC v5.x+ actually works on Travis CI
9+
# - gcc
1210
# different C Standard versions - test on C99 and C11
1311
env:
14-
matrix:
15-
- LIBSAXBOSPIRAL_C_STANDARD=99
16-
- LIBSAXBOSPIRAL_C_STANDARD=11
12+
- LIBSAXBOSPIRAL_C_STANDARD=99
13+
- LIBSAXBOSPIRAL_C_STANDARD=11
14+
# exclude gcc on osx as this always points to clang
1715
matrix:
18-
# exclude gcc on osx as this always points to clang
1916
exclude:
2017
- os: osx
2118
compiler: gcc
@@ -25,22 +22,18 @@ addons:
2522
apt:
2623
sources:
2724
- george-edison55-precise-backports # cmake 3.2.3 / doxygen 1.8.3
28-
# add PPAs with more up-to-date toolchains
29-
- ubuntu-toolchain-r-test
3025
packages:
3126
- cmake
3227
- cmake-data
28+
# commenting out gcc until installing GCC v5.x+ actually works on Travis CI
3329
# want gcc 5.x as 4.x gives incorrect warnings
34-
- gcc-5
30+
# - gcc-5
3531
# branches safelist
3632
branches:
3733
only:
3834
- master
3935
- develop
4036
- /^test\/.*$/
41-
install:
42-
# override $CC to use gcc-5
43-
- if [ "$CC" = "gcc" ]; then export CC="gcc-5"; fi
4437
before_script:
4538
- cmake .
4639
script:

CMakeLists.txt

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,8 @@ cmake_minimum_required(VERSION 3.0)
2525

2626
project(libsaxbospiral VERSION 0.21.0 LANGUAGES C)
2727

28-
# set default C standard to use (C99) if not already set
29-
if(NOT DEFINED SAXBOSPIRAL_C_STANDARD)
30-
set(SAXBOSPIRAL_C_STANDARD "99")
31-
endif()
28+
# set default C standard to use (C99)
29+
set(SAXBOSPIRAL_C_STANDARD "99")
3230
# if env variable LIBSAXBOSPIRAL_C_STANDARD is set and valid, override version
3331
if(DEFINED ENV{LIBSAXBOSPIRAL_C_STANDARD})
3432
# not a very robust regex but ok for most purposes

README.md

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -61,17 +61,11 @@ cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON .
6161
make
6262
```
6363

64-
The above builds in C99 mode by default. The standard to use can be controlled by the `SAXBOSPIRAL_C_STANDARD` CMake variable or by the `LIBSAXBOSPIRAL_C_STANDARD` environment variable.
64+
The above builds in C99 mode by default. The standard to use is controlled by the `LIBSAXBOSPIRAL_C_STANDARD` environment variable.
6565

66-
You can build in C11 mode if you want with either of the following:
66+
You can build in C11 mode if you want with the following:
6767

6868
```sh
69-
# using CMake variable
70-
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON -DSAXBOSPIRAL_C_STANDARD=11 .
71-
```
72-
73-
```sh
74-
# using environment variable
7569
LIBSAXBOSPIRAL_C_STANDARD=11 cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON .
7670
```
7771

0 commit comments

Comments
 (0)