Issue 3438 in reviewboard: registration notification false positive?

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3438 by matthias...@famsik.de: registration notification false positive? http://code.google.com/p/reviewboard/issues/detail?id=3438 *** READ THIS BEFORE POSTING! *** *** You must complete this form in its entirety, or your

Issue 3439 in reviewboard: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE'

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3439 by david.ga...@gmail.com: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE' http://code.google.com/p/reviewboard/issues/detail?id=3439 What version are you running? RBTools 6.0.0 What's the URL of the

Re: Issue 3439 in reviewboard: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE'

Comment #1 on issue 3439 by david.ga...@gmail.com: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE' http://code.google.com/p/reviewboard/issues/detail?id=3439 I cannot edit the description, but the proper steps to reproduce is: 1. 'rbt post -d file1@@/main/1:file2@@/main/2'

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #1 on issue 3433 by determin...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 hi, i got the same issue. It's good I checked it's already reported. Here is a dirty workaround to unblock you while waiting for the

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #2 on issue 3433 by nick.mai...@rockwellcollins.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 Thanks! Actually have the exact same hack for now and it seems to be doing the trick. -- You received this message because

Issue 3440 in reviewboard: AssertionError displaying changes from diff rev 1 to 2

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3440 by bruce.c...@gmail.com: AssertionError displaying changes from diff rev 1 to 2 http://code.google.com/p/reviewboard/issues/detail?id=3440 What version are you running? 2.0.1 What's the URL of the page containing the

Re: Issue 3440 in reviewboard: AssertionError displaying changes from diff rev 1 to 2

Updates: Status: NeedInfo Comment #1 on issue 3440 by chip...@gmail.com: AssertionError displaying changes from diff rev 1 to 2 http://code.google.com/p/reviewboard/issues/detail?id=3440 Huh, haven't seen that before. I'll unfortunately need the diff in question in order to be able

Re: Issue 3427 in reviewboard: ValueError: too many values to unpack when upload a pdf file

Comment #3 on issue 3427 by chip...@gmail.com: ValueError: too many values to unpack when upload a pdf file http://code.google.com/p/reviewboard/issues/detail?id=3427 I don't know off-hand, but the very first thing I'd want you to do before looking into it further is to upgrade to a modern

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Updates: Status: NeedInfo Comment #3 on issue 3433 by chip...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 So the CVS server and RB are on the same box, with the repository configured as a local path? -- You

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #4 on issue 3433 by determin...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 hi, in my case they are on separate servers. RBT used to work very good and I was able to see created review requests and their diffs.

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #5 on issue 3433 by chip...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 We do normalize paths now to work around issues elsewhere. Seems like CVS broke along with that. Would you be able to supply a formal

Re: Issue 2269 in reviewboard: Allow inline review for non-text file types

Comment #12 on issue 2269 by chip...@gmail.com: Allow inline review for non-text file types http://code.google.com/p/reviewboard/issues/detail?id=2269 The original report was about adding a feature to do review of text files, which we did not have before. We never closed the bug when we

Re: Issue 3432 in reviewboard: Empty files are not pushed to review board, nor pulled with 'patch'

Comment #1 on issue 3432 by chip...@gmail.com: Empty files are not pushed to review board, nor pulled with 'patch' http://code.google.com/p/reviewboard/issues/detail?id=3432 We have this fixed in Review Board. We're waiting to do the big update on RBCommons, and then this will work for Git

Re: Issue 3434 in reviewboard: Search results don't include file names.

Updates: Status: NeedInfo Comment #1 on issue 3434 by chip...@gmail.com: Search results don't include file names. http://code.google.com/p/reviewboard/issues/detail?id=3434 Try prefixing the file path with 'file:' and see if that gets any further. -- You received this message

Re: Issue 3435 in reviewboard: Example crontab shows deprecated index rb-site command

Comment #1 on issue 3435 by chip...@gmail.com: Example crontab shows deprecated index rb-site command http://code.google.com/p/reviewboard/issues/detail?id=3435 I've confirmed that the newer crontabs have the correct contents. We should consider just overwriting the generated crontab in

Re: Issue 3436 in reviewboard: Clicking and dragging on linenumbers in diffviewer doesn't allow user to comment

Updates: Labels: -Priority-Medium Priority-Critical Milestone-Release2.1 Component-DiffViewer Comment #1 on issue 3436 by chip...@gmail.com: Clicking and dragging on linenumbers in diffviewer doesn't allow user to comment http://code.google.com/p/reviewboard/issues/detail?id=3436 Ick.

Re: Issue 3434 in reviewboard: Search results don't include file names.

Comment #2 on issue 3434 by peter.sn...@gmail.com: Search results don't include file names. http://code.google.com/p/reviewboard/issues/detail?id=3434 Ah, adding file: works fine! Was that documented somewhere? You can close this issue. -- You received this message because this project is

Re: Issue 3434 in reviewboard: Search results don't include file names.

Updates: Status: UserError Comment #3 on issue 3434 by chip...@gmail.com: Search results don't include file names. http://code.google.com/p/reviewboard/issues/detail?id=3434 Yep, plus some other fields that may be handy:

Re: Issue 3437 in reviewboard: Double-clicking to Add a Comment in the Diff View Discards Any Comments Made

Updates: Status: Confirmed Labels: Component-DiffViewer EasyFix Milestone-Release2.0.x Comment #1 on issue 3437 by chip...@gmail.com: Double-clicking to Add a Comment in the Diff View Discards Any Comments Made http://code.google.com/p/reviewboard/issues/detail?id=3437 Looked

Re: Issue 3438 in reviewboard: registration notification false positive?

Updates: Status: Confirmed Labels: Component-Accounts EasyFix Milestone-Release2.0.x Comment #1 on issue 3438 by chip...@gmail.com: registration notification false positive? http://code.google.com/p/reviewboard/issues/detail?id=3438 (No comment was entered for this change.)

Re: Issue 3439 in reviewboard: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE'

Updates: Status: Confirmed Owner: trowb...@gmail.com Labels: Milestone-Release2.0.x Comment #2 on issue 3439 by chip...@gmail.com: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE' http://code.google.com/p/reviewboard/issues/detail?id=3439 Looks to be

Re: Issue 3439 in reviewboard: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE'

Updates: Labels: -Milestone-Release2.0.x Milestone-RBTools-Release0.6.x Project-RBTools Comment #3 on issue 3439 by chip...@gmail.com: 'ClearCaseClient' object has no attribute 'REVISION_BRANCH_BASE' http://code.google.com/p/reviewboard/issues/detail?id=3439 (No comment was entered for

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #6 on issue 3433 by determin...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 Hi, to be honest I am new in contributing but I can try. Do you think the 'fix' in

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #7 on issue 3433 by chip...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 I think the fix is reasonable. You'd need a development environment to confirm the unit test suite and to write a new test. It'd take a

Re: Issue 2269 in reviewboard: Allow inline review for non-text file types

Comment #13 on issue 2269 by ymik...@gmail.com: Allow inline review for non-text file types http://code.google.com/p/reviewboard/issues/detail?id=2269 Yes and no. My bug is about not being able to review java and perl files as text files. I found a workaround for java files using IE to

Re: Issue 3433 in reviewboard: CVS file path stored incorrectly in database

Comment #8 on issue 3433 by determin...@gmail.com: CVS file path stored incorrectly in database http://code.google.com/p/reviewboard/issues/detail?id=3433 hi, just I did two screenshot before and after the change proposed for the same review request. Btw there is a detailed error stack. If