From 5b23666e67c1c96d5ffc87f8e3291686d78c9b85 Mon Sep 17 00:00:00 2001 From: Adrian Grange Date: Mon, 16 Sep 2013 11:46:00 -0700 Subject: [PATCH] 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 --- libs.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs.mk b/libs.mk index 233863108..43545e38a 100644 --- a/libs.mk +++ b/libs.mk @@ -395,7 +395,7 @@ libvpx_test_srcs.txt: @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))