Merge "Compare tidy_disabled_srcs list by string values" am: ec76c2f072
am: 9f60d66aa9
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1987286 Change-Id: Ie9873f6bb3b0464ee80b6db0c50eb07b38df54ae
This commit is contained in:
@@ -464,12 +464,12 @@ func transformSourceToObj(ctx ModuleContext, subdir string, srcFiles, noTidySrcs
|
||||
// Source files are one-to-one with tidy, coverage, or kythe files, if enabled.
|
||||
objFiles := make(android.Paths, len(srcFiles))
|
||||
var tidyFiles android.Paths
|
||||
noTidySrcsMap := make(map[android.Path]bool)
|
||||
noTidySrcsMap := make(map[string]bool)
|
||||
var tidyVars string
|
||||
if flags.tidy {
|
||||
tidyFiles = make(android.Paths, 0, len(srcFiles))
|
||||
for _, path := range noTidySrcs {
|
||||
noTidySrcsMap[path] = true
|
||||
noTidySrcsMap[path.String()] = true
|
||||
}
|
||||
tidyTimeout := ctx.Config().Getenv("TIDY_TIMEOUT")
|
||||
if len(tidyTimeout) > 0 {
|
||||
@@ -675,7 +675,7 @@ func transformSourceToObj(ctx ModuleContext, subdir string, srcFiles, noTidySrcs
|
||||
}
|
||||
|
||||
// Even with tidy, some src file could be skipped by noTidySrcsMap.
|
||||
if tidy && !noTidySrcsMap[srcFile] {
|
||||
if tidy && !noTidySrcsMap[srcFile.String()] {
|
||||
tidyFile := android.ObjPathWithExt(ctx, subdir, srcFile, "tidy")
|
||||
tidyDepFile := android.ObjPathWithExt(ctx, subdir, srcFile, "tidy.dep")
|
||||
tidyFiles = append(tidyFiles, tidyFile)
|
||||
|
Reference in New Issue
Block a user