Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread sakthi
Hi Chris, I accept what you said, I can see some logic there. I dont want to load test scripts and docs for review.. I will have to find a way.. thanks again. sakthi On Aug 24, 2:03 am, Chris Clark chris.cl...@ingres.com wrote: sakthi wrote: Could find i have to use post-review tool to post

Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread Jan Koprowski
There could be -exclude or -include switch to post-review but I think better way is keep all in two separate checkouts which appropriate ignore rulez - one for docs second for source code. On Tue, Aug 24, 2010 at 8:11 AM, sakthi listsak...@gmail.com wrote: Hi Chris, I accept what you said, I can

Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread Christian Hammond
If you can build the file list separately of *.c and *.h files, you can just pass those to post-review. It takes file paths and lists of files. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Mon, Aug 23,

Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread sakthi
Hello Christian, Can you tell me how to mention the path? I am stuck up.. i am getting this error as the repository contains doc files.. Failed to execute command: ['svn', 'info', 'Design Specs/ s1020_design.doc'] May i know how to mention only the path that is required? This is the sample

Re: keeping track of large number of files

2010-08-24 Thread Scott Quesnelle
John, This isn't a software answer, but if the changes is that large, then we encourage the author to try and split it into pieces which provide a part of the whole solution. I.e a review of the server side change, another with all the database related changes, and a last one with the client side

Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread sakthi
I tried to give a path that resides inside the repository.. I get Got API Error 207 (HTTP code 200): The file was not found in the repository post-review doesnt take path inside repository? Do i have to touch the file which have to be reviewed and check in to post into review board? Kindly

Re: post-review tool: How to post only *.c and *.h files

2010-08-24 Thread sakthi
does review board handle if the files are deleted and new file that is added in the repository, while doing --revision-range? On Aug 24, 7:29 pm, sakthi listsak...@gmail.com wrote: I tried to give a path that resides inside the repository.. I get Got API Error 207 (HTTP code 200): The file was

ReviewBoard and TFS - working

2010-08-24 Thread Timothy Pinet
Good afternoon, It has taken me some time but I have ReviewBoard working with our TFS repository at my workplace. I want to post here so that others can get TFS working or refine my process. I have posted to my blog all of my findings.

Re: ReviewBoard and TFS - working

2010-08-24 Thread David Trowbridge
This is an awesome article! I'm super impressed to see TFS working, even if it's by doing an end-run around TFS itself. Thanks so much for this detailed guide. -David On Tue, Aug 24, 2010 at 11:32 AM, Timothy Pinet tim.pi...@gmail.com wrote: Good afternoon, It has taken me some time but I

reviewboard.settings lost?

2010-08-24 Thread tag_98007work
I've been running Reviewboard 1.5 RC1 for about a week. According to the logs sometime Sunday morning there was a restart, after that I'm getting the messages: Error log: [Sun Aug 22 08:03:41 2010] [notice] SIGUSR1 received. Doing graceful restart

Re: reviewboard.settings lost?

2010-08-24 Thread Christian Hammond
reviewboard.settings is part of Review Board itself. It could be that your install is broken or your web server is pointing to the wrong path. The file itself does not change, so the syntax errors warning shouldn't be a concern. Another thing to make sure is that Review Board is installed for

Re: Issue 1784 in reviewboard: user setting for plaintext emails

2010-08-24 Thread reviewboard
Comment #2 on issue 1784 by latchkey: user setting for plaintext emails http://code.google.com/p/reviewboard/issues/detail?id=1784 Great response! I totally should have thought of that before I posted. You are absolutely right. Thanks! -- You received this message because you are subscribed

Issue 1785 in reviewboard: customizable logo

2010-08-24 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 1785 by latchkey: customizable logo http://code.google.com/p/reviewboard/issues/detail?id=1785 1.5rc1 As absurd as it sounds, we would like to be able to easily customize the logo image and width/height that shows up

Issue 1786 in reviewboard: Cannot change Server name under Site Settings

2010-08-24 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1786 by wleibe: Cannot change Server name under Site Settings http://code.google.com/p/reviewboard/issues/detail?id=1786 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.5 RC 1

Re: Issue 1750 in reviewboard: Unable to dd svn+ssh repository when host requires rsa cert auth disallows password auth

2010-08-24 Thread reviewboard
Comment #9 on issue 1750 by roy.jean.francois: Unable to dd svn+ssh repository when host requires rsa cert auth disallows password auth http://code.google.com/p/reviewboard/issues/detail?id=1750 The server I am trying to connect to does allow password authentication, so it definitely