Browse Source

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

This reverts commit bcfadce598a3603b238a06c6e28559d669354076.
Because now test_compiler.py copies clang-dev instead of creating a
symlink.

Bug: 288327832
Test: run test_compiler.py
Change-Id: Ib676a41c152046d710d73fdaac678d8a98de5af2
Yabin Cui 10 months ago
parent
commit
f8622b732a
1 changed files with 0 additions and 12 deletions
  1. 0 12
      cmd/soong_build/main.go

+ 0 - 12
cmd/soong_build/main.go

@@ -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
 }