Reland "mark platform un-availability""
This reverts commit6e1fa4452b
. Reason for revert: relanding a reverted changed Bug: 153073816 Test: m Exempt-From-Owner-Approval: cherry-pick from AOSP Merged-In: I3735866776cf6eb0320c9fb8c6be8ac0c48baa1c (cherry picked from commit403c255b6f
) Change-Id: I3735866776cf6eb0320c9fb8c6be8ac0c48baa1c
This commit is contained in:
parent
f068a60e8a
commit
7d40b59c4d
3 changed files with 47 additions and 0 deletions
|
@ -811,6 +811,16 @@ ifdef LOCAL_PICKUP_FILES
|
|||
ALL_MODULES.$(my_register_name).PICKUP_FILES := \
|
||||
$(ALL_MODULES.$(my_register_name).PICKUP_FILES) $(LOCAL_PICKUP_FILES)
|
||||
endif
|
||||
# Record the platform availability of this module. Note that the availability is not
|
||||
# meaningful for non-installable modules (e.g., static libs) or host modules.
|
||||
# We only care about modules that are installable to the device.
|
||||
ifeq (true,$(LOCAL_NOT_AVAILABLE_FOR_PLATFORM))
|
||||
ifneq (true,$(LOCAL_UNINSTALLABLE_MODULE))
|
||||
ifndef LOCAL_IS_HOST_MODULE
|
||||
ALL_MODULES.$(my_register_name).NOT_AVAILABLE_FOR_PLATFORM := true
|
||||
endif
|
||||
endif
|
||||
endif
|
||||
|
||||
my_required_modules := $(LOCAL_REQUIRED_MODULES) \
|
||||
$(LOCAL_REQUIRED_MODULES_$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH))
|
||||
|
|
|
@ -192,6 +192,7 @@ LOCAL_NO_PIC:=
|
|||
LOCAL_NOSANITIZE:=
|
||||
LOCAL_NO_STANDARD_LIBRARIES:=
|
||||
LOCAL_NO_STATIC_ANALYZER:=
|
||||
LOCAL_NOT_AVAILABLE_FOR_PLATFORM:=
|
||||
LOCAL_NOTICE_FILE:=
|
||||
LOCAL_ODM_MODULE:=
|
||||
LOCAL_OEM_MODULE:=
|
||||
|
|
36
core/tasks/platform_availability_check.mk
Normal file
36
core/tasks/platform_availability_check.mk
Normal file
|
@ -0,0 +1,36 @@
|
|||
#
|
||||
# Copyright (C) 2020 The Android Open Source Project
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||
# you may not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
# See the License for the specific language governing permissions and
|
||||
# limitations under the License.
|
||||
#
|
||||
|
||||
# Check whether there is any module that isn't available for platform
|
||||
# is installed to the platform.
|
||||
|
||||
# Filter FAKE and NON_INSTALLABLE modules out and then collect those are not
|
||||
# available for platform
|
||||
_modules_not_available_for_platform := \
|
||||
$(strip $(foreach m,$(product_MODULES),\
|
||||
$(if $(filter-out FAKE,$(ALL_MODULES.$(m).CLASS)),\
|
||||
$(if $(ALL_MODULES.$(m).INSTALLED),\
|
||||
$(if $(filter true,$(ALL_MODULES.$(m).NOT_AVAILABLE_FOR_PLATFORM)),\
|
||||
$(m))))))
|
||||
|
||||
_violators_with_path := $(foreach m,$(sort $(_modules_not_available_for_platform)),\
|
||||
$(m):$(word 1,$(ALL_MODULES.$(m).PATH))\
|
||||
)
|
||||
|
||||
$(call maybe-print-list-and-error,$(_violators_with_path),\
|
||||
Following modules are requested to be installed. But are not available \
|
||||
for platform because they do not have "//apex_available:platform" or \
|
||||
they depend on other modules that are not available for platform)
|
Loading…
Reference in a new issue