[whimsy] branch master updated: Actually implement archive checking ...

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new e3099af  Actually implement archive checking ...
e3099af is described below

commit e3099af88989afee574f8421a2e0da87a237232e
Author: Sebb <se...@apache.org>
AuthorDate: Mon Jun 1 11:59:18 2020 +0100

    Actually implement archive checking ...
---
 tools/download_check.rb | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/tools/download_check.rb b/tools/download_check.rb
index 403fb72..ddaf2e3 100755
--- a/tools/download_check.rb
+++ b/tools/download_check.rb
@@ -496,7 +496,11 @@ def _checkDownloadPage(path, tlp, version)
     if h =~ %r{\.(asc|sha256|sha512)$}
       host, stem, ext = check_hash_loc(h,tlp)
       if host == 'archive'
-        I "Ignoring archive hash #{h}"
+        if $ARCHIVE_CHECK
+          check_head(h, :E, "200", true, true)
+        else
+          I "Ignoring archive hash #{h}"
+        end
       elsif host
         if $NOFOLLOW
           I "Skipping archive hash #{h}"
@@ -521,8 +525,10 @@ def _checkDownloadPage(path, tlp, version)
       name = $1
       ext = $2
       if h =~ %r{https?://archive\.apache\.org/}
-        I "Ignoring archive artifact #{h}"
-        next
+        unless $ARCHIVE_CHECK
+            I "Ignoring archive artifact #{h}"
+            next
+        end
       end
       if h =~ %r{https?://(www\.)?apache\.org/dist}
         E "Must use mirror system #{h}"