New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

travis: remove clang-analyze #2917

Merged
merged 1 commit into from Sep 9, 2017

Conversation

Projects
None yet
2 participants
@stapelberg
Member

stapelberg commented Sep 9, 2017

• The output currently contains a large number of false-positives and — AFAICT —
no actual issues.

• Upstream shows little interest in addressing the long-standing issues with the
TAILQ macros, so the false-positive situation probably won’t change soon:
https://bugs.llvm.org/show_bug.cgi?id=18222

Currently, we largely spend travis CPU cycles on this, for no additional value.

travis: remove clang-analyze
• The output currently contains a large number of false-positives and — AFAICT —
  no actual issues.

• Upstream shows little interest in addressing the long-standing issues with the
  TAILQ macros, so the false-positive situation probably won’t change soon:
  https://bugs.llvm.org/show_bug.cgi?id=18222

Currently, we largely spend travis CPU cycles on this, for no additional value.

@stapelberg stapelberg requested a review from Airblader Sep 9, 2017

@stapelberg stapelberg merged commit 1a2ce3e into i3:next Sep 9, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@stapelberg stapelberg deleted the stapelberg:remove-clang-analyze branch Sep 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment