Merge lp:~oldmanuk/bzr-autoreview/support-dot-plugins-install into lp:bzr-autoreview

Proposed by Dominic Evans
Status: Rejected
Rejected by: Paul Hummer
Proposed branch: lp:~oldmanuk/bzr-autoreview/support-dot-plugins-install
Merge into: lp:bzr-autoreview
Diff against target: None lines
To merge this branch: bzr merge lp:~oldmanuk/bzr-autoreview/support-dot-plugins-install
Reviewer Review Type Date Requested Status
Paul Hummer Needs Fixing
Review via email: mp+4303@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Dominic Evans (oldmanuk) wrote :

Small patch to allow bzr-autoreview to be installed via:

$ cd ~/.bazaar/plugins
$ bzr branch lp:bzr-autoreview autoreview

Rather than having to install it globally using distutils.

Revision history for this message
Paul Hummer (rockstar) wrote :

Dominic-

  Thanks for the branch. I hadn't even released 24 hours previous, and I've
already gotten your contribution. Thanks for you work!

  I've been running this plugin from ~/.bazaar/plugins What issues have you
been having? I don't think this specific change is really the direction we
want to go in.

  To install, I usually do:

  cd ~/.bazaar/plugins; bzr branch lp:bzr-autoreview autoreview

  If you're having problems after doing that, then I think we need to
investigate it more.

Cheers,
Paul

review: Needs Fixing
Revision history for this message
Dominic Evans (oldmanuk) wrote :

Hmm. When I tested it on my Jaunty laptop, doing the exactly same steps as you mention, it failed to find autoreview via bzrlib so I had to make these changes.

However, repeating the steps on my Jaunty server, all worked out of the box this way.

I'll cancel this merge :)

Unmerged revisions

22. By Dominic Evans

Reference autoreview locally rather than via bzrlib.plugins. Seemed to be the only way to get it to work in ~/.bazaar/plugins/autoreview

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '__init__.py'
2--- __init__.py 2009-02-24 04:00:46 +0000
3+++ __init__.py 2009-03-09 12:16:55 +0000
4@@ -1,5 +1,5 @@
5 # Copyright (c) 2009 Paul Hummer <paul@eventuallyanyway.com>
6-from bzrlib.plugins.autoreview.commands import *
7+from commands import *
8
9 commands.register_command(cmd_autoreview, 'autoreview')
10
11
12=== modified file 'commands.py'
13--- commands.py 2009-03-09 06:37:56 +0000
14+++ commands.py 2009-03-09 12:16:55 +0000
15@@ -10,7 +10,7 @@
16 from bzrlib.diff import _get_trees_to_diff, show_diff_trees
17 from bzrlib.email_message import EmailMessage
18 from bzrlib.msgeditor import edit_commit_message
19-from bzrlib.plugins.autoreview.utils import get_launchpad_api_object
20+from utils import get_launchpad_api_object
21 from bzrlib.workingtree import WorkingTree
22 from bzrlib.smtp_connection import SMTPConnection
23 from launchpadlib.launchpad import EDGE_SERVICE_ROOT

Subscribers

People subscribed via source and target branches

to all changes: