You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2020/12/11 22:59:03 UTC

[whimsy] branch master updated: Version must include '.'

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 cd5c632  Version must include '.'
cd5c632 is described below

commit cd5c632fa004ed888b6d415a6f3eb57966305a84
Author: Sebb <se...@apache.org>
AuthorDate: Fri Dec 11 22:58:54 2020 +0000

    Version must include '.'
---
 tools/download_check.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/download_check.rb b/tools/download_check.rb
index 985b350..a8a0f3c 100755
--- a/tools/download_check.rb
+++ b/tools/download_check.rb
@@ -443,7 +443,8 @@ def _checkDownloadPage(path, tlp, version)
             ext = $2 # save for use after RE match
             $vercheck[base] = [h =~ %r{^https?://archive.apache.org/} ? 'archive' : (h =~ %r{https?://repo\d?\.maven\.org/} ? 'maven' : 'live')]
             unless $vercheck[base].first == 'archive'
-              if base[0..-(ext.size+2)] =~ %r{^(.+?)-(\d.+)$}
+              # version must include '.'
+              if base[0..-(ext.size+2)] =~ %r{^(.+?)-(\d+\..+)$}
                 $versions[$2][$1] << ext
               else
                 W "Cannot parse #{base} for version"