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

Commit 068d1010 authored by nelsonli's avatar nelsonli
Browse files

Build: Fix test data no present in test suite zip files. 

 my_installed_test_data (LOCAL_TEST_DATA) no present in test
 suite zip files for device tests

Bug: 135957483
Test: 1. Remove the patch aosp/1008195
      2. m general-tests
      3. unzip general-tests.zip
      4. data files should exist in target/testcases/toybox-tests

Change-Id: I3f0a4d87e284c809625843d55dc5d35ef77e79bb
parent 28142690

core/base_rules.mk

+0 −13
Original line number Original line Diff line number Diff line
@@ -710,18 +710,6 @@ ifneq (,$(filter $(SOONG_OUT_DIR)%,$(LOCAL_FULL_TEST_CONFIG))) 
  endif

  endif

endif

endif







ifeq ($(use_testcase_folder),true)

ifneq ($(my_test_data_file_pairs),)

$(foreach pair, $(my_test_data_file_pairs), \

  $(eval parts := $(subst :,$(space),$(pair))) \

  $(eval src_path := $(word 1,$(parts))) \

  $(eval file := $(word 2,$(parts))) \

  $(foreach suite, $(LOCAL_COMPATIBILITY_SUITE), \

    $(eval my_compat_dist_$(suite) += $(foreach dir, $(call compatibility_suite_dirs,$(suite),$(arch_dir)), \

      $(call filter-copy-pair,$(src_path),$(call append-path,$(dir),$(file)),$(my_installed_test_data))))))

endif

else

ifneq ($(my_test_data_file_pairs),)

ifneq ($(my_test_data_file_pairs),)

$(foreach pair, $(my_test_data_file_pairs), \

$(foreach pair, $(my_test_data_file_pairs), \

  $(eval parts := $(subst :,$(space),$(pair))) \

  $(eval parts := $(subst :,$(space),$(pair))) \
@@ -731,7 +719,6 @@ $(foreach pair, $(my_test_data_file_pairs), \ 
    $(eval my_compat_dist_$(suite) += $(foreach dir, $(call compatibility_suite_dirs,$(suite),$(arch_dir)), \

    $(eval my_compat_dist_$(suite) += $(foreach dir, $(call compatibility_suite_dirs,$(suite),$(arch_dir)), \

      $(src_path):$(call append-path,$(dir),$(file))))))

      $(src_path):$(call append-path,$(dir),$(file))))))

endif

endif

endif

core/definitions.mk

+0 −10
Original line number Original line Diff line number Diff line
@@ -2417,16 +2417,6 @@ define compat-copy-pair 
$(if $(filter-out $(2), $(LOCAL_INSTALLED_MODULE)), $(1):$(2))

$(if $(filter-out $(2), $(LOCAL_INSTALLED_MODULE)), $(1):$(2))

endef

endef





# Create copy pair for $(1) $(2)

# If $(2) is substring of $(3) do nothing.

# $(1): source path

# $(2): destination path

# $(3): filter-out target

# The format of copy pair is src:dst

define filter-copy-pair

$(if $(findstring $(2), $(3)),,$(1):$(2))

endef



# Copies many files.

# Copies many files.

# $(1): The files to copy.  Each entry is a ':' separated src:dst pair

# $(1): The files to copy.  Each entry is a ':' separated src:dst pair

# $(2): An optional directory to prepend to the destination

# $(2): An optional directory to prepend to the destination