From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Tue, 4 Apr 2017 23:37:43 +0000 (+0100)
Subject: ownsource: do not include files more than once in srcbomb
X-Git-Url: https://git.distorted.org.uk/~mdw/hippotat/commitdiff_plain/9693ff43ed669ec93b40b3bb2b62482ec5dbfdf7

ownsource: do not include files more than once in srcbomb

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/hippotatlib/ownsource.py b/hippotatlib/ownsource.py
index 385c9ea..8ecc7f2 100644
--- a/hippotatlib/ownsource.py
+++ b/hippotatlib/ownsource.py
@@ -342,11 +342,16 @@ class SourceShipmentPreparer():
 
   def mk_inner_portmanteau(s):
     outputs = [s.manifest_name]
+    outputs_done = { }
     mfh = s.open_output_fh(s.manifest_name,'w')
     for me in s._manifest:
       try: fname = me['file']
       except KeyError: fname = me.get('file_print','')
-      else: outputs.append(fname)
+      else:
+        try: outputs_done[fname]
+        except KeyError:
+          outputs.append(fname)
+          outputs_done[fname] = 1
       print('%s\t%s' % (fname, me['info']), file=mfh)
     mfh.close()