Revert "Revert "Disable hiddenapi check for building from prebuilts.""
This reverts commit bb6b813b1d
.
Reason for revert: resubmit
Change-Id: I0ded78ed1a355654fa142351b95cd51f48c2186e
This commit is contained in:
committed by
Gerrit Code Review
parent
bb6b813b1d
commit
34ab8792c5
@@ -167,7 +167,8 @@ func (c Config) RunningInsideUnitTest() bool {
|
||||
}
|
||||
|
||||
// DisableHiddenApiChecks returns true if hiddenapi checks have been disabled.
|
||||
// For 'eng' target variant hiddenapi checks are disabled by default for performance optimisation,
|
||||
// For 'eng' target variant hiddenapi checks are disabled by default for performance optimisation
|
||||
// Hiddenapi checks are also disabled when RELEASE_DEFAULT_MODULE_BUILD_FROM_SOURCE is set to false
|
||||
// but can be enabled by setting environment variable ENABLE_HIDDENAPI_FLAGS=true.
|
||||
// For other target variants hiddenapi check are enabled by default but can be disabled by
|
||||
// setting environment variable UNSAFE_DISABLE_HIDDENAPI_FLAGS=true.
|
||||
@@ -176,7 +177,8 @@ func (c Config) RunningInsideUnitTest() bool {
|
||||
func (c Config) DisableHiddenApiChecks() bool {
|
||||
return !c.IsEnvTrue("ENABLE_HIDDENAPI_FLAGS") &&
|
||||
(c.IsEnvTrue("UNSAFE_DISABLE_HIDDENAPI_FLAGS") ||
|
||||
Bool(c.productVariables.Eng))
|
||||
Bool(c.productVariables.Eng) ||
|
||||
!c.ReleaseDefaultModuleBuildFromSource())
|
||||
}
|
||||
|
||||
// MaxPageSizeSupported returns the max page size supported by the device. This
|
||||
@@ -221,6 +223,13 @@ func (c Config) ReleaseAconfigFlagDefaultPermission() string {
|
||||
return c.config.productVariables.ReleaseAconfigFlagDefaultPermission
|
||||
}
|
||||
|
||||
// The flag indicating behavior for the tree wrt building modules or using prebuilts
|
||||
// derived from RELEASE_DEFAULT_MODULE_BUILD_FROM_SOURCE
|
||||
func (c Config) ReleaseDefaultModuleBuildFromSource() bool {
|
||||
return c.config.productVariables.ReleaseDefaultModuleBuildFromSource == nil ||
|
||||
Bool(c.config.productVariables.ReleaseDefaultModuleBuildFromSource)
|
||||
}
|
||||
|
||||
// A DeviceConfig object represents the configuration for a particular device
|
||||
// being built. For now there will only be one of these, but in the future there
|
||||
// may be multiple devices being built.
|
||||
|
@@ -481,6 +481,8 @@ type ProductVariables struct {
|
||||
|
||||
ReleaseAconfigFlagDefaultPermission string `json:",omitempty"`
|
||||
|
||||
ReleaseDefaultModuleBuildFromSource *bool `json:",omitempty"`
|
||||
|
||||
KeepVndk *bool `json:",omitempty"`
|
||||
|
||||
CheckVendorSeappViolations *bool `json:",omitempty"`
|
||||
|
Reference in New Issue
Block a user