Skip to content

Commit

Permalink
Test gcc-4.8 as part of CI (open-telemetry#19)
Browse files Browse the repository at this point in the history
* Add CI test for gcc48.

* Add gcc_48_test to circleci

* Use bazel to test old compiler.

* Fix gcc-48 CI.

* Use CC instead of CXX
  • Loading branch information
rnburn authored Jan 4, 2020
1 parent 15bc3c5 commit 901f87a
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 0 deletions.
12 changes: 12 additions & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,17 @@ jobs:
path: ~/build/Testing/Temporary/LastTest.log
destination: Test.log

gcc_48_test:
resource_class: xlarge
docker:
- image: ubuntu:18.04
steps:
- checkout
- run: ./ci/setup_ci_environment.sh
- run: ./ci/install_bazelisk.sh
- run: ./ci/install_gcc48.sh
- run: CC=/usr/bin/g++-4.8 ./ci/do_ci.sh bazel.test

bazel_test:
resource_class: xlarge
docker:
Expand Down Expand Up @@ -48,6 +59,7 @@ workflows:
build_and_test:
jobs:
- cmake_test
- gcc_48_test
- bazel_test
- osx_test
- windows
1 change: 1 addition & 0 deletions ci/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,5 @@ ADD install_bazelisk.sh /setup-ci

RUN /setup-ci/setup_ci_environment.sh \
&& /setup-ci/setup_cmake.sh \
&& /setup-ci/install_gcc48.sh \
&& /setup-ci/install_bazelisk.sh
6 changes: 6 additions & 0 deletions ci/install_gcc48.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
#!/bin/bash

set -e
apt-get update
apt-get install --no-install-recommends --no-install-suggests -y \
g++-4.8

0 comments on commit 901f87a

Please sign in to comment.