Skip to content

attempt to resolve issues with mixins in CI#1835

Merged
SteveMacenski merged 5 commits intomasterfrom
circle
Jun 30, 2020
Merged

attempt to resolve issues with mixins in CI#1835
SteveMacenski merged 5 commits intomasterfrom
circle

Conversation

@SteveMacenski
Copy link
Copy Markdown
Member

@SteveMacenski SteveMacenski commented Jun 30, 2020

Removing the release / debug elements from the mixins list for CircleCI

@SteveMacenski SteveMacenski requested a review from ruffsl June 30, 2020 20:02
@SteveMacenski
Copy link
Copy Markdown
Member Author

SteveMacenski commented Jun 30, 2020

@ruffsl any input you have on my solution would be good

ruffsl
ruffsl reviewed Jun 30, 2020
View reviewed changes
@SteveMacenski
Copy link
Copy Markdown
Member Author

SteveMacenski commented Jun 30, 2020

@ruffsl this look good?

@SteveMacenski
Copy link
Copy Markdown
Member Author

SteveMacenski commented Jun 30, 2020

Debug fails with:

Finished .info-file creation

+ lcov --capture --directory build --output-file lcov/test_coverage.info --rc lcov_branch_coverage=0

Capturing coverage data from build

Found gcov version: 9.3.0

Using intermediate gcov format

Scanning build for .gcda files ...

geninfo: WARNING: no .gcda files found in build - skipping!

Finished .info-file creation

+ lcov --add-tracefile lcov/initial_coverage.info --add-tracefile lcov/test_coverage.info --output-file lcov/full_coverage.info --rc lcov_branch_coverage=0

Combining tracefiles.

Reading tracefile lcov/initial_coverage.info

Reading tracefile lcov/test_coverage.info

lcov: ERROR: no valid records found in tracefile lcov/test_coverage.info


Exited with code exit status 255

CircleCI received exit code 255

and release

#!/bin/bash -eo pipefail
mkdir test_results/
cp -rH build/*/test_results/* test_results

cp: cannot stat 'build/*/test_results/*': No such file or directory


Exited with code exit status 1

CircleCI received exit code 1

these are a little odd

ruffsl
ruffsl reviewed Jun 30, 2020
View reviewed changes
@SteveMacenski
Copy link
Copy Markdown
Member Author

SteveMacenski commented Jun 30, 2020

Tests failed from the dump issue described in #1836 unrelated to this problem. Merging.

@SteveMacenski SteveMacenski merged commit c33cc89 into master Jun 30, 2020
@SteveMacenski SteveMacenski deleted the circle branch June 30, 2020 23:27
This was referenced Jun 30, 2020
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ruffsl ruffsl ruffsl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SteveMacenski @ruffsl