[calm - Cygwin server-side packaging maintenance script] branch master, updated. 20200401-27-g6d92f86

Jon TURNEY via Cygwin-apps-cvs <[email protected]>
Newsgroups gmane.os.cygwin.cvs.apps
Message-ID <[email protected]>


https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/calm.git;h=6d92f86649c1bc385949fda0ad74bb644d80fc65

commit 6d92f86649c1bc385949fda0ad74bb644d80fc65
Author: Jon Turney <[email protected]>
Date:   Mon May 25 16:28:59 2020 +0100

    Ignore versions overriden by version: when checking if a hint is stale
    
    When uploading a replacement hint with a version: override, the overriden
    version doesn't exist anymore, so the hint will be considered stale and
    removed.
    
    This causes the upload to fail as movelist validation notices that the
    hint is both uploaded and automatically vaulted.


Diff:
---
 calm/package.py | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/calm/package.py b/calm/package.py
index 487c3ed..b789f44 100755
--- a/calm/package.py
+++ b/calm/package.py
@@ -394,6 +394,8 @@ def read_one_package(packages, p, relpath, dirpath, files, remove, kind):
         # apply a version override
         if 'version' in pvr_hint:
             ovr = pvr_hint['version']
+            # also record the version before the override
+            pvr_hint['original-version'] = vr
         else:
             ovr = vr
 
@@ -1409,7 +1411,14 @@ def stale_packages(packages):
                     else:
                         all_stale[v] = False
 
+        # skip over any versions which have a version: override
+        effective_versions = list(po.hints.keys())
         for v in po.hints:
+            ov = po.hints[v].hints.get('original-version', v)
+            if ov != v and ov in effective_versions:
+                effective_versions.remove(ov)
+
+        for v in effective_versions:
             # if there's a pvr.hint without a fresh source or install of the
             # same version, move it as well
             if all_stale.get(v, True):
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.