[8.0.0] Fix crashing issue with WORKSPACE (#24510)

Fixes #24498 RELNOTES: None PiperOrigin-RevId: 700720057 Change-Id: Ieb656821c41d5dde64c0807f2220f8b212267880 Commit dbb9457 Co-authored-by: Googler <[email protected]>
bazelbuild · Nov 27, 2024 · d64d7c3 · d64d7c3
1 parent ea0d27a
commit d64d7c3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/RepoFileFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/RepoFileFunction.java
@@ -75,6 +75,10 @@ public SkyValue compute(SkyKey skyKey, Environment env)
       if (repoDirValue == null) {
         return null;
       }
+      if (!repoDirValue.repositoryExists()) {
+        throw new RepoFileFunctionException(
+            new IOException(repoDirValue.getErrorMsg()), Transience.PERSISTENT);
+      }
       repoRoot = repoDirValue.getPath();
     }
     RootedPath repoFilePath =

diff --git a/src/test/shell/bazel/workspace_test.sh b/src/test/shell/bazel/workspace_test.sh
@@ -66,6 +66,10 @@ EOF
   assert_contains "bye" bazel-genfiles/external/x/out
 }
 
+function test_build_non_existing_repo() {
+  bazel build @blah &>$TEST_log && fail "Failure expected" || true
+  expect_log "Repository '@@blah' is not defined"
+}
 
 function test_path_with_spaces() {
   ws="a b"