Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 2958 by ceesjan@gmail.com: post-review with bzr crashes when
using --revision-range
http://code.google.com/p/reviewboard/issues/detail?id=2958
What version are you running?
Development versions of ReviewBoard+RBTools, checked out at 2013-04-19
What steps will reproduce the problem?
1. Enter a bazaar repository
2. Execute `post-review --revision-range=123..126`
What is the expected output? What do you see instead?
I get a stacktrace:
quinox@duivenkracht:~/projects/x/y ^2 $ post-review
--revision-range=297..299
==> HTTP Authentication Required
Enter authorization information for "Web API" at
Username: quinox
Password:
Traceback (most recent call last):
File "/home/quinox/projects/tooling/usr/local/bin/post-review", line 3,
in
postreview.main()
File "/home/quinox/projects/tooling/usr/local/lib/python/rbtools/postreview.py",
line 1292, in main
repository_info)
ValueError: too many values to unpack
Please provide any additional information below.
I solved it by patching rbtools/clients/bazaar.py:
@@ -85,7 +85,7 @@ class BazaarClient(SCMClient):
self._set_summary(last_revision)
self._set_description(revision_range)
-return diff
+return (diff, None)
def _get_range_diff(self, revision_range, files):
"""
(Not uploading to your reviewboard because I wrote this patch by guessing,
I haven't looked at the code that calls this function)
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
You received this message because you are subscribed to the Google Groups
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.