Re: git config issues

2013-12-17 Thread Thomas Dahlin
Hi Christian, Thanks for the feedback! I would like to say of course I already did that!, but after trying out so many different configurations I'm not sure any more... :) I will get back to you as soon as I have tried it out. /Thomas On Monday, December 16, 2013 11:17:33 PM UTC+1,

Post Review Request filtering by a directory

2013-12-17 Thread Bruno Carneiro
Hi everybody, I have a GIT repository which has many subprojects (modules) inside. Different teams work in each subproject. So, I would like to know if it is possible to post a review request filtering by a path. I tried to use Posting Changeset

File change listing the same file multiple times?

2013-12-17 Thread Bluefox
I'm not sure if its just me or something weird about our setup. But sometimes when uploading a diff, multiple entries of the same file would appear in reviewboard, this makes it very hard to review the code. Does anyone have similar experience or how can I fix this with our setup? Review Board

Re: File change listing the same file multiple times?

2013-12-17 Thread Christian Hammond
Hi, How are you generating the diff? The diff viewer trusts whatever entries are in the diff file. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Tue, Dec 17, 2013 at 9:01 AM, Bluefox

Re: Issue 2646 in reviewboard: Make Auto-Submitted: auto-generated header optional with a settings flag

2013-12-17 Thread reviewboard
Comment #3 on issue 2646 by peter.ho...@gmail.com: Make Auto-Submitted: auto-generated header optional with a settings flag http://code.google.com/p/reviewboard/issues/detail?id=2646 It would be nice to have this. I actually could use any workaround. This option would make reviewboard to

Re: Issue 3152 in reviewboard: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries

2013-12-17 Thread reviewboard
Comment #15 on issue 3152 by amit.pra...@gmail.com: Comment window doesn't show TextArea in the right color. Text itself scrolls on making entries http://code.google.com/p/reviewboard/issues/detail?id=3152 When is rb2b2 expected? -- You received this message because this project is

Re: Issue 513 in reviewboard: Improvements for authentication using REMOTE_USER

2013-12-17 Thread reviewboard
Comment #10 on issue 513 by mra...@rapitasystems.com: Improvements for authentication using REMOTE_USER http://code.google.com/p/reviewboard/issues/detail?id=513 Is there any update on this? -- You received this message because this project is configured to send all issue notifications to

Issue 3170 in reviewboard: RBTools: A critical bug for handling P4Proxy

2013-12-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3170 by eyuw...@gmail.com: RBTools: A critical bug for handling P4Proxy http://code.google.com/p/reviewboard/issues/detail?id=3170 What version are you running? RBTools 0.5.2 We have multiple sites in our company where

Issue 3171 in reviewboard: RBTools: A critical bug for determining SCM type?

2013-12-17 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3171 by eyuw...@gmail.com: RBTools: A critical bug for determining SCM type? http://code.google.com/p/reviewboard/issues/detail?id=3171 What version are you running? RBTools 0.5.2 What's the URL of the page containing the

Re: Issue 3171 in reviewboard: RBTools: A critical bug for determining SCM type?

2013-12-17 Thread reviewboard
Comment #1 on issue 3171 by chip...@gmail.com: RBTools: A critical bug for determining SCM type? http://code.google.com/p/reviewboard/issues/detail?id=3171 We can use an OrderedDict, but it's not a guaranteed fix, as we can't expect a certain order from Python Entrypoints. The

Re: Issue 3170 in reviewboard: RBTools: A critical bug for handling P4Proxy

2013-12-17 Thread reviewboard
Updates: Status: NeedInfo Labels: Component-RBTools Comment #1 on issue 3170 by chip...@gmail.com: RBTools: A critical bug for handling P4Proxy http://code.google.com/p/reviewboard/issues/detail?id=3170 We can't delete that code, as other companies depend on it, but what we

Re: Issue 3170 in reviewboard: RBTools: A critical bug for handling P4Proxy

2013-12-17 Thread reviewboard
Comment #2 on issue 3170 by chip...@gmail.com: RBTools: A critical bug for handling P4Proxy http://code.google.com/p/reviewboard/issues/detail?id=3170 Actually, let me get you a more detailed patch to try. -- You received this message because this project is configured to send all issue

Re: Issue 3170 in reviewboard: RBTools: A critical bug for handling P4Proxy

2013-12-17 Thread reviewboard
Comment #3 on issue 3170 by chip...@gmail.com: RBTools: A critical bug for handling P4Proxy http://code.google.com/p/reviewboard/issues/detail?id=3170 Apply this patch and see if things work better. Attachments: 0001-Add-the-requested-hostname-to-the-Perforce-repo-path.patch 2.2 KB

Re: Issue 3171 in reviewboard: RBTools: A critical bug for determining SCM type?

2013-12-17 Thread reviewboard
Comment #2 on issue 3171 by eyuw...@gmail.com: RBTools: A critical bug for determining SCM type? http://code.google.com/p/reviewboard/issues/detail?id=3171 What you suggest might work for a small group, but doesn't really scale for a company with 13K+ employees. You are suggesting to ask

Re: Issue 513 in reviewboard: Improvements for authentication using REMOTE_USER

2013-12-17 Thread reviewboard
Comment #11 on issue 513 by trowb...@gmail.com: Improvements for authentication using REMOTE_USER http://code.google.com/p/reviewboard/issues/detail?id=513 The patches that were initially written for this are very, very old and no longer work. If someone wants this, we'll need them to

Re: Issue 3171 in reviewboard: RBTools: A critical bug for determining SCM type?

2013-12-17 Thread reviewboard
Comment #4 on issue 3171 by eyuw...@gmail.com: RBTools: A critical bug for determining SCM type? http://code.google.com/p/reviewboard/issues/detail?id=3171 OK, let me confirm if I understand your suggestion. You meant when we provision a git repo, we should drop in a .reviewboardrc file in

Re: Issue 3171 in reviewboard: RBTools: A critical bug for determining SCM type?

2013-12-17 Thread reviewboard
Comment #7 on issue 3171 by chip...@gmail.com: RBTools: A critical bug for determining SCM type? http://code.google.com/p/reviewboard/issues/detail?id=3171 Yep, it'll take both into account. Any user configuration will also override the repository configuration. -- You received this