Merge lp:~brian-murray/debian/sid/lptools/lp-grab-attach into lp:~debian-lptools/debian/sid/lptools/sid

[Brian Murray]

Ubuntu has been carrying the "FailedToDecompressContent" patch for a while and OSError change is one I made locally recently.

Unmerged revisions

106. By Brian Murray on 2021-02-16

bin/lp-grab-attachments: Do not crash when an attachment cannot be
downloaded because it fails to decompress. If the bug folder already
exists download attachments anyway as the bug may have new ones.

=== modified file 'bin/lp-grab-attachments'
--- bin/lp-grab-attachments	2012-10-24 14:00:22 +0000
+++ bin/lp-grab-attachments	2021-02-16 22:30:33 +0000
@@ -38,7 +38,7 @@
         os.mkdir(bug_folder_name)
     except OSError, error:
         if error.errno == errno.EEXIST:
-            return
+            pass
 
     if descriptions:
         description = bug.description
@@ -49,7 +49,11 @@
         local_file.close()
 
     for attachment in bug.attachments:
-        f = attachment.data.open()
+        try:
+            f = attachment.data.open()
+        except FailedToDecompressContent:
+            print("Failed to download %s" % attachment.title)
+            continue
         filename = os.path.join(os.getcwd(), bug_folder_name, f.filename)
         local_file = open(filename, "w")
         local_file.write(f.read())

=== modified file 'debian/changelog'
--- debian/changelog	2020-08-29 20:36:32 +0000
+++ debian/changelog	2021-02-16 22:30:33 +0000
@@ -1,3 +1,11 @@
+lptools (0.2.0-8) UNRELEASED; urgency=medium
+
+  * bin/lp-grab-attachments: Do not crash when an attachment cannot be
+    downloaded because it fails to decompress. If the bug folder already
+    exists download attachments anyway as the bug may have new ones.
+
+ -- Brian Murray <[e-mail]>  Tue, 16 Feb 2021 14:23:15 -0800
+
 lptools (0.2.0-7) unstable; urgency=medium
 
   [ dann frazier ]