Re: Issue 1793 in reviewboard: Authentication failure with full gecos

2010-08-30 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 1793 by chipx86: Authentication failure with full gecos http://code.google.com/p/reviewboard/issues/detail?id=1793 This doesn't really show any code in Review Board, so I'm not able to see what's going on. Where did that specific error in

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

2010-08-30 Thread reviewboard
Updates: Labels: -Type-Defect -Milestone-Release1.5 Type-Support Comment #7 on issue 1737 by chipx86: Exception is thrown when adding a private GitHub repository http://code.google.com/p/reviewboard/issues/detail?id=1737 (No comment was entered for this change.) -- You received this

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

2010-08-30 Thread reviewboard
Comment #6 on issue 1737 by chipx86: Exception is thrown when adding a private GitHub repository http://code.google.com/p/reviewboard/issues/detail?id=1737 Okay, it's a problem with your installation then. Do you remember what steps you did to install it? And was the database from this fres

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

2010-08-30 Thread reviewboard
Comment #5 on issue 1737 by kevin.go...@gmail.com: Exception is thrown when adding a private GitHub repository http://code.google.com/p/reviewboard/issues/detail?id=1737 Adding Subversion doesn't work either. Tool is set to '1' in this case. , POST:u'bug_tracker_type': [u'none'], u'encoding'

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

2010-08-30 Thread reviewboard
Comment #4 on issue 1737 by kevin.go...@gmail.com: Exception is thrown when adding a private GitHub repository http://code.google.com/p/reviewboard/issues/detail?id=1737 Adding a normal git repository doesn't work either, with the same error: , POST:u'bug_tracker_type': [u'none'], u'encoding

Re: Issue 1727 in reviewboard: post-review wrongly detects client class if few source control tools installed on server (e.g mercurial and clearcase)

2010-08-30 Thread reviewboard
Comment #1 on issue 1727 by trowbrds: post-review wrongly detects client class if few source control tools installed on server (e.g mercurial and clearcase) http://code.google.com/p/reviewboard/issues/detail?id=1727 Issue 1704 has been merged into this issue. -- You received this message b

Re: Issue 1704 in reviewboard: Problem with clearcase and post-review....

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1727 Comment #3 on issue 1704 by trowbrds: Problem with clearcase and post-review http://code.google.com/p/reviewboard/issues/detail?id=1704 Issue 1727 has a deeper analysis of what's going on. -- You received this message because yo

Re: Issue 1704 in reviewboard: Problem with clearcase and post-review....

2010-08-30 Thread reviewboard
Comment #2 on issue 1704 by trowbrds: Problem with clearcase and post-review http://code.google.com/p/reviewboard/issues/detail?id=1704 Issue 1711 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To

Re: Issue 1711 in reviewboard: Bug in RBTools with Clearcase and post-review

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1704 Comment #1 on issue 1711 by trowbrds: Bug in RBTools with Clearcase and post-review http://code.google.com/p/reviewboard/issues/detail?id=1711 (No comment was entered for this change.) -- You received this message because you are su

Re: Issue 1707 in reviewboard: Description should not be required

2010-08-30 Thread reviewboard
Updates: Labels: -Type-Defect Type-Enhancement Component-Reviews Comment #1 on issue 1707 by trowbrds: Description should not be required http://code.google.com/p/reviewboard/issues/detail?id=1707 (No comment was entered for this change.) -- You received this message because you are sub

Issue 1793 in reviewboard: Authentication failure with full gecos

2010-08-30 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1793 by alleonik: Authentication failure with full gecos http://code.google.com/p/reviewboard/issues/detail?id=1793 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.0.5.1 What

Re: Issue 1772 in reviewboard: Allow reviewers to check off files they've looked at

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 774 Comment #2 on issue 1772 by trowbrds: Allow reviewers to check off files they've looked at http://code.google.com/p/reviewboard/issues/detail?id=1772 (No comment was entered for this change.) -- You received this message because you

Re: Issue 774 in reviewboard: Review tracker for user

2010-08-30 Thread reviewboard
Comment #5 on issue 774 by trowbrds: Review tracker for user http://code.google.com/p/reviewboard/issues/detail?id=774 Issue 1772 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, sen

Re: Issue 1338 in reviewboard: Allow repository-id to be used as a parameter instead of repository-path

