StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Use only single dash for tidy flags" am: def9bf2c1c am: df8fe32206

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1972547

Change-Id: I12f4c1acbfc61dd1d0f70cf4c9871a028182f3a2
This commit is contained in:
Chih-hung Hsieh
2022-02-08 22:47:59 +00:00
committed by Automerger Merge Worker
parent 0959ae13e7 df8fe32206
commit 31989351ac
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cc/tidy.go
View File

@@ -102,6 +102,12 @@ func (tidy *tidyFeature) flags(ctx ModuleContext, flags Flags) Flags {
} }
flags.TidyFlags = append(flags.TidyFlags, headerFilter) flags.TidyFlags = append(flags.TidyFlags, headerFilter)
} }
// Work around RBE bug in parsing clang-tidy flags, replace "--flag" with "-flag".
// Some C/C++ modules added local tidy flags like --header-filter= and --extra-arg-before=.
doubleDash := regexp.MustCompile("^('?)--(.*)$")
for i, s := range flags.TidyFlags {
flags.TidyFlags[i] = doubleDash.ReplaceAllString(s, "$1-$2")
}
// If clang-tidy is not enabled globally, add the -quiet flag. // If clang-tidy is not enabled globally, add the -quiet flag.
if !ctx.Config().ClangTidy() { if !ctx.Config().ClangTidy() {