public inbox for cygwin-apps-cvs@sourceware.org
help / color / mirror / Atom feed
* [calm - Cygwin server-side packaging maintenance script] branch master, updated. 60c749590746bf7ffc6222bf17e0191f025ca017
@ 2016-06-15 16:47 jturney
  0 siblings, 0 replies; only message in thread
From: jturney @ 2016-06-15 16:47 UTC (permalink / raw)
  To: cygwin-apps-cvs




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

commit 60c749590746bf7ffc6222bf17e0191f025ca017
Author: Jon Turney <jon.turney@dronecode.org.uk>
Date:   Thu May 19 12:15:51 2016 +0000

    Removing setup.hint files isn't allowed
    
    Removing a package doesn't make much sense currently, all we can do is
    replace it with an empty, obsolete one, so existing installs are updated to
    that.


Diff:
---
 calm/calm.py    |    3 ---
 calm/uploads.py |    3 +++
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/calm/calm.py b/calm/calm.py
index 9fc350e..97746fb 100755
--- a/calm/calm.py
+++ b/calm/calm.py
@@ -149,9 +149,6 @@ def process(args):
                         break
 
                     # remove files which are to be removed
-                    #
-                    # XXX: this doesn't properly account for removing setup.hint
-                    # files
                     for p in scan_result[arch].to_vault:
                         for f in scan_result[arch].to_vault[p]:
                             package.delete(merged_packages[arch], p, f)
diff --git a/calm/uploads.py b/calm/uploads.py
index 10b5514..59c3d70 100644
--- a/calm/uploads.py
+++ b/calm/uploads.py
@@ -158,6 +158,9 @@ def scan(m, all_packages, arch, args):
                 elif os.path.getsize(fn) != 0:
                     logging.error("remove file %s is not empty" % fn)
                     error = True
+                elif f == 'setup.hint':
+                    logging.error("remove file %s is not permitted" % fn)
+                    error = True
                 else:
                     vault[relpath].append(f[1:])
                     remove_success.append(fn)


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2016-06-15 16:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-15 16:47 [calm - Cygwin server-side packaging maintenance script] branch master, updated. 60c749590746bf7ffc6222bf17e0191f025ca017 jturney

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).