Re: Issue 1735 in reviewboard: Perforce clients not adding the charset

2012-03-29 Thread reviewboard
Comment #5 on issue 1735 by steven...@gmail.com: Perforce clients not adding the charset http://code.google.com/p/reviewboard/issues/detail?id=1735 Hi, I got this error in version 1.6.4.1. According to workaround, I modified perforce.py which is under ~/reviewboard/scmtools folder. Then the

Re: Issue 1735 in reviewboard: Perforce clients not adding the charset

2012-03-29 Thread reviewboard
Comment #6 on issue 1735 by steven...@gmail.com: Perforce clients not adding the charset http://code.google.com/p/reviewboard/issues/detail?id=1735 Does anyone can tell me how to define "repository" as global name ? -- You received this message because you are subscribed to the Google Groups

Re: Issue 1735 in reviewboard: Perforce clients not adding the charset

2012-03-29 Thread reviewboard
Comment #7 on issue 1735 by carodo...@gmail.com: Perforce clients not adding the charset http://code.google.com/p/reviewboard/issues/detail?id=1735 The only lines I added on the top portion were: if (str(repository.encoding)): self.p4.charset = str(repository.encoding) so, you should

Issue 2557 in reviewboard: Ability to only send updates in emails, not entire reviews

2012-03-29 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2557 by marcin.z...@gmail.com: Ability to only send updates in emails, not entire reviews http://code.google.com/p/reviewboard/issues/detail?id=2557 * NOTE: Do not post confidential information in this bug report. * *

Re: Issue 2557 in reviewboard: Ability to only send updates in emails, not entire reviews

2012-03-29 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 2557 by trowb...@gmail.com: Ability to only send updates in emails, not entire reviews http://code.google.com/p/reviewboard/issues/detail?id=2557 I'm confused. The emails don't contain the contents of the diffs. What's making it so big?

Re: Issue 2557 in reviewboard: Ability to only send updates in emails, not entire reviews

2012-03-29 Thread reviewboard
Comment #2 on issue 2557 by marcin.z...@gmail.com: Ability to only send updates in emails, not entire reviews http://code.google.com/p/reviewboard/issues/detail?id=2557 Weird. For me they do contain the entire diff. Is it possible an installation is configured extra to do that (I am not the

Issue 2558 in reviewboard: post-review permission denied searching for repository

2012-03-29 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2558 by ivanthet...@gmail.com: post-review permission denied searching for repository http://code.google.com/p/reviewboard/issues/detail?id=2558 * NOTE: Do not post confidential information in this bug report. * * If