Configuration problem: get_repository_info sees C: as root instead of cygwin\root

2012-10-15 Thread Lemuel Tito
When I try to post-review, I get an error after these calls: python\Scripts\post-review-script.py calls rbtools~.egg\rbtools\postreview.py calls rbtools~.egg\rbtools\clients\__init__.py calls rbtools~.egg\rbtools\clients\git.py where I get an this: line 58, in get_repository_info WindowsError:

Re: Configuration problem: get_repository_info sees C: as root instead of cygwin\root

2012-10-15 Thread Christian Hammond
Hi, Looks like this is what git is reporting as the top-level directory. Unfortunately, that means it's beyond us. Just to verify, see what this says: git rev-parse --show-toplevel Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc.

Re: Configuration problem: get_repository_info sees C: as root instead of cygwin\root

2012-10-15 Thread Lemuel Tito
Thanks for the quick response. It says /home/my.user.name/my-working-directory On Monday, October 15, 2012 5:10:45 PM UTC+8, Christian Hammond wrote: Hi, Looks like this is what git is reporting as the top-level directory. Unfortunately, that means it's beyond us. Just to verify, see

Re: Review Board 1.6.12 Issue with view diff

2012-10-15 Thread Peer
Hi Christian, the diff uploaded with post-review also show err with similar scenario (i.e view diff works fine for a while and get the err after some time) could you please help assist in resolving this error. Thanks. Peer On Friday, October 12, 2012 10:38:33 PM UTC+5:30, Christian Hammond

Re: Review Board 1.6.12 Issue with view diff

2012-10-15 Thread Tomasz Moń
On Mon, Oct 15, 2012 at 3:22 PM, Peer peermoh...@gmail.com wrote: the diff uploaded with post-review also show err with similar scenario (i.e view diff works fine for a while and get the err after some time) could you please help assist in resolving this error. Was the file renamed or removed

Re: Review Board 1.6.12 Issue with view diff

2012-10-15 Thread Peer
Hi Tomasz, The file was not renamed/moved after the review submitted. the diff view simply breaks after some time with the above error. i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time)

Re: Review Board 1.6.12 Issue with view diff

2012-10-15 Thread Tomasz Moń
On Mon, Oct 15, 2012 at 4:06 PM, Peer peermoh...@gmail.com wrote: i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time) If you submitted patch using post-review, the extended_path should

Re: Configuration problem: get_repository_info sees C: as root instead of cygwin\root

2012-10-15 Thread hbCyber
Hi Christian, I'm a colleague of Lemuel. After testing, we confirmed that the issue resides within RBTools v0.4.2 specifically. Re-installing v0.4.1 allowed us to start posting reviews; no change to Cygwin or Git. We will stick to 0.4.1 for now, but I'd be curious to know why upgrading to

Re: Issue 2720 in reviewboard: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/

2012-10-15 Thread reviewboard
Comment #1 on issue 2720 by chris.cl...@actian.com: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/ http://code.google.com/p/reviewboard/issues/detail?id=2720 I'm seeing this with the same RB version, a few times a week. Appears to be the same as issue 2223. -- You received this

Re: Issue 2223 in reviewboard: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/

2012-10-15 Thread reviewboard
Comment #7 on issue 2223 by chris.cl...@actian.com: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/ http://code.google.com/p/reviewboard/issues/detail?id=2223 I'm seeing this with the same RB version, a few times a week. Appears to be the same as issue 2720. -- You received this

Re: Issue 2223 in reviewboard: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/

2012-10-15 Thread reviewboard
Comment #8 on issue 2223 by chris.cl...@actian.com: ERROR (EXTERNAL IP): Internal Server Error: /dashboard/ http://code.google.com/p/reviewboard/issues/detail?id=2223 I'm seeing this with the RB version 1.6.11, a few times a week. Appears to be the same as issue 2720. -- You received

Re: Issue 1491 in reviewboard: LDAP group support

2012-10-15 Thread reviewboard
Comment #6 on issue 1491 by liam.rei...@gmail.com: LDAP group support http://code.google.com/p/reviewboard/issues/detail?id=1491 Any progress on this? It's been New for a long time. This would be extremely handy. -- You received this message because you are subscribed to the Google Groups

Re: Issue 2696 in reviewboard: Add comment missing towards the end of a review

2012-10-15 Thread reviewboard
Comment #5 on issue 2696 by jtvea...@gmail.com: Add comment missing towards the end of a review http://code.google.com/p/reviewboard/issues/detail?id=2696 A fix is up for review: http://reviews.reviewboard.org/r/3415/ -- You received this message because you are subscribed to the Google

Re: Issue 2335 in reviewboard: post-review basedir

2012-10-15 Thread reviewboard
Comment #5 on issue 2335 by j...@purestorage.com: post-review basedir http://code.google.com/p/reviewboard/issues/detail?id=2335 This is absolutely a bug. There's no way for me to upload a diff that's generated in a child directory of the svn repo without manipulating the diff to include

Re: Issue 2335 in reviewboard: post-review basedir

2012-10-15 Thread reviewboard
Comment #6 on issue 2335 by j...@purestorage.com: post-review basedir http://code.google.com/p/reviewboard/issues/detail?id=2335 This is absolutely a bug. There's no way for me to upload a diff that's generated in a child directory of the svn repo without manipulating the diff to include

Re: Issue 2335 in reviewboard: post-review basedir

2012-10-15 Thread reviewboard
Updates: Status: Confirmed Labels: Component-RBTools Milestone-RBTools-Release1.0 Comment #7 on issue 2335 by chip...@gmail.com: post-review basedir http://code.google.com/p/reviewboard/issues/detail?id=2335 Seems there are some legitimate use cases we should address. -- You