Re: post-review and CVS

2013-10-04 Thread Christian Hammond
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

Re: post-review and CVS

2013-10-04 Thread Luiz Henrique Teixeira Balloti
Hello Christian, Thanks for responding. The repository has a single CVSROOT directory, inside /srv/cvs, along with all the modules, like so: /srv/cvs \- CVSROOT \- productA \- productB Therefore I'm not sure how to create the entries for each CVSROOT as you suggested. On Friday,

post-review and CVS

2013-10-02 Thread Luiz Henrique Teixeira Balloti
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

Re: Post-review thinks CVS diff file is empty

2013-07-02 Thread James Oravec
Hi Christian, Did you guys ever figure out the issue with this? I'm receiving the same errors both on the client and server side. Any help would be greatly appreciated. Thanks, James On Monday, September 26, 2011 4:59:39 PM UTC-7, Christian Hammond wrote: Hi Matthew, So I've seen this

Re: Post-review thinks CVS diff file is empty

2013-07-02 Thread Christian Hammond
Hi James, Can you show me the Error uploading diff error when using post-review with --debug? Christian James Oravec wrote: Got past this issue. It was related to my connection string in my CVS directory's Root file. I was modifying it, trying to get past a different error. The summary is

Re: Post-review thinks CVS diff file is empty

2013-07-02 Thread Christian Hammond
Hi James, The command Review Board executes to get the file is: cvs -f -d cvsroot checkout -r revision -p filename In your case: cvs -f -d CVS-SERVER-DNS-NAME:/home/cvsroot/ checkout -r 1.3 -p /home/cvsroot/path/on/cvs/server/blob.cpp (I believe -- something like that.) Can you

Re: Post-review thinks CVS diff file is empty

2013-07-02 Thread Christian Hammond
Hi James, It will do this automatically, if your repository path in Review Board is configured correctly. I'll need to see what it is to diagnose this. Christian James Oravec wrote: Hi Christian, I reverted the file to get rid of the conflict then I made another simple change. After doing

Unable to post review from cvs commitinfo trigger

2012-08-27 Thread cmuser
Hi, Opening a fresh post to avoid any old mess..need urgent help. I am trying to use post review with every cvs commit. Applying a trigger in cvs commitinfo file as follows $cat commitinfo /usr/bin/post-review --server=http://172.16.7.100:81 -- username=admin --password=admin --submit

Re: Unable to post review from cvs commitinfo trigger

2012-08-27 Thread cmuser
kapilanar...@gmail.com wrote: Hi,   Opening a fresh post to avoid any old mess..need urgent help. I am trying to use post review with every cvs commit. Applying a trigger in cvs commitinfo file as follows $cat commitinfo /usr/bin/post-review  --server=http://172.16.7.100:81  -- username=admin

Re: Unable to post review from cvs commitinfo trigger

2012-08-27 Thread Christian Hammond
repository could be accessed at the supplied url. On Aug 27, 2:52 pm, cmuser kapilanar...@gmail.com wrote: Hi, Opening a fresh post to avoid any old mess..need urgent help. I am trying to use post review with every cvs commit. Applying a trigger in cvs commitinfo file as follows $cat

Re: Using post-review with CVS commitinfo

2012-01-20 Thread zhibin wang
reviewed. just have a test. thank you. On 1月12日, 上午4时52分, Tom tom.o...@gmail.com wrote: I need help with a problem with post-review when it's invoked within commitinfo under CVS. CVS and RB 1.6.3 are running on separate Linux servers. RBtools are installed on the CVS server. I have an

Using post-review with CVS commitinfo

2012-01-13 Thread Tom
I need help with a problem with post-review when it's invoked within commitinfo under CVS. CVS and RB 1.6.3 are running on separate Linux servers. RBtools are installed on the CVS server. I have an apparently valid repository definition in the RB admin web interface, the path for which is:

Re: Post-review thinks CVS diff file is empty

2011-09-26 Thread Christian Hammond
Hi Matthew, It's entirely possible that that error is misleading. Looking at the code, it's possible that the same error code could be generated when there's also an unexpected error on Review Board's end. If you have access to the reviewboard server logs, look for a line starting with Error

Re: Post-review thinks CVS diff file is empty

2011-09-26 Thread Matthew
I appreciate the quick help. After enabling logging and running the commands again I get the output below. I notice the absolute module reference points into the cvs repository. After checking in the repository, there is no Version.java, although there is Version.java,v. I don't know if it's

Re: Post-review thinks CVS diff file is empty

2011-09-26 Thread Christian Hammond
Hi Matthew, So I've seen this issue mentioned before with Review Board, but also with CVS in general. It's possible it's us. Can you perhaps privately e-mail me with the Path and Mirror Path field info on your Repository entry? Here's a link to someone who's hit it with just plain ol' CVS and