Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 05fe5012 authored by Chaitanya Cheemala (xWF)'s avatar Chaitanya Cheemala (xWF) Committed by Gerrit Code Review
Browse files

Revert^3 "Reland linking shared_libs to their modules" 

This reverts commit 47d30424.

Reason for revert: Likely culprit for b/382182759  - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Change-Id: I6daf2d6f4130c237009a106053d5bd3450fdd56e
parent 47d30424

core/tasks/general-tests-shared-libs.mk

0 → 100644
+52 −0
Original line number Diff line number Diff line 
# Copyright (C) 2024 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.



.PHONY: general-tests-shared-libs



intermediates_dir := $(call intermediates-dir-for,PACKAGING,general-tests-shared-libs)



general_tests_shared_libs_zip := $(PRODUCT_OUT)/general-tests_host-shared-libs.zip



# Filter shared entries between general-tests and device-tests's HOST_SHARED_LIBRARY.FILES,

# to avoid warning about overriding commands.

my_host_shared_lib_for_general_tests := \

  $(foreach m,$(filter $(COMPATIBILITY.device-tests.HOST_SHARED_LIBRARY.FILES),\

	   $(COMPATIBILITY.general-tests.HOST_SHARED_LIBRARY.FILES)),$(call word-colon,2,$(m)))

my_general_tests_shared_lib_files := \

  $(filter-out $(COMPATIBILITY.device-tests.HOST_SHARED_LIBRARY.FILES),\

	 $(COMPATIBILITY.general-tests.HOST_SHARED_LIBRARY.FILES))



my_host_shared_lib_for_general_tests += $(call copy-many-files,$(my_general_tests_shared_lib_files))



$(general_tests_shared_libs_zip) : PRIVATE_INTERMEDIATES_DIR := $(intermediates_dir)

$(general_tests_shared_libs_zip) : PRIVATE_HOST_SHARED_LIBS := $(my_host_shared_lib_for_general_tests)

$(general_tests_shared_libs_zip) : PRIVATE_general_host_shared_libs_zip := $(general_tests_shared_libs_zip)

$(general_tests_shared_libs_zip) : $(my_host_shared_lib_for_general_tests) $(SOONG_ZIP)

	rm -rf $(PRIVATE_INTERMEDIATES_DIR)

	mkdir -p $(PRIVATE_INTERMEDIATES_DIR) $(PRIVATE_INTERMEDIATES_DIR)/tools

	$(hide) for shared_lib in $(PRIVATE_HOST_SHARED_LIBS); do \

	  echo $$shared_lib >> $(PRIVATE_INTERMEDIATES_DIR)/shared-libs.list; \

	done

	grep $(HOST_OUT_TESTCASES) $(PRIVATE_INTERMEDIATES_DIR)/shared-libs.list > $(PRIVATE_INTERMEDIATES_DIR)/host-shared-libs.list || true

	$(SOONG_ZIP) -d -o $(PRIVATE_general_host_shared_libs_zip) \

	  -P host -C $(HOST_OUT) -l $(PRIVATE_INTERMEDIATES_DIR)/host-shared-libs.list



general-tests-shared-libs: $(general_tests_shared_libs_zip)

$(call dist-for-goals, general-tests-shared-libs, $(general_tests_shared_libs_zip))



$(call declare-1p-container,$(general_tests_shared_libs_zip),)

$(call declare-container-license-deps,$(general_tests_shared_libs_zip),$(my_host_shared_lib_for_general_tests),$(PRODUCT_OUT)/:/)



intermediates_dir :=

general_tests_shared_libs_zip :=

core/tasks/general-tests.mk

+3 −51
Original line number Diff line number Diff line
@@ -27,61 +27,19 @@ general_tests_list_zip := $(PRODUCT_OUT)/general-tests_list.zip 
# Create an artifact to include all test config files in general-tests.

general_tests_configs_zip := $(PRODUCT_OUT)/general-tests_configs.zip



# Filter shared entries between general-tests and device-tests's HOST_SHARED_LIBRARY.FILES,

# to avoid warning about overriding commands.

my_host_shared_lib_for_general_tests := \

  $(foreach m,$(filter $(COMPATIBILITY.device-tests.HOST_SHARED_LIBRARY.FILES),\

	   $(COMPATIBILITY.general-tests.HOST_SHARED_LIBRARY.FILES)),$(call word-colon,2,$(m)))

my_general_tests_shared_lib_files := \

  $(filter-out $(COMPATIBILITY.device-tests.HOST_SHARED_LIBRARY.FILES),\

	 $(COMPATIBILITY.general-tests.HOST_SHARED_LIBRARY.FILES))



my_host_shared_lib_for_general_tests += $(call copy-many-files,$(my_general_tests_shared_lib_files))



my_host_shared_lib_symlinks := \

    $(filter $(COMPATIBILITY.host-unit-tests.SYMLINKS),\

	$(COMPATIBILITY.general-tests.SYMLINKS))



my_general_tests_symlinks := \

    $(filter-out $(COMPATIBILITY.camera-hal-tests.SYMLINKS),\

    $(filter-out $(COMPATIBILITY.host-unit-tests.SYMLINKS),\

	 $(COMPATIBILITY.general-tests.SYMLINKS)))



my_symlinks_for_general_tests := $(foreach f,$(my_general_tests_symlinks),\

	$(strip $(eval _cmf_tuple := $(subst :, ,$(f))) \

	$(eval _cmf_dep := $(word 1,$(_cmf_tuple))) \

	$(eval _cmf_src := $(word 2,$(_cmf_tuple))) \

	$(eval _cmf_dest := $(word 3,$(_cmf_tuple))) \

	$(call symlink-file,$(_cmf_dep),$(_cmf_src),$(_cmf_dest)) \

	$(_cmf_dest)))



# In this one directly take the overlap into the zip since we can't rewrite rules

my_symlinks_for_general_tests += $(foreach f,$(my_host_shared_lib_symlinks),\

        $(strip $(eval _cmf_tuple := $(subst :, ,$(f))) \

        $(eval _cmf_dep := $(word 1,$(_cmf_tuple))) \

        $(eval _cmf_src := $(word 2,$(_cmf_tuple))) \

        $(eval _cmf_dest := $(word 3,$(_cmf_tuple))) \

        $(_cmf_dest)))

general_tests_shared_libs_zip := $(PRODUCT_OUT)/general-tests_host-shared-libs.zip



$(general_tests_zip) : $(general_tests_shared_libs_zip)

$(general_tests_zip) : PRIVATE_general_tests_list_zip := $(general_tests_list_zip)

$(general_tests_zip) : .KATI_IMPLICIT_OUTPUTS := $(general_tests_list_zip) $(general_tests_configs_zip)

$(general_tests_zip) : PRIVATE_TOOLS := $(general_tests_tools)

$(general_tests_zip) : PRIVATE_INTERMEDIATES_DIR := $(intermediates_dir)

$(general_tests_zip) : PRIVATE_HOST_SHARED_LIBS := $(my_host_shared_lib_for_general_tests)

$(general_tests_zip) : PRIVATE_SYMLINKS := $(my_symlinks_for_general_tests)

$(general_tests_zip) : PRIVATE_general_tests_configs_zip := $(general_tests_configs_zip)

$(general_tests_zip) : $(COMPATIBILITY.general-tests.FILES) $(my_host_shared_lib_for_general_tests) $(COMPATIBILITY.general-tests.SOONG_INSTALLED_COMPATIBILITY_SUPPORT_FILES) $(general_tests_tools) $(my_symlinks_for_general_tests) $(SOONG_ZIP)

$(general_tests_zip) : $(COMPATIBILITY.general-tests.FILES) $(COMPATIBILITY.general-tests.SOONG_INSTALLED_COMPATIBILITY_SUPPORT_FILES) $(general_tests_tools) $(SOONG_ZIP)

	rm -rf $(PRIVATE_INTERMEDIATES_DIR)

	rm -f $@ $(PRIVATE_general_tests_list_zip)

	mkdir -p $(PRIVATE_INTERMEDIATES_DIR) $(PRIVATE_INTERMEDIATES_DIR)/tools

	echo $(sort $(COMPATIBILITY.general-tests.FILES) $(COMPATIBILITY.general-tests.SOONG_INSTALLED_COMPATIBILITY_SUPPORT_FILES)) | tr " " "\n" > $(PRIVATE_INTERMEDIATES_DIR)/list

	for symlink in $(PRIVATE_SYMLINKS); do \

	  echo $$symlink >> $(PRIVATE_INTERMEDIATES_DIR)/list; \

	done

	$(hide) for shared_lib in $(PRIVATE_HOST_SHARED_LIBS); do \

	  echo $$shared_lib >> $(PRIVATE_INTERMEDIATES_DIR)/shared-libs.list; \

	done

	grep $(HOST_OUT_TESTCASES) $(PRIVATE_INTERMEDIATES_DIR)/shared-libs.list > $(PRIVATE_INTERMEDIATES_DIR)/host-shared-libs.list || true

	grep $(HOST_OUT_TESTCASES) $(PRIVATE_INTERMEDIATES_DIR)/list > $(PRIVATE_INTERMEDIATES_DIR)/host.list || true

	grep $(TARGET_OUT_TESTCASES) $(PRIVATE_INTERMEDIATES_DIR)/list > $(PRIVATE_INTERMEDIATES_DIR)/target.list || true

	grep -e .*\\.config$$ $(PRIVATE_INTERMEDIATES_DIR)/host.list > $(PRIVATE_INTERMEDIATES_DIR)/host-test-configs.list || true
@@ -91,7 +49,6 @@ $(general_tests_zip) : $(COMPATIBILITY.general-tests.FILES) $(my_host_shared_lib 
	  -P host -C $(PRIVATE_INTERMEDIATES_DIR) -D $(PRIVATE_INTERMEDIATES_DIR)/tools \

	  -P host -C $(HOST_OUT) -l $(PRIVATE_INTERMEDIATES_DIR)/host.list \

	  -P target -C $(PRODUCT_OUT) -l $(PRIVATE_INTERMEDIATES_DIR)/target.list \

	  -P host -C $(HOST_OUT) -l $(PRIVATE_INTERMEDIATES_DIR)/host-shared-libs.list \

	  -sha256

	$(SOONG_ZIP) -d -o $(PRIVATE_general_tests_configs_zip) \

	  -P host -C $(HOST_OUT) -l $(PRIVATE_INTERMEDIATES_DIR)/host-test-configs.list \
@@ -112,8 +69,3 @@ general_tests_zip := 
general_tests_list_zip :=

general_tests_configs_zip :=

general_tests_shared_libs_zip :=
 
my_host_shared_lib_for_general_tests :=

my_symlinks_for_general_tests :=

my_general_tests_shared_lib_files :=

my_general_tests_symlinks :=

my_host_shared_lib_symlinks :=