Labels: Type-Defect Priority-Medium
New issue 1700 by gordon.morehouse: RB fails to get raw output from
Mercurial HTTP repo
What version are you running?
1.5 beta 2
What steps will reproduce the problem?
1. Install Review Board on Ubuntu 10.04 LTS or Debian Lenny
2. Configure Mercurial repo with path containing a https URL
3. Post reviews using Mercurial plugin or 'post-review'
4. Go to Review Board and select View Diff
Expected output: the RB diff viewer working properly.
Actual output: 'patch' fails to apply the diff 100% of the time because it
is attempting to apply the diff to cooked HTML output from the browseable
Mercurial repo. This completely breaks the diff viewer and renders RB
Traceback (most recent call last):
views.py", line 153, in view_diff
interdiffset, highlighting, True)
diffutils.py", line 881, in get_diff_files
line 166, in cache_memoize
data = lookup_callable()
diffutils.py", line 880, in <lambda>
diffutils.py", line 531, in get_chunks
new = get_patched_file(old, filediff)
diffutils.py", line 356, in get_patched_file
return patch(filediff.diff, buffer, filediff.dest_file)
diffutils.py", line 224, in patch
(filename, tempdir, patch_output))
Exception: The patch to 'install.py' didn't apply cleanly. The
temporary files have been left in '/tmp/reviewboard.9HV3Hs' for
`patch` returned: patching file /tmp/reviewboard.9HV3Hs/tmpYvMLPF
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file /tmp/reviewboard.
In this case, 'tmpYvMLPF' contains HTML output from the Mercurial repo, not
the raw contents of the file that should be patched.
You received this message because you are subscribed to the Google Groups
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to
For more options, visit this group at