diff options
author | 2020-11-23 00:22:30 -0500 | |
---|---|---|
committer | 2020-11-23 00:29:18 -0500 | |
commit | cda22c9bb92a5fc4fa5063103afde34805161c7a (patch) | |
tree | f7304d81543a6c3319ce32ca71639241dc718ddd /android/module.go | |
parent | b26070efef296bdd26c9d2ddadd0b4083587c484 (diff) |
Refactor 'in_make' to mean Kati is not skipped.
In Nougat and before, Make wrote a marker file to indicate that
soong_build was invoked from Make to change certain behaviors of Soong
at build time.
https://cs.android.com/android/platform/superproject/+/android-7.1.2_r36:build/core/soong.mk;l=70-73;drc=ae18638b0406ad107b0882a02a13cdd8b92f2a4e
Things have changed, and now soong_build is invoked from soong_ui, which
supports a --skip-make configuration flag:
https://cs.android.com/android/platform/superproject/+/master:build/soong/ui/build/build.go;l=31-33;drc=680387bf1d3ce7cbc77f535be7c42cec411b1687
Thus, the various remnants of 'EmbeddedInMake' and 'inMake'
configuration are misleading, since soong_build is no longer invoked
from Make. This CL refactors all instances to actually mean that
Kati is enabled (not skipped with --skip-make), and will run after
soong_build finishes, so Kati-specific behavior like the AndroidMk
singleton should run.
Test: TH presubmit
Change-Id: I576ab8e54f99f5c8ddf9feaf9a828019b279e266
Diffstat (limited to 'android/module.go')
-rw-r--r-- | android/module.go | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/android/module.go b/android/module.go index 6b659d2cd..70c649787 100644 --- a/android/module.go +++ b/android/module.go @@ -1440,7 +1440,7 @@ func (m *ModuleBase) generateModuleTarget(ctx ModuleContext) { if len(deps) > 0 { suffix := "" - if ctx.Config().EmbeddedInMake() { + if ctx.Config().KatiEnabled() { suffix = "-soong" } @@ -2320,7 +2320,7 @@ func (m *moduleContext) skipInstall(fullInstallPath InstallPath) bool { } if m.Device() { - if m.Config().EmbeddedInMake() && !m.InstallBypassMake() { + if m.Config().KatiEnabled() && !m.InstallBypassMake() { return true } @@ -2373,7 +2373,7 @@ func (m *moduleContext) installFile(installPath InstallPath, name string, srcPat Input: srcPath, Implicits: implicitDeps, OrderOnly: orderOnlyDeps, - Default: !m.Config().EmbeddedInMake(), + Default: !m.Config().KatiEnabled(), }) m.installFiles = append(m.installFiles, fullInstallPath) @@ -2405,7 +2405,7 @@ func (m *moduleContext) InstallSymlink(installPath InstallPath, name string, src Description: "install symlink " + fullInstallPath.Base(), Output: fullInstallPath, Input: srcPath, - Default: !m.Config().EmbeddedInMake(), + Default: !m.Config().KatiEnabled(), Args: map[string]string{ "fromPath": relPath, }, @@ -2436,7 +2436,7 @@ func (m *moduleContext) InstallAbsoluteSymlink(installPath InstallPath, name str Rule: Symlink, Description: "install symlink " + fullInstallPath.Base() + " -> " + absPath, Output: fullInstallPath, - Default: !m.Config().EmbeddedInMake(), + Default: !m.Config().KatiEnabled(), Args: map[string]string{ "fromPath": absPath, }, @@ -2665,7 +2665,7 @@ func (c *buildTargetSingleton) GenerateBuildActions(ctx SingletonContext) { }) suffix := "" - if ctx.Config().EmbeddedInMake() { + if ctx.Config().KatiEnabled() { suffix = "-soong" } @@ -2673,7 +2673,7 @@ func (c *buildTargetSingleton) GenerateBuildActions(ctx SingletonContext) { ctx.Phony("checkbuild"+suffix, checkbuildDeps...) // Make will generate the MODULES-IN-* targets - if ctx.Config().EmbeddedInMake() { + if ctx.Config().KatiEnabled() { return } |