Re: AD Settings in RB

2010-04-29 Thread Christian Hammond
Hi,

I was hoping someone else would respond to this. AD is a
contributor-supplied feature, which I have little personal knowledge of.

We make use of the Python ldap and DNS modules in order to handle
authentication. I don't know what is needed to support multiple security
groups, whether we need to modify code to allow for it or what. I'd love to
give you a better answer, but this is just outside my area of expertise. I'd
be fine with providing such support in our ActiveDirectory backend if
someone with the ability to test this could provide a patch.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Wed, Apr 28, 2010 at 9:43 PM, Kunjal kunjal.par...@gmail.com wrote:

 Can someone please replay as soon as possible?


 On Apr 27, 5:42 pm, Kunjal kunjal.par...@gmail.com wrote:
  We have RB 1.0.5.1 installed.
  We have configured AD settings so all the users can login easily.
  Now, the issue is that we want to restrict users to only few
  departments in our company.
  So, we have created new windows security groups which has nested
  multiple department names in it.
  After doing this, the users of this departments can not access RB.
  If we put one department group then it works. But nesting of
  departments does not work.
  Does the RB 1.0.5.1 has ability to put multiple windows security
  groups for access/authentication? Is nesting possible?
 
  --
  Want to help the Review Board project? Donate today athttp://
 www.reviewboard.org/donate/
  Happy user? Let us know athttp://www.reviewboard.org/users/
  -~--~~~~--~~--~--~---
  To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
  For more options, visit this group athttp://
 groups.google.com/group/reviewboard?hl=en

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en


-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: ereviewboard plugin for Eclipse

2010-04-29 Thread Markus Knittig
On 03/24/2010 05:23 PM, Stodge wrote:
 Ah! I'm definitely using the right username so maybe it doesn't
 support accessing an RB site using Apache's basic auth. Thanks for the
 update!

The newest version should support this. Please update.
If it still doesn't work feel free to raise an issue on the github issue
tracker. Sorry for the late answer, I don't read this list every day...

Best regards,
 Markus

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: Install ReviewBoard 1.5. Stucked when create site

2010-04-29 Thread Christian Hammond
Hi,

How did you install Review Board? You never installed another version of it?

Did you manually install Django Evolution?

I've never seen this problem before. It almost looks like it's trying to use
an older version of Review Board's model sets at some point in the
installation.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Thu, Apr 29, 2010 at 1:49 AM, Le lostinheaven@gmail.com wrote:

 This is my first time install Reveiw Board.
 My install environment:
 Windows XP SP2
Apache 2.2.11
MySQL 5.0.51b

 I installed followed dependencies
python-2.5.4.msi
setuptools-0.6c11.win32-py2.5.exe ( Python Setuptools
 Installer)
patch-2.5.9-7-setup.exe (GNU patch)
memcached-1.2.6-win32-bin.zip (and python-memcached binding
 also)
PIL-1.1.7.win32-py2.5.exe
pycrypto-2.1.0.win32-py2.5.exe
MySQL-python-1.2.2.win32-py2.5.exe
py25-pysvn-svn161-1.7.0-1177.exe
mod_python-3.3.1.win32-py2.5-Apache2.2.exe

 after first few times failed, I install those two
RBTools
simplejson

 when i run the command to create the site ( rb-site install D:/tools/
 reviewboardsite),  to the last step, it always shows followed waring,
 
 * Installing the site...
 Building site directories ... OK
 Building site configuration files ... OK
 Creating database ... Project signature has changed - an evolution is
 required
 OK
 Performing migrations ... The stored evolutions do not completely
 resolve all mo
 del changes.
 Run `./manage.py evolve --hint` to see a suggestion for the changes
 required.

 The following are the changes that could not be resolved:
 In model reviews.DefaultReviewer:
Field 'repository' has been deleted
 In model scmtools.Repository:
Field 'visible' has been deleted
Field 'raw_file_url' has been deleted
 In model diffviewer.DiffSet:
Field 'basedir' has been deleted
 _

 Does any one have the same experiences?
 Any one knows how to fix it ?

 Thanks

 Tracy

 --
 Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to
 reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: AD Settings in RB

