1101: View Diff error on demo site

aqua*****@gmai***** (Google Code) (Is this you? Claim this profile.)
May 2, 2009
What version are you running?
whatever's on the demo site

What's the URL of the page containing the problem?
http://demo.review-board.org/r/1399/diff/#index_header

What steps will reproduce the problem?
1. go to http://demo.review-board.org/r/1399/
2. click "View Diff" in the top right
3.

What is the expected output? What do you see instead?
A diff

What operating system are you using? What browser?
I'm on Fedora 10 using Firefox.  

Please provide any additional information below.
This is the error's output: 

The file
'svn+ssh://sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
(r9) could not be found in the repository: PROPFIND request failed on
'/svn/!svn/bc/9/svn+ssh:/sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
'/svn/!svn/bc/9/svn+ssh:/sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
path not found

Traceback (most recent call last):
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/views.py",
line 152, in view_diff
    interdiffset, highlighting, True)
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 620, in get_diff_files
    large_data=True)
  File
"/usr/lib/python2.5/site-packages/Djblets-0.5beta2.dev_20090426-py2.5.egg/djblets/util/misc.py",
line 143, in cache_memoize
    data = lookup_callable()
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 619, in <lambda>
    enable_syntax_highlighting),
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 341, in get_chunks
    old = get_original_file(filediff)
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 248, in get_original_file
    large_data=True)[0]
  File
"/usr/lib/python2.5/site-packages/Djblets-0.5beta2.dev_20090426-py2.5.egg/djblets/util/misc.py",
line 143, in cache_memoize
    data = lookup_callable()
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 247, in <lambda>
    data = cache_memoize(key, lambda: [fetch_file(file, revision)],
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/diffviewer/diffutils.py",
line 227, in fetch_file
    data = tool.get_file(file, revision)
  File
"/usr/lib/python2.5/site-packages/ReviewBoard-1.0beta3.dev_20090426-py2.5.egg/reviewboard/scmtools/svn.py",
line 107, in get_file
    raise FileNotFoundError(path, revision, str(e))
FileNotFoundError: The file
'svn+ssh://sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
(r9) could not be found in the repository: PROPFIND request failed on
'/svn/!svn/bc/9/svn+ssh:/sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
'/svn/!svn/bc/9/svn+ssh:/sun1669.daytonoh.ncr.com/ES/ES_1.0_DEV/ncrx/12.0.0/install/db/account_status_lookups.sql'
path not found

Loading...
chipx86
#1 chipx86
It looks like you're uploading a diff that's for your server, but on our demo server.
We don't have a repository configured for your server, so this is expected. You can
only upload diffs against the repositories we have configured.
  • +UserError
#2 aqua*****@gmai***** (Google Code) (Is this you? Claim this profile.)
Hey.  I was just browsing through the demo thing. 

I suppose I just feel like the demo server shouldn't be displaying backtraces to the
user for a diff they might upload, but would rather present some useful error message
(perhaps including the backtrace in an HTML comment block?)

Cheers. 
chipx86
#3 chipx86
Ah, thought you were uploading that. We get a lot of people trying random stuff on
the demo server.

A future version will verify diffs before uploading so that this doesn't happen, and
we'll have better errors for those cases as well.
#4 aqua*****@gmai***** (Google Code) (Is this you? Claim this profile.)
Cool :D  Thanks for Review Board by the way.  It simplifies life.