Re: post-review fails to login / uses wrong URL

2010-06-12 Thread Joachim Worringen
On 3 Jun., 10:48, Christian Hammond chip...@chipx86.com wrote: The value passed to --server must be a full URL, not just a file path. Thanks, Christian, using --server=http://testsrv/reviews actually helped. But I still wonder why the incomplete URL testsrv/reviews is then expanded to

Re: post-review fails to login / uses wrong URL

2010-06-12 Thread Christian Hammond
Hi, We're using Python's URL building functions for building that path. I don't know why it was built that way. We should probably do some validation for that (care to file a bug?) As for the config file, .reviewboardrc is actually a Python file, so it must use Python syntax. That means in this

Issue 1683 in reviewboard: PendingDeprecationWarning while rbsite install

2010-06-12 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1683 by Jan.Koprowski: PendingDeprecationWarning while rbsite install http://code.google.com/p/reviewboard/issues/detail?id=1683 What version are you running? ReviewBoard 1.5 beta 2 What steps will reproduce the problem?

Issue 1684 in reviewboard: Error: Please correct the error below django.utils.functional.__proxy__ object at 0x257dfd0

2010-06-12 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1684 by Jan.Koprowski: Error: Please correct the error below django.utils.functional.__proxy__ object at 0x257dfd0 http://code.google.com/p/reviewboard/issues/detail?id=1684 What version are you running? ReviewBoard 1.5

Re: Issue 1684 in reviewboard: Error: Please correct the error below django.utils.functional.__proxy__ object at 0x257dfd0

2010-06-12 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 1684 by chipx86: Error: Please correct the error below django.utils.functional.__proxy__ object at 0x257dfd0 http://code.google.com/p/reviewboard/issues/detail?id=1684 Can you verify the version of Django installed? Does this only happen

Re: Issue 1683 in reviewboard: PendingDeprecationWarning while rbsite install

2010-06-12 Thread reviewboard
Updates: Status: Confirmed Labels: -Priority-Medium Priority-Low Milestone-Release1.5 Component-RB-Site Comment #1 on issue 1683 by chipx86: PendingDeprecationWarning while rbsite install http://code.google.com/p/reviewboard/issues/detail?id=1683 We should see about turning off

Re: Issue 1541 in reviewboard: Multibyte characters in diff fragments are broken with syntax highlighting.

2010-06-12 Thread reviewboard
Comment #3 on issue 1541 by chipx86: Multibyte characters in diff fragments are broken with syntax highlighting. http://code.google.com/p/reviewboard/issues/detail?id=1541 How did you post it? Did you use post-review? -- You received this message because you are subscribed to the Google

Re: Issue 1541 in reviewboard: Multibyte characters in diff fragments are broken with syntax highlighting.

2010-06-12 Thread reviewboard
Comment #4 on issue 1541 by KoRoN.KaoriYa: Multibyte characters in diff fragments are broken with syntax highlighting. http://code.google.com/p/reviewboard/issues/detail?id=1541 No I didn't. I had made it by git diff HEAD. -- You received this message because you are subscribed to the

Re: Issue 1541 in reviewboard: Multibyte characters in diff fragments are broken with syntax highlighting.

2010-06-12 Thread reviewboard
Comment #5 on issue 1541 by chipx86: Multibyte characters in diff fragments are broken with syntax highlighting. http://code.google.com/p/reviewboard/issues/detail?id=1541 Try using post-review. Most likely, it's broken due to your diff using a short SHA1. Long SHA1s are required.

Re: Issue 1541 in reviewboard: Multibyte characters in diff fragments are broken with syntax highlighting.

2010-06-12 Thread reviewboard
Comment #6 on issue 1541 by KoRoN.KaoriYa: Multibyte characters in diff fragments are broken with syntax highlighting. http://code.google.com/p/reviewboard/issues/detail?id=1541 Thank you chipx86! I could post a new diff file which be generated with '--ful-index', and it seems good in diff