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...@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...@googlegr > > > > > > oups.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 - > > > > -- > > > 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...@googlegr > > > oups.com> > > > For more options, visit this group at > > >http://groups.google.com/group/reviewboard?hl=en-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 For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en