Skip to content
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

Remove LCOV merger dependency of cc_test without coverage #17004

Merged
merged 1 commit into from
Dec 13, 2022

Commits on Dec 13, 2022

  1. Remove LCOV merger dependency of cc_test without coverage

    When coverage is disabled, `cc_test` should not depend on the Java LCOV merger tool. This is achieved by using `configuration_field`.
    
    Also depend on coverage tools in `exec` rather than `target` configuration, matching Java rules as well as the once-per-build coverage report generator.
    
    Fixes bazelbuild#16961
    
    Closes bazelbuild#16994.
    
    PiperOrigin-RevId: 494941601
    Change-Id: Ie614de713b87bb66d869515676698f1a71cb106e
    fmeum committed Dec 13, 2022
    Configuration menu
    Copy the full SHA
    d097f82 View commit details
    Browse the repository at this point in the history