ref: 5b23666e67c1c96d5ffc87f8e3291686d78c9b85
parent: e7f2aa0fb86855c617aca1586e10722eed21d2d1
author: Adrian Grange <agrange@google.com>
date: Mon Sep 16 07:46:00 EDT 2013
Fix failure to copy data files if content changes Jenkins was failing to detect the case where an existing file is recreated with new content. In this case, thinking that the file already existed, Jenkins did not re-copy the file as it should have. By adding the file test-data.sha1 as a dependendency to the LIBVPX_TEST_DATA build target the files will be recopied if the MD5 of an existing file changes. This could be further improved to only copy files that have changed rather than copying the whole set as done in this patch. (Thanks to jzern@ who diagnozed ithe problem and suggested this fix). Change-Id: Icea7c61a95189bc639fec83020c28c70da5b2b41
--- a/libs.mk
+++ b/libs.mk
@@ -395,7 +395,7 @@
@echo $(LIBVPX_TEST_SRCS) | xargs -n1 echo | sort -u > $@
CLEAN-OBJS += libvpx_test_srcs.txt
-$(LIBVPX_TEST_DATA):
+$(LIBVPX_TEST_DATA): $(SRC_PATH_BARE)/test/test-data.sha1
@echo " [DOWNLOAD] $@"
$(qexec)trap 'rm -f $@' INT TERM &&\
curl -L -o $@ $(call libvpx_test_data_url,$(@F))