2010-08-30 Thread reviewboard
Updates: Labels: Component-RBTools PendingReview Comment #2 on issue 1338 by trowbrds: Allow repository-id to be used as a parameter instead of repository-path http://code.google.com/p/reviewboard/issues/detail?id=1338 (No comment was entered for this change.) -- You received this me

Re: Issue 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-08-30 Thread reviewboard
Comment #12 on issue 1431 by trowbrds: Need support for TortoiseSVN diffs http://code.google.com/p/reviewboard/issues/detail?id=1431 Issue 1608 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To post to t

Re: Issue 1608 in reviewboard: Valid looking patch did not apply cleanly

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1431 Comment #1 on issue 1608 by trowbrds: Valid looking patch did not apply cleanly http://code.google.com/p/reviewboard/issues/detail?id=1608 (No comment was entered for this change.) -- You received this message because you are subscr

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

2010-08-30 Thread reviewboard
Updates: Status: NeedInfo Comment #1 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 Sorry, this seems to have slipped through the cracks. Do you have the full error and traceback?

Re: Issue 1247 in reviewboard: Support providing my own diff rather than have postreview always generate it for me.

2010-08-30 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 1247 by trowbrds: Support providing my own diff rather than have postreview always generate it for me. http://code.google.com/p/reviewboard/issues/detail?id=1247 (No comment was entered for this change.) -- You received this message because

Re: Issue 477 in reviewboard: Per reviewer email control

2010-08-30 Thread reviewboard
Comment #13 on issue 477 by trowbrds: Per reviewer email control http://code.google.com/p/reviewboard/issues/detail?id=477 Issue 1214 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,

Re: Issue 1214 in reviewboard: Make the email script smarter so it does not email the user that issued the action for the email.

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 477 Comment #3 on issue 1214 by trowbrds: Make the email script smarter so it does not email the user that issued the action for the email. http://code.google.com/p/reviewboard/issues/detail?id=1214 This is covered in issue 477. -- You r

Re: Issue 1106 in reviewboard: Markup for text fields

2010-08-30 Thread reviewboard
Updates: Status: WontFix Comment #7 on issue 1106 by trowbrds: Markup for text fields http://code.google.com/p/reviewboard/issues/detail?id=1106 The more I think about this, the more it seems like it's way too much complexity. SCMs generally don't let you use markups in change descript

Re: Issue 861 in reviewboard: Review dialog is XBox-huge can't be resized, and can't be moved

2010-08-30 Thread reviewboard
Updates: Status: WontFix Comment #4 on issue 861 by trowbrds: Review dialog is XBox-huge can't be resized, and can't be moved http://code.google.com/p/reviewboard/issues/detail?id=861 I'm going to close this out for now. We haven't heard of any other demand for changing this, so unl

Re: Issue 1779 in reviewboard: Better error messaging when running post-review on an empty change

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 832 Comment #1 on issue 1779 by trowbrds: Better error messaging when running post-review on an empty change http://code.google.com/p/reviewboard/issues/detail?id=1779 (No comment was entered for this change.) -- You received this messag

Re: Issue 832 in reviewboard: post-review for perforce. don't submit if changelist is empty

2010-08-30 Thread reviewboard
Comment #2 on issue 832 by trowbrds: post-review for perforce. don't submit if changelist is empty http://code.google.com/p/reviewboard/issues/detail?id=832 Issue 1779 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-

Re: Issue 1783 in reviewboard: how to configure LDAP with group authentication

2010-08-30 Thread reviewboard
Updates: Status: NotABug Comment #1 on issue 1783 by trowbrds: how to configure LDAP with group authentication http://code.google.com/p/reviewboard/issues/detail?id=1783 This doesn't really seem like a bug report, but more of a support request. I'm not sure if what you want to do is

Re: Issue 1456 in reviewboard: Uses should be able to remove reviewed changes from my Inbox

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1453 Comment #4 on issue 1456 by trowbrds: Uses should be able to remove reviewed changes from my Inbox http://code.google.com/p/reviewboard/issues/detail?id=1456 I'm going to turn 1453 into a kind of master bug for this. -- You received

Re: Issue 1453 in reviewboard: Show pending reviews on the dashboard page

