make work again

src/parse/asp/builtins.go

@@ -402,11 +402,6 @@ func subincludeTarget(s *scope, l core.BuildLabel) *core.BuildTarget {
 				s.Error("Failed to activate subinclude target: %v", err)
 			}
 		}
-	} else if isLocal && s.pkg == nil {
-		// The subinclude is local to this package but we didn't hit the part above.
-		// This is almost certainly going to cause problems sooner or later; we will probably
-		// lock up having not activated it.
-		s.Error("Cannot make package-local subincludes from within another subinclude")
 	}
 
 	t := s.WaitForSubincludedTarget(l, pkgLabel)

test/double_local_subinclude/BUILD

@@ -2,7 +2,6 @@ subinclude("//test/build_defs")
 
 please_repo_e2e_test(
     name = "double_local_subinclude_test",
-    plz_command = "plz build -p -v 2 //...",
-    expected_failure = True,
+    plz_command = "for i in `seq 1 20`; do plz build -p -v 2 //...; done",
     repo = "test_repo",
 )