2010-04-29 Thread grimbeaver
You might be able to do it by setting the group name or a custom
search root in the settings but I'm not real familiar with the AD
scheme.  Currently I'm just using the Find DC from DNS option.  The
other thing to remember is that a user is not created in reviewboard
until after the user logs in once.  This makes assigning privileges
kind of difficult.

On Apr 29, 2:50 am, Christian Hammond chip...@chipx86.com wrote:
 Hi,

 I was hoping someone else would respond to this. AD is a
 contributor-supplied feature, which I have little personal knowledge of.

 We make use of the Python ldap and DNS modules in order to handle
 authentication. I don't know what is needed to support multiple security
 groups, whether we need to modify code to allow for it or what. I'd love to
 give you a better answer, but this is just outside my area of expertise. I'd
 be fine with providing such support in our ActiveDirectory backend if
 someone with the ability to test this could provide a patch.

 Christian

 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com





 On Wed, Apr 28, 2010 at 9:43 PM, Kunjal kunjal.par...@gmail.com wrote:
  Can someone please replay as soon as possible?

  On Apr 27, 5:42 pm, Kunjal kunjal.par...@gmail.com wrote:
   We have RB 1.0.5.1 installed.
   We have configured AD settings so all the users can login easily.
   Now, the issue is that we want to restrict users to only few
   departments in our company.
   So, we have created new windows security groups which has nested
   multiple department names in it.
   After doing this, the users of this departments can not access RB.
   If we put one department group then it works. But nesting of
   departments does not work.
   Does the RB 1.0.5.1 has ability to put multiple windows security
   groups for access/authentication? Is nesting possible?

   --
   Want to help the Review Board project? Donate today athttp://
 www.reviewboard.org/donate/
   Happy user? Let us know athttp://www.reviewboard.org/users/
   -~--~~~~--~~--~--~---
   To unsubscribe from this group, send email to
  reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr 
  oups.com
   For more options, visit this group athttp://
  groups.google.com/group/reviewboard?hl=en

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
  Happy user? Let us know athttp://www.reviewboard.org/users/
  -~--~~~~--~~--~--~---
  To unsubscribe from this group, send email to
  reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr 
  oups.com
  For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

 --
 Want to help the Review Board project? Donate today 
 athttp://www.reviewboard.org/donate/
 Happy user? Let us know athttp://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to 
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group 
 athttp://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: HTTPS support

2010-04-29 Thread Kunjal
What I found is that if I do not change post-review.py script at all,
I can post the review but I can not post the diff.

kapar...@ltsdo-kaparikh ~
$ post-review.cygwin 275221 -d

This tool looks at the given Pending-CL#  posts it for review.
[Version: 200911
13_1141]

 Repository info 'Path: pf-sj1-mob.sj.broadcom.com:1668, Base path: None, Sup
ports changesets: True'
 Generating diff for changenum 275221
 P4CLIENT is LTSDO-KAPARIKH
 status is pending
 Lines are: ['//depot/Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/
ccb_cq_analysis.pl#5 - edit change 275221 (text) by kapar...@ltsdo-
KAPARIKH\r\n'
]
 Client LTSDO-KAPARIKH depot path //depot/Sources/Integration/Automation_Scri
pts/CCB_CQ_Analysis/ccb_cq_analysis.pl
 Writing //depot/Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/ccb_
cq_analysis.pl#5 to /cygdrive/c/Temp/tmpE2AEBQ
 Old file: /cygdrive/c/Temp/tmpE2AEBQ New file c:/Perforce/sysrel_scripts/CCB
_CQ_Analysis/ccb_cq_analysis.pl
 Attempting to create review request for 275221
repository path pf-sj1-mob.sj.broadcom.com:1668
 HTTP POSTing to http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/n
