SVN urls don't get the revision properly

[GoogleCodeExporter]

What steps will reproduce the problem?
Go to http://codereview.appspot.com/5076/diff/1/6

What is the expected output? What do you see instead?
Expected diff. Instead see "error: old chunk mismatch"

Please provide any additional information below.
I believe that the SVN URLs aren't correct.

When using an invalid repository base, I saw how the URL was constructed,
and it didn't seem correct.

http://codereview.appspot.com/5078/diff/1/6
has an error in repository base, implies that repository is using URL
http://svn.apache.org/repos/asf/incubator/shindig/trunk/java/server/src/test/jav
a/org/apache/shindig/server/endtoend/EndToEndServer.java?rev=685757
for the old version.

This URL doesn't respect the version number (remove rev= and the content is
the same)

Original issue reported on code.google.com by evan.gil...@gmail.com on 17 Sep 2008 at 8:02

[GoogleCodeExporter]

It looks like the correct URL is

http://svn.apache.org/repos/asf/!svn/bc/685757/incubator/shindig/trunk/java/serv
er/src/test/java/org/apache/shindig/server/endtoend/EndToEndServer.java

IMO it's better to use the push method (default with most recent upload.py, with
older versions use the "-l" flag) here instead of adding another repository 
specific
way to compute the correct download URL on the server side.

Original comment by albrecht.andi on 17 Sep 2008 at 2:11

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

This is fixed now? Clicking the link above shows the proper diff. There is no 
error: 
old chunk mismatch.

Original comment by drak...@gmail.com on 18 Jun 2009 at 6:08

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

The old chunk mismatch is still there. 

Original comment by albrecht.andi on 18 Jun 2009 at 6:54

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

>This URL doesn't respect the version number (remove rev= and the content is
> the same)
Use ?p=685757 insteed ?rev=685757 . http://svn/somefile.txt?p=<rev> is default 
url
for mod_dav_svn in Subversion-1.6 and higher. Rieveld dosn't support this type 
of URLs

Original comment by crchemis...@gtempaccount.com on 18 Mar 2010 at 12:18

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Issue 187 has been merged into this issue.

Original comment by albrecht.andi on 1 Apr 2010 at 6:20

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Happening for me as well: http://codereview.chromium.org/3453029/diff/9001/10016

Original comment by andyb...@google.com on 28 Sep 2010 at 3:58

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

I am seeing this with
http://codereview.chromium.org/4157005/show

The potentially interesting part is that I got this error on git cl upload:

Got exception while uploading -- saving description to 
/home/rochberg/.git_cl_description_backup

AppEngine is misbehaving and returned HTTP 500, again. Keep faith and retry or 
visit [internal url]
HTTP Error 500: Internal Server Error

Original comment by rochb...@chromium.org on 27 Oct 2010 at 9:06

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Original comment by albrecht.andi on 6 Apr 2012 at 7:12

• Added labels: Component-Engine
• Removed labels: ****