Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread chuck j
Hi David, As you mentioned that adding prefix ssl to --p4-port is not required, i did that but i got message which ask me add ssl to P4PORT. DEBUG:root:Checking for a Perforce repository... DEBUG:root:Running: p4 info DEBUG:root:Running: diff --version DEBUG:root:repository info: Path:

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread David Trowbridge
post-review will execute the 'p4' command using the port that you're passing into --p4-port. If you have a valid P4PORT environment variable that you'd prefer to use, don't pass --p4-port. Since it looks like the server at pf-sa-perforce:19773 refuses non-SSL connections, then you probably do need

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread chuck j
But adding ssl gives me HTTP POSTing to http://svrb1.company.com/svrb/api/review-requests/: {'changenum': '191316', 'submit_as': 'chuck', 'repository': 'pf-sa-perforce:19773'} Got API Error 206 (HTTP code 400): The repository path specified is not in the list of known repositories Error

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread David Trowbridge
This is where you'd use a .reviewboardrc file with the REPOSITORY setting or the --repository-url parameter. Please read the post-review documentation for details. -David On Sat, Apr 20, 2013 at 12:26 AM, chuck j cjerry2...@gmail.com wrote: But adding ssl gives me HTTP POSTing to

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread chuck j
Thanks i will try that option too. Please correct me if the .reviewboardrc content are correct. REPOSITORY = 'http://ssl:pf-sa-perforce:19773/' On Saturday, April 20, 2013 12:59:52 PM UTC+5:30, David Trowbridge wrote: This is where you'd use a .reviewboardrc file with the REPOSITORY setting

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread David Trowbridge
The contents within the quotes on that line should be the name that you assigned to your repository entry in the reviewboard admin. If you named the repository http://ssl:pf-sa-perforce:19773/ then that would be correct, but that's kind of a funny name. I really can't say whether or not it's

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-20 Thread chuck j
Thanks David and I really appreciate your responses. I did searched review board forum to get the response for such basic questions when there are complex system are in place. Failing to get the answer we have emailed you for the answer. Unless we start using the tool within the organization

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-19 Thread David Trowbridge
post-review (and rbt post) shouldn't require the ssl: prefix, but you probably will want to create a .reviewboardrc with the repository name. -David On Fri, Apr 19, 2013 at 1:31 AM, chuck j cjerry2...@gmail.com wrote: Hi All, For ssl enabled perforce, while posting a review request, is it

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-19 Thread chuck j
Hi David, But while posting to non ssl base reviewboard i didn't require .reviewbaordrc. why it's require for this? where this file should be present. What should be content of .reviewboardrc. -chuck. On Friday, April 19, 2013 2:05:56 PM UTC+5:30, David Trowbridge wrote: post-review (and

Re: post-review is it mandatory to pass ssl prefix in --p4-port option.

2013-04-19 Thread David Trowbridge
If the p4port on the client matches that on the server, it will use that to figure out which repository to post against.  If it doesn't (which is what you're describing), you need to specify it manually, either with command-line arguments or a .reviewboardrc file. -David On Fri, Apr 19, 2013