Re: Issue 1566 in reviewboard: Posting new diff returns error page even though it seems to work

2010-09-26 Thread reviewboard

Updates:
Status: Incomplete

Comment #2 on issue 1566 by trowbrds: Posting new diff returns error page  
even though it seems to work

http://code.google.com/p/reviewboard/issues/detail?id=1566

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1603 in reviewboard: Encoded URLs are not handled correctly

2010-09-26 Thread reviewboard

Updates:
Status: Incomplete

Comment #3 on issue 1603 by trowbrds: Encoded URLs are not handled correctly
http://code.google.com/p/reviewboard/issues/detail?id=1603

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1737 in reviewboard: Exception is thrown when adding a private GitHub repository

2010-09-26 Thread reviewboard

Updates:
Status: SetupIssue

Comment #9 on issue 1737 by trowbrds: Exception is thrown when adding a  
private GitHub repository

http://code.google.com/p/reviewboard/issues/detail?id=1737

I think every shared host is different in that regards; if there's  
something janky in their apache/python installs, options are pretty  
limited. I'm not sure there's anything we can document except specific  
workarounds for specific hosts, which seems like it's not worth the trouble.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1748 in reviewboard: error posting to review board

2010-09-26 Thread reviewboard

Updates:
Status: Incomplete

Comment #2 on issue 1748 by trowbrds: error posting to review board
http://code.google.com/p/reviewboard/issues/detail?id=1748

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1811 in reviewboard: Failing to publish each diff comment forgets them when publishing the whole diff.

2010-09-26 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 831

Comment #2 on issue 1811 by trowbrds: Failing to publish each diff comment  
forgets them when publishing the whole diff.

http://code.google.com/p/reviewboard/issues/detail?id=1811

This actually sounds exactly like bug 831.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 831 in reviewboard: Can't leave comments before you publish your own newly created review request

2010-09-26 Thread reviewboard


Comment #19 on issue 831 by trowbrds: Can't leave comments before you  
publish your own newly created review request

http://code.google.com/p/reviewboard/issues/detail?id=831

Issue 1811 has been merged into this issue.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1789 in reviewboard: .rc diff appears incorrectly on Firefox

2010-09-26 Thread reviewboard

Updates:
Status: Incomplete

Comment #3 on issue 1789 by trowbrds: .rc diff appears incorrectly on  
Firefox

http://code.google.com/p/reviewboard/issues/detail?id=1789

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1805 in reviewboard: Error while uploading diff

2010-09-26 Thread reviewboard

Updates:
Status: Incomplete

Comment #5 on issue 1805 by trowbrds: Error while uploading diff
http://code.google.com/p/reviewboard/issues/detail?id=1805

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1572 in reviewboard: git repository matching is too strict (should ignore username)

2010-09-26 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 1803

Comment #1 on issue 1572 by trowbrds: git repository matching is too strict  
(should ignore username)

http://code.google.com/p/reviewboard/issues/detail?id=1572

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1803 in reviewboard: post-review : git http or https remote with username name in remote url.

2010-09-26 Thread reviewboard


Comment #6 on issue 1803 by trowbrds: post-review : git http or https  
remote with username name in remote url.

http://code.google.com/p/reviewboard/issues/detail?id=1803

Issue 1572 has been merged into this issue.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1155 in reviewboard: links in request list drop other customizations

2010-09-26 Thread reviewboard

Updates:
Labels: Component-Dashboard

Comment #2 on issue 1155 by trowbrds: links in request list drop other  
customizations

http://code.google.com/p/reviewboard/issues/detail?id=1155

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1127 in reviewboard: post-review creates '\' in diff pathname and it failed view diff on unix repository

2010-09-26 Thread reviewboard

Updates:
Labels: Component-RBTools

Comment #4 on issue 1127 by trowbrds: post-review creates '\' in diff  
pathname and it failed view diff on unix repository

http://code.google.com/p/reviewboard/issues/detail?id=1127

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1166 in reviewboard: view diff failed for files contain Mac newline char (i.e. CR \r) -- use SVN client

2010-09-26 Thread reviewboard

Updates:
Labels: Component-RBTools

Comment #9 on issue 1166 by trowbrds: view diff failed for files contain  
Mac newline char (i.e. CR \r)  -- use SVN client

http://code.google.com/p/reviewboard/issues/detail?id=1166

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1191 in reviewboard: post-review with no arguments generated an empty review request

2010-09-26 Thread reviewboard

Updates:
Labels: Component-RBTools

