Patch Set 2:

I think this is plain wrong, citing a private mail from Shawn:


2010/5/29 Shawn O. Pearce <spearce@spearce.org>:
> Matthias Sohn <matthias.sohn@googlemail.com> wrote:
>> 2010/5/29 Shawn O. Pearce <spearce@spearce.org>
>> >
>> > Yea, I found that bug myself today, and fixed it.  So I closed
>> > 314964.  Sorry you spent time on a bug, thinking its user error.
>> >
>>
>> no problem
>>
>> I think the review URL you added in
>> http://egit.eclipse.org/r/#patch,unified,770,1,.eclipse_iplog
>> <http://egit.eclipse.org/r/#patch,unified,770,1,.eclipse_iplog>has an extra
>> trailing r/ :
>>     http://egit.eclipse.org/r/r/
>> which should be removed.
>
> Nope, its correct.  It looks funny, yes, but its correct.
>
> The first /r/ tells our Apache to redirect the request to Gerrit.
>
> The second /r/ tells Gerrit that the thing following is a Git
> SHA-1 and it should try to locate the change(s) that use that
> commit object.
>
> Nothing I can easily do about it now.  The second /r/ is historical
> and comes from Gerrit 1.x days.  The first /r/ was a foolish
> mistake when I setup the egit.eclipse.org Apache configuration.
> Its modeled after how I set it up within Google... so I've made
> that stupid mistake twice now.  Yay.
>
> --
> Shawn.