Re: I have some problems about Languages.

2011-04-11 Thread Christian Hammond
Hi, Sorry, I've been out the past week so I haven't been around to offer much advice here. I'm glad you found a solution. I think we need a better way to factor this all out, though. Otherwise you'll have to maintain a custom patch for every upgrade. Can you tell me where you placed the first

Re: I have some problems about Languages.

2011-04-11 Thread jh5774....@samsung.com
Hi, My English is not as good as you, so, I'm afraid of talk about question. 1. Can you tell me where you placed the first bit of code (encoding changeset0['desc'])? Well, I don't understand perfectly, but I can tell you about that array. When I debugging problems, this is an sample I found.

Re: Customization of Review Board?

2011-04-11 Thread Akash Nawani
Hi David, There is one more query, can we make bug-id field as a required field while creating a review request? On Fri, Apr 8, 2011 at 8:42 AM, Akash Nawani akash.naw...@gmail.com wrote: Thanks a lot David. On Fri, Apr 8, 2011 at 2:49 AM, David Trowbridge trowb...@gmail.comwrote: These

Re: Realized web-based post-commit reviews

2011-04-11 Thread Jan Koprowski
On Mon, Apr 11, 2011 at 9:06 AM, Christian Hammond chip...@chipx86.com wrote: Hey, I'll be honest, I'm not wild about Review Board's codebase knowing about post-review at all. It introduces some compatibility concerns and makes it harder to move forward on either end without breaking the

Re: I have some problems about Languages.

2011-04-11 Thread Chris Clark
Christian Hammond wrote: I think we need a better way to factor this all out, though. Otherwise you'll have to maintain a custom patch for every upgrade. Can you tell me where you placed the first bit of code (encoding changeset0['desc'])? We probably can reuse the repository's Encoding

review comments by default received by submitter

2011-04-11 Thread Michael Keller
Hi, We create reviews using post-review (through a git hook). And we set up default review groups according to code ownership. Now it turns out, that if someone posts a review comment, the default reviewers receive an e-mail notification, but not necessarily the submitter (who may not be in

Re: I have some problems about Languages.

2011-04-11 Thread Christian Hammond
On Monday, April 11, 2011, Chris Clark chris.cl...@ingres.com wrote: Christian Hammond wrote: I think we need a better way to factor this all out, though. Otherwise you'll have to maintain a custom patch for every upgrade. Can you tell me where you placed the first bit of code (encoding

Re: installing Reviewboard on Centos 5.5 64-bit

2011-04-11 Thread Dean
I went back through my notes and backed out everything I had installed following the original install instructions and then added the EPEL repo. The EPEL repo and its ReviewBoard installed smoothly. I found that I needed to re-install mysql-server. That completed, I set the permissions on the

Re: installing Reviewboard on Centos 5.5 64-bit

2011-04-11 Thread Stephen Gallagher
* Required modules are missing You are missing Python modules that are needed before the installation process. You will need to install the necessary modules and restart the install. Databases (required): * sqlite3 (pysqlite2, sqlite3) * MySQL (MySQLdb) *

Issue 2068 in reviewboard: starring an outgoing review adds it to the incoming review list

2011-04-11 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2068 by david.mi...@gmail.com: starring an outgoing review adds it to the incoming review list http://code.google.com/p/reviewboard/issues/detail?id=2068 What version are you running? 1.5.4 What's the URL of the page