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

Commit f3a43404 authored by Colin Cross's avatar Colin Cross
Browse files

Fix test suite zip generation on mac

Don't use sed -i, it requires an argument on mac and is eating the
-e parameter, which causes the command to not be applied.

Also clean up the temporary list files so they don't clutter up
$OUT.

Test: m -j device-tests
Bug: 64226094
Change-Id: Ie71820fc2d1cd812717bf0c156af011ca723e9f6
parent 218a0c0c
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -17,11 +17,11 @@

device-tests-zip := $(PRODUCT_OUT)/device-tests.zip
$(device-tests-zip): $(COMPATIBILITY.device-tests.FILES) $(SOONG_ZIP)
	echo $(sort $(COMPATIBILITY.device-tests.FILES)) > $@.list
	sed -i -e 's/\s\+/\n/g' $@.list
	echo $(sort $(COMPATIBILITY.device-tests.FILES)) | tr " " "\n" > $@.list
	grep $(HOST_OUT_TESTCASES) $@.list > $@-host.list || true
	grep $(TARGET_OUT_TESTCASES) $@.list > $@-target.list || true
	$(hide) $(SOONG_ZIP) -d -o $@ -P host -C $(HOST_OUT) -l $@-host.list -P target -C $(PRODUCT_OUT) -l $@-target.list
	rm -f $@.list $@-host.list $@-target.list

device-tests: $(device-tests-zip)
$(call dist-for-goals, device-tests, $(device-tests-zip))
+2 −2
Original line number Diff line number Diff line
@@ -16,11 +16,11 @@

general-tests-zip := $(PRODUCT_OUT)/general-tests.zip
$(general-tests-zip): $(COMPATIBILITY.general-tests.FILES) $(SOONG_ZIP)
	echo $(sort $(COMPATIBILITY.general-tests.FILES)) > $@.list
	sed -i -e 's/\s\+/\n/g' $@.list
	echo $(sort $(COMPATIBILITY.general-tests.FILES)) | tr " " "\n" > $@.list
	grep $(HOST_OUT_TESTCASES) $@.list > $@-host.list || true
	grep $(TARGET_OUT_TESTCASES) $@.list > $@-target.list || true
	$(hide) $(SOONG_ZIP) -d -o $@ -P host -C $(HOST_OUT) -l $@-host.list -P target -C $(PRODUCT_OUT) -l $@-target.list
	rm -f $@.list $@-host.list $@-target.list

general-tests: $(general-tests-zip)
$(call dist-for-goals, general-tests, $(general-tests-zip))