Skip to content

Conversation

@eramongodb
Copy link
Contributor

Addresses CXX-2338 by avoiding the need to run yum install parallel in the run-clang-tidy.sh script. DEVPROD-23486 is still in-progress, but verified via patch build that parallel has been deployed to the rhel96 distro (current target for the rhel9-latest distro alias).

As a drive-by improvement, formats the output of the list of files being analyzed to contain a - prefix:

# Before
Scanning the following files:
src/bsoncxx/include/bsoncxx/docs/top.hpp
src/bsoncxx/include/bsoncxx/docs/topics.hpp
src/bsoncxx/include/bsoncxx/docs/v1.hpp
...

# After
Scanning the following files:
 - src/bsoncxx/include/bsoncxx/docs/top.hpp
 - src/bsoncxx/include/bsoncxx/docs/topics.hpp
 - src/bsoncxx/include/bsoncxx/docs/v1.hpp
...

@eramongodb eramongodb requested a review from kevinAlbs November 11, 2025 16:37
@eramongodb eramongodb self-assigned this Nov 11, 2025
@eramongodb eramongodb requested a review from a team as a code owner November 11, 2025 16:37
@eramongodb
Copy link
Contributor Author

eramongodb commented Nov 11, 2025

verified via patch build that parallel has been deployed

Sorry, looks like I spoke too soon: the command check revealed a workaround for not-yet-audited clang-tidy configuration was also silencing "command not found" errors and diagnostic output. parallel doesn't appear to have been deployed yet. It looks like this workaround was added by #1049 with a TODO comment as part of EVG refactors, but did not have a proper followup since.

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.

2 participants