[whimsy] branch master updated: Ensure RE refs are used immediately

[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 8fc52a0  Ensure RE refs are used immediately
8fc52a0 is described below

commit 8fc52a0c41d0cb5065f6a71c91b8daa889ade4aa
Author: Sebb <se...@apache.org>
AuthorDate: Tue Apr 13 15:55:34 2021 +0100

    Ensure RE refs are used immediately
    
    Also improve the log output
---
 tools/download_check.rb | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tools/download_check.rb b/tools/download_check.rb
index 955c41e..8ffc117 100755
--- a/tools/download_check.rb
+++ b/tools/download_check.rb
@@ -579,10 +579,6 @@ def _checkDownloadPage(path, tlp, version)
           check_closer_down(h)
       end
     elsif h =~ ARTIFACT_RE
-      if $NOFOLLOW
-        I "Skipping archive artifact #{h}"
-        next
-      end
       name = $1
       _ext = $2
       if h =~ %r{https?://archive\.apache\.org/}
@@ -590,6 +586,11 @@ def _checkDownloadPage(path, tlp, version)
             I "Ignoring archived artifact #{h}"
             next
         end
+      else
+        if $NOFOLLOW
+          I "Skipping artifact #{h}"
+          next
+        end
       end
       # Ideally would like to check for use of closer.lua/.cgi, but some projects pre-populate the pages
       # TODO: would it help to check host against mirrors.list?