ew/: {'repository_path': 'pf-sj1-mob.sj.broadcom.com:1668',
'changenum': '275221
'}
 Review request created
 --- c:\Perforce\sysrel_scripts\CCB_CQ_Analysis\ccb_cq_analysis.pl   //depot/
Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/
ccb_cq_analysis.pl#5
+++ c:\Perforce\sysrel_scripts\CCB_CQ_Analysis
\ccb_cq_analysis.pl   2010-04-
29 09:57:09
@@ -13,7 +13,16 @@ use Win32::OLE::Const 'Microsoft Excel';
 $Win32::OLE::Warn = 3;  # die on errors...
 use Tie::File;
 
###
-#All Variables declaration
 Uploading diff
 HTTP POSTing to http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/5
4/diff/new/: {}
Error uploading diff: One or more fields had errors (105)
Your review request still exists, but the diff is not attached.


On Apr 28, 5:11 pm, Christian Hammond chip...@chipx86.com wrote:
 It should work with https fine. I use it with https every day. Can you paste
 the output when using --debug with post-review?

 Christian

 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com





 On Wed, Apr 28, 2010 at 4:23 PM, Kunjal kunjal.par...@gmail.com wrote:
  Here is the scenairo happening.
  We are using post-review.py file to post review.
  We close down HTTP port and open up HTTPS port.
  Now, when I post the review, I can not post review at all !!
  What I ovserve that in post-review.py, there are HTTP post methods are
  used and
  methonds like: self.http_post, http_get etc. are used.

  Do we have post-review.py which supports HTTPS?

  Thanks
  Kunjal

  On Apr 27, 10:59 pm, Christian Hammond chip...@chipx86.com wrote:
   Review Board doesn't know or care anything about http vs. https. If
  you're
   having issues with post-review, I suspect it's due to your existing
   configuration referencing the http URL, which presumably doesn't work
  now. I
   don't know how you have it configured, so it may be post-review.py, or a
   .reviewboardrc, or a property on the repository.

   Christian

   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.reviewboard.org
   VMware, Inc. -http://www.vmware.com

   On Tue, Apr 27, 2010 at 10:57 PM, Kunjal kunjal.par...@gmail.com
  wrote:
Chris,

Changes to Apache installation is done and we have moved our RB
website to HTTPS.
However, now when I post the review now, I can not post the diff.
Does the HTTPS interact the way diff is done ?
Do I need to update post-review.py script to update the url field to
point to https?

Thanks a lot.
Kunjal

On Apr 27, 10:45 pm, Christian Hammond chip...@chipx86.com wrote:
 Do you mean the Apache installation? That should work fine. Review
  Board
 doesn't really care whether it's being used with https or http. It's
  a
 matter of configuring your Apache installation and updating your
 .reviewboardrc files (or whatever mechanism you're using for
  post-review
 support) to point to the https URL instead of the http URL.

 Christian

 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com

 On Tue, Apr 27, 2010 at 5:38 PM, Kunjal kunjal.par...@gmail.com
  wrote:
  We have RB 1.0.5.1 in production and it is http now.
  We want to move it to https.
  So, we did apache server settings and move RB to https.
  But now I can post the review but I can not post the diff.
  It gives me error.
  Is HTTTPS supported in 1.0.5.1?

  Because I see that in the roadmap of RB 1.5, it says ...

  SSH and HTTPS support -- commited.

  Thanks
  Kunjal

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
  

Re: AD Settings in RB

2010-04-29 Thread Kunjal
Thanks.
Is this feature will get supported in near future?

On Apr 29, 4:52 am, grimbeaver tbrez...@gmail.com wrote:
 You might be able to do it by setting the group name or a custom
 search root in the settings but I'm not real familiar with the AD
 scheme.  Currently I'm just using the Find DC from DNS option.  The
 other thing to remember is that a user is not created in reviewboard
 until after the user logs in once.  This makes assigning privileges
 kind of difficult.

 On Apr 29, 2:50 am, Christian Hammond chip...@chipx86.com wrote:





  Hi,

  I was hoping someone else would respond to this. AD is a
  contributor-supplied feature, which I have little personal knowledge of.

  We make use of the Python ldap and DNS modules in order to handle
  authentication. I don't know what is needed to support multiple security
  groups, whether we need to modify code to allow for it or what. I'd love to
  give you a better answer, but this is just outside my area of expertise. I'd
  be fine with providing such support in our ActiveDirectory backend if
  someone with the ability to test this could provide a patch.

  Christian

  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.reviewboard.org
  VMware, Inc. -http://www.vmware.com

  On Wed, Apr 28, 2010 at 9:43 PM, Kunjal kunjal.par...@gmail.com wrote:
   Can someone please replay as soon as possible?

   On Apr 27, 5:42 pm, Kunjal kunjal.par...@gmail.com wrote:
We have RB 1.0.5.1 installed.
We have configured AD settings so all the users can login easily.
Now, the issue is that we want to restrict users to only few
departments in our company.
So, we have created new windows security groups which has nested
multiple department names in it.
After doing this, the users of this departments can not access RB.
If we put one department group then it works. But nesting of
departments does not work.
Does the RB 1.0.5.1 has ability to put multiple windows security
groups for access/authentication? Is nesting possible?

--
Want to help the Review Board project? Donate today athttp://
  www.reviewboard.org/donate/
Happy user? Let us know athttp://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to
   reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr
oups.com
For more options, visit this group athttp://
   groups.google.com/group/reviewboard?hl=en

   --
   Want to help the Review Board project? Donate today at
  http://www.reviewboard.org/donate/
   Happy user? Let us know athttp://www.reviewboard.org/users/
   -~--~~~~--~~--~--~---
   To unsubscribe from this group, send email to
   reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr
oups.com
   For more options, visit this group at
  http://groups.google.com/group/reviewboard?hl=en

  --
  Want to help the Review Board project? Donate today 
  athttp://www.reviewboard.org/donate/
  Happy user? Let us know athttp://www.reviewboard.org/users/
  -~--~~~~--~~--~--~---
  To unsubscribe from this group, send email to 
  reviewboard+unsubscr...@googlegroups.com
  For more options, visit this group 
  athttp://groups.google.com/group/reviewboard?hl=en

 --
 Want to help the Review Board project? Donate today 
 athttp://www.reviewboard.org/donate/
 Happy user? Let us know athttp://www.reviewboard.org/users/
 -~--~~~~--~~--~--~---
 To unsubscribe from this group, send email to 
 reviewboard+unsubscr...@googlegroups.com
 For more options, visit this group 
 athttp://groups.google.com/group/reviewboard?hl=en- Hide quoted text -

 - Show quoted text -

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: No new line at end of file crash diffviewer

2010-04-29 Thread Chris Clark

Christian Hammond wrote:

What version of patch are you guys running?


I'm running Ubuntu 8.04.3 LTS - I have patch 2.5.9


   ing...@reviews:~$ cat /etc/*elease*
   DISTRIB_ID=Ubuntu
   DISTRIB_RELEASE=8.04
   DISTRIB_CODENAME=hardy
   DISTRIB_DESCRIPTION=Ubuntu 8.04.3 LTS
   ing...@reviews:~$ patch -v
   patch 2.5.9
   Copyright (C) 1988 Larry Wall
   Copyright (C) 2003 Free Software Foundation, Inc.

   This program comes with NO WARRANTY, to the extent permitted by law.
   You may redistribute copies of this program
   under the terms of the GNU General Public License.
   For more information about these matters, see the file named COPYING.

   written by Larry Wall and Paul Eggert




And do the diffs have any indication at the end saying that there's no 
newline at the end?


Yes and no ;-) For me it depends on the diff tool used. E.g. some 
diff tools dump:


   \ No newline at end of file

at the point the newline is missing to stdout, for a single file diff 
this is at the end. For multiple file diffs this could be in the middle 
of the diff.


Some diff tools (e.g. Solaris) dump that information to stderr, if you 
redirect both stdout and stderr the position of that message can vary 
wildly.


Some diff tools do nothing. Which means that multi-file diffs with the 
bad file in the middle look corrupted as the next file starts on the 
last line of the previous files diff.





Also, what SCM?


I'm using p2 which has the option to use the platform provided diff (so 
sort of a lottery on the newline detection) OR perform diffs server side 
(which gives consistent output). The default diff format that is used 
for p2 is the default diff output (i.e. not a unified diff).


I'm not sure the SCM relevant, I think the issue is the diff and patch 
tools.


Chris

--
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


.reviewboardrc on windows

2010-04-29 Thread Noah Campbell
Hi Folks,

The docs (http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/) 
are incorrect when specifying the location of .reviewboardrc on windows.  It 
should be: %USERPROFILE%\Application Data instead of 
$USERPROFILE\LocalSettings\Application Data

Great product, keep up the good work!

Thank you,
-Noah

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Using Reviewboard with Mercurial

2010-04-29 Thread Andrew
Hi.  I have a few questions for the reviewboard gurus.  I'm interested
in using reviewboard with Mercurial.

1. Does the --parent option work with Mercurial?  It's documented as
only working with Git.  If so, is there a workaround for usage with
Mercurial or a suggestion on how to review changesets whose parents
are not in the repository that Reviewboard is pointing to?

2. Often we will have a developer work on a series of changesets on a
branch.  Is it possible with Reviewboard to automatically create
review requests for a series of consecutive changesets, but tag them
so that they are all linked in some manner?

Thanks.

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: HTTPS support

2010-04-29 Thread Christian Hammond
Ah so you are running a custom one. It looks like it's hard-coded to use the
http:// path. You need to make it set to use https:// for your Review Board
URL.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Thu, Apr 29, 2010 at 10:05 AM, Kunjal kunjal.par...@gmail.com wrote:

 What I found is that if I do not change post-review.py script at all,
 I can post the review but I can not post the diff.

 kapar...@ltsdo-kaparikh ~
 $ post-review.cygwin 275221 -d

 This tool looks at the given Pending-CL#  posts it for review.
 [Version: 200911
 13_1141]

  Repository info 'Path: pf-sj1-mob.sj.broadcom.com:1668, Base path:
 None, Sup
 ports changesets: True'
  Generating diff for changenum 275221
  P4CLIENT is LTSDO-KAPARIKH
  status is pending
  Lines are:
 ['//depot/Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/
 ccb_cq_analysis.pl#5 - edit change 275221 (text) by kapar...@ltsdo-
 KAPARIKH\r\n'
 ]
  Client LTSDO-KAPARIKH depot path
 //depot/Sources/Integration/Automation_Scri
 pts/CCB_CQ_Analysis/ccb_cq_analysis.pl
  Writing
 //depot/Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/ccb_
 cq_analysis.pl#5 to /cygdrive/c/Temp/tmpE2AEBQ
  Old file: /cygdrive/c/Temp/tmpE2AEBQ New file
 c:/Perforce/sysrel_scripts/CCB
 _CQ_Analysis/ccb_cq_analysis.pl
  Attempting to create review request for 275221
 repository path pf-sj1-mob.sj.broadcom.com:1668
  HTTP POSTing to
 http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/n
 ew/ http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/n%0Aew/:
 {'repository_path': 'pf-sj1-mob.sj.broadcom.com:1668',
 'changenum': '275221
 '}
  Review request created
  --- c:\Perforce\sysrel_scripts\CCB_CQ_Analysis\ccb_cq_analysis.pl
 //depot/
 Sources/Integration/Automation_Scripts/CCB_CQ_Analysis/
 ccb_cq_analysis.pl#5
 +++ c:\Perforce\sysrel_scripts\CCB_CQ_Analysis
 \ccb_cq_analysis.pl   2010-04-
 29 09:57:09
 @@ -13,7 +13,16 @@ use Win32::OLE::Const 'Microsoft Excel';
  $Win32::OLE::Warn = 3;  # die on errors...
  use Tie::File;

 ###
 -#All Variables declaration
  Uploading diff
  HTTP POSTing to
 http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/5
 4/diff/new/http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/5%0A4/diff/new/:
 {}
 Error uploading diff: One or more fields had errors (105)
 Your review request still exists, but the diff is not attached.


 On Apr 28, 5:11 pm, Christian Hammond chip...@chipx86.com wrote:
  It should work with https fine. I use it with https every day. Can you
 paste
  the output when using --debug with post-review?
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.reviewboard.org
  VMware, Inc. -http://www.vmware.com
 
 
 
 
 
  On Wed, Apr 28, 2010 at 4:23 PM, Kunjal kunjal.par...@gmail.com wrote:
   Here is the scenairo happening.
   We are using post-review.py file to post review.
   We close down HTTP port and open up HTTPS port.
   Now, when I post the review, I can not post review at all !!
   What I ovserve that in post-review.py, there are HTTP post methods are
   used and
   methonds like: self.http_post, http_get etc. are used.
 
   Do we have post-review.py which supports HTTPS?
 
   Thanks
   Kunjal
 
   On Apr 27, 10:59 pm, Christian Hammond chip...@chipx86.com wrote:
Review Board doesn't know or care anything about http vs. https. If
   you're
having issues with post-review, I suspect it's due to your existing
configuration referencing the http URL, which presumably doesn't work
   now. I
don't know how you have it configured, so it may be post-review.py,
 or a
.reviewboardrc, or a property on the repository.
 
Christian
 
--
Christian Hammond - chip...@chipx86.com
Review Board -http://www.reviewboard.org
VMware, Inc. -http://www.vmware.com
 
On Tue, Apr 27, 2010 at 10:57 PM, Kunjal kunjal.par...@gmail.com
   wrote:
 Chris,
 
 Changes to Apache installation is done and we have moved our RB
 website to HTTPS.
 However, now when I post the review now, I can not post the diff.
 Does the HTTPS interact the way diff is done ?
 Do I need to update post-review.py script to update the url field
 to
 point to https?
 
 Thanks a lot.
 Kunjal
 
 On Apr 27, 10:45 pm, Christian Hammond chip...@chipx86.com
 wrote:
  Do you mean the Apache installation? That should work fine.
 Review
   Board
  doesn't really care whether it's being used with https or http.
 It's
   a
  matter of configuring your Apache installation and updating your
  .reviewboardrc files (or whatever mechanism you're using for
   post-review
  support) to point to the https URL instead of the http URL.
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.reviewboard.org
  VMware, 

Issue 1632 in reviewboard: Alphabetial sorting of review requests should not be case sensitive

2010-04-29 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 1632 by yodan.ta...@gmail.com: Alphabetial sorting of review  
requests should not be case sensitive

http://code.google.com/p/reviewboard/issues/detail?id=1632

*NOTE: Do not post confidential information in this bug report.*

What version are you running?
1.0.6

What's the URL of the page containing the problem?
It's on the intranet, so it probably won't help much.
http://[review-board-host]/reviews/r/?page=4sort=submitter

What steps will reproduce the problem?
1. Go to a list of review requests, e.g. All review requests
2. Sort the list by the submitter name (click on Submitter title)

What is the expected output? What do you see instead?
I can see the requests sorted by the submitter names in this order (note
the capitalization):
Alexey, Jenia, Lev, alex, boris, dan...
They should obviously be sorted this way:
alex, Alexey, boris, dan, Jenia, Lev...

What operating system are you using? What browser?
Client: Firefox 3.6.3 on Windows XP 64bit
Server: Apache/2.2.12 (Ubuntu) Server


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1444 in reviewboard: DLL load failed on trying to create SVN repository

2010-04-29 Thread reviewboard


Comment #5 on issue 1444 by evan.leonard: DLL load failed on trying to  
create SVN repository

http://code.google.com/p/reviewboard/issues/detail?id=1444

Dang... ran through a whole annoying installation on windows and then  
finally hit this.  Would you atleast add a

prominent notice that SVN on Windows Server 2003 isn't working due to pysvn?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Issue 1634 in reviewboard: New group request for replication team

2010-04-29 Thread reviewboard

Status: New
Owner: 
Labels: Type-Enhancement Priority-Medium

New issue 1634 by itisashu: New group request for replication team
http://code.google.com/p/reviewboard/issues/detail?id=1634

*NOTE: If you have a patch, please submit it to
http://reviews.reviewboard.org/


What version are you running?


What's the URL of the page this enhancement relates to, if any?


Describe the enhancement and the motivation for it.
It will be useful to create a new group specific for replication team. It
will enable use to send review request to only replication team.

What operating system are you using? What browser?


Please provide any additional information below.


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1437 in reviewboard: ReviewBoard Installation failed on Windows XP SP2 VMWare.

2010-04-29 Thread reviewboard


Comment #6 on issue 1437 by vera1171: ReviewBoard Installation failed on  
Windows XP SP2 VMWare.

http://code.google.com/p/reviewboard/issues/detail?id=1437

I encounter the same problem.Is there anybody install reviewboard manually  
in LAN

without usting setuptool?

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.