Skip to content

Commit 9a547e2

Browse files
authored
Merge pull request #24 from meroton/fix-apply-get-containing-digests
Fix panic for unimplemented ApplyGetContainingDigests
2 parents 759419b + 20c222c commit 9a547e2

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

pkg/filesystem/virtual/bazel_output_service_directory.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,9 @@ func (d *BazelOutputServiceDirectory) filterMissingChildren(ctx context.Context,
184184
// this file or directory depends.
185185
var p virtual.ApplyGetContainingDigests
186186
if !node.GetNode().VirtualApply(&p) {
187-
panic("output path contains nodes that don't support ApplyGetContainingDigests")
187+
// This node type does not depend on any digests, e.g. a file in the
188+
// file pool.
189+
return true
188190
}
189191
if p.Err != nil {
190192
// Can't compute the set of digests underneath

0 commit comments

Comments
 (0)