I don't see any error in here. That line indicates that we're doing an HTTP
POST to that URL but aren't sending any fields, which is correct. The files
are likely sent, but we don't log that on the console.

Does it just stop after that? Is there error information I'm just not
seeing?

Christian

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


On Tue, Dec 8, 2009 at 10:47 AM, Kunjal <kunjal.par...@gmail.com> wrote:

>
> Chris,
>
> We have now set up production server which will released in few days.
> During my final testing I found that when I post review, I can not
> post the diff. Looks like RB-Perforce integration issue.
> I read few blogs and you said that P4 should be in web server's path.
> However, I check httpd.conf and I do not see any P4 specific
> settings.
> Where else I can debug?
>
> Here is the error...
>
>
> >>> Uploading diff
> >>> HTTP POSTing to
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di
> ff/new/<http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di%0Aff/new/>:
> {}
>
> -Kunjal
>
>
>
>
> kapar...@ltsdo-kaparikh ~
> $ post-review.cygwin 250188 -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 250188
> >>> P4CLIENT is LTSDO-KAPARIKH
> >>> status is pending
> >>> Lines are:
> ['//depot/Sources/Integration/Automation_Scripts/1DDR_Test_Reques
> t_Script/1DDR_TestRequest.pl#1 - edit change 250188 (text) by
> kapar...@ltsdo-kap
> ARIKH\r\n']
> >>> Client LTSDO-KAPARIKH depot path
> //depot/Sources/Integration/Automation_Scri
> pts/1DDR_Test_Request_Script/1DDR_TestRequest.pl
> >>> Writing
> "//depot/Sources/Integration/Automation_Scripts/1DDR_Test_Request_Sc
> ript/1DDR_TestRequest.pl#1" to "/cygdrive/c/Temp/tmpBiyySU"
> >>> Old file: /cygdrive/c/Temp/tmpBiyySU New file
> c:/Perforce/Sources/Integratio
> n/Automation_Scripts/1DDR_Test_Request_Script/1DDR_TestRequest.pl
> >>> Attempting to create review request for 250188
> repository path pf-sj1-mob.sj.broadcom.com:1668
> >>> HTTP POSTing to
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/new/
> : {'repository_path': 'pf-sj1-mob.sj.broadcom.com:1668', 'changenum':
> '250188'}
> >>> Review request already exists. Updating it...
> >>> HTTP POSTing to
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/up
> date_from_changenum/<http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/up%0Adate_from_changenum/>:
> {}
> >>> Review request created
> >>> ---
> c:\Perforce\Sources\Integration\Automation_Scripts\1DDR_Test_Request_Scr
> ipt\1DDR_TestRequest.pl //depot/Sources/Integration/Automation_Scripts/
> 1DDR_Test
> _Request_Script/1DDR_TestRequest.pl#1
> +++ c:\Perforce\Sources\Integration\Automation_Scripts
> \1DDR_Test_Request_Script\
> 1DDR_TestRequest.pl     2009-12-08 10:23:30
> @@ -12,6 +12,20 @@
>  use MIME::Lite;
>  use CGI qw/:standard/;
>  use File::Basename;
> +#!/usr/bin/perl
>
> +#------------------------------------------------------------------------------
>
> +#Script Name: 1DDR_TestRequest.pl
> +#Usage:The script is used to get list of zip files from the latest CL
> +#from the path "\\\\brcm-irv\\dfs\\projects\\mcsi_2ddr\\programname"
> +#and send the mail notification for the 1DDR Test Request.
> +#Arguments: The script takes one argument which is the
> +#Branch/Program name/SDB name.
>
> +#------------------------------------------------------------------------------
>
> +#use warnings;
> +#use strict;
> +use MIME::Lite;
> +use CGI qw/:standard/;
> +use File::Basename;
>
>  use Data::Dumper;
>
>
> #------------------------------------------------------------------------------
>
>
> >>> Uploading diff
> >>> HTTP POSTing to
> http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di
> ff/new/<http://rb-sj1-1.sj.broadcom.com/api/json/reviewrequests/5/di%0Aff/new/>:
> {}
>
>
>
>
> --
> 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<reviewboard%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

Reply via email to