From 68fb7cb68da47e36d80285e69b1d597ea7d6dbea Mon Sep 17 00:00:00 2001 From: Spandan Das Date: Mon, 3 Feb 2025 23:49:27 +0000 Subject: Fixes for missing rules in m --soong-only This CL 1. Disables filesystem diff test if there is no kati invocation, since there will not be a file_list.txt to do the comparison 2. Disables installation for goBinary if it is in an unexported namespace. Test: m checkbuild --soong-build (top of stack) Bug: 394096939 Change-Id: I5057c2cc431f5e8db12816c3dacf69b3159e18d5 --- golang/golang.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'golang') diff --git a/golang/golang.go b/golang/golang.go index 3422f8bdf..9e0744aaf 100644 --- a/golang/golang.go +++ b/golang/golang.go @@ -101,7 +101,7 @@ func (g *GoBinary) GenerateAndroidBuildActions(ctx android.ModuleContext) { // Modules in an unexported namespace have no install rule, only add modules in the exported namespaces // to the blueprint_tools phony rules. - if (!ctx.Config().KatiEnabled() || g.ExportedToMake()) && !usedByBootstrap(ctx.ModuleName()) { + if g.ExportedToMake() && !usedByBootstrap(ctx.ModuleName()) { // Don't add the installed file of bootstrap tools to the deps of `blueprint_tools`. // The install command line will differ from what was used during bootstrap, // which will cause ninja to rebuild the module on the next run, -- cgit v1.2.3-59-g8ed1b