rbt post no diff posted

2013-12-09 Thread Lloyd
I have setup a review board (latest 1.7.19) server on a windows machine using bitnami review board installer (It is a prebuilt setup, we need not worry about anything). I have successfully configured my svn repository in review board. I have installed rbt tools in my client system, then made

Re: rbt post no diff posted

2013-12-09 Thread David Trowbridge
Can you run with the --debug flag and see what's listed in the output? -David On Mon, Dec 9, 2013 at 1:57 AM, Lloyd lloydkl.t...@gmail.com wrote: I have setup a review board (latest 1.7.19) server on a windows machine using bitnami review board installer (It is a prebuilt setup, we need not

Re: rbt post no diff posted

2013-12-09 Thread Lloyd
This is the output, I could not find any clue... G:\Project\xxxrbt post --debug DEBUG:root:Checking for a Subversion repository... DEBUG:root:Running: svn info --non-interactive DEBUG:root:Running: diff --version DEBUG:root:repository info: Path: https://myserver/svn/xxx, Base path: /br

Re: Interesting (js or css) bug in 2.0 beta1

2013-12-09 Thread Vlad S.
Hi Chris! here is the file attached. четверг, 5 декабря 2013 г., 23:20:15 UTC+4 пользователь Christian Hammond написал: Thanks Vlad. This has been widely reported, yet not everyone seems to hit it. It appears due to some missing or broken CSS. We're trying to figure out the cause for

'rbt diff' gives ValueError: too many values

