Skip to content

Use gcc-4.8 for GHA legacy tests#207

Merged
reyang merged 4 commits into
open-telemetry:masterfrom
pyohannes:use-gcc-4.8
Jul 23, 2020
Merged

Use gcc-4.8 for GHA legacy tests#207
reyang merged 4 commits into
open-telemetry:masterfrom
pyohannes:use-gcc-4.8

Conversation

@pyohannes
Copy link
Copy Markdown
Contributor

Setting CC=/usr/bin/gcc-4.8 was omitted for our GHA legacy test, resulting in the use of the default Ubuntu 18.04 gcc version.

This PR fixes this and makes our legacy tests actually using gcc-4.8 again (and pass). Merging this will fix other PR builder failures.

@pyohannes pyohannes requested a review from a team July 23, 2020 15:40
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 23, 2020

Codecov Report

Merging #207 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #207   +/-   ##
=======================================
  Coverage   95.69%   95.69%           
=======================================
  Files          96       96           
  Lines        2878     2878           
=======================================
  Hits         2754     2754           
  Misses        124      124           
Impacted Files Coverage Δ
exporters/ostream/src/span_exporter.cc 100.00% <100.00%> (ø)

Copy link
Copy Markdown
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

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

LGTM.

@reyang reyang merged commit 94cdc70 into open-telemetry:master Jul 23, 2020
GerHobbelt pushed a commit to GerHobbelt/opentelemetry-cpp that referenced this pull request Jun 17, 2025
[SEMANTIC CONVENTIONS] Upgrade semantic conventions to 1.31.0 (open-telemetry#3297)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants