lptools

Merge lp:~jelmer/lptools/fix-imports into lp:~dobey/lptools/trunk

  1. fix-imports
  2. Merge into trunk
Proposed by Jelmer Vernooij
Status: Merged
Approved by: dobey
Approved revision: 13
Merged at revision: not available
Proposed branch: lp:~jelmer/lptools/fix-imports
Merge into: lp:~dobey/lptools/trunk
Diff against target: 23 lines (+2/-3)
1 file modified
bin/lp-review-list (+2/-3)
To merge this branch: bzr merge lp:~jelmer/lptools/fix-imports
Reviewer Review Type Date Requested Status
dobey Approve
Review via email: mp+21188@code.launchpad.net

Commit message

Fix missing and unused imports in lp-review-list

Description of the change

This removes some unused imports in lp-review-list and adds a missing one for EDGE_SERVICE_ROOT.

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/lp-review-list'
2--- bin/lp-review-list 2010-02-20 04:10:17 +0000
3+++ bin/lp-review-list 2010-03-11 22:06:18 +0000
4@@ -18,18 +18,17 @@
5
6 from __future__ import with_statement
7 from ConfigParser import ConfigParser
8+from launchpadlib.launchpad import EDGE_SERVICE_ROOT
9 import os
10 import re
11 import subprocess
12-import sys
13 from threading import Thread
14
15 import pygtk
16 pygtk.require('2.0')
17 import gobject
18 import gtk
19-import pango
20-from xdg.BaseDirectory import xdg_cache_home, xdg_config_home
21+from xdg.BaseDirectory import xdg_config_home
22
23 from lptools import config
24

Subscribers

People subscribed via source and target branches

to all changes: