Hi Luiz, Sounds like a configuration issue.
You should have a repository entry for every CVSROOT you're working with. It sounds like you're instead creating one for the parent of the CVS modules? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Wed, Oct 2, 2013 at 11:42 AM, Luiz Henrique Teixeira Balloti < lhball...@gmail.com> wrote: > Hello! > > I'm a little confused with the way Reviewboard works with a CVS repository. > > The repository sits at /srv/cvs and there are a number of modules inside > it, one for each software my company develops; I'm planning to use > Reviewboard and automate review request creation for a couple of modules > only. > > In my test setup, Reviewboard runs in the same machine which hosts the CVS > repository, so the repository path in Reviewboard is configured as > /srv/cvs. On that same machine, I checked out a module "mod" to > /home/user/src/mod, modified a file and ran post-review in that directory > (the module "root"). The review request is created successfully, but when > viewing the diffs, the list of files shows something lilke: > > Files Changed: > > src/file.properties (was mod/src/file.properties): 4 changes [ 1 2 3 4 ] > > > It seems Reviewboard thinks the file name changed! If I generate the diff > myself with cvs diff and prepend the module name to each path, so that > > --- src/file.properties <timestamp> 1.1 > +++ src/file.properties <timestamp> > > becomes > > --- mod/src/file.properties <timestamp> 1.1 > +++ mod/src/file.properties <timestamp> > > and then post the review request, the changed files list only shows the > file name. > > In both diffs, the "RCS file" line is correct: > /srv/cvs/mod/src/file.properties > > So, can I simply ignore this? Is the CVS repository configured incorrectly? > > Thanks in advance for any clarification. > > -- > Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ > --- > Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ > --- > Happy user? Let us know at http://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/groups/opt_out. > -- Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ --- Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ --- Happy user? Let us know at http://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/groups/opt_out.
