Merge "Revert "Exclude prebuilts/clang/host/linux-x86/clang-dev/BUILD from the symlink forest.""

This commit is contained in:
Yabin Cui
2023-06-28 16:26:54 +00:00
committed by Gerrit Code Review

View File

@@ -725,18 +725,6 @@ func excludedFromSymlinkForest(ctx *android.Context, verbose bool) []string {
// FIXME: 'frameworks/compile/slang' has a filegroup error due to an escaping issue
"frameworks/compile/slang",
// FIXME(b/260809113): 'prebuilts/clang/host/linux-x86/clang-dev' is a tool-generated symlink
// directory that contains a BUILD file. The bazel files finder code doesn't traverse into symlink dirs,
// and hence is not aware of this BUILD file and exclude it accordingly during symlink forest generation
// when checking against keepExistingBuildFiles allowlist.
//
// This is necessary because globs in //prebuilts/clang/host/linux-x86/BUILD
// currently assume no subpackages (keepExistingBuildFile is not recursive for that directory).
//
// This is a bandaid until we the symlink forest logic can intelligently exclude BUILD files found in
// source symlink dirs according to the keepExistingBuildFile allowlist.
"prebuilts/clang/host/linux-x86/clang-dev",
)
return excluded
}