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 <priteshpaga...@gmail.com> 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+unsubscr...@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+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
