Merge lp:~brian-murray/lptools/grab-descriptions into lp:lptools

[Brian Murray]

lp-grab-attachments: add option to download bug descriptions too

update-manager receives some terrible bug reports where people just copy and paste terminal error messages into the bug description. I thought it'd be useful to download all of the descriptions and search them for a specifc error message so I modified lp-grab-attachments to also grab descriptions.

[dobey (dobey)]

review: Approve

[Ubuntu One Auto Pilot (otto-pilot)]

There was a problem validating some authors of the branch. Authors must be either one of the listed Launchpad users, or a member of one of the listed teams on Launchpad.

Persons or Teams:

    bdmurray
    contributor-agreement-canonical
    lptools

Unaccepted Authors:

    Brian Murray <email address hidden>

[Ubuntu One Auto Pilot (otto-pilot)]

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0, Needs Fixing == 0, Needs Information == 0, Resubmit == 0, Pending == 0. Got: 1 Approve, 1 Pending.

[Ubuntu One Auto Pilot (otto-pilot)]

Voting does not meet specified criteria. Required: Approved >= 1. Got: 1 Approve, 1 Pending.

=== modified file 'bin/lp-grab-attachments'
--- bin/lp-grab-attachments	2012-02-10 18:41:41 +0000
+++ bin/lp-grab-attachments	2012-05-18 21:07:21 +0000
@@ -21,6 +21,7 @@
 # ##################################################################
 
 from optparse import OptionParser
+import codecs
 import errno
 import os
 
@@ -29,7 +30,7 @@
 USAGE = "lp-grab-attachments <bug numbers>"
 
 
-def download_attachments(bug):
+def download_attachments(bug, descriptions):
 
     bug_folder_name = 'bug-%s' % bug.id
 
@@ -39,6 +40,14 @@
         if error.errno == errno.EEXIST:
             return
 
+    if descriptions:
+        description = bug.description
+        filename = os.path.join(os.getcwd(), bug_folder_name,
+            'Description.txt')
+        local_file = codecs.open(filename, encoding="utf-8", mode="w")
+        local_file.write(description)
+        local_file.close()
+
     for attachment in bug.attachments:
         f = attachment.data.open()
         filename = os.path.join(os.getcwd(), bug_folder_name, f.filename)
@@ -59,6 +68,9 @@
     parser.add_option('-P', '--project',
                       help='Download attachments from all bugs with a '
                            'open task for this project')
+    parser.add_option('-D', '--descriptions', default=False,
+                      action='store_true',
+                      help='Also download bug descriptions as Description.txt')
 
     opts, args = parser.parse_args()
     if len(args) < 1 and not opts.package and not opts.project:
@@ -88,11 +100,11 @@
             parser.error("'%s' is not a valid bug number." % arg)
 
         bug = launchpad.bugs[bug_number]
-        download_attachments(bug)
+        download_attachments(bug, opts.descriptions)
 
         if opts.duplicates is True:
             for bug in bug.duplicates:
-                download_attachments(bug)
+                download_attachments(bug, opts.descriptions)
 
 if __name__ == '__main__':
     main()

=== modified file 'doc/lp-grab-attachments.1'
--- doc/lp-grab-attachments.1	2012-02-10 18:41:41 +0000
+++ doc/lp-grab-attachments.1	2012-05-18 21:07:21 +0000
@@ -37,6 +37,9 @@
 .TP
 .B \-P \fIPROJECT\fR, \fB\-\-project\fR=\fIPROJECT\fR
 Download attachments from all bugs with an open task for this project.
+.TP
+.BR \-D ", " \-\-descriptions
+Also download bug descriptions as Description.txt.
 .SH AUTHOR
 \fBlp-grab\-attachments\fR was written by Daniel Holbach and this manual page
 was written by Jonathan Patrick Davies.