Merge lp:~brendan-donegan/lpltk/lp-file-bug-fixes into lp:lpltk

[Brendan Donegan]

Few small fixes for the lp-file-bug script. I have given them a good test, but you might want to double-check the scenarios you saw failing before.

[Bryce Harrington (bryce)]

Looks good, this solves the editor problem I noticed.

review: Approve (code)

=== modified file 'scripts/lp-file-bug'
--- scripts/lp-file-bug	2012-09-19 21:09:50 +0000
+++ scripts/lp-file-bug	2012-09-19 22:49:27 +0000
@@ -22,17 +22,18 @@
     lp = LaunchpadService()
 
     # TODO: Get the description for 'your favourite editor'
-    editor = os.environ.get('EDITOR','nano')
+    editor = os.environ.get('EDITOR','nano').split()
     tempfile = NamedTemporaryFile(delete=False)
+    editor.append(tempfile.name)
     try:
-        check_call([editor,tempfile.name])
-    except CalledProcessError:
-        print("Failed to open text editor")
+        check_call(editor)
+    except OSError:
+        print("Failed to open preferred editor '%s'" % editor)
     description = tempfile.read()
 
     try:
         bug = lp.create_bug(args.project, args.package, title=args.title, description=description)
-        print(bug.lp_bug.web_link)
+        print(bug.lpbug.web_link)
     except:
         print("Unable to file bug in:")
         print("\tproject: %s" % args.project)
@@ -40,5 +41,7 @@
         if args.package:
             print("\tpackage: %s" % args.package)
 
+        print("Please check that the project and/or package name are correct.")
+
 if __name__ == "__main__":
     sys.exit(main())