public inbox for cygwin-apps-cvs@sourceware.org
help / color / mirror / Atom feed
From: Jon TURNEY <jturney@sourceware.org>
To: cygwin-apps-cvs@sourceware.org
Subject: [calm - Cygwin server-side packaging maintenance script] branch master, updated. 20210626-44-g6ef8bde
Date: Tue, 24 May 2022 14:18:41 +0000 (GMT)	[thread overview]
Message-ID: <20220524141841.80F7B38425BE@sourceware.org> (raw)




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

commit 6ef8bdef5f8f9dd76dec89150d0845521fa04b96
Author: Jon Turney <jon.turney@dronecode.org.uk>
Date:   Tue May 24 13:38:00 2022 +0100

    Warn if source and binary packages diff in test: label
    
    This happens due to a bug in cygport 0.35.0, and breaks assumptions made
    by the deprecated so report (and maybe elsewhere?)


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

diff --git a/calm/package.py b/calm/package.py
index 45be182..2f318b2 100755
--- a/calm/package.py
+++ b/calm/package.py
@@ -780,6 +780,14 @@ def validate_packages(args, packages, valid_requires_extra=None):
                     packages[es_p].is_used_by.add(p)
                     missing_source = False
 
+                    # also check that they match in presence or absence test: label
+                    #
+                    # (this is needed if we are going to compare best_version
+                    # between install and source packages for some information,
+                    # as we do in some places...)
+                    if ('test' in packages[p].version_hints[v]) != ('test' in packages[es_p].version_hints[v]):
+                        logging.error("package '%s' version '%s' test: label mismatches source package '%s'" % (p, v, es_p))
+
             if missing_source:
                 # unless the install tarfile is empty
                 if packages[p].tar(v).is_empty:



                 reply	other threads:[~2022-05-24 14:18 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220524141841.80F7B38425BE@sourceware.org \
    --to=jturney@sourceware.org \
    --cc=cygwin-apps-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).