Hi Christian,

The git url you are seeing is hosted on local server running SCM Manager.

Note: Reviewboard is also hosted on same server.

Regards,
Pritesh

On Friday, January 20, 2017 at 1:23:26 PM UTC+5:30, Christian Hammond wrote:
>
> Hi Pritesh,
>
> A couple important notes:
>
> 1) Git and Mercurial are not compatible, and Git repositories cannot be 
> used with Mercurial repository configurations. It may have appeared to be 
> working due to Mercurial understanding a variant of Git diffs, and with the 
> local developer using Mercurial and generating Git diffs, but using a local 
> Git clone and a Mercurial configuration is not possible. They would have 
> different SHAs, and you would not be able to fetch the files from the 
> repository.
>
> Unless this is some kind of Git/Mercurial bridge, but I don't know 
> anything about your setup. I'll need to know more.
>
> 2) Git's remoting protocol is limited and does not support the kind of 
> fine-grained access we require from a remote repository. To use a remote 
> Git repository, you need to configure the "raw file URL mask" field, which 
> is basically a way to point to some URL that, given a file path and Git 
> blob SHA, will return the contents of a file. This requires a service like 
> cgit or gitweb.
>
>
> Based on my current understanding, I'm not sure how this would have worked 
> before. What service is hosting that URL shown in the screenshot?
>
> Christian
>
> -- 
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Thu, Jan 19, 2017 at 9:38 PM, pritesh pagar <pritesh...@gmail.com 
> <javascript:>> wrote:
>
>> Hi Christian,
>>
>> Below are the Repo configuration of reviewboard:
>>
>>
>> <https://lh3.googleusercontent.com/-Aplt-B8gYms/WIGeDtDrqYI/AAAAAAAAAkw/GNF-wC2wnoIPyHacBU-kkwhtwjXed1_xwCLcB/s1600/rb_conf.png>
>>
>> Note:
>> 1. In repo configuration, You can see Repo Type is Mercurial even though 
>> repo is a git repo. This was requirement from some developer and was 
>> working correctly before upgrade.
>> I tried changing repo type to GIT as well, still got same error.
>>
>> 2. Git repo is hosted on our Inhouse SCM server, not on git cloud.
>>
>> Regards,
>> Pritesh
>>
>> On Friday, January 20, 2017 at 9:35:39 AM UTC+5:30, Christian Hammond 
>> wrote:
>>>
>>> Hi Pritesh,
>>>
>>> Can you show me the configuration for your repository in Review Board?
>>>
>>> Do existing review requests work?
>>>
>>> How are you generating/uploading the diff?
>>>
>>> Christian
>>>
>>> -- 
>>> Christian Hammond
>>> President/CEO of Beanbag <https://www.beanbaginc.com/>
>>> Makers of Review Board <https://www.reviewboard.org/>
>>>
>>> On Thu, Jan 19, 2017 at 5:41 AM, pritesh pagar <pritesh...@gmail.com> 
>>> wrote:
>>>
>>>> Hi,
>>>>
>>>> Recently I upgraded reviewboard from 2.0.15 to 2.5.7.
>>>>
>>>> After upgrade, diff upload for git repo is failng.
>>>>
>>>> *ERROR ON UI:*
>>>> The patch to 'index.js' didn't apply cleanly. The temporary files have 
>>>> been left in '/tmp/reviewboard.cMAy_Q' for debugging purposes.
>>>> `patch` returned: 
>>>>
>>>> This may be a bug in the software, a temporary outage, or an issue with 
>>>> the format of your diff.
>>>>
>>>> *After debugging, I found following error in reviewboard error logs:*
>>>> Traceback (most recent call last):
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/views.py",
>>>>  
>>>> line 299, in get
>>>>     response = renderer.render_to_response(request)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/renderers.py",
>>>>  
>>>> line 56, in render_to_response
>>>>     return HttpResponse(self.render_to_string(request))
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/renderers.py",
>>>>  
>>>> line 74, in render_to_string
>>>>     large_data=True)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 295, in cache_memoize
>>>>     compress_large_data))
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 249, in cache_memoize_iter
>>>>     items = items_or_callable()
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 292, in <lambda>
>>>>     lambda: [lookup_callable()],
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/renderers.py",
>>>>  
>>>> line 73, in <lambda>
>>>>     lambda: self.render_to_string_uncached(request),
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/renderers.py",
>>>>  
>>>> line 87, in render_to_string_uncached
>>>>     request=request)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/diffutils.py",
>>>>  
>>>> line 667, in populate_diff_chunks
>>>>     chunks = list(generator.get_chunks())
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/chunk_generator.py",
>>>>  
>>>> line 756, in get_chunks
>>>>     for chunk in super(DiffChunkGenerator, self).get_chunks(cache_key):
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/chunk_generator.py",
>>>>  
>>>> line 107, in get_chunks
>>>>     large_data=True)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 295, in cache_memoize
>>>>     compress_large_data))
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 249, in cache_memoize_iter
>>>>     items = items_or_callable()
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/Djblets-0.9.4-py2.7.egg/djblets/cache/backend.py",
>>>>  
>>>> line 292, in <lambda>
>>>>     lambda: [lookup_callable()],
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/chunk_generator.py",
>>>>  
>>>> line 106, in <lambda>
>>>>     lambda: list(self.get_chunks_uncached()),
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/chunk_generator.py",
>>>>  
>>>> line 763, in get_chunks_uncached
>>>>     new = get_patched_file(old, self.filediff, self.request)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/diffutils.py",
>>>>  
>>>> line 233, in get_patched_file
>>>>     return patch(diff, buffer, filediff.dest_file, request)
>>>>   File 
>>>> "/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.5.7-py2.7.egg/reviewboard/diffviewer/diffutils.py",
>>>>  
>>>> line 169, in patch
>>>>     'output': stderr,
>>>> Exception: The patch to 'index.js' didn't apply cleanly. The temporary 
>>>> files have been left in '/tmp/reviewboard.cMAy_Q' for debugging purposes.
>>>> `patch` returned: 
>>>>
>>>>    
>>>> *Could this error be caused because of GIT version and REVIEBOARD 
>>>> version incompatibility, or could it be something else?*
>>>>
>>>> *Note: This issue is only in case of git patch upload, patch of 
>>>> mercurial repo is uploaded correctly.*
>>>>
>>>> Regards,
>>>> Pritesh
>>>>
>>>> -- 
>>>> Supercharge your Review Board with Power Pack: 
>>>> https://www.reviewboard.org/powerpack/
>>>> Want us to host Review Board for you? Check out RBCommons: 
>>>> https://rbcommons.com/
>>>> Happy user? Let us know! https://www.reviewboard.org/users/
>>>> --- 
>>>> You received this message because you are subscribed to the Google 
>>>> Groups "reviewboard" group.
>>>> To unsubscribe from this group and stop receiving emails from it, send 
>>>> an email to reviewboard...@googlegroups.com.
>>>> For more options, visit https://groups.google.com/d/optout.
>>>>
>>>
>>> -- 
>> Supercharge your Review Board with Power Pack: 
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons: 
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com <javascript:>.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to