Thanks for the suggestion, David, but it doesn't work either. I've now
tried it both ways around (Path == proxy, Mirror == info and Path ==
info, Mirror =proxy) but the result is the same in either case. post-
review successfully creates the request, but the server is unable to
display diffs. Here's an example error (with depot path information
sanitized):
The patch to '//XXXXX/clear.cpp' didn't apply cleanly. The temporary
files have been left in '/tmp/reviewboard.UVgy2H' for debugging
purposes. `patch` returned: patching file /tmp/reviewboard.UVgy2H/
tmpB2EamP Hunk #1 FAILED at 171. 1 out of 1 hunk FAILED -- saving
rejects to file /tmp/reviewboard.UVgy2H/tmpB2EamP-new.rej
Traceback (most recent call last):
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/views.py", line 153, in view_diff
interdiffset, highlighting, True)
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/diffutils.py", line 1066, in get_diff_files
large_data=True)
File "/usr/lib/python2.5/site-packages/Djblets-0.6.7-py2.5.egg/
djblets/util/misc.py", line 166, in cache_memoize
data = lookup_callable()
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/diffutils.py", line 1065, in <lambda>
enable_syntax_highlighting)),
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/diffutils.py", line 552, in get_chunks
new = get_patched_file(old, filediff)
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/diffutils.py", line 374, in get_patched_file
return patch(filediff.diff, buffer, filediff.dest_file)
File "/usr/lib/python2.5/site-packages/ReviewBoard-1.5.5-py2.5.egg/
reviewboard/diffviewer/diffutils.py", line 242, in patch
(filename, tempdir, patch_output))
Exception: The patch to '//XXXXX/clear.cpp' didn't apply cleanly. The
temporary files have been left in '/tmp/reviewboard.UVgy2H' for
debugging purposes.
`patch` returned: patching file /tmp/reviewboard.UVgy2H/tmpB2EamP
Hunk #1 FAILED at 171.
1 out of 1 hunk FAILED -- saving rejects to file /tmp/
reviewboard.UVgy2H/tmpB2EamP-new.rej
When I go and look in the temporary directory, I find a perfectly
valid-looking patch plus empty source and destination files. So the
server is clearly failing to retrieve the contents of the files, but
since there's no useful logging, I really don't have any further
information about what is going wrong. We stumbled into the proxy
issue by looking at Perforce server logs...
-Chris
On Jul 20, 3:31 pm, David Trowbridge <[email protected]> wrote:
> This configuration is supported with the following:
>
> Set the repository path to the results of "p4 info"
> Set the mirror path to the proxy geographically closest to the reviewboard
> server.
>
> - David
>
> On Jul 20, 2011, at 8:29 AM, chrisn <[email protected]> wrote:
>
>
>
>
>
>
>
> > I just filedhttp://code.google.com/p/reviewboard/issues/detail?id=2182
> > to describe an issue with ReviewBoard and Perforce proxy servers that
> > is effectively blocking us from using ReviewBoard.
>
> > I'd appreciate any thoughts or BTDT advice about how I might be able
> > to work around the problem.
>
> > Thanks.
>
> > --
> > Want to help the Review Board project? Donate today
> > athttp://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://www.reviewboard.org/users/
> > -~----------~----~----~----~------~----~------~--~---
> > To unsubscribe from this group, send email to
> > [email protected]
> > For more options, visit this group
> > athttp://groups.google.com/group/reviewboard?hl=en
--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
