Using cgit is the preferred way. It's faster, easier to set up, requires
less space, and is less error-prone.
The URL mask referenced in the docs is an actual mask, not a path to a
specific file. You can't use this for Path or Mirror Path. It needs to go in
the Raw File URL Mask field, as described in the documentation. You can try
browsing your cgit repository and viewing a raw file to see if you have it
correct.
Your Path or Mirror Path must point to the result of:
$ git remote show origin
On the client machines. post-review is going to use this to match the
repository on Review Board. Review Board in turn is going to plug the
filename and revision into the Raw File URL mask and use that to fetch the
file from cgit.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Fri, Jun 18, 2010 at 5:03 PM, Kunjal <kunjal.par...@gmail.com> wrote:
> So..should I create clone on the server OR should I use CGIT? Which
> one is MOST easy.
> I am hold up on this issue almost 8 months now..Any help is greatly
> appreciated.
>
> Also, can some one answer how to use CGIT here?
>
> For example: If I have repository in CGIT as
>
> http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_kernel/prebuilt.git/
> and I put this path in MIRROR PATH then will it work?
>
> Thanks
> Kunjal
>
>
>
> On Jun 18, 10:00 am, Dan Savilonis <d...@n-cube.org> wrote:
> > On Jun 16, 6:58 pm, Kunjal <kunjal.par...@gmail.com> wrote:
> >
> > > Now I get the point..
> >
> > > But If keep clone on the server, I need to keep the clone in-sync with
> > > every commit happening in developers workspace. Is that correct?
> >
> > No. This is where the parent branch comes in. There are two
> > possiblities: the developer wants to review code based off a revision
> > that exists in the central repository, or the developer wants to
> > review code based off a local revision (which in turn is based off of
> > the central repository at some point). For the first case, post-review
> > simply submits the diff and the file blobs in the diff already exist
> > on the server. In the second case the review diff references files
> > that are *not* in the repository, so a corresponding "parent diff" is
> > required to specify the differences between the base revision in the
> > central repository and the base revision upon which the review is
> > based.
> >
> > > We already have CGIT configured. How I can re-sue CGIT here?
> >
> > > Should I say PATH ashttp://
> mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
> > > and will it work?
> >
> > > Thanks for your great help..
> >
> > > Kunjal
> >
> > Dan
> >
> >
> >
> > > On Jun 16, 11:46 am, Christian Hammond <chip...@chipx86.com> wrote:
> >
> > > > It's a clone of the repository on the Review Board server itself.
> Path
> > > > points to that clone, and Mirror Path would be the remote origin that
> the
> > > > other developers are using for the central repository. Think of Path
> as
> > > > being what Review Board actually uses for fetching files, and Mirror
> Path as
> > > > just being an alias that is matched against for the central server.
> >
> > > > Using a local clone is not great, though, unless Review Board is
> running on
> > > > the same server as the central Git repository (or has that server
> mounted).
> > > > You really should use something like cgit/gitweb and then the raw
> file URL
> > > > mask field. It's easier to maintain and more scalable.
> >
> > > > Christian
> >
> > > > --
> > > > Christian Hammond - chip...@chipx86.com
> > > > Review Board -http://www.reviewboard.org
> > > > VMware, Inc. -http://www.vmware.com
> >
> > > > On Wed, Jun 16, 2010 at 9:57 AM, Kunjal <kunjal.par...@gmail.com>
> wrote:
> > > > > Chris,
> >
> > > > > I do not understand how GIT can be supported for 300+ users if the
> > > > > Path field is the full path of the .git directory inside the
> > > > > checkout.
> > > > > How would we know the checkout directory for each and every
> developer?
> > > > > and how it is possible to use the same PATH variable for N number
> of
> > > > > users?
> > > > > It looks like I am missing somehting here.
> >
> > > > > Please elobarate...
> >
> > > > > Thanks for your time
> >
> > > > > Kunjal
> >
> > > > > The Path field should be the full path of the .git directory inside
> > > > > this checkout.
> >
> > > > > On Jun 15, 10:53 pm, Jan Koprowski <jan.koprow...@gmail.com>
> wrote:
> > > > > > This looks like Christain said - git can't find repo. Read this:
> > > > >
> http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
> > > > > > to know how get patches for Git to enter.
> >
> > > > > > On Jun 16, 1:06 am, Christian Hammond <chip...@chipx86.com>
> wrote:
> >
> > > > > > > Git only works with local file paths on the Review Board
> server, unless
> > > > > you
> > > > > > > have the Raw URL field set. For that, you need a web-based Git
> viewer
> > > > > such
> > > > > > > as cgit or gitweb. See the Raw File URL Mask section athttp://
> > > > >www.reviewboard.org/docs/manual/dev/admin/management/repositor...
> >
> > > > > > > Basically, Git has no way to fetch a remote file with a given
> revision
> > > > > from
> > > > > > > a server, requiring instead local clones. However, our Raw File
> URL
> > > > > mask, if
> > > > > > > set correctly, allows us to fetch the file from a web service
> that is
> > > > > on the
> > > > > > > central Git server that has access to all files and revisions.
> >
> > > > > > > Christian
> >
> > > > > > > --
> > > > > > > Christian Hammond - chip...@chipx86.com
> > > > > > > Review Board -http://www.reviewboard.org
> > > > > > > VMware, Inc. -http://www.vmware.com
> >
> > > > > > > On Tue, Jun 15, 2010 at 3:30 PM, Kunjal <
> kunjal.par...@gmail.com>
> > > > > wrote:
> > > > > > > > Thanks !
> >
> > > > > > > > My path variable is set to git://
> > > > > > > > mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git
> >
> > > > > > > > My Mirror Path variable is empty. What should be the value in
> mirror
> > > > > > > > path?
> >
> > > > > > > > On Jun 15, 2:09 pm, Christian Hammond <chip...@chipx86.com>
> wrote:
> > > > > > > > > Hi,
> >
> > > > > > > > > First thing to check is that git is in your path,
> accessible by
> > > > > your web
> > > > > > > > > server.
> >
> > > > > > > > > What are you using for your Path and Mirror Path?
> >
> > > > > > > > > Christian
> >
> > > > > > > > > --
> > > > > > > > > Christian Hammond - chip...@chipx86.com
> > > > > > > > > Review Board -http://www.reviewboard.org
> > > > > > > > > VMware, Inc. -http://www.vmware.com
> >
> > > > > > > > > On Tue, Jun 15, 2010 at 2:05 PM, Kunjal <
> kunjal.par...@gmail.com>
> > > > > wrote:
> > > > > > > > > > Hi chris,
> >
> > > > > > > > > > I have reviewborad 1.5Beta 2 configured for GIT.
> > > > > > > > > > When I create review request, I get below error. any
> pointers?
> >
> > > > > > > > > > ImportError at /r/56/
> > > > > > > > > > Request Method: GET
> > > > > > > > > > Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
> > > > > > > > > > Exception Type: ImportError
> > > > > > > > > > Exception Value:
> > > > > > > > > > Exception Location:
> /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/
> >
> > > > > python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
> > > > > > > > > > scmtools/git.py in __init__, line 202
> > > > > > > > > > Python Executable: /opt/lampp/bin/python
> > > > > > > > > > Python Version: 2.5.2
> > > > > > > > > > Python Path: ['/opt/lampp/htdocs/reviewboard/conf',
> > > > > '/opt/lampp/lib/
> > > > > > > > > > python2.5/site-packages/django',
> '/opt/lampp/lib/python2.5/site-
> > > > > > > > > > packages/reviewboard',
> '/opt/lampp/lib/python2.5/site-packages/
> > > > > > > > > > MySQL_python-1.2.2-py2.5-linux-i686.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/setuptools-0.6c9-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/pytz-2009l-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/site-
> > > > > > > > > > packages/flup-1.0.3.dev_20090716-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/Pygments-1.1.1-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/django_evolution-0.0.0-py2.5.egg',
> '/opt/lampp/lib/
> > > > > > > > > > python2.5/site-packages/PIL-1.1.6-py2.5-linux-i686.egg',
> > > > > '/opt/lampp/
> >
> > > > > lib/python2.5/site-packages/simplejson-2.0.9-py2.5-linux-i686.egg',
> '/
> >
> > > > >
> opt/lampp/lib/python2.5/site-packages/python_ldap-2.3.9-py2.5-linux-
> > > > > > > > > > i686.egg',
> '/opt/lampp/lib/python2.5/site-packages/Django-1.1.1-
> > > > > > > > > > py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/site-packages/JCC-2.5-py2.5-
> > > > > > > > > > linux-i686.egg',
> > > > > '/opt/lampp/lib/python2.5/site-packages/lucene-3.0.0-
> > > > > > > > > > py2.5-linux-i686.egg',
> '/opt/lampp/lib/python2.5/site-packages/
> > > > > > > > > > RBTools-0.2-py2.5.egg',
> '/opt/lampp/lib/python2.5/site-packages/
> > > > > > > > > > ReviewBoard-1.5beta2-py2.5.egg',
> '/opt/lampp/lib/python2.5/site-
> > > > > > > > > > packages/recaptcha_client-1.0.5-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/paramiko-1.7.6-py2.5.egg',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > site-packages/pycrypto-2.0.1-py2.5-linux-i686.egg',
> > > > > '/opt/lampp/lib/
> > > > > > > > > > python2.5/site-packages/Djblets-0.6.2-py2.5.egg',
> > > > > '/opt/lampp/lib/
> > > > > > > > > > python25.zip', '/opt/lampp/lib/python2.5',
> > > > > '/opt/lampp/lib/python2.5/
> > > > > > > > > > plat-linux2', '/opt/lampp/lib/python2.5/lib-tk',
> '/opt/lampp/lib/
> > > > > > > > > > python2.5/lib-dynload',
> '/opt/lampp/lib/python2.5/site-packages']
> > > > > > > > > > Server time: Tue, 15 Jun 2010 14:04:07 -0700
> >
> > > > > > > > > > --
> > > > > > > > > > Want to help the Review Board project? Donate today at
> > > > > > > > > >http://www.reviewboard.org/donate/
> > > > > > > > > > Happy user? Let us know athttp://
> www.reviewboard.org/users/
> > > > > > > > > > -~----------~----~----~----~------~----~------~--~---
> > > > > > > > > > To unsubscribe from this group, send email to
> > > > > > > > > > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> <reviewboard%2bunsubscr...@googlegr oups.com>
> > > > > <reviewboard%2bunsubscr...@googlegr oups.com>
> > > > > > > > <reviewboard%2bunsubscr...@googlegroups.com>
> > > > > > > > > > For more options, visit this group at
> > > > > > > > > >
> http://groups.google.com/group/reviewboard?hl=en-Hidequotedtext
> > > > > -
> >
> > > > > > > > > - Show quoted text -
> >
> > > > > > > > --
> > > > > > > > Want to help the Review Board project? Donate today at
> > > > > > > >http://www.reviewboard.org/donate/
> > > > > > > > Happy user? Let us know athttp://www.reviewboard.org/users/
> > > > > > > > -~----------~----~----~----~------~----~------~--~---
> > > > > > > > To unsubscribe from this group, send email to
> > > > > > > > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> <reviewboard%2bunsubscr...@googlegr oups.com>
> > > > > <reviewboard%2bunsubscr...@googlegr oups.com>
> > > > > > > > For more options, visit this group at
> > > > > > > >
> http://groups.google.com/group/reviewboard?hl=en-Hidequotedtext -
> >
> > > > > > - Show quoted text -
> >
> > > > > --
> > > > > Want to help the Review Board project? Donate today at
> > > > >http://www.reviewboard.org/donate/
> > > > > Happy user? Let us know athttp://www.reviewboard.org/users/
> > > > > -~----------~----~----~----~------~----~------~--~---
> > > > > To unsubscribe from this group, send email to
> > > > > reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> <reviewboard%2bunsubscr...@googlegr oups.com>
> > > > > For more options, visit this group at
> > > > >http://groups.google.com/group/reviewboard?hl=en-Hidequoted text -
> >
> > > > - Show quoted text -- Hide quoted text -
> >
> > - Show quoted text -
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en
>
--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
