History log of /openbmc/openbmc-build-scripts/tools/config-clang-tidy (Results 1 – 4 of 4)
Revision Date Author Comments
# aeb73bc5 30-May-2025 Patrick Williams <patrick@stwcx.xyz>

config-clang-tidy: add format subcommand

The 'merge' subcommand can be used for formatting, if the `--reference`
is the same clang-tidy file as the input, but this is not convenient
for users. Add

config-clang-tidy: add format subcommand

The 'merge' subcommand can be used for formatting, if the `--reference`
is the same clang-tidy file as the input, but this is not convenient
for users. Add a 'format' subcommand that does the expected.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: If51ded84e85f6f285a6b5d1df0764ff20b1b2064

show more ...


# b8ce3818 30-May-2025 Patrick Williams <patrick@stwcx.xyz>

config-clang-tidy: better handle wildcards

Some repositories had category wildcards; handle those better:
- Remove category disables since we already disable global.
- Allow category enables t

config-clang-tidy: better handle wildcards

Some repositories had category wildcards; handle those better:
- Remove category disables since we already disable global.
- Allow category enables to act as a single check.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ibe320717331c0627439c54846465786b9316f186

show more ...


# 23362867 30-May-2025 Patrick Williams <patrick@stwcx.xyz>

config-clang-tidy: strip comments better

Some repositories have a Checks line with embedded comments. This
is unsupported by the python YAML parser and results in strings where
comment lines can no

config-clang-tidy: strip comments better

Some repositories have a Checks line with embedded comments. This
is unsupported by the python YAML parser and results in strings where
comment lines can no longer be identified because it becomes one big
paragraph. Strip these out prior to YAML processing.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I455ea4584d71d1ff94710ad8a5c02d5eabb05b82

show more ...


# b516729d 23-May-2025 Patrick Williams <patrick@stwcx.xyz>

config-clang-tidy: introduce tool

There have been a few efforts to enable clang-tidy across the
organization, many of which have either been partially applied or
inconsistently applied. Part of the

config-clang-tidy: introduce tool

There have been a few efforts to enable clang-tidy across the
organization, many of which have either been partially applied or
inconsistently applied. Part of the issue has been that enabling
options is done by hand by humans.

Introduce a tool that allows us to more automatically manipulate
the clang-tidy configs. There are currently 3 sub-commands:
- merge
- enable
- disable

The merge command can be used to merge an existing repository
clang-tidy config with a reference, such as the one in docs:
```
tools/config-clang-tidy --repo ../phosphor-virtual-sensor \
merge --reference ../docs/style/cpp/.clang-tidy
```

Similarly, the enable / disable commands can be used to enable or
disable an individual check.
```
./tools/config-clang-tidy --repo ../phosphor-virtual-sensor \
enable modernize-use-nullptr
```

This tool will enable us to create scripts jobs that:
- Synchronize the clang-tidy config across a large set of
repositories.
- Enable a single check across a large set of repositories and
examine the fallout.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib338f7dcbc7f1049122399e0b616f77ec5eb7dc2

show more ...