diff options
| author | 2021-04-23 13:55:49 +0100 | |
|---|---|---|
| committer | 2021-04-26 17:22:28 +0100 | |
| commit | c7d1644b0bc5281c8f1a6c19bb071527e7628b9f (patch) | |
| tree | 03816a7c9929a92df5336abe6747f7e6473cff6d /java/bootclasspath_fragment.go | |
| parent | 46e4a9e6b189ceec44157fa0e0602cdfd39078cd (diff) | |
Add coverage specific properties to bootclasspath_fragment
This allows a bootclasspath_fragment (specifically the
art-bootclasspath-fragment) to specify additional contents to be
appended when coverage is enabled.
The art-bootclasspath-fragment will use this to add jacocoagent to its
contents to ensure that it is always consistent with the configuration.
Bug: 177892522
Test: m nothing
Change-Id: I50d05fe5e0e9b8c14bdf3dfd63bba0ac97e31d48
Diffstat (limited to 'java/bootclasspath_fragment.go')
| -rw-r--r-- | java/bootclasspath_fragment.go | 27 |
1 files changed, 22 insertions, 5 deletions
diff --git a/java/bootclasspath_fragment.go b/java/bootclasspath_fragment.go index 90383af60..8bb5cb18e 100644 --- a/java/bootclasspath_fragment.go +++ b/java/bootclasspath_fragment.go @@ -69,16 +69,23 @@ func IsBootclasspathFragmentContentDepTag(tag blueprint.DependencyTag) bool { return tag == bootclasspathFragmentContentDepTag } +// Properties that can be different when coverage is enabled. +type BootclasspathFragmentCoverageAffectedProperties struct { + // The contents of this bootclasspath_fragment, could be either java_library, or java_sdk_library. + // + // The order of this list matters as it is the order that is used in the bootclasspath. + Contents []string +} + type bootclasspathFragmentProperties struct { // The name of the image this represents. // // If specified then it must be one of "art" or "boot". Image_name *string - // The contents of this bootclasspath_fragment, could be either java_library, java_sdk_library, or boot_image. - // - // The order of this list matters as it is the order that is used in the bootclasspath. - Contents []string + // Properties whose values need to differ with and without coverage. + BootclasspathFragmentCoverageAffectedProperties + Coverage BootclasspathFragmentCoverageAffectedProperties Hidden_api HiddenAPIFlagFileProperties } @@ -97,8 +104,18 @@ func bootclasspathFragmentFactory() android.Module { android.InitSdkAwareModule(m) android.InitAndroidArchModule(m, android.HostAndDeviceSupported, android.MultilibCommon) - // Initialize the contents property from the image_name. android.AddLoadHook(m, func(ctx android.LoadHookContext) { + // If code coverage has been enabled for the framework then append the properties with + // coverage specific properties. + if ctx.Config().IsEnvTrue("EMMA_INSTRUMENT_FRAMEWORK") { + err := proptools.AppendProperties(&m.properties.BootclasspathFragmentCoverageAffectedProperties, &m.properties.Coverage, nil) + if err != nil { + ctx.PropertyErrorf("coverage", "error trying to append coverage specific properties: %s", err) + return + } + } + + // Initialize the contents property from the image_name. bootclasspathFragmentInitContentsFromImage(ctx, m) }) return m |