]> granicus.if.org Git - libvpx/commitdiff
Fix failure to copy data files if content changes
authorAdrian Grange <agrange@google.com>
Mon, 16 Sep 2013 18:46:00 +0000 (11:46 -0700)
committerAdrian Grange <agrange@google.com>
Mon, 16 Sep 2013 23:14:39 +0000 (16:14 -0700)
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

diff --git a/libs.mk b/libs.mk
index 2338631085ed680c024afc22f8925456ccd81337..43545e38a35ef96dc1f05b277d8a7bac83defe8d 100644 (file)
--- 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))