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

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