example output nothing

[honeway]

The current master (9e9bfc5) code doesn't work.

❯ bazel build //example/... --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report


INFO: Analyzed 5 targets (3 packages loaded, 31 targets configured).
INFO: Found 5 targets...
INFO: Elapsed time: 0.121s, Critical Path: 0.04s
INFO: 7 processes: 2 internal, 5 processwrapper-sandbox.
INFO: Build completed successfully, 7 total actions

If checked into (0b86881), it works fine.

❯ bazel build //example/... --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report


INFO: Analyzed 4 targets (1 packages loaded, 8 targets configured).
ERROR: /home/xxx/Workspace/bazel_clang_tidy/example/BUILD:1:11: Run clang-tidy on example/lib.cpp failed: (Exit 1): clang_tidy failed: error executing ClangTidy command (from target //example:lib) bazel-out/k8-fastbuild/bin/clang_tidy/clang_tidy clang-tidy bazel-out/k8-fastbuild/bin/example/bazel_clang_tidy_example/lib.cpp.lib.clang-tidy.yaml .clang-tidy --export-fixes ... (remaining 19 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
8987 warnings generated.
/home/xxx/.cache/bazel/_bazel_xxx/90ed30fac4a9b2ab2f943ec475e6d263/sandbox/processwrapper-sandbox/29/execroot/_main/example/lib.cpp:4:43: error: the parameter 'name' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
    4 | std::string lib_get_greet_for(std::string name)
      |                                           ^
      |                               const      &
Suppressed 8986 warnings (8986 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
1 warning treated as error
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.681s, Critical Path: 0.57s
INFO: 7 processes: 4 internal, 3 processwrapper-sandbox.
ERROR: Build did NOT complete successfully

BTW, Bazel 6.5 version doesn't work.

# Bazel 6 output errorError: 'CompilationContext' value has no field or method 'external_includes'
Error: 'CompilationContext' value has no field or method 'external_includes'

[Wheest]

I also have this issue with the current head (`9e9bfc5).

The aspect build just says "everything is fine".

The older commit does give the errors I'm expecting.

I did a binary search over the commits:

Which makes me think the issue was introduced in 43618e3

For 07bc38524e61d9501d772726d6e27cb980db42c7, I get a warning as error output:

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
1 warning generated.
my_file.hpp:1:9: error: #pragma once in main file [clang-diagnostic-pragma-once-outside-header,-warnings-as-errors]
    1 | #pragma once
      |         ^
1 warning treated as error

For 43618e3748af0ee675eced41b32f63293f590383 I get no output.

For 0b86881921e89fc8848d7747a645541ca83b171f I get more of the output I'm looking for:

another_one_of_my_files.hpp:8:15: error: no header providing "std::size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
    3 | #include <utility>
    4 | 
    5 | namespace detail {
    6 | 
    7 | // Create a compile-time constant which holds the value of template parameter N
    8 | template<std::size_t N>

Suppressed 2960 warnings (2960 in non-user code).

[erenon]

Thanks for reporting. Please check if the issue is still present with the latest commit.

[honeway]

Thanks for reporting. Please check if the issue is still present with the latest commit.

The problem still exists. 267b0bb

[mattlangford]

I was running into similar issues, this seemed to help