RB1.7.7.1 Upgrade Bug: Hide WhiteSpace changes in viewdiff is not functioning.

2013-06-05 Thread satish singh
I looked at this issue today. The issue here is, a class called "whitespace-chunk" is not being generated to html page: It should be like: But its coming rendering like this: As per my anaysis, this calss is getting rendered through: reviewboard/templates/diffviewer/diff_file_fragment.ht

Listing all groups a user belongs to

2013-06-05 Thread Steve
I'm using ReviewBoard 1.7.6. I was hoping the new UI would have some way to list the ReviewBoard groups that a user belongs to, but I can't find it. Is there a way to do that? If not, are there plans to add it? When someone leaves the company we want to remove them from ReviewBoard groups a

Re: Unable to create review using P4 repository

2013-06-05 Thread Steve
I have these set in my htdocs/reviewboard.wsgi file: os.environ['P4PORT'] = "" os.environ['P4USER'] = "" os.environ['P4CLIENT'] = "" os.environ['P4TICKETS'] = "path to ticket file" os.environ['PATH'] = os.environ['PATH'] + ":/usr/local/bin" I need P4TICKETS because we use p4 passwords

Re: Listing all groups a user belongs to

2013-06-05 Thread David Trowbridge
If you go to the site admin, and browse to the desired user, there's a "Groups" box where you can unselect everything. -David On Wed, Jun 5, 2013 at 7:30 AM, Steve wrote: > I'm using ReviewBoard 1.7.6. I was hoping the new UI would have some way > to list the ReviewBoard groups that a user be

upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
# easy_install -U ReviewBoard Searching for ReviewBoard Reading http://pypi.python.org/simple/ReviewBoard/ Reading http://www.review-board.org/ Reading http://downloads.review-board.org/releases/ Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.7/ Reading http://downloads.reviewboar

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
On Wed, Jun 5, 2013 at 7:44 PM, Christian Hammond wrote: > Hi, > > Can you provide the following info for me: > > 1) The version of django-evolution installed. > # pip list distribute (0.6.35) Django (1.4.5) django-evolution (0.6.9) django-pipeline (1.2.24) Djblets (0.7.15) docutils (0.10) feedp

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
Hi, Can you provide the following info for me: 1) The version of django-evolution installed. 2) The result of: rb-site manage /path/to/site evolve -- --hint 3) The result of: rb-site manage /path/to/site list-evolutions 4) Type of database being used. Christian On Jun 5, 2013, at 12:40, kegsta

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
mysql> select * from reviews_reviewrequest_depends_on ; Empty set (0.00 sec) On Wed, Jun 5, 2013 at 7:50 PM, kegstand wrote: > On Wed, Jun 5, 2013 at 7:44 PM, Christian Hammond wrote: > >> Hi, >> >> Can you provide the following info for me: >> >> 1) The version of django-evolution installed.

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
Did you at any point attempt an upgrade before to this version? Also, did you install Review Boars using pip, or easy_install? You can try dropping the extra_data field from accounts_profile and re-running rb-site upgrade. Christian On Jun 5, 2013, at 12:58, kegstand wrote: > mysql> select

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
On Wed, Jun 5, 2013 at 8:14 PM, Christian Hammond wrote: > Did you at any point attempt an upgrade before to this version? > Never. > > Also, did you install Review Boars using pip, or easy_install? > > I cannot remember. > You can try dropping the extra_data field from accounts_profile and

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
mysql> alter table accounts_profile drop column extra_data ; Query OK, 25 rows affected (0.00 sec) Records: 25 Duplicates: 0 Warnings: 0 mysql> quit Bye [root@rb reviewboard]# rb-site upgrade /var/www/reviewboard Rebuilding directory structure Updating database. This may take a while. The log o

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
You'll have to repeat the process for all new fields listed in the --hint, and the intermediate tables (reviews_reviewrequestdraft_depends_on, reviews_reviewrequest_depends_on). I recommend also backing up your database first, just in case. Christian -- Christian Hammond - chip...@chipx86.com R

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
db backed up. mysql: mysql> alter table accounts_profile drop column extra_data ; Query OK, 25 rows affected (0.00 sec) Records: 25 Duplicates: 0 Warnings: 0 mysql> alter table hostingsvcs_hostingserviceaccount drop column hosting_url ; ERROR 1091 (42000): Can't DROP 'hosting_url'; check that

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
You need to delete all the fields, not just the next set. It's applying part of the evolutions, then failing at the next step. Because of the failure, it's already applied the previous steps but didn't record that, so it tries to apply again next time. Delete extra_data, hosting_url, and all the

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
Here I show I deleted what I needed to: mysql> select extra_data from accounts_profile ; ERROR 1054 (42S22): Unknown column 'extra_data' in 'field list' mysql> select hosting_url from hostingsvcs_hostingserviceaccount; ERROR 1054 (42S22): Unknown column 'hosting_url' in 'field list' mysql> show cr

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
> > 2) The result of: rb-site manage /path/to/site evolve -- --hint > (edited slightly) MUTATIONS = [ AddField('Profile', 'extra_data', JSONField, null=True) ] MUTATIONS = [ AddField('HostingServiceAccount', 'hosting_url', models.CharField, max_length=255, null=True) ] MUTATIONS = [ Add

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
You have a mix of InnoDB tables and MyISAM tables. You need to make sure they're all consistent. Either convert all your old tables, or configure MySQL to create new tables of the same type as existing ones (you'll have to look up how to configure this as I don't know off-hand). Christian --

Re: Listing all groups a user belongs to

2013-06-05 Thread Steve
Do you mean the 'Groups' box in the 'Permissions' section on the 'Change User' page? For every user I've checked, the only entry in the 'Groups' box is 'default reviewer'. It doesn't correctly show the groups they belong to, which may be why I missed it before. A bug maybe? --Steve On Wedn

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread kegstand
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 converted a few tables to MyISAM: mysql> ALTER TABLE diffviewer_filediffdata ENGINE = MyISAM ; Query OK, 4582 rows affected (0.12 sec) Records: 4582 Duplicates: 0 War

Re: Listing all groups a user belongs to

2013-06-05 Thread David Trowbridge
Oh, I guess I just cursorily looked at this and was confused. You're right that we don't have an admin-visible UI. This is something that's probably worth adding, but we're pretty bogged down right now. Mind filing an enhancement request in the issue tracker? Thanks, -David On Wed, Jun 5, 2013

Re: upgrade from 1.7.7.1 to 1.7.9 error

2013-06-05 Thread Christian Hammond
On Jun 5, 2013, at 2:43 PM, kegstand 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 converted a few tables to MyISAM: > > mysql> ALTER TABLE diffviewer_filediffdata ENGINE = MyISAM ; > Query O

Re: Users are automatically created if authenticated through ldap

2013-06-05 Thread Joe Reyna
Hi Eric, can you post your modified custom authenticator? On Wednesday, March 23, 2011 1:02:10 PM UTC-7, Eric Johnson wrote: > > I achieved your aim for our local copy of ReviewBoard by snagging the > existing LDAP code, and modifying it slightly. I then used the modified > code as a custom auth

Re: Listing all groups a user belongs to

2013-06-05 Thread Steve
Okay, will do. On Wednesday, June 5, 2013 3:57:15 PM UTC-7, David Trowbridge wrote: > > Oh, I guess I just cursorily looked at this and was confused. You're right > that we don't have an admin-visible UI. > > This is something that's probably worth adding, but we're pretty bogged > down right n

Re: RB1.7.7.1 Upgrade Bug: Hide WhiteSpace changes in viewdiff is not functioning.

2013-06-05 Thread satish singh
Hi Team, Please provide some input So, that I can fix this bug. On Wednesday, 5 June 2013 19:29:40 UTC+5:30, satish singh wrote: > > I looked at this issue today. The issue here is, a class called > "whitespace-chunk" > is not being generated to html page: > > It should be like: > > > > > But

Re: RB1.7.7.1 Upgrade Bug: Hide WhiteSpace changes in viewdiff is not functioning.

2013-06-05 Thread Christian Hammond
Hi Satish, I don't know off-hand, and haven't had the time to look into this. I'll put it on my plate for the next release to investigate. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Jun 5, 2013, a