Skip to content

ci: Correct the status code check so Coverity doesn't force-fail Travis#4764

Merged
ethomson merged 1 commit into
libgit2:masterfrom
tiennou:ci/coverity-only
Aug 14, 2018
Merged

ci: Correct the status code check so Coverity doesn't force-fail Travis#4764
ethomson merged 1 commit into
libgit2:masterfrom
tiennou:ci/coverity-only

Conversation

@tiennou
Copy link
Copy Markdown
Contributor

@tiennou tiennou commented Aug 14, 2018

As per https://travis-ci.org/libgit2/libgit2/jobs/415921269, which is the cron job.

Emitted 525 C/C++ compilation units (100%) successfully

525 C/C++ compilation units (100%) are ready for analysis
The cov-build utility completed successfully.
Build successfully submitted.
Received error code 200 from Coverity

travis_time:end:14cf6373:start=1534254309066933889,finish=1534254728190974302,duration=419124040413
The command "if [ -n "$COVERITY" ]; then ../ci/coverity.sh; fi" exited with 1.
travis_time:start:01ed61d4
$ if [ -z "$COVERITY" ]; then ../ci/build.sh && ../ci/test.sh; fi

travis_time:end:01ed61d4:start=1534254728197560961,finish=1534254728202711214,duration=5150253
The command "if [ -z "$COVERITY" ]; then ../ci/build.sh && ../ci/test.sh; fi" exited with 0.

Done. Your build exited with 1.

Otherwise you get something like

Emitted 525 C/C++ compilation units (100%) successfully

525 C/C++ compilation units (100%) are ready for analysis
The cov-build utility completed successfully.
Build successfully submitted.
Received error code 200 from Coverity

travis_time:end:14cf6373:start=1534254309066933889,finish=1534254728190974302,duration=419124040413
�The command "if [ -n "$COVERITY" ]; then ../ci/coverity.sh; fi" exited with 1.�
travis_time:start:01ed61d4
�$ if [ -z "$COVERITY" ]; then ../ci/build.sh && ../ci/test.sh; fi

travis_time:end:01ed61d4:start=1534254728197560961,finish=1534254728202711214,duration=5150253
��The command "if [ -z "$COVERITY" ]; then ../ci/build.sh && ../ci/test.sh; fi" exited with 0.�

Done. Your build exited with 1.
@ethomson
Copy link
Copy Markdown
Member

Oops, thanks.

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.

3 participants