Re: httpd remains in 'starting' state when starting via systemctl

2014-09-24 Thread Paul Fee
Hi Stephen, I don't recall exact details, but I've seen the rb-site upgrade command fail on occasion. I think in my case the sequence was: * yum update [Included a reviewboard update] * reboot * While httpd service was starting rb-site upgrade was invoked. * rb-site upgrade failed

Re: Reviewboard 2.0.6 hungs on new request with non latin symbols

2014-09-24 Thread Sara Rayburn
2.0.8 fixes this for us. Thanks so much! On Tuesday, September 23, 2014 6:32:55 PM UTC-5, Christian Hammond wrote: Hi Sara, David has a repro case for this now, so we're looking into a fix. There will be a 2.0.8 release tonight. Christian -- Christian Hammond -

Re: rb-site fails

2014-09-24 Thread Pierre Leroy
Hi Chris, thanks a lot, the --console flag helped me in sorting out my installation issue. Now, I've got a ReviewBoard site running correclty, I think. Going a bit further, though, I'm a bit stuck in my workflow with RB and Mercurial and maybe you could help again. On the RB website, I've

Re: httpd remains in 'starting' state when starting via systemctl

2014-09-24 Thread David Carson
Paul, This is not the issue for me. I am demonstrating below that (a) the 'restart httpd' succeeds when the hook is *not* in place, and (b) that even if mariadb is running, the 'restart httpd' hangs when the hook is in place. The (0) at the beginning of my prompt is the return code ($?).

Re: httpd remains in 'starting' state when starting via systemctl

2014-09-24 Thread David Carson
By the way, I can do the 'rb-site upgrade --all-sites' as a stand-alone command, whether or not the hook is in place. So, the problem has something to do with the rb-site being called by the httpd restart (or start). On Wednesday, September 24, 2014 9:59:59 AM UTC-4, David Carson wrote:

Re: The file something/orother/withsomeextens.ion (revision hexadecimal joy) was not found in the repository

2014-09-24 Thread Kevin Jones
In my attempts to get the command used, the developer informed me that they can see the diffs now. I'm going to assume I fat-fingered the password to the repo in the setup and fixed it by reentering it yesterday. Christian, your help is amazing, and I feel somewhat silly for having wasted

please, please, please, please help me with installation

2014-09-24 Thread neel roy
It's 8:10 pm and I have been fighting with reviewboard installation for last SIX hours. I tried with apache2 and I ran into 000-default site problem.. from https://www.reviewboard.org/docs/manual/dev/admin/installation/creating-sites/#creating-sites: Some Apache installations (such as the

Review Board 2.0.8 is released!

2014-09-24 Thread Christian Hammond
Hi everyone, Early this morning, we put out a release of Review Board 2.0.8. This fixes some of the major issues recently brought up on this list, including several issues in the New Review Request page, installation problems due to a new release of Python-Markdown, and more. Announcement:

rbt patch failing with perforce

2014-09-24 Thread Henrique de Miranda Gontijo
Hi, I want to ask help with an issue with rbt patch + perforce. When I try to apply a patch from review board by using rbt patch, the tool fails with a can't find the file to patch error. $ rbt patch --server https://reviewboard.eng.mydomain.com/ --debug --username hgontijo --repository

Re: please, please, please, please help me with installation

2014-09-24 Thread Christian Hammond
Hi Neel, lighttpd is not well-supported, which is why it's considered an advanced option. Very few people seem to use it. I'd *strongly* suggest using Apache + mod_wsgi (daemon mode works well these days), if at all possible. You'll have the best support that way. FastCGI is a bad option.

Re: rb-site fails

2014-09-24 Thread Christian Hammond
Hi Pierre, Can you try using RBTools to post a change? Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On September 24, 2014 at 6:29:41 AM, Pierre Leroy (pierre.lero...@gmail.com) wrote: Hi

Re: Reviewboard 2.0.6 hungs on new request with non latin symbols

2014-09-24 Thread Christian Hammond
Glad it works, Sara! Thanks for your patience and feedback on this! Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On September 24, 2014 at 6:10:36 AM, Sara Rayburn (sararayb...@gmail.com) wrote:

Re: please, please, please, please help me with installation

