Re: how to review code files that are not in repository yet

2009-09-29 Thread Deniz
Christian, Where should I look into for this? is it postreview.py? thanks, Deniz On Sep 25, 3:17 pm, Christian Hammond chip...@chipx86.com wrote: You being Deniz, not Chris, unless Chris wants to take this on too. :) Christian -- Christian Hammond - chip...@chipx86.com Review Board

Re: how to review code files that are not in repository yet

2009-09-29 Thread Christian Hammond
Yep. You'll want to look at the CVSClient class, specifically the diff code. It's not too big and is hopefully mostly obvious (in terms of how the commands are constructed and all). Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. -

how to review code files that are not in repository yet

2009-09-25 Thread Deniz
Is there any way to create review request for code files that are not added into CVS repository yet? I would like to be able to send review requests for classes that I am writing first time and not yet added into CVS. thanks, Deniz --~--~-~--~~~---~--~~ You

Re: how to review code files that are not in repository yet

2009-09-25 Thread Christian Hammond
You just need to cvs add these files and then generate a diff. This model (pre-commit reviews) are actually the model we had in mind when writing Review Board. We highly recommend using post-review for diff generation and uploading. It will make life much easier. You can get this by:

Re: how to review code files that are not in repository yet

2009-09-25 Thread Deniz
Thank you, Christian. I tried cvs add but then I got the following result: cvs diff: file_name is a new entry, no comparison available I am trying review board for facilitating our code review practice. We have a primitive mechanism around cvs and bugzilla for post-commit reviews but we do not

Re: how to review code files that are not in repository yet

2009-09-25 Thread Christian Hammond
Yeah, sounds like a bug in the CVS support. If you can figure out what's causing it and contribute a fix, that would help. I won't be able to look at it for a little while. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. -

Re: how to review code files that are not in repository yet

2009-09-25 Thread Christian Hammond
You being Deniz, not Chris, unless Chris wants to take this on too. :) Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Fri, Sep 25, 2009 at 3:17 PM, Christian Hammond chip...@chipx86.comwrote: Yeah,