Hi Sapna, With the current ReviewBoard, you need to have a local copy of the git repository you are attempting to review against.
eg. git clone <parent-git-repository> I have a cron job that then does 'git pull <parent-git-repository>' every minute to keep this copy up to date, and this works reasonably well. a 1 min delay (at most) doesn't appear to cause problems as it seems that it takes most developers at least this amount of time to make their changes and create the diff for the review. If you have control over the 'upstream repo' then a 'post-commit' hook that pushes changes as they are made would be a better solution, but I haven't done this, or seen any documentation on this. (Anyone here done this? It would be good to have this in the Reviewboard documentation, or at least referenced.) Some more documentation: http://www.reviewboard.org/docs/manual/1.0/admin/management/repositories/#git Some references, after some googling. See: http://groups.google.com/group/reviewboard/browse_thread/thread/fce7abb11375cac7?pli=1 Cheers, Paul On Fri, Jul 2, 2010 at 7:37 AM, Sapna Vempati <sapnavemp...@gmail.com> wrote: > Hi Christian, > > I am trying to configure a repository on our RB test server, and am having > trouble filling these fields below, > > When I am under--- admin ---> repositories, > > path: (It says) For git, this should be the path to the .git repository on > the local disk. > But the repo is not on the local disk, but on a central repository. Can I > have the unix path to our central server on ubuntu ? (Since I am going to > use cgit, I assume I don't have to clone the central repository to my local > ?) > > mirror path: <Blank> ? > > Raw file URL Mask: > for example: The upper-level directory url with all paths to the .gits is: > http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say under here > I need to set to this repo: git_repos/repo_tools/scripts.git) > How will I get the <filename> (can be a lot of file in this repo) & > <revision>, as suggested on your website for this field. > > Thanks for the help ! > Sapna > > On Fri, Jun 18, 2010 at 5:53 PM, Christian Hammond <chip...@chipx86.com> > wrote: >> >> 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...@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-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> >>> > > > > 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 >>> 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 > > -- > 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 -- 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