2013-12-09 Thread Thomas Dahlin
Hi, I recently installed RB on a Ubuntu server to evaluate it in our company. I finally managed to get it all in place. I am however running into problems when trying to use RBTools from my local windows machine. After spending quite some time trying to find people reporting similar problems (

How to post review comment using changed file line number and WebAPI

2013-12-09 Thread Mahen
Hi, I am trying to post a review comment using WebAPI and python script. I first find out filediff_id and then I send review comment using line number which is with respect to modified file but what I am observing in GUI is that, my comment is posted with respect to original file and not with

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-12-09 Thread Adam Daughterson
On Wednesday, June 5, 2013 5:00:38 PM UTC-6, Christian Hammond wrote: On Jun 5, 2013, at 2:43 PM, kegstand kegs...@gmail.com javascript: wrote: Thank you for your help. I noticed most of my tables were MyISAM. I made the default engine for my mysql install be MyISAM. I

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-12-09 Thread Christian Hammond
Hi Adam, Most likely, new tables are being created as InnoDB, which is where those errors tend to come from. It's a table with a relation to a MyISAM table, which has that sort of name, and causes MySQL to be quite grumpy. You should check that you don't have any new tables in InnoDB. You can

Re: 'rbt diff' gives ValueError: too many values

2013-12-09 Thread Christian Hammond
This is a bug in the current release of rbt diff. We're actually putting together a new RBTools release, which will likely go out either tonight or tomorrow. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com

Re: Interesting (js or css) bug in 2.0 beta1

2013-12-09 Thread Christian Hammond
Thanks. So I see what's wrong now, and it turns out we did have a fix for this that slipped the release. We're getting things in shape for a beta 2, so it'll definitely be fixed for then. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag,

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-12-09 Thread Adam Daughterson
Thanks for the quick response! I wound up experimenting with engine types and (since MySQL 5.5 wants InnoDB) converted all my existing tables to InnoDB, and voila! it worked. I was originally trying to convert everything to MYISAM. -- Get the Review Board Power Pack at

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-12-09 Thread Christian Hammond
Glad to hear it, Adam! Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Mon, Dec 9, 2013 at 12:27 PM, Adam Daughterson adam.daughter...@gmail.com wrote: Thanks for the quick response! I wound up

Re: How to post review comment using changed file line number and WebAPI

2013-12-09 Thread David Trowbridge
The ReviewBot codebase does this same thing. I suggest looking at _translate_line_num in https://github.com/reviewboard/ReviewBot/blob/master/bot/reviewbot/processing/review.py -David On Mon, Dec 9, 2013 at 5:04 AM, Mahen mahenk...@gmail.com wrote: Hi, I am trying to post a review comment

Re: rbt post no diff posted

2013-12-09 Thread David Trowbridge
Hmm. It definitely looks like it's adding a diff to the review request. Can you show us a screenshot of /r/3 ? -David On Mon, Dec 9, 2013 at 4:20 AM, Lloyd lloydkl.t...@gmail.com wrote: This is the output, I could not find any clue... G:\Project\xxxrbt post --debug DEBUG:root:Checking for

Re: rbt post no diff posted

2013-12-09 Thread Lloyd
Extremely sorry for the trouble :( . When I checked today, it is present, yesterday it was not there! Thanks a lot, Lloyd On Tuesday, December 10, 2013 3:54:34 AM UTC+5:30, David Trowbridge wrote: Hmm. It definitely looks like it's adding a diff to the review request. Can you show us a

Re: Issue 3152 in reviewboard: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries

2013-12-09 Thread reviewboard
Comment #11 on issue 3152 by amit.pra...@gmail.com: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries http://code.google.com/p/reviewboard/issues/detail?id=3152 Any updates on this? -- You received this message because this project is configured

Re: Issue 3152 in reviewboard: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries

2013-12-09 Thread reviewboard
Comment #12 on issue 3152 by chip...@gmail.com: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries http://code.google.com/p/reviewboard/issues/detail?id=3152 We are unable to reproduce it, but we know other people have hit it as well. Can you

Re: Issue 3152 in reviewboard: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries

2013-12-09 Thread reviewboard
Comment #13 on issue 3152 by amit.pra...@gmail.com: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries http://code.google.com/p/reviewboard/issues/detail?id=3152 Attached common.min.ccacd15c8a12.css Attachments: common.min.ccacd15c8a12.css

Re: Issue 1426 in reviewboard: Reply on the diff page is confusing and error-prone.

2013-12-09 Thread reviewboard
Comment #17 on issue 1426 by eshara...@gmail.com: Reply on the diff page is confusing and error-prone. http://code.google.com/p/reviewboard/issues/detail?id=1426 I agree 1000% with comment #13. Having the reply button cause you to navigate away from the diff is the cause of the confusion;

Issue 3161 in reviewboard: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username'

2013-12-09 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3161 by tjhug...@gmail.com: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username' http://code.google.com/p/reviewboard/issues/detail?id=3161 *** READ THIS BEFORE

Re: Issue 1426 in reviewboard: Reply on the diff page is confusing and error-prone.

2013-12-09 Thread reviewboard
Comment #18 on issue 1426 by matt...@salsitasoft.com: Reply on the diff page is confusing and error-prone. http://code.google.com/p/reviewboard/issues/detail?id=1426 Also agree 1000%. -- You received this message because this project is configured to send all issue notifications to this

Re: Issue 3058 in reviewboard: file not found in repository

2013-12-09 Thread reviewboard
Comment #13 on issue 3058 by thomas.d...@aphelion.se: file not found in repository http://code.google.com/p/reviewboard/issues/detail?id=3058 When trying to set username/password in the URL as suggested in #9 I get the following error: Unexpected error fetching file from

Issue 3162 in reviewboard: Allow reviews to fix and drop own issues

2013-12-09 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 3162 by bhava...@cloudera.com: Allow reviews to fix and drop own issues http://code.google.com/p/reviewboard/issues/detail?id=3162 *** For customer support, please post to reviewbo...@googlegroups.com *** If you have

Re: Issue 3149 in reviewboard: reviewboard doesn't use sender email setting

2013-12-09 Thread reviewboard
Comment #3 on issue 3149 by leo.shkl...@gmail.com: reviewboard doesn't use sender email setting http://code.google.com/p/reviewboard/issues/detail?id=3149 Thanks for the response. You're correct about the RFC. I re-read the reviewboard doc I linked to and it's a little confusing. With

Re: Issue 3149 in reviewboard: reviewboard doesn't use sender email setting

2013-12-09 Thread reviewboard
Comment #4 on issue 3149 by chip...@gmail.com: reviewboard doesn't use sender email setting http://code.google.com/p/reviewboard/issues/detail?id=3149 We won't be doing the incorrect thing, since that'll just lead to more confusion for users, issues in environments where different e-mail

Re: Issue 3161 in reviewboard: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username'

2013-12-09 Thread reviewboard
Updates: Status: Confirmed Owner: chip...@gmail.com Labels: -Priority-Medium Priority-High Milestone-Release1.7.x Comment #1 on issue 3161 by chip...@gmail.com: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username'

Re: Issue 2888 in reviewboard: Allow the creator of an issue to mark it as fixed or drop it

2013-12-09 Thread reviewboard
Comment #5 on issue 2888 by trowb...@gmail.com: Allow the creator of an issue to mark it as fixed or drop it http://code.google.com/p/reviewboard/issues/detail?id=2888 Issue 3162 has been merged into this issue. -- You received this message because this project is configured to send all

Re: Issue 3158 in reviewboard: Admin topbar menu hidden under changelist due to z-index

2013-12-09 Thread reviewboard
Updates: Labels: EasyFix Comment #1 on issue 3158 by trowb...@gmail.com: Admin topbar menu hidden under changelist due to z-index http://code.google.com/p/reviewboard/issues/detail?id=3158 (No comment was entered for this change.) -- You received this message because this project is

Re: Issue 3119 in reviewboard: Error publishing draft

2013-12-09 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 3085 Comment #7 on issue 3119 by trowb...@gmail.com: Error publishing draft http://code.google.com/p/reviewboard/issues/detail?id=3119 (No comment was entered for this change.) -- You received this message because this project is

Re: Issue 3085 in reviewboard: 'Server' configuration setting is not used for absolute URIs in webapi

2013-12-09 Thread reviewboard
Comment #1 on issue 3085 by trowb...@gmail.com: 'Server' configuration setting is not used for absolute URIs in webapi http://code.google.com/p/reviewboard/issues/detail?id=3085 Issue 3119 has been merged into this issue. -- You received this message because this project is configured to

Re: Issue 3149 in reviewboard: reviewboard doesn't use sender email setting

2013-12-09 Thread reviewboard
Comment #5 on issue 3149 by leo.shkl...@gmail.com: reviewboard doesn't use sender email setting http://code.google.com/p/reviewboard/issues/detail?id=3149 Fair. Thanks for considering the issue. Unfortunately, I can't filter on any text in the email itself since someone replying to it will

Re: Issue 3156 in reviewboard: New Review page is blank

2013-12-09 Thread reviewboard
Comment #21 on issue 3156 by tjbut...@gmail.com: New Review page is blank http://code.google.com/p/reviewboard/issues/detail?id=3156 I have run rb-site upgrade, using first Djblets 0.8alpha1, then tried again with Djblets 0.8alpha2-dev. I had to remember to create the link from the RB site.

Re: Issue 3161 in reviewboard: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username'

2013-12-09 Thread reviewboard
Updates: Status: NeedInfo Comment #2 on issue 3161 by chip...@gmail.com: self.cleaned_data['hosting_account'].username AttributeError: 'NoneType' object has no attribute 'username' http://code.google.com/p/reviewboard/issues/detail?id=3161 This should actually have been fixed in

Re: Issue 3155 in reviewboard: After Postgres and MySQL migration - reviewboard color and text alignments are messed up.

2013-12-09 Thread reviewboard
Comment #3 on issue 3155 by chip...@gmail.com: After Postgres and MySQL migration - reviewboard color and text alignments are messed up. http://code.google.com/p/reviewboard/issues/detail?id=3155 Okay, well this is unrelated to the database. It's either a caching problem or the result of

Re: Issue 3152 in reviewboard: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries

2013-12-09 Thread reviewboard
Updates: Status: Fixed Comment #14 on issue 3152 by chip...@gmail.com: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries http://code.google.com/p/reviewboard/issues/detail?id=3152 Fixed for beta 2. -- You received this message because

Re: Issue 1426 in reviewboard: Reply on the diff page is confusing and error-prone.

2013-12-09 Thread reviewboard
Comment #19 on issue 1426 by chip...@gmail.com: Reply on the diff page is confusing and error-prone. http://code.google.com/p/reviewboard/issues/detail?id=1426 Issue 3148 has been merged into this issue. -- You received this message because this project is configured to send all issue

Re: Issue 3148 in reviewboard: Replying to a review in the diff view creates a new review instead of a reply

2013-12-09 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1426 Comment #1 on issue 3148 by chip...@gmail.com: Replying to a review in the diff view creates a new review instead of a reply http://code.google.com/p/reviewboard/issues/detail?id=3148 (No comment was entered for this change.) --

Re: Issue 3136 in reviewboard: depends_on field is removed after a change

2013-12-09 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 3136 by chip...@gmail.com: depends_on field is removed after a change http://code.google.com/p/reviewboard/issues/detail?id=3136 This was due to the display of this field in a draft. It has been fixed in more recent versions. -- You

Re: Issue 3155 in reviewboard: After Postgres and MySQL migration - reviewboard color and text alignments are messed up.

2013-12-09 Thread reviewboard
Comment #4 on issue 3155 by munsaya...@yahoo.com: After Postgres and MySQL migration - reviewboard color and text alignments are messed up. http://code.google.com/p/reviewboard/issues/detail?id=3155 Apache not needed to be updated. I will state this problem has been fixed. I found

Re: Issue 3155 in reviewboard: After Postgres and MySQL migration - reviewboard color and text alignments are messed up.

2013-12-09 Thread reviewboard
Updates: Status: SetupIssue Comment #5 on issue 3155 by trowb...@gmail.com: After Postgres and MySQL migration - reviewboard color and text alignments are messed up. http://code.google.com/p/reviewboard/issues/detail?id=3155 (No comment was entered for this change.) -- You received