From df0b83966161d767714909dff032648c0d83d4c8 Mon Sep 17 00:00:00 2001 From: Yu Liu Date: Wed, 12 Feb 2025 18:27:03 +0000 Subject: Convert CheckMinSdkVersion to use providers. Next step is to convert it to use ModuleProxy once IsDepInSameApex is converted to use providers. Bug: 377723687 Test: Unit tests and compare the ninja and mk files generated. Change-Id: I20cb052666fa95eb964e63e3302ac892d3324183 --- java/base.go | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'java/base.go') diff --git a/java/base.go b/java/base.go index 730554896..21ad73f84 100644 --- a/java/base.go +++ b/java/base.go @@ -2267,20 +2267,16 @@ func (m JavaDepInSameApexChecker) OutgoingDepIsInSameApex(tag blueprint.Dependen } // Implements android.ApexModule -func (j *Module) ShouldSupportSdkVersion(ctx android.BaseModuleContext, sdkVersion android.ApiLevel) error { +func (j *Module) MinSdkVersionSupported(ctx android.BaseModuleContext) android.ApiLevel { sdkVersionSpec := j.SdkVersion(ctx) minSdkVersion := j.MinSdkVersion(ctx) - if !minSdkVersion.Specified() { - return fmt.Errorf("min_sdk_version is not specified") - } + // If the module is compiling against core (via sdk_version), skip comparison check. if sdkVersionSpec.Kind == android.SdkCore { - return nil + return android.MinApiLevel } - if minSdkVersion.GreaterThan(sdkVersion) { - return fmt.Errorf("newer SDK(%v)", minSdkVersion) - } - return nil + + return minSdkVersion } func (j *Module) Stem() string { -- cgit v1.2.3-59-g8ed1b