Getting total number of reviewers and displaying on dashboard

2010-01-14 Thread daniel.j.la...@googlemail.com
All,

I have just started using this tool - I am very impressed.
I had previously written my own tool that was built on top of
Telelogic Synergy and wanted a review tool when a change in employer
resulted in a change in SCM system (SVN)

I have a number of questions:
1. Is there anyone who can help me with the necessary changes to get a
column added to the dashboard that shows the total number of people
asked to do a review.  Without this its very hard to know if X/Y
people have marked ship it.

2. As a developer I want as much stuff as possible to be automated.
Does anyone else see any value in getting review board to post the
changes to the repository?
What I mean is that the users all review the code and then the author
changes the status to 'submitted' - at this point review board could
apply the patches or change set to the repository.
If this works great, if it does not inform the user that further
rework is required.

I have no idea how hard this would be! But it means that developers
write code, get it reviewed, mark review as complete and changes are
posted.  Otherwise these is that danger period where you get code
reviewed, every one accepts and you then try to commit and fail.

Great tool, and I have high hopes of rolling this out to other
developers and teams!
Cheers
Dan
-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Install reviewboard on IIS 7

2010-01-14 Thread slav
just to follow up.

was sleepy. I created directory by hand, and that's what gave the
error. rb-site creates directory on it's own.

still couldn't get it to run on iis. I tried several approaches, main
one looks like: 
http://code.djangoproject.com/wiki/DjangoOnWindowsWithIISAndSQLServer

almost positive it's possible to get it to run on iis 7, but can't
quite figure out how. any help would be appreciated.


On Jan 13, 10:07 pm, slav  wrote:
> hello. I'm wondering if it's possible to install reviewboard under iis
> 7, and if not, why not?
>
> if it is possible, how?
>
> I 
> followedhttp://www.reviewboard.org/docs/manual/dev/admin/installation/windows/
> and got all the way to creating a site using rb-site, but it fails
> with an error, telling me to ensure I'm admin (which I am).
>
> I'm on Windows 2008 r2 x64 and Python 2.6 x64
>
> thank you
-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

What is the status of ClearCase integration on Windows?

2010-01-14 Thread Sassan

Does anyone know what is and is not currently supported for ClearCase
as an SCM tool on Windows?

For example I see instructions for posting commited code reviews using
post-review.exe under ClearCase section in
http://www.reviewboard.org/docs/manual/1.0/users/tools/post-review/

Does the "Todo" mean it is only on the wish list and not yet
implemented?

I also see a --label= under ClearCase options for post-
review.exe but I am not clear as to how a single label can designate a
diff of two distinct revisions.

I tried this anyway and I got::

Traceback (most recent call last):
  File "C:\Python25\Scripts\post-review-script.py", line 8, in
load_entry_point('RBTools==0.2beta2', 'console_scripts',
'post-review')()
  File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py", line 2509, in main
  File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py", line 954, in diff_label
  File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py", line 947, in diff
  File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py", line 811, in get_extended_namespace
  File "C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py", line 790, in get_previous_version
ValueError: too many values to unpack

Do you know how I get to the postreview.py script inside the rbtools
egg to modify it?

On the other hand it is easy enough for me to generate the diffs
myself out of ClearCase, but is there any way for me post these
directly into the RB server ?

Maybe by passing them to post-review.exe but only to be posted to the
RB server ?

Thanks,

Sassan


-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Will post-review generate ReviewRequest for CVS removed files?

2010-01-14 Thread JohnHenry
Hi, all.
I am trying using post-review to generate review request for cvs
removed files. Once a file has been removed with "cvs remove file "
"cvs commit", Then the new version will be the removed version.
Suppose the file version before remove is V1.1 , Then after execution
"cvs remove" , It will be V1.2; I use the following command:
post-review --target-people=rbroot --submit-as=new rb --
summary="Testing wrong" --file-name=new.c --revision-range=1.1:1.2 -
p
But the execution result is:
Error uploading diff: One or more fields had errors (105)
Your review request still exists, but the diff is not attached.
"""
>>> {"stat": "ok"}
>>> Review request draft saved
>>> Uploading diff, size: 223
>>> HTTP POSTing to 
>>> http://192.168.3.20:8080/api/json/reviewrequests/204/diff/new/: {}
>>> {"fields": {"path": ["Unable to parse diff revision header 'Fri Jan 15 
>>> 11:57:39 2010'"]}, "stat": "fail", "err": {"msg": "One or more fields had 
>>> errors", "code": 105}}
Error uploading diff: One or more fields had errors (105)
"""
What is the "Unable to parse diff revision header " mean?

Best Regards!
-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en