2010-08-30 Thread reviewboard
Comment #3 on issue 1453 by trowbrds: Show pending reviews on the dashboard page http://code.google.com/p/reviewboard/issues/detail?id=1453 Issue 1456 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To

Re: Issue 1453 in reviewboard: Show pending reviews on the dashboard page

2010-08-30 Thread reviewboard
Comment #2 on issue 1453 by trowbrds: Show pending reviews on the dashboard page http://code.google.com/p/reviewboard/issues/detail?id=1453 Issue 904 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To

Re: Issue 904 in reviewboard: Remove reviews from my Dashboard that I have chosen "Ship It" for

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1453 Comment #5 on issue 904 by trowbrds: Remove reviews from my Dashboard that I have chosen "Ship It" for http://code.google.com/p/reviewboard/issues/detail?id=904 I'm going to merge this with 1453, which I think is a better bigger pict

Re: Issue 1453 in reviewboard: Show pending reviews on the dashboard page

2010-08-30 Thread reviewboard
Comment #1 on issue 1453 by trowbrds: Show pending reviews on the dashboard page http://code.google.com/p/reviewboard/issues/detail?id=1453 Issue 1775 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To

Re: Issue 1775 in reviewboard: "Pending action" view on the dashboard

2010-08-30 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1453 Comment #2 on issue 1775 by trowbrds: "Pending action" view on the dashboard http://code.google.com/p/reviewboard/issues/detail?id=1775 (No comment was entered for this change.) -- You received this message because you are subscribed t

Re: Issue 1766 in reviewboard: Automatically ping reviewers after a configurable amount of time

2010-08-30 Thread reviewboard
Updates: Status: WontFix Comment #1 on issue 1766 by trowbrds: Automatically ping reviewers after a configurable amount of time http://code.google.com/p/reviewboard/issues/detail?id=1766 Thanks for your suggestion. I disagree with the request, though. People already think we send to

Re: Issue 1765 in reviewboard: Git should show the use ra Permission Denied error when using local repositories with invalid permissions

2010-08-30 Thread reviewboard
Comment #5 on issue 1765 by trowbrds: Git should show the use ra Permission Denied error when using local repositories with invalid permissions http://code.google.com/p/reviewboard/issues/detail?id=1765 I don't actually see how it would be possible to give a better error. All we really know

Re: Issue 1762 in reviewboard: Publish/Discard buttons don't load with Safari 5.0.1

2010-08-30 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 1762 by trowbrds: Publish/Discard buttons don't load with Safari 5.0.1 http://code.google.com/p/reviewboard/issues/detail?id=1762 Please look in Safari's debug tools to see if there are any errors. For what it's worth, other webkit-based

Re: Issue 1756 in reviewboard: Download Diff button doesn't download the diff file in Safari

2010-08-30 Thread reviewboard
Updates: Status: ThirdParty Comment #1 on issue 1756 by trowbrds: Download Diff button doesn't download the diff file in Safari http://code.google.com/p/reviewboard/issues/detail?id=1756 We give the file type as text/x-patch (which is correct). If safari wants to display that, there

Re: Issue 1755 in reviewboard: LDAP does not appear to support multiple login names for the same account (common with Mac OS X Server)

2010-08-30 Thread reviewboard
Updates: Status: WontFix Comment #1 on issue 1755 by trowbrds: LDAP does not appear to support multiple login names for the same account (common with Mac OS X Server) http://code.google.com/p/reviewboard/issues/detail?id=1755 While the LDAP server may authenticate against it, Review B

Re: Issue 1463 in reviewboard: reviewboard's post-review does not support p4 move

2010-08-30 Thread reviewboard
Updates: Status: Fixed Owner: chipx86 Labels: Component-RBTools Milestone-RBTools-Release1.0 Comment #1 on issue 1463 by chipx86: reviewboard's post-review does not support p4 move http://code.google.com/p/reviewboard/issues/detail?id=1463 This bug's a bit old, but thi

Re: Issue 1399 in reviewboard: Mark review as "submitted" only if all remarks are commented

2010-08-30 Thread reviewboard
Updates: Status: WontFix Comment #2 on issue 1399 by trowbrds: Mark review as "submitted" only if all remarks are commented http://code.google.com/p/reviewboard/issues/detail?id=1399 I think defect tracking would address this in a better way. I don't really like imposing too much st