Comment #5 on issue 1191 by trowbrds: post-review with no arguments  
generated an empty review request

http://code.google.com/p/reviewboard/issues/detail?id=1191

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1626 in reviewboard: ClearCase doesn't support directory changes

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 1626 by trowbrds: ClearCase doesn't support directory  
changes

http://code.google.com/p/reviewboard/issues/detail?id=1626

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1414 in reviewboard: gt; expanded to

2010-09-26 Thread reviewboard

Updates:
Labels: Component-Reviews

Comment #1 on issue 1414 by trowbrds: gt; expanded to 
http://code.google.com/p/reviewboard/issues/detail?id=1414

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1421 in reviewboard: post-review doesn't gracefully handle ClearCase deployments where VOBs are mounted as subdirectories of /vobs

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #4 on issue 1421 by trowbrds: post-review doesn't gracefully handle  
ClearCase deployments where VOBs are mounted as subdirectories of /vobs

http://code.google.com/p/reviewboard/issues/detail?id=1421

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1465 in reviewboard: post-review with revision-range posts all files in tree not the ones listed

2010-09-26 Thread reviewboard

Updates:
Status: NotABug

Comment #1 on issue 1465 by trowbrds: post-review with revision-range posts  
all files in tree not the ones listed

http://code.google.com/p/reviewboard/issues/detail?id=1465

There's not really such a thing as mixing --revision-range and file lists.  
It's one or the other.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1607 in reviewboard: post-review parameter to reopen discarded review

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #1 on issue 1607 by trowbrds: post-review parameter to reopen  
discarded review

http://code.google.com/p/reviewboard/issues/detail?id=1607

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1597 in reviewboard: Provide --repository-urlfor ClearCase

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #3 on issue 1597 by trowbrds: Provide --repository-urlfor ClearCase
http://code.google.com/p/reviewboard/issues/detail?id=1597

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1598 in reviewboard: Provide --cc-root parameter for Clear Case

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 1598 by trowbrds: Provide --cc-root parameter for Clear  
Case

http://code.google.com/p/reviewboard/issues/detail?id=1598

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1651 in reviewboard: New Review Request screen does not offer option to attach a parent diff to a git repo patch.

2010-09-26 Thread reviewboard

Updates:
Status: Fixed

Comment #1 on issue 1651 by trowbrds: New Review Request screen does not  
offer option to attach a parent diff to a git repo patch.

http://code.google.com/p/reviewboard/issues/detail?id=1651

This is fixed in 1.5.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1731 in reviewboard: Can not create new review when no repository is set

2010-09-26 Thread reviewboard

Updates:
Status: NotABug

Comment #1 on issue 1731 by trowbrds: Can not create new review when no  
repository is set

http://code.google.com/p/reviewboard/issues/detail?id=1731

There's no such thing as a diff without a repository (we wouldn't be able  
to show the side-by-side view without it). Someone got TFS working with a  
third-party product called SVNBridge. His process is documented here:  
http://blogs.pinet.ca/tim/2010/08/24/code-review-goodness-with-reviewboard/


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1651 in reviewboard: New Review Request screen does not offer option to attach a parent diff to a git repo patch.

2010-09-26 Thread reviewboard

Updates:
Status: Confirmed
Labels: Component-Reviews Milestone-Release1.5.x

Comment #2 on issue 1651 by chipx86: New Review Request screen does not  
offer option to attach a parent diff to a git repo patch.

http://code.google.com/p/reviewboard/issues/detail?id=1651

Are you sure? I think it's still broken :(

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1674 in reviewboard: LDAP: username in django admin has too strict of validation rules

2010-09-26 Thread reviewboard


Comment #2 on issue 1674 by john.p.c...@gmail.com: LDAP:  username in  
django admin has too strict of validation rules

http://code.google.com/p/reviewboard/issues/detail?id=1674

I believe part of the point was missed.  The auto account creation for ldap  
users actually creates the invalid account name, that then can no longer be  
edited in the admin view.


I would suggest a code modification on the reviewboard ldap/login side that  
automatically strips @.*$ from the username.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1478 in reviewboard: Username handling for case-insensitive LDAP login

2010-09-26 Thread reviewboard


Comment #2 on issue 1478 by john.p.c...@gmail.com: Username handling for  
case-insensitive LDAP login

http://code.google.com/p/reviewboard/issues/detail?id=1478

For this and issue 1674, I would suggest perhaps allowing for a user  
supplied function to process the login after form submittal.  An admin  
could create a simple function that calls lc() on the username.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.