Skip to content

Commit

Permalink
fix(qsync): Include runtime_deps in the target graph
Browse files Browse the repository at this point in the history
  • Loading branch information
Tomasz Pasternak committed Nov 29, 2024
1 parent b7edbea commit d93ba16
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ public Optional<Label> sourceFileToLabel(Path sourceFile) {
@Memoized
public DepsGraph<Label> depsGraph() {
DepsGraph.Builder<Label> builder = new DepsGraph.Builder<>();
targetMap().values().stream().forEach(target -> builder.add(target.label(), target.deps()));
targetMap().values().stream().forEach(target -> builder.add(target.label(),ImmutableSet.<Label>builder().addAll(target.deps()).addAll(target.runtimeDeps()).build()));
return builder.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,7 @@ public abstract class QuerySummary {
// Runtime dependency attributes
private static final ImmutableSet<String> RUNTIME_DEP_ATTRIBUTES =
ImmutableSet.of(
"runtime_deps",
// From android_binary rules used in android_instrumentation_tests
"instruments",
// From android_instrumentation_test rules
Expand Down

0 comments on commit d93ba16

Please sign in to comment.