2014-09-24 Thread Christian Hammond
Hi Neel, You definitely want to remove the 000-default file. If the site isn't showing, that's often an indicator that Apache either isn't seeing the Review Board config file, or the VirtualHost setup is wrong. Can you provide the reviewboard.conf file along with Apache's httpd.conf? Can you

Re: please, please, please, please help me with installation

2014-09-24 Thread neel roy
Okay, I fixed the problem. here are the couple of changes I did: sudo apt-get install apache2 sudo apt-get install libapache2-mod-wsgi sudo a2enmod *wsgi* sudo apt-get install mysql-server sudo apt-get install python-mysqldb sudo apt-get install memcached sudo apt-get install memcached-dev sudo

Re: Resetting open issue counters

2014-09-24 Thread Alexey Neyman
Hi Christian, Just upgraded to 2.0.8: [[[ bash-4.1# rb-site manage /var/www.reviewboard reset-issue-counts --recalculate --verbosity=2 2050 Usage: rb-site command [options] path rb-site: error: no such option: --recalculate bash-4.1# rb-site --version rb-site 2.0.8 ]]] BTW, release notes for

Re: Resetting open issue counters

2014-09-24 Thread Christian Hammond
Thanks, I'll fix the release notes. Make sure to put a -- before any --arguments, like:     rb-site manage /var/www/reviewboard reset-issue-counts -- --verbosity=2 --recalculate Not sure what you mean by the help output being wrong though? Christian -- Christian Hammond -

Re: Resetting open issue counters

2014-09-24 Thread Alexey Neyman
Figured it out, needs -- to have --recalculate argument applied to 'reset-issue-counts' rather than rb-site itself. Here is the output: DEBUG:root:Calculated issue counts for review request ID 2050 across 5 review(s): Resulting counts = {u'R': 0, u'D': 0, u'O': 1}; DB values =

Re: Resetting open issue counters

2014-09-24 Thread Alexey Neyman
Re help: the order of the 'manage' and path arguments is swapped. Correct order: rb-site manage path Help message: rb-site path manage Regards, Alexey. On Wed, Sep 24, 2014 at 12:41 PM, Christian Hammond christ...@beanbaginc.com wrote: Thanks, I'll fix the release notes. Make sure to put a

Re: Resetting open issue counters

2014-09-24 Thread Christian Hammond
Thanks for the debug info! Yeah, as I mentioned in an earlier reply, a reset command will *not* fix the problem, because the calculation is broken. Best I could do is to get this command in to reset and provide debug information, so that I can hopefully figure out the cause. I'll look into

Re: Resetting open issue counters

2014-09-24 Thread Alexey Neyman
Re release notes: you've fixed only one instance of the two where 'manage' was missing: A new rb-site path reset-issue-counts management command has been added... On Wed, Sep 24, 2014 at 12:41 PM, Christian Hammond christ...@beanbaginc.com wrote: Thanks, I'll fix the release notes. Make

Re: Resetting open issue counters

2014-09-24 Thread Christian Hammond
Can you tell it's been a very long few days? Fixed. New docs are building now. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On September 24, 2014 at 12:47:39 PM, Alexey Neyman

Re: Resetting open issue counters

2014-09-24 Thread Christian Hammond
So according to that debug output, there is one comment that has an issue open. I want to do some probing into this. Can you run this and provide the full output, commands included?     $ rb-site manage /path/to/site shell     from reviewboard.reviews.models import Comment, Review     comment

Re: Review Board 2.0.8 is released!

2014-09-24 Thread Stephen Gallagher
On 09/24/2014 01:05 PM, Christian Hammond wrote: Hi everyone, Early this morning, we put out a release of Review Board 2.0.8. This fixes some of the major issues recently brought up on this list, including several issues in the New Review Request page, installation problems due to a new

Re: Resetting open issue counters

2014-09-24 Thread Alexey Neyman
Here it is: from reviewboard.reviews.models import Comment, Review comment = Comment.objects.get(pk=8199) print comment.reply_to_id None print comment.issue_opened False print comment.issue_status O review = comment.review.get() print review.pk 6722 print review.public True print

Re: Resetting open issue counters

2014-09-24 Thread Christian Hammond
Hi Alexey, Thanks, that helps a lot. I think I know what's going on then. I'll send a patch your way for testing once I have a fix. Christian -- Christian Hammond - christ...@beanbaginc.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On September

Re: Perl Syntax Highlighting working for .pm, but not .pl extensions

2014-09-24 Thread Steve
Upgrading Pygments to 1.6 fixed the problem. --Steve On Thursday, August 28, 2014 2:25:51 PM UTC-7, Steve wrote: Knowing it should work helps. The .pl files I'm looking at are smaller than the .pm files that work, so I'm going to rule out file size as the cause. I'll try to isolate the

Re: httpd remains in 'starting' state when starting via systemctl

2014-09-24 Thread David Carson
Stephen, Yep, removing the ...oneshot line fixed it for me. Thanks a lot. ~David On Wednesday, September 24, 2014 3:39:34 PM UTC-4, Stephen Gallagher wrote: On 09/24/2014 10:06 AM, David Carson wrote: By the way, I can do the 'rb-site upgrade --all-sites' as a stand-alone command,

Issue 3581 in reviewboard: If a user exists in a local site they will not get emails from the normal site if they are part of a reveiw group in the normal site.

2014-09-24 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3581 by whyisyou...@gmail.com: If a user exists in a local site they will not get emails from the normal site if they are part of a reveiw group in the normal site.

Re: Issue 421 in reviewboard: Enhancement request: nag owners about stale review requests

2014-09-24 Thread reviewboard
Comment #15 on issue 421 by da.s...@gmail.com: Enhancement request: nag owners about stale review requests https://code.google.com/p/reviewboard/issues/detail?id=421 This is tangential/related, I'd like to be able to abstain or remove myself from a review request. I would do this regularly

Re: Issue 3484 in reviewboard: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member

2014-09-24 Thread reviewboard
Comment #4 on issue 3484 by john.sirois: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member https://code.google.com/p/reviewboard/issues/detail?id=3484 Afaict there has been a regression. I see currently for

Issue 3582 in reviewboard: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member

2014-09-24 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3582 by john.sirois: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member https://code.google.com/p/reviewboard/issues/detail?id=3582 What version are you

Re: Issue 3484 in reviewboard: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member

2014-09-24 Thread reviewboard
Comment #5 on issue 3484 by john.sirois: `rbt patch -c` fails for patches to public github repos with an RBCommons team when the patch if from a non-team member https://code.google.com/p/reviewboard/issues/detail?id=3484 Filed a new bug with the regression detail in case that works better

Re: Issue 3452 in reviewboard: RR updated altough no changes made / warn when no changes are made

2014-09-24 Thread reviewboard
Updates: Status: Fixed Comment #4 on issue 3452 by trowb...@gmail.com: RR updated altough no changes made / warn when no changes are made https://code.google.com/p/reviewboard/issues/detail?id=3452 Fixed in release-2.0.x (f29c57f). Thanks! -- You received this message because this

Issue 3583 in reviewboard: RB2: No way to post patch with parent-diff in initial web pages

2014-09-24 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3583 by jared.gr...@gmail.com: RB2: No way to post patch with parent-diff in initial web pages https://code.google.com/p/reviewboard/issues/detail?id=3583 What version are you running? ReviewBoard 2.0.6 What's the URL of

Re: Issue 3583 in reviewboard: RB2: No way to post patch with parent-diff in initial web pages

2014-09-24 Thread reviewboard
Comment #1 on issue 3583 by jared.gr...@gmail.com: RB2: No way to post patch with parent-diff in initial web pages https://code.google.com/p/reviewboard/issues/detail?id=3583 Even having a post no patch option, where I could then use the UI in the review editor would be nice. -- You

Re: Issue 3583 in reviewboard: RB2: No way to post patch with parent-diff in initial web pages

2014-09-24 Thread reviewboard
Updates: Status: Fixed Comment #2 on issue 3583 by trowb...@gmail.com: RB2: No way to post patch with parent-diff in initial web pages https://code.google.com/p/reviewboard/issues/detail?id=3583 This is already fixed in git, and will ship with the 2.1 release. rbt is fine with ports