Re: Post-review error when uploading the diff

2014-07-31 Thread Christian Hammond
Looks like you’re using Review Board with direct access to a git repository. It 
could be on that end or on the RBTools side.

For Review Board, make sure that:

1) The git repository Review Board is using is local, and not some remote URL 
(or you have the raw URL configured correctly).
2) If it is a local git repository, it’s up-to-date.

For the RBTools side, check that:

1) You’re using —tracking-branch and —parent if needed. If your change is based 
on commits from a remote other than origin/master, then you need 
—tracking-branch.
2) Your refs on the tracking branches are pushed.
3) Your branch has the latest from the tracking branch merged into it.

Also, knowing what versions of Review Board and RBTools you’re using would be 
helpful.

Christian

-- 
Christian Hammond - christ...@beanbaginc.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

On July 31, 2014 at 12:31:59 PM, Zhongting Hu (zhongting...@gmail.com) wrote:

Hi, Guys

This sounds like a problem which has been resolved before by some of the guys, 
but I just wonder how to figure it out.

My problem is got some failed when uploading the diff..  Some people here says 
it's related with Permissions, so how can I figure out the files which has the 
permission issues here?

Many Thanks.

---
DEBUG:root:Checking for a Git repository...
DEBUG:root:Running: git rev-parse --git-dir
DEBUG:root:Running: git config core.bare
DEBUG:root:Running: git rev-parse --show-toplevel
DEBUG:root:Running: git symbolic-ref -q HEAD
DEBUG:root:Running: git config --get git-p4.port
DEBUG:root:Command exited with rc 1: ['git', 'config', '--get', 'git-p4.port']
---
DEBUG:root:Running: git config --get branch.reviewboard_testing.merge
DEBUG:root:Command exited with rc 1: ['git', 'config', '--get', 
'branch.reviewboard_testing.merge']
---
DEBUG:root:Running: git config --get branch.reviewboard_testing.remote
DEBUG:root:Command exited with rc 1: ['git', 'config', '--get', 
'branch.reviewboard_testing.remote']
---
DEBUG:root:Running: git config --get remote.origin.url
DEBUG:root:repository info: Path:X, Base path: , Supports changesets: False
 Finished checking the repository type.
DEBUG:root:Running: git config --get reviewboard.url
 HTTP GETting api/info/
== HTTP Authentication Required
Enter authorization information for Web API at reviewboard.loyal3.com
Password: 
DEBUG:root:Running: git rev-parse refs/heads/reviewboard_testing
DEBUG:root:Running: git merge-base 810d037f97eab6a4b6d405bf57f86e52cccab043 
origin/master
DEBUG:root:Running: git rev-parse f2561b898eb4288e4e073765be94971f2d02d1ee
DEBUG:root:Running: git status --porcelain
DEBUG:root:Running: git diff --no-color --full-index --no-ext-diff 
--ignore-submodules --no-renames 
f2561b898eb4288e4e073765be94971f2d02d1ee..810d037f97eab6a4b6d405bf57f86e52cccab043
 HTTP GETting api/
 HTTP GETting http://reviewboard.loyal3.com/api/info/
 Using the new web API
 Attempting to create review request on:iphone for None
 HTTP POSTing to http://reviewboard.loyal3.com/api/review-requests/: 
 {'repository': ''}
 Review request created
 Uploading diff, size: 58248
 HTTP POSTing to X/api/review-requests/3528/diffs/: {}
 Got API Error 105 (HTTP code 400): One or more fields had errors
 Error data: {u'fields': {u'path': [u'error: unable to find 
 b09893aae2dfabe6e9591330a9e00be920f0b503\nfatal: git cat-file 
 b09893aae2dfabe6e9591330a9e00be920f0b503: bad file\n']}, u'stat': u'fail', 
 u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
--
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: post-review: Error uploading diff

2014-01-20 Thread Vlad Dovlekaev
We are currently using the RB 1.7.6 and having the similar problem using 
rbt post.

nntopencl046el$ rbt post -d
DEBUG:root:Checking for a Git repository...
DEBUG:root:Running: git rev-parse --git-dir
DEBUG:root:Running: git config core.bare
DEBUG:root:Running: git rev-parse --show-toplevel
DEBUG:root:Running: git symbolic-ref -q HEAD
DEBUG:root:Running: git config --get git-p4.port
DEBUG:root:Command exited with rc 1: ['git', 'config', '--get', 
'git-p4.port']
---
DEBUG:root:Running: git config --get branch.xx.merge
DEBUG:root:Running: git config --get branch.xx.remote
DEBUG:root:Running: git config --get remote.origin.url
DEBUG:root:repository info: Path: xx, Base path: , Supports changesets: 
False
DEBUG:root:Making HTTP GET request to http://xxx.xxx.com/cr/api/
DEBUG:root:Making HTTP GET request to http://xxx.xxx.com/cr/api/info/
DEBUG:root:Running: git rev-parse refs/heads/xx
DEBUG:root:Running: git merge-base e1d20e90c03a7591195ff08cb99934c0ea1b1934 
origin/xx
DEBUG:root:Running: git rev-parse e83d0646802305ec98b60606f907a655a92723b1
DEBUG:root:Running: git status --porcelain --untracked-files=no
DEBUG:root:Running: git diff --no-color --full-index --no-ext-diff 
--ignore-submodules --no-renames 
e83d0646802305ec98b60606f907a655a92723b1..e1d20e90c03a7591195ff08cb99934c0ea1b1934
 
-M
DEBUG:root:Making HTTP GET request to 
http://xxx.xxx.com/cr/api/review-requests/
DEBUG:root:Making HTTP POST request to 
http://xxx.xxx.com/cr/api/review-requests/
DEBUG:root:Making HTTP GET request to 
http://xxx.xxx.com/cr/api/review-requests/3852/diffs/
DEBUG:root:Making HTTP POST request to 
http://xxx.xxx.com/cr/api/review-requests/3852/diffs/
DEBUG:root:Making HTTP GET request to 
http://xxx.xxx.com/cr/api/review-requests/3852/draft/
Traceback (most recent call last):
  File /opt/tools/bin/rbt, line 8, in module
load_entry_point('RBTools==0.5.5', 'console_scripts', 'rbt')()
  File 
/opt/tools/lib/python2.7/site-packages/RBTools-0.5.5-py2.7.egg/rbtools/commands/main.py,
 
line 127, in main
command.run_from_argv([RB_MAIN] + args)
  File 
/opt/tools/lib/python2.7/site-packages/RBTools-0.5.5-py2.7.egg/rbtools/commands/__init__.py,
 
line 158, in run_from_argv
exit_code = self.main(*args) or 0
  File 
/opt/tools/lib/python2.7/site-packages/RBTools-0.5.5-py2.7.egg/rbtools/commands/post.py,
 
line 659, in main
submit_as=self.options.submit_as)
  File 
/opt/tools/lib/python2.7/site-packages/RBTools-0.5.5-py2.7.egg/rbtools/commands/post.py,
 
line 585, in post_request
return review_request.id, review_request.absolute_url
  File 
/opt/tools/lib/python2.7/site-packages/RBTools-0.5.5-py2.7.egg/rbtools/api/resource.py,
 
line 278, in __getattr__
raise AttributeError
AttributeError

However the draft review is created in the RB. The only problem is this 
error message that rbt post command is generating.  
Is it a known issue?  

Thanks.
Vlad

On Tuesday, August 20, 2013 6:35:03 PM UTC+3, Steven MacLeod wrote:

 The minimum Review Board version which fully supports the rbt commands is 
 RB 1.6.12. There were some changes to the web API in this version which 
 rbt relies on. Unfortunately, you won't be able to make use of most of rbt 
 without a newer version of Review Board.

 Admittedly, the error reporting here was terrible, but is something I'm 
 working on imporoving. 


 On Tue, Aug 20, 2013 at 8:43 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com javascript: wrote:

 Thank for the response David .

 I had rbtool 0.4.1 so rbt was not there so i installed 0.5.2 and got rbt 
 script  .

 To attach a arbitary file to existing review ,as per you i am using *rbt 
 attach 359 --filename=diff* (where 359 is ReviewID), it giving below 
 usage messge 


 hostname:/vobs/ms 167 % *rbt attach 359 --filename=diff*
 *Usage: rbt attach [options] review-request-id file*

 *rbt: error: Invalid number of arguments provided*


 So i tried the command as *rbt attach 359 diff* (where 359 is the 
 reviewID and diff is the filename) but it giving error * 
 CRITICAL:root: .Please have look and let me know what could be the issue .*

 *Note:The reviewBoard server version 1.6.3 and Rbtools is 0.5.2 ,Python 
 2.7 *

 rbt attach 359 diff
 CRITICAL:root:

 *below is the debug output :*

  169 % rbt attach 359 diff -d
 DEBUG:root:Checking for a Subversion repository...
 DEBUG:root:Checking for a Git repository...
 DEBUG:root:Checking for a Mercurial repository...
 DEBUG:root:Checking for a CVS repository...
 DEBUG:root:Checking for a Perforce repository...
 DEBUG:root:Checking for a Plastic repository...
 DEBUG:root:Checking for a ClearCase repository...
 DEBUG:root:Running: cleartool pwv -short
 DEBUG:root:Running: diff --version
 DEBUG:root:Running: cleartool lsview -full -properties -cview
 DEBUG:root:Running: cleartool describe -short vob:.
 DEBUG:root:Running: cleartool pwv -root
 DEBUG:root:repository info: Path: /vobs/ms-test, Base path: 
 /vobs/ms-test, Supports changesets: False
 DEBUG:root:Making 

Re: post-review error: WindowsError: [Error 2] The system cannot find the file specified

2013-09-10 Thread Safi Rehaman
Hi Akilesh,

I tried doing this, adding the DIFF.exe under tha %PATH% however no luck,,, 
i have my directory structure with a space C:\DEV SETPUP. could this be 
the reason.. if so please suggest a solution, i tried changing all the ENV 
variable to include double codes...

Regards/-
Safi

On Saturday, November 7, 2009 4:27:28 AM UTC+5:30, Akhilesh wrote:

 I have setup Reviewboard on Windows and now setting up post-review on 
 client machine so that I can start creating review requests. 

 I'm getting following error: 

 C:\p4post-review  25159 -d 
  p4 info 
  repository info: Path: ssb-sc.mimosasystems.com:1666, Base path: 
 None, Supports changesets: True 
  p4 counters 
  Generating diff for changenum 25159 
  p4 describe -s 25159 
  Processing edit of 
 //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/IndexDisposer.cs
  

  Writing 
 //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/IndexDisposer.cs#4
  
 to c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3- 
  p4 print -o c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3- -q 
 //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/IndexDisposer.cs#4
  

  Writing 
 //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/IndexDisposer.cs#5
  
 to c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh 
  p4 print -o c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh -q 
 //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/IndexDisposer.cs#5
  

  diff -urNp c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3- 
 c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh 
 Traceback (most recent call last): 
   File C:\Python25\Scripts\post-review-script.py, line 8, in 
 module 
 load_entry_point('RBTools==0.2beta2', 'console_scripts', 'post- 
 review')() 
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 
 2511, in main 
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 
 1358, in diff 
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 
 1609, in _changenum_diff 
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 
 1639, in _do_diff 
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 
 2112, in execute 
   File C:\Python25\lib\subprocess.py, line 593, in __init__ 
 errread, errwrite) 
   File C:\Python25\lib\subprocess.py, line 793, in _execute_child 
 startupinfo) 
 WindowsError: [Error 2] The system cannot find the file specified 

 C:\p4 


 Any ideas? 



-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: post-review: Error uploading diff

2013-08-20 Thread Akshaya Samantaray
Thank for the response David .

I had rbtool 0.4.1 so rbt was not there so i installed 0.5.2 and got rbt 
script  .

To attach a arbitary file to existing review ,as per you i am using *rbt 
attach 359 --filename=diff* (where 359 is ReviewID), it giving below usage 
messge 


hostname:/vobs/ms 167 % *rbt attach 359 --filename=diff*
*Usage: rbt attach [options] review-request-id file*
*
*
*rbt: error: Invalid number of arguments provided*


So i tried the command as *rbt attach 359 diff* (where 359 is the 
reviewID and diff is the filename) but it giving error * CRITICAL:root: 
.Please have look and let me know what could be the issue .*
*
*
*Note:The reviewBoard server version 1.6.3 and Rbtools is 0.5.2 ,Python 2.7 
*

rbt attach 359 diff
CRITICAL:root:

*below is the debug output :*

 169 % rbt attach 359 diff -d
DEBUG:root:Checking for a Subversion repository...
DEBUG:root:Checking for a Git repository...
DEBUG:root:Checking for a Mercurial repository...
DEBUG:root:Checking for a CVS repository...
DEBUG:root:Checking for a Perforce repository...
DEBUG:root:Checking for a Plastic repository...
DEBUG:root:Checking for a ClearCase repository...
DEBUG:root:Running: cleartool pwv -short
DEBUG:root:Running: diff --version
DEBUG:root:Running: cleartool lsview -full -properties -cview
DEBUG:root:Running: cleartool describe -short vob:.
DEBUG:root:Running: cleartool pwv -root
DEBUG:root:repository info: Path: /vobs/ms-test, Base path: /vobs/ms-test, 
Supports changesets: False
DEBUG:root:Making HTTP GET request to https://ppsappdev02:443/api/
Traceback (most recent call last):
  File /clearcase/tools/bin/rbt, line 9, in module
load_entry_point('RBTools==0.5.2', 'console_scripts', 'rbt')()
  File 
/usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/main.py,
 
line 106, in main
command.run_from_argv([RB_MAIN] + args)
  File 
/usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/__init__.py,
 
line 157, in run_from_argv
exit_code = self.main(*args) or 0
  File 
/usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 
line 67, in main
request = self.get_review_request(request_id, api_root)
  File 
/usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 
line 55, in get_review_request
request = api_root.get_review_request(review_request_id=request_id)
  File 
/usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/api/resource.py,
 
line 278, in __getattr__
raise AttributeError
AttributeError











On Tuesday, August 20, 2013 1:01:04 AM UTC+5:30, David Trowbridge wrote:

 You can't use post-review --diff-filename to upload arbitrary files. The 
 newer rbt commands will let you upload file attachments with rbt attach 
 review request ID --filename=filename

 For your existing 


 On Mon, Aug 19, 2013 at 4:35 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com javascript: wrote:

 Hi David ,

 My object is just to upload a file which  can be any file (*.gif  or 
 *.bmp ) (need not be a diff file) using commandline post-review 
 --diff-filename=file_name  to review-board server in our clear-case 
 environment 

 How to do it using post-review in clearcase environment ? let me know if 
  any other option to post-review  is there ?

 Thanks



 On Monday, August 19, 2013 4:39:59 PM UTC+5:30, Akshaya Samantaray wrote:

 HI David ,

 Thanks for the response .

 d1.diff is just diff file i have created manually i.e the diff between 
 my dev branch changes  Integration branch changes redirected to file 
 called  d1.diff and i am trying to upload this manually created diff file 
 d1.diff using  *post-review --diff-filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .*
 *
 *
 I can upload this manually created diff file d1.diff file from GUI in 
 reviewboard  page  but not sure why its not uploading using  from command 
 line using  post-review diff-**filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .I am not getting any clue 
 from the post-review debug log or reviewboard server log .

 Do i need to have any setup for apache or reviewboard server side 
  configuration ?where can i check to get the exact clue for this error ? 

 Please have a look and get back to me .I am waiting for your response .

 Thanks

  HTTP POSTing to 
  https://MyRbServer/api/**review-requests/370/diffs/https://myrbserver/api/review-requests/370/diffs/:
   
 {'basedir': u'/view/ppreleng_**reviewboardAdmin/vobs/ms-test'**}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. 

Re: post-review: Error uploading diff

2013-08-20 Thread Steven MacLeod
The minimum Review Board version which fully supports the rbt commands is RB
1.6.12. There were some changes to the web API in this version which rbt
relies on. Unfortunately, you won't be able to make use of most of rbt
without a newer version of Review Board.

Admittedly, the error reporting here was terrible, but is something I'm
working on imporoving.


On Tue, Aug 20, 2013 at 8:43 AM, Akshaya Samantaray 
akshaya.samanta...@gmail.com wrote:

 Thank for the response David .

 I had rbtool 0.4.1 so rbt was not there so i installed 0.5.2 and got rbt
 script  .

 To attach a arbitary file to existing review ,as per you i am using *rbt
 attach 359 --filename=diff* (where 359 is ReviewID), it giving below
 usage messge


 hostname:/vobs/ms 167 % *rbt attach 359 --filename=diff*
 *Usage: rbt attach [options] review-request-id file*
 *
 *
 *rbt: error: Invalid number of arguments provided*


 So i tried the command as *rbt attach 359 diff* (where 359 is the
 reviewID and diff is the filename) but it giving error * CRITICAL:root:
 .Please have look and let me know what could be the issue .*
 *
 *
 *Note:The reviewBoard server version 1.6.3 and Rbtools is 0.5.2 ,Python
 2.7 *

 rbt attach 359 diff
 CRITICAL:root:

 *below is the debug output :*

  169 % rbt attach 359 diff -d
 DEBUG:root:Checking for a Subversion repository...
 DEBUG:root:Checking for a Git repository...
 DEBUG:root:Checking for a Mercurial repository...
 DEBUG:root:Checking for a CVS repository...
 DEBUG:root:Checking for a Perforce repository...
 DEBUG:root:Checking for a Plastic repository...
 DEBUG:root:Checking for a ClearCase repository...
 DEBUG:root:Running: cleartool pwv -short
 DEBUG:root:Running: diff --version
 DEBUG:root:Running: cleartool lsview -full -properties -cview
 DEBUG:root:Running: cleartool describe -short vob:.
 DEBUG:root:Running: cleartool pwv -root
 DEBUG:root:repository info: Path: /vobs/ms-test, Base path: /vobs/ms-test,
 Supports changesets: False
 DEBUG:root:Making HTTP GET request to https://ppsappdev02:443/api/
 Traceback (most recent call last):
   File /clearcase/tools/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.5.2', 'console_scripts', 'rbt')()
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/main.py,
 line 106, in main
 command.run_from_argv([RB_MAIN] + args)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/__init__.py,
 line 157, in run_from_argv
 exit_code = self.main(*args) or 0
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 line 67, in main
 request = self.get_review_request(request_id, api_root)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/attach.py,
 line 55, in get_review_request
 request = api_root.get_review_request(review_request_id=request_id)
   File
 /usr/local/project/tools/ActivePython-2.7/lib/python2.7/site-packages/RBTools-0.5.2-py2.7.egg/rbtools/api/resource.py,
 line 278, in __getattr__
 raise AttributeError
 AttributeError











 On Tuesday, August 20, 2013 1:01:04 AM UTC+5:30, David Trowbridge wrote:

 You can't use post-review --diff-filename to upload arbitrary files.
 The newer rbt commands will let you upload file attachments with rbt
 attach review request ID --filename=filename

 For your existing



 On Mon, Aug 19, 2013 at 4:35 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com wrote:

 Hi David ,

 My object is just to upload a file which  can be any file (*.gif  or
 *.bmp ) (need not be a diff file) using commandline post-review
 --diff-filename=file_name  to review-board server in our clear-case
 environment

 How to do it using post-review in clearcase environment ? let me know if
  any other option to post-review  is there ?

 Thanks



 On Monday, August 19, 2013 4:39:59 PM UTC+5:30, Akshaya Samantaray wrote:

 HI David ,

 Thanks for the response .

 d1.diff is just diff file i have created manually i.e the diff between
 my dev branch changes  Integration branch changes redirected to file
 called  d1.diff and i am trying to upload this manually created diff file
 d1.diff using  *post-review --diff-filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .*
 *
 *
 I can upload this manually created diff file d1.diff file from GUI in
 reviewboard  page  but not sure why its not uploading using  from command
 line using  post-review diff-**filen**ame=d1.diff --server=
 https://MyRbServer https://myrbserver/ .I am not getting any clue
 from the post-review debug log or reviewboard server log .

 Do i need to have any setup for apache or reviewboard server side
  configuration ?where can i check to get the exact clue for this error ?

 Please have a look and get back to me .I am waiting for your response .

 Thanks

  HTTP POSTing 

Re: post-review: Error uploading diff

2013-08-19 Thread Akshaya Samantaray
HI David ,

Thanks for the response .

d1.diff is just diff file i have created manually i.e the diff between my 
dev branch changes  Integration branch changes redirected to file called 
 d1.diff and i am trying to upload this manually created diff file d1.diff 
using  *post-review --diff-filename=d1.diff 
--server=https://MyRbServerhttps://myrbserver/
 .*
*
*
I can upload this manually created diff file d1.diff file from GUI in 
reviewboard  page  but not sure why its not uploading using  from command 
line using  post-review diff-filename=d1.diff 
--server=https://MyRbServerhttps://myrbserver/ 
.I am not getting any clue from the post-review debug log or reviewboard 
server log .

Do i need to have any setup for apache or reviewboard server side 
 configuration ?where can i check to get the exact clue for this error ? 

Please have a look and get back to me .I am waiting for your response .

Thanks

 HTTP POSTing to 
 https://MyRbServer/api/review-requests/370/diffs/https://myrbserver/api/review-requests/370/diffs/:
  
{'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
* Got API Error 105 (HTTP code 400): One or more fields had errors*
* Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
u'code': 105}}*

Error uploading diff

The generated diff file was empty. This usually means no files were
modified in this change.

Try running with --output-diff and --debug for more information.



*
*
*
*
*
*
*
*
On Friday, August 16, 2013 11:15:13 PM UTC+5:30, David Trowbridge wrote:

 What is in d1.diff? The debug output says it's 27 bytes, which doesn't 
 sound like a diff file.

 -David


 On Fri, Aug 16, 2013 at 3:28 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com javascript: wrote:

 HI,

 Ours is base clearcase Enviroment .Our Reviewboard  server is LINUX rhel 
 machine with RB version 1.6.3 .i am running post-review from Linux hosts 
 having rb tools 0.4.1  

 I am facing same  issue while uploading existing diff file using 
 *post-review 
 --diff-filename* from linux host ..The review is getting created but 
 the diff file is not getting uploaded from command line post-review  .I can 
 upload the diff file from GUI to the appropriate review ID but failing 
  from commandline *post-review --diff-filename*

 Let me know if any specfic Server side configuration missing  ?  Please 
 help .

 Error:

 14 % *post-review --diff-filename=d1.diff --server=https://MyRbServer; 
  -d*
  RBTools 0.4.1
  Home = /emc/ppreleng
  HTTP GETting api/
  HTTP GETting https://MyRbServer/api/info/
  Using the new web API
  HTTP GETting https://MyRbServer/api/repositories/
  HTTP GETting https://MyRbServer/api/repositories/1/
  HTTP GETting https://MyRbServer/api/repositories/1/info/
  HTTP GETting https://MyRbServer/api/repositories/2/
  HTTP GETting https://MyRbServer/api/repositories/2/info/
  HTTP GETting https://MyRbServer/api/repositories/3/
  HTTP GETting https://MyRbServer/api/repositories/3/info/
  Attempting to create review request on 
 /view/ppreleng_reviewboardAdmin/vobs/ms-test for None
  HTTP POSTing to https://MyRbServer/api/review-requests/: 
 {'repository': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
  Review request created
  Uploading diff, size: 27
  HTTP POSTing to https://MyRbServer/api/review-requests/370/diffs/: 
 {'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more information.

 Your review request still exists, but the diff is not attached.

 On Tuesday, October 30, 2012 2:14:29 PM UTC+5:30, Alvin NG wrote:

 Hi, 

 I could not attach the difference between revisions to my review. 
 It gives me the error Got API Error 105 (HTTP code 400): One or more 
 fields had errors. 
 I have also tried to output the difference by using svn diff and 
 attach the file to my review, but it still gives me the same error. 

 My command: post-review 
 --server=http://localhost/**reviewboardhttp://localhost/reviewboard--debug
  
 --disable-proxy --target-group=proj_test -- revision=0:11 --diff- 
 filename=diff.txt 

 (I'm not sure if this would also help) The repository server of my 
 Reviewboard is like this svn://localhost/test_proj but not the local 
 path to the reviewboard server 

 Many thanks, 
 Alvin 

  -- 
 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...@googlegroups.com javascript:
 For more 

Re: post-review: Error uploading diff

2013-08-19 Thread Akshaya Samantaray
Hi David ,

My object is just to upload a file which  can be any file (*.gif  or *.bmp 
) (need not be a diff file) using commandline post-review 
--diff-filename=file_name  to review-board server in our clear-case 
environment 

How to do it using post-review in clearcase environment ? let me know if 
 any other option to post-review  is there ?

Thanks



On Monday, August 19, 2013 4:39:59 PM UTC+5:30, Akshaya Samantaray wrote:

 HI David ,

 Thanks for the response .

 d1.diff is just diff file i have created manually i.e the diff between my 
 dev branch changes  Integration branch changes redirected to file called 
  d1.diff and i am trying to upload this manually created diff file d1.diff 
 using  *post-review --diff-filename=d1.diff 
 --server=https://MyRbServerhttps://myrbserver/
  .*
 *
 *
 I can upload this manually created diff file d1.diff file from GUI in 
 reviewboard  page  but not sure why its not uploading using  from command 
 line using  post-review diff-filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .I am not getting any clue 
 from the post-review debug log or reviewboard server log .

 Do i need to have any setup for apache or reviewboard server side 
  configuration ?where can i check to get the exact clue for this error ? 

 Please have a look and get back to me .I am waiting for your response .

 Thanks

  HTTP POSTing to 
  https://MyRbServer/api/review-requests/370/diffs/https://myrbserver/api/review-requests/370/diffs/:
   
 {'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more information.



 *
 *
 *
 *
 *
 *
 *
 *
 On Friday, August 16, 2013 11:15:13 PM UTC+5:30, David Trowbridge wrote:

 What is in d1.diff? The debug output says it's 27 bytes, which doesn't 
 sound like a diff file.

 -David


 On Fri, Aug 16, 2013 at 3:28 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com wrote:

 HI,

 Ours is base clearcase Enviroment .Our Reviewboard  server is LINUX rhel 
 machine with RB version 1.6.3 .i am running post-review from Linux hosts 
 having rb tools 0.4.1  

 I am facing same  issue while uploading existing diff file using 
 *post-review 
 --diff-filename* from linux host ..The review is getting created but 
 the diff file is not getting uploaded from command line post-review  .I can 
 upload the diff file from GUI to the appropriate review ID but failing 
  from commandline *post-review --diff-filename*

 Let me know if any specfic Server side configuration missing  ?  Please 
 help .

 Error:

 14 % *post-review --diff-filename=d1.diff --server=https://MyRbServer; 
  -d*
  RBTools 0.4.1
  Home = /emc/ppreleng
  HTTP GETting api/
  HTTP GETting https://MyRbServer/api/info/
  Using the new web API
  HTTP GETting https://MyRbServer/api/repositories/
  HTTP GETting https://MyRbServer/api/repositories/1/
  HTTP GETting https://MyRbServer/api/repositories/1/info/
  HTTP GETting https://MyRbServer/api/repositories/2/
  HTTP GETting https://MyRbServer/api/repositories/2/info/
  HTTP GETting https://MyRbServer/api/repositories/3/
  HTTP GETting https://MyRbServer/api/repositories/3/info/
  Attempting to create review request on 
 /view/ppreleng_reviewboardAdmin/vobs/ms-test for None
  HTTP POSTing to https://MyRbServer/api/review-requests/: 
 {'repository': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
  Review request created
  Uploading diff, size: 27
  HTTP POSTing to https://MyRbServer/api/review-requests/370/diffs/: 
 {'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more information.

 Your review request still exists, but the diff is not attached.

 On Tuesday, October 30, 2012 2:14:29 PM UTC+5:30, Alvin NG wrote:

 Hi, 

 I could not attach the difference between revisions to my review. 
 It gives me the error Got API Error 105 (HTTP code 400): One or more 
 fields had errors. 
 I have also tried to output the difference by using svn diff and 
 attach the file to my review, but it still gives me the same error. 

 My command: post-review 
 --server=http://localhost/**reviewboardhttp://localhost/reviewboard--debug
  
 --disable-proxy --target-group=proj_test -- revision=0:11 --diff- 
 filename=diff.txt 

 (I'm not sure if this would also help) The repository 

Re: post-review: Error uploading diff

2013-08-19 Thread David Trowbridge
On Mon, Aug 19, 2013 at 4:09 AM, Akshaya Samantaray 
akshaya.samanta...@gmail.com wrote:

 d1.diff is just diff file i have created manually i.e the diff between my
 dev branch changes  Integration branch changes redirected to file called
  d1.diff


Please check the contents of the file. If it's 27 bytes, it's probably not
what you expect it is.

-David

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: post-review: Error uploading diff

2013-08-19 Thread David Trowbridge
You can't use post-review --diff-filename to upload arbitrary files. The
newer rbt commands will let you upload file attachments with rbt attach
review request ID --filename=filename

For your existing


On Mon, Aug 19, 2013 at 4:35 AM, Akshaya Samantaray 
akshaya.samanta...@gmail.com wrote:

 Hi David ,

 My object is just to upload a file which  can be any file (*.gif  or *.bmp
 ) (need not be a diff file) using commandline post-review
 --diff-filename=file_name  to review-board server in our clear-case
 environment

 How to do it using post-review in clearcase environment ? let me know if
  any other option to post-review  is there ?

 Thanks



 On Monday, August 19, 2013 4:39:59 PM UTC+5:30, Akshaya Samantaray wrote:

 HI David ,

 Thanks for the response .

 d1.diff is just diff file i have created manually i.e the diff between my
 dev branch changes  Integration branch changes redirected to file called
  d1.diff and i am trying to upload this manually created diff file d1.diff
 using  *post-review --diff-filename=d1.diff 
 --server=https://MyRbServerhttps://myrbserver/
  .*
 *
 *
 I can upload this manually created diff file d1.diff file from GUI in
 reviewboard  page  but not sure why its not uploading using  from command
 line using  post-review diff-**filename=d1.diff --server=
 https://MyRbServer https://myrbserver/ .I am not getting any clue
 from the post-review debug log or reviewboard server log .

 Do i need to have any setup for apache or reviewboard server side
  configuration ?where can i check to get the exact clue for this error ?

 Please have a look and get back to me .I am waiting for your response .

 Thanks

  HTTP POSTing to 
  https://MyRbServer/api/**review-requests/370/diffs/https://myrbserver/api/review-requests/370/diffs/:
 {'basedir': u'/view/ppreleng_**reviewboardAdmin/vobs/ms-test'**}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']},
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors',
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more information.



 *
 *
 *
 *
 *
 *
 *
 *
 On Friday, August 16, 2013 11:15:13 PM UTC+5:30, David Trowbridge wrote:

 What is in d1.diff? The debug output says it's 27 bytes, which doesn't
 sound like a diff file.

 -David


 On Fri, Aug 16, 2013 at 3:28 AM, Akshaya Samantaray 
 akshaya.s...@gmail.com wrote:

 HI,

 Ours is base clearcase Enviroment .Our Reviewboard  server is LINUX
 rhel machine with RB version 1.6.3 .i am running post-review from Linux
 hosts having rb tools 0.4.1

 I am facing same  issue while uploading existing diff file using 
 *post-review
 --diff-filename* from linux host ..The review is getting created but
 the diff file is not getting uploaded from command line post-review  .I can
 upload the diff file from GUI to the appropriate review ID but failing
  from commandline *post-review --diff-filename*

 Let me know if any specfic Server side configuration missing  ?  Please
 help .

 Error:

 14 % *post-review --diff-filename=d1.diff --server=https://MyRbServer;
  -d*
  RBTools 0.4.1
  Home = /emc/ppreleng
  HTTP GETting api/
  HTTP GETting https://MyRbServer/api/info/
  Using the new web API
  HTTP GETting 
  https://MyRbServer/api/**repositories/https://MyRbServer/api/repositories/
  HTTP GETting 
  https://MyRbServer/api/**repositories/1/https://MyRbServer/api/repositories/1/
  HTTP GETting 
  https://MyRbServer/api/**repositories/1/info/https://MyRbServer/api/repositories/1/info/
  HTTP GETting 
  https://MyRbServer/api/**repositories/2/https://MyRbServer/api/repositories/2/
  HTTP GETting 
  https://MyRbServer/api/**repositories/2/info/https://MyRbServer/api/repositories/2/info/
  HTTP GETting 
  https://MyRbServer/api/**repositories/3/https://MyRbServer/api/repositories/3/
  HTTP GETting 
  https://MyRbServer/api/**repositories/3/info/https://MyRbServer/api/repositories/3/info/
  Attempting to create review request on 
  /view/ppreleng_**reviewboardAdmin/vobs/ms-test
 for None
  HTTP POSTing to 
  https://MyRbServer/api/review-**requests/https://MyRbServer/api/review-requests/:
 {'repository': u'/view/ppreleng_**reviewboardAdmin/vobs/ms-test'**}
  Review request created
  Uploading diff, size: 27
  HTTP POSTing to 
  https://MyRbServer/api/review-**requests/370/diffs/https://MyRbServer/api/review-requests/370/diffs/:
 {'basedir': u'/view/ppreleng_**reviewboardAdmin/vobs/ms-test'**}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']},
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors',
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more 

Re: post-review: Error uploading diff

2013-08-16 Thread Akshaya Samantaray
HI,

Ours is base clearcase Enviroment .Our Reviewboard  server is LINUX rhel 
machine with RB version 1.6.3 .i am running post-review from Linux hosts 
having rb tools 0.4.1  

I am facing same  issue while uploading existing diff file using *post-review 
--diff-filename* from linux host ..The review is getting created but the 
diff file is not getting uploaded from command line post-review  .I can 
upload the diff file from GUI to the appropriate review ID but failing 
 from commandline *post-review --diff-filename*

Let me know if any specfic Server side configuration missing  ?  Please 
help .

Error:

14 % *post-review --diff-filename=d1.diff --server=https://MyRbServer;  -d*
 RBTools 0.4.1
 Home = /emc/ppreleng
 HTTP GETting api/
 HTTP GETting https://MyRbServer/api/info/
 Using the new web API
 HTTP GETting https://MyRbServer/api/repositories/
 HTTP GETting https://MyRbServer/api/repositories/1/
 HTTP GETting https://MyRbServer/api/repositories/1/info/
 HTTP GETting https://MyRbServer/api/repositories/2/
 HTTP GETting https://MyRbServer/api/repositories/2/info/
 HTTP GETting https://MyRbServer/api/repositories/3/
 HTTP GETting https://MyRbServer/api/repositories/3/info/
 Attempting to create review request on 
/view/ppreleng_reviewboardAdmin/vobs/ms-test for None
 HTTP POSTing to https://MyRbServer/api/review-requests/: {'repository': 
u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
 Review request created
 Uploading diff, size: 27
 HTTP POSTing to https://MyRbServer/api/review-requests/370/diffs/: 
{'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
* Got API Error 105 (HTTP code 400): One or more fields had errors*
* Error data: {u'fields': {u'path': [u'The diff file is empty']}, 
u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
u'code': 105}}*

Error uploading diff

The generated diff file was empty. This usually means no files were
modified in this change.

Try running with --output-diff and --debug for more information.

Your review request still exists, but the diff is not attached.

On Tuesday, October 30, 2012 2:14:29 PM UTC+5:30, Alvin NG wrote:

 Hi, 

 I could not attach the difference between revisions to my review. 
 It gives me the error Got API Error 105 (HTTP code 400): One or more 
 fields had errors. 
 I have also tried to output the difference by using svn diff and 
 attach the file to my review, but it still gives me the same error. 

 My command: post-review --server=http://localhost/reviewboard --debug 
 --disable-proxy --target-group=proj_test -- revision=0:11 --diff- 
 filename=diff.txt 

 (I'm not sure if this would also help) The repository server of my 
 Reviewboard is like this svn://localhost/test_proj but not the local 
 path to the reviewboard server 

 Many thanks, 
 Alvin 


-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: post-review: Error uploading diff

2013-08-16 Thread David Trowbridge
What is in d1.diff? The debug output says it's 27 bytes, which doesn't
sound like a diff file.

-David


On Fri, Aug 16, 2013 at 3:28 AM, Akshaya Samantaray 
akshaya.samanta...@gmail.com wrote:

 HI,

 Ours is base clearcase Enviroment .Our Reviewboard  server is LINUX rhel
 machine with RB version 1.6.3 .i am running post-review from Linux hosts
 having rb tools 0.4.1

 I am facing same  issue while uploading existing diff file using *post-review
 --diff-filename* from linux host ..The review is getting created but the
 diff file is not getting uploaded from command line post-review  .I can
 upload the diff file from GUI to the appropriate review ID but failing
  from commandline *post-review --diff-filename*

 Let me know if any specfic Server side configuration missing  ?  Please
 help .

 Error:

 14 % *post-review --diff-filename=d1.diff --server=https://MyRbServer;
  -d*
  RBTools 0.4.1
  Home = /emc/ppreleng
  HTTP GETting api/
  HTTP GETting https://MyRbServer/api/info/
  Using the new web API
  HTTP GETting https://MyRbServer/api/repositories/
  HTTP GETting https://MyRbServer/api/repositories/1/
  HTTP GETting https://MyRbServer/api/repositories/1/info/
  HTTP GETting https://MyRbServer/api/repositories/2/
  HTTP GETting https://MyRbServer/api/repositories/2/info/
  HTTP GETting https://MyRbServer/api/repositories/3/
  HTTP GETting https://MyRbServer/api/repositories/3/info/
  Attempting to create review request on
 /view/ppreleng_reviewboardAdmin/vobs/ms-test for None
  HTTP POSTing to https://MyRbServer/api/review-requests/:
 {'repository': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
  Review request created
  Uploading diff, size: 27
  HTTP POSTing to https://MyRbServer/api/review-requests/370/diffs/:
 {'basedir': u'/view/ppreleng_reviewboardAdmin/vobs/ms-test'}
 * Got API Error 105 (HTTP code 400): One or more fields had errors*
 * Error data: {u'fields': {u'path': [u'The diff file is empty']},
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors',
 u'code': 105}}*

 Error uploading diff

 The generated diff file was empty. This usually means no files were
 modified in this change.

 Try running with --output-diff and --debug for more information.

 Your review request still exists, but the diff is not attached.

 On Tuesday, October 30, 2012 2:14:29 PM UTC+5:30, Alvin NG wrote:

 Hi,

 I could not attach the difference between revisions to my review.
 It gives me the error Got API Error 105 (HTTP code 400): One or more
 fields had errors.
 I have also tried to output the difference by using svn diff and
 attach the file to my review, but it still gives me the same error.

 My command: post-review 
 --server=http://localhost/**reviewboardhttp://localhost/reviewboard--debug
 --disable-proxy --target-group=proj_test -- revision=0:11 --diff-
 filename=diff.txt

 (I'm not sure if this would also help) The repository server of my
 Reviewboard is like this svn://localhost/test_proj but not the local
 path to the reviewboard server

 Many thanks,
 Alvin

  --
 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
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.


-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Re: post-review: Error uploading diff

2012-10-30 Thread Christian Hammond
Make sure Review Board is configured to point to the root of the SVN
server, not a subdirectory within. I'm assuming test_proj is a subdirectory
inside the repo?

Christian

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


On Tue, Oct 30, 2012 at 1:44 AM, Alvin NG asang...@gmail.com wrote:

 Hi,

 I could not attach the difference between revisions to my review.
 It gives me the error Got API Error 105 (HTTP code 400): One or more
 fields had errors.
 I have also tried to output the difference by using svn diff and
 attach the file to my review, but it still gives me the same error.

 My command: post-review --server=http://localhost/reviewboard --debug
 --disable-proxy --target-group=proj_test -- revision=0:11 --diff-
 filename=diff.txt

 (I'm not sure if this would also help) The repository server of my
 Reviewboard is like this svn://localhost/test_proj but not the local
 path to the reviewboard server

 Many thanks,
 Alvin

 --
 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




-- 
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: post-review: Error uploading diff

2012-10-30 Thread Alvin NG
test_proj is the root of the SVN server.
I kick my SVN server by
svnserve -d -r /svn/test_proj/

On 10月30日, 下午4時47分, Christian Hammond chip...@chipx86.com wrote:
 Make sure Review Board is configured to point to the root of the SVN
 server, not a subdirectory within. I'm assuming test_proj is a subdirectory
 inside the repo?

 Christian

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







 On Tue, Oct 30, 2012 at 1:44 AM, Alvin NG asang...@gmail.com wrote:
  Hi,

  I could not attach the difference between revisions to my review.
  It gives me the error Got API Error 105 (HTTP code 400): One or more
  fields had errors.
  I have also tried to output the difference by using svn diff and
  attach the file to my review, but it still gives me the same error.

  My command: post-review --server=http://localhost/reviewboard--debug
  --disable-proxy --target-group=proj_test -- revision=0:11 --diff-
  filename=diff.txt

  (I'm not sure if this would also help) The repository server of my
  Reviewboard is like this svn://localhost/test_proj but not the local
  path to the reviewboard server

  Many thanks,
  Alvin

  --
  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.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: post-review: Error uploading diff

2012-10-30 Thread Alvin NG
FYI, here is the setting in my reviewboard
Path: svn://localhost
Mirror path: (empty)

I saw a post which is talking about the same problem (I think) here
https://groups.google.com/group/reviewboard/browse_thread/thread/d770407ab6d34fb8/923d3f8b7de4d85a

I followed the setting suggested by the above post and set
Path: /svn/test_proj
Mirror path: svn@server:localhost

But it still doesn't work. Hope this would help.

On 10月30日, 下午4時47分, Christian Hammond chip...@chipx86.com wrote:
 Make sure Review Board is configured to point to the root of the SVN
 server, not a subdirectory within. I'm assuming test_proj is a subdirectory
 inside the repo?

 Christian

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







 On Tue, Oct 30, 2012 at 1:44 AM, Alvin NG asang...@gmail.com wrote:
  Hi,

  I could not attach the difference between revisions to my review.
  It gives me the error Got API Error 105 (HTTP code 400): One or more
  fields had errors.
  I have also tried to output the difference by using svn diff and
  attach the file to my review, but it still gives me the same error.

  My command: post-review --server=http://localhost/reviewboard--debug
  --disable-proxy --target-group=proj_test -- revision=0:11 --diff-
  filename=diff.txt

  (I'm not sure if this would also help) The repository server of my
  Reviewboard is like this svn://localhost/test_proj but not the local
  path to the reviewboard server

  Many thanks,
  Alvin

  --
  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.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: post-review error Unable to find root /api/

2012-01-13 Thread Jigar
Hi Chrisitan,

C:\Python25\Scriptspost-review -d --server=http://127.0.0.1/reviews --
summary=533900 --description=533900 --username=jigars --
password=jigars --p4-
client=jigars_JIGAR02V --p4-port=p4jira:1666 --p4-
passwd=EAF62925210A08CC4896D0B80E026501 533900
 RBTools 0.3.4
 Home = C:\Users\jigars\AppData\Roaming
 p4 info
 repository info: Path: illin660:1666, Base path: None, Supports changesets: 
 True
 HTTP GETting api/
 HTTP GETting http://127.0.0.1/reviews/api/info/
 Using the new web API
 Generating diff for changenum 533900
 p4 -P EAF62925210A08CC4896D0B80E026501 describe -s 533900
 Processing edit of 
 //depot/Applications/SE/v80_0_SP2/product/product.epcimport/src/com/company/product/epcimport/ant/AntStep.java
 Writing 
 //depot/Applications/SE/v80_0_SP2/product/product.epcimport/src/com/company/product/epcimport/ant/AntStep.java#26
  to c:\us
ers\jigars\appdata\local\temp\tmpgwvxzu
 p4 print -o c:\users\jigars\appdata\local\temp\tmpgwvxzu -q 
 //depot/Applications/SE/v80_0_SP2/product/product.epcimport/src/com/company/product/epcimport/ant/AntStep.java#26
Client 'jigars_JIGAR02V' unknown - use 'client' command to create it.

Failed to execute command: ['p4', '-G', 'where', '//depot/Applications/
SE/v80_0_SP2/company/product/src/com/company/product/epc
import/ant/AntStep.java']


~JIGAR

On Dec 18 2011, 11:17 pm, Christian Hammond chip...@gmail.com wrote:
 It's the Squid proxy running. It's interfering with the communication. You'll 
 need to access Review Board directly without the proxy.

 Christian

 On Dec 17, 2011, at 22:07,Jigarjigar23m...@gmail.com wrote:



  Hi,

  I sent you details with debug options

  On Dec 10, 12:56 am, Christian Hammond chip...@chipx86.com wrote:
  Can you show me the output of post-review when also using --debug?

  Christian

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

  On Thu, Dec 8, 2011 at 8:42 AM,JigarSutaria jigarsuta...@gmail.comwrote:

  I am running post-review and receiving the error root /api/ not found

  I am using

  reviewboard 1.6
  apache 2.0
  python25
  postgre9

  my root folder is reviews

  I am stucked with this since few days and no clue how to move forward.
  Please help. If I am trying the URL in the browser, it not working
  there. How can I make sure thathttp://localhost/reviews/apiexists.

  --
  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.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- 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: post-review error Unable to find root /api/

2011-12-18 Thread Christian Hammond
It's the Squid proxy running. It's interfering with the communication. You'll 
need to access Review Board directly without the proxy.

Christian



On Dec 17, 2011, at 22:07, Jigar jigar23m...@gmail.com wrote:

 Hi,
 
 I sent you details with debug options
 
 On Dec 10, 12:56 am, Christian Hammond chip...@chipx86.com wrote:
 Can you show me the output of post-review when also using --debug?
 
 Christian
 
 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com
 
 On Thu, Dec 8, 2011 at 8:42 AM, Jigar Sutaria jigarsuta...@gmail.comwrote:
 
 
 
 
 
 
 
 I am running post-review and receiving the error root /api/ not found
 
 I am using
 
 reviewboard 1.6
 apache 2.0
 python25
 postgre9
 
 my root folder is reviews
 
 I am stucked with this since few days and no clue how to move forward.
 Please help. If I am trying the URL in the browser, it not working
 there. How can I make sure thathttp://localhost/reviews/apiexists.
 
 --
 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.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

-- 
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: post-review error Unable to find root /api/

2011-12-17 Thread Jigar
Hi Chris,

Can you tell what could be the reason for this error?

~JIGAR

On Dec 12, 11:58 am, Jigar jigar23m...@gmail.com wrote:
 bIf I am using jigars02v /b
 C:\Python25\Scriptspost-review -d --server=http://jigars02v/reviews--
 summary=533900 --description=533900 --username=admin --password=admin
 --p4-cl
 ient=jigars_JIGAR02V --p4-port=p4server:1666 --p4-passwd=01Nov2011 --
 http-username=admin --http-password=admin 533900 RBTools 0.3.4
  Home = C:\Users\jigars\AppData\Roaming
  p4 info
  repository info: Path: illin660:1666, Base path: None, Supports 
  changesets: True
  HTTP GETting api/
  Got HTTP error: 503: TITLE503 Service Unavailable/TITLE

 H1503 Service Unavailable/H1
 Failed to resolve the name of server Bjigars02v/B to connect

 Unable to access the root /api/ URL on the server.

 C:\Python25\Scripts

 bIf I am using localhost /b

 C:\Python25\Scriptspost-review -d --server=http://localhost/reviews--
 summary=533900 --description=533900 --username=admin --password=admin
 --p4-cl
 ient=jigars_JIGAR02V --p4-port=p4server:1666 --p4-passwd=01Nov2011 --
 http-username=admin --http-password=admin 533900 RBTools 0.3.4
  Home = C:\Users\jigars\AppData\Roaming
  p4 info
  repository info: Path: illin660.corp.amdocs.com:1666, Base path: None, 
  Supports changesets: True
  HTTP GETting api/
  Got HTTP error: 502: !DOCTYPE HTML PUBLIC -//W3C//DTD HTML 4.01 
  Transitional//EN http://www.w3.org/TR/html4/loose.dtd;

 HTMLHEADMETA HTTP-EQUIV=Content-Type CONTENT=text/html;
 charset=iso-8859-1
 TITLEERROR: The requested URL could not be retrieved/TITLE
 STYLE type=text/css!--BODY{background-color:#ff;font-
 family:verdana,sans-serif}PRE{font-family:sans-serif}--/STYLE
 /HEADBODY
 H1ERROR/H1
 H2The requested URL could not be retrieved/H2
 HR noshade size=1px
 P
 While trying to retrieve the URL:
 A HREF=http://localhost/reviews/api/;http://localhost/reviews/api//
 A
 P
 The following error was encountered:
 UL
 LI
 STRONG
 Zero Sized Reply
 /STRONG
 /UL

 P
 Squid did not receive any data for this request.
 PYour cache administrator is A HREF=mailto:webmaster;webmaster/
 A.

 BR clear=all
 HR noshade size=1px
 ADDRESS
 Generated Mon, 12 Dec 2011 06:56:21 GMT by iproxy2n (squid/
 2.7.STABLE2)
 /ADDRESS
 /BODY/HTML

 Unable to access the root /api/ URL on the server.

 C:\Python25\Scripts

 On Dec 9, 9:56 pm, Christian Hammond chip...@chipx86.com wrote:



  Can you show me the output of post-review when also using --debug?

  Christian

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

  On Thu, Dec 8, 2011 at 8:42 AM,JigarSutaria jigarsuta...@gmail.comwrote:

   I am running post-review and receiving the error root /api/ not found

   I am using

   reviewboard 1.6
   apache 2.0
   python25
   postgre9

   my root folder is reviews

   I am stucked with this since few days and no clue how to move forward.
   Please help. If I am trying the URL in the browser, it not working
   there. How can I make sure thathttp://localhost/reviews/apiexists.

   --
   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.com
   For more options, visit this group at
  http://groups.google.com/group/reviewboard?hl=en-Hide quoted text -

  - Show quoted text -- 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: post-review error Unable to find root /api/

2011-12-11 Thread Jigar
bIf I am using jigars02v /b
C:\Python25\Scriptspost-review -d --server=http://jigars02v/reviews --
summary=533900 --description=533900 --username=admin --password=admin
--p4-cl
ient=jigars_JIGAR02V --p4-port=p4server:1666 --p4-passwd=01Nov2011 --
http-username=admin --http-password=admin 533900
 RBTools 0.3.4
 Home = C:\Users\jigars\AppData\Roaming
 p4 info
 repository info: Path: illin660:1666, Base path: None, Supports changesets: 
 True
 HTTP GETting api/
 Got HTTP error: 503: TITLE503 Service Unavailable/TITLE
H1503 Service Unavailable/H1
Failed to resolve the name of server Bjigars02v/B to connect

Unable to access the root /api/ URL on the server.

C:\Python25\Scripts

bIf I am using localhost /b


C:\Python25\Scriptspost-review -d --server=http://localhost/reviews --
summary=533900 --description=533900 --username=admin --password=admin
--p4-cl
ient=jigars_JIGAR02V --p4-port=p4server:1666 --p4-passwd=01Nov2011 --
http-username=admin --http-password=admin 533900
 RBTools 0.3.4
 Home = C:\Users\jigars\AppData\Roaming
 p4 info
 repository info: Path: illin660.corp.amdocs.com:1666, Base path: None, 
 Supports changesets: True
 HTTP GETting api/
 Got HTTP error: 502: !DOCTYPE HTML PUBLIC -//W3C//DTD HTML 4.01 
 Transitional//EN http://www.w3.org/TR/html4/loose.dtd;
HTMLHEADMETA HTTP-EQUIV=Content-Type CONTENT=text/html;
charset=iso-8859-1
TITLEERROR: The requested URL could not be retrieved/TITLE
STYLE type=text/css!--BODY{background-color:#ff;font-
family:verdana,sans-serif}PRE{font-family:sans-serif}--/STYLE
/HEADBODY
H1ERROR/H1
H2The requested URL could not be retrieved/H2
HR noshade size=1px
P
While trying to retrieve the URL:
A HREF=http://localhost/reviews/api/;http://localhost/reviews/api//
A
P
The following error was encountered:
UL
LI
STRONG
Zero Sized Reply
/STRONG
/UL

P
Squid did not receive any data for this request.
PYour cache administrator is A HREF=mailto:webmaster;webmaster/
A.

BR clear=all
HR noshade size=1px
ADDRESS
Generated Mon, 12 Dec 2011 06:56:21 GMT by iproxy2n (squid/
2.7.STABLE2)
/ADDRESS
/BODY/HTML

Unable to access the root /api/ URL on the server.

C:\Python25\Scripts


On Dec 9, 9:56 pm, Christian Hammond chip...@chipx86.com wrote:
 Can you show me the output of post-review when also using --debug?

 Christian

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

 On Thu, Dec 8, 2011 at 8:42 AM, Jigar Sutaria jigarsuta...@gmail.comwrote:



  I am running post-review and receiving the error root /api/ not found

  I am using

  reviewboard 1.6
  apache 2.0
  python25
  postgre9

  my root folder is reviews

  I am stucked with this since few days and no clue how to move forward.
  Please help. If I am trying the URL in the browser, it not working
  there. How can I make sure thathttp://localhost/reviews/apiexists.

  --
  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.com
  For more options, visit this group at
 http://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: post-review error Unable to find root /api/

2011-12-09 Thread Christian Hammond
Can you show me the output of post-review when also using --debug?

Christian

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


On Thu, Dec 8, 2011 at 8:42 AM, Jigar Sutaria jigarsuta...@gmail.comwrote:

 I am running post-review and receiving the error root /api/ not found

 I am using

 reviewboard 1.6
 apache 2.0
 python25
 postgre9

 my root folder is reviews

 I am stucked with this since few days and no clue how to move forward.
 Please help. If I am trying the URL in the browser, it not working
 there. How can I make sure that http://localhost/reviews/api exists.

 --
 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

-- 
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: post-review error

2011-11-15 Thread Maciek Borzecki
On Thu, Nov 10, 2011 at 4:16 PM, Krles antonin.nechva...@gmail.com wrote:
 Hello

 I would like you to ask about post-review.

 I've needed install RBtools on RHEL 3. RHEL in default has only python
 2.2.3. Therefore I've installed python 2.4.6, setuptools 0.6c11 fo 2.4
 and RBtools 0.34.

 After that I typed post-review --help and  I  got the error

 Traceback (most recent call last):
  File /usr/local/bin/post-review, line 7, in ?
    sys.exit(
  File /usr/local/lib/python2.3/site-packages/setuptools-0.6c11-
 py2.4.egg/pkg_resources.py, line 318, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File /usr/local/lib/python2.3/site-packages/setuptools-0.6c11-
 py2.4.egg/pkg_resources.py, line 2221, in load_entry_point
    return ep.load()
  File /usr/local/lib/python2.3/site-packages/setuptools-0.6c11-
 py2.4.egg/pkg_resources.py, line 1954, in load
    entry = __import__(self.module_name, globals(),globals(),
 ['__name__'])
  File /usr/local/lib/python2.3/site-packages/RBTools-0.3.4-py2.4.egg/
 rbtools/postreview.py, line 2507
    @property
    ^
 SyntaxError: invalid syntax


 Do you know what's wrong ?

Looks like you're running python 2.3 after all. Have you installed
python 2.4 side by side with python 2.3? Are you calling the right
binary?

maciek

-- 
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: post-review error

2011-08-19 Thread Deva
Hi Christian,
Reviewboard Server what i have is in the same LAN or the network.
Still am seeing this problem.

Please help.

Regards,
Deva

On Aug 19, 9:47 am, Christian Hammond chip...@chipx86.com wrote:
 Hi Deva,

 Are you behind a proxy?

 Christian

 On Thursday, August 18, 2011, Deva devara...@photoninfotech.net wrote:
  Hi,
  Am getting below error while posting the review in windows. It says
  'Unable to accesshttp://x.x.x.x/api/json/accounts/login/. The host
  path may be invalid
  urlopen error [Errno 10054] An existing connection was forcibly
  closed by the r
  emote host'

  C:\workspace\BDBJEpython C:\rbtools\rbtools\postreview.py --debug
  svn info
  repository info: Path:http://x.x.x.x/wds, Base path: /projects/NavApp/
  devBranches/1.8.X-BDB/BDBJE, Supports changesets: False
  svn diff --diff-cmd=diff
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio

  n.java
  Looking for 'x.x.x.x /' cookie in C:\Documents and Settings\devaraj_m\
  Local Settings\Application Data\.post-review-cookies.txt
  Couldn't load cookie file: [Errno 2] No such file or directory:
 'C:\\Documen
  ts and Settings\\devaraj_m\\Local Settings\\Application Data\\.post-
  review-cooki
  es.txt'
  == Review Board Login Required
  Enter username and password for Review Board athttp://x.x.x.x/
  Username: deva
  Password:
  Logging in with username deva
  HTTP POSTing tohttp://x.x.x.x/api/json/accounts/login/:{'userna
  me': 'deva', 'password': '**'}
  Unable to accesshttp://x.x.x.x/api/json/accounts/login/. The host pa
  th may be invalid
  urlopen error [Errno 10054] An existing connection was forcibly
  closed by the r
  emote host

  Can anyone please help me ?

  Regards,
  Deva

  --
  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.com For more options, visit this group 
 at

 http://groups.google.com/group/reviewboard?hl=en

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

-- 
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: post-review error

2011-08-19 Thread Christian Hammond
Yes, but is your computer you're running post-review on configured at all to
use a proxy?

Christian


On Friday, August 19, 2011, Deva devara...@photoninfotech.net wrote:
 Hi Christian,
 Reviewboard Server what i have is in the same LAN or the network.
 Still am seeing this problem.

 Please help.

 Regards,
 Deva

 On Aug 19, 9:47 am, Christian Hammond chip...@chipx86.com wrote:
 Hi Deva,

 Are you behind a proxy?

 Christian

 On Thursday, August 18, 2011, Deva devara...@photoninfotech.net wrote:
  Hi,
  Am getting below error while posting the review in windows. It says
  'Unable to accesshttp://x.x.x.x/api/json/accounts/login/. The host
  path may be invalid
  urlopen error [Errno 10054] An existing connection was forcibly
  closed by the r
  emote host'

  C:\workspace\BDBJEpython C:\rbtools\rbtools\postreview.py --debug
  svn info
  repository info: Path:http://x.x.x.x/wds, Base path:
/projects/NavApp/
  devBranches/1.8.X-BDB/BDBJE, Supports changesets: False
  svn diff --diff-cmd=diff
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
  svn info

 src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio

  n.java
  Looking for 'x.x.x.x /' cookie in C:\Documents and
Settings\devaraj_m\
  Local Settings\Application Data\.post-review-cookies.txt
  Couldn't load cookie file: [Errno 2] No such file or directory:
 'C:\\Documen
  ts and Settings\\devaraj_m\\Local Settings\\Application Data\\.post-
  review-cooki
  es.txt'
  == Review Board Login Required
  Enter username and password for Review Board athttp://x.x.x.x/
  Username: deva
  Password:
  Logging in with username deva
  HTTP POSTing tohttp://x.x.x.x/api/json/accounts/login/:{'userna
  me': 'deva', 'password': '**'}
  Unable to accesshttp://x.x.x.x/api/json/accounts/login/. The host pa
  th may be invalid
  urlopen error [Errno 10054] An existing connection was forcibly
  closed by the r
  emote host

  Can anyone please help me ?

  Regards,
  Deva

  --
  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.com For more options, visit this
group at

 http://groups.google.com/group/reviewboard?hl=en

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

 --
 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

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

-- 
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: post-review error: Unable to find a Review Board server for this source code tree.

2011-08-19 Thread Christian Hammond
You have REPOSITORY_URL set to the Review Board server, which is the
problem. You should just remove that entry.

Christian


On Thursday, August 18, 2011, Clutch mhightower.c...@gmail.com wrote:
 clutch@fx09:~/workspace/example$ post-review --debug
 RBTools 0.3.3
 Home = /home/clutch
 svn info
 diff --version
 repository info: Path:
https://svnserver.example.com:447/repos/example.com, Base path: /trunk,
Supports changesets: False
 svn propget reviewboard:url /home/clutch/workspace/example
 svn propget reviewboard:url
https://svnserver.example.com:447/repos/example.com
 Unable to find a Review Board server for this source code tree.

 Here's my reviewboardrc
 clutch@fx09:~/workspace/example$ cat /home/clutch/.reviewboardrc
 REPOSITORY = 'https://svnserver.example.com:447/repos/example.com/
 trunk'
 REPOSITORY_URL = http://reviews.local/reviews.local;

 svn info
 clutch@fx09:~/workspace/example$ svn info
 Path: .
 URL: https://svnserver.example.com:447/repos/example.com/trunk
 Repository Root: https://svnserver.example.com:447/repos/example.com
 Repository UUID: ab757fae-6e61-4142-9f3e-588728cd6790
 Revision: 332
 Node Kind: directory
 Schedule: normal

 Here's my repository as defined in Review Board
 Name: examplerepo
 Hosting service: Custom
 Repository type: Subversion
 Path: https://svnserver.example.com:447/repos/example.com
 Mirror path: https://svnserver.example.com:447/repos/example.com/trunk


 I'm stumped does any one see the problem?

 --
 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

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

-- 
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: post-review error

2011-08-18 Thread Christian Hammond
Hi Deva,

Are you behind a proxy?

Christian


On Thursday, August 18, 2011, Deva devara...@photoninfotech.net wrote:
 Hi,
 Am getting below error while posting the review in windows. It says
 'Unable to access http://x.x.x.x/api/json/accounts/login/. The host
 path may be invalid
 urlopen error [Errno 10054] An existing connection was forcibly
 closed by the r
 emote host'

 C:\workspace\BDBJEpython C:\rbtools\rbtools\postreview.py --debug
 svn info
 repository info: Path: http://x.x.x.x/wds, Base path: /projects/NavApp/
 devBranches/1.8.X-BDB/BDBJE, Supports changesets: False
 svn diff --diff-cmd=diff
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
 n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
 n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
 n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
 n.java
 Looking for 'x.x.x.x /' cookie in C:\Documents and Settings\devaraj_m\
 Local Settings\Application Data\.post-review-cookies.txt
 Couldn't load cookie file: [Errno 2] No such file or directory:
'C:\\Documen
 ts and Settings\\devaraj_m\\Local Settings\\Application Data\\.post-
 review-cooki
 es.txt'
 == Review Board Login Required
 Enter username and password for Review Board at http://x.x.x.x/
 Username: deva
 Password:
 Logging in with username deva
 HTTP POSTing to http://x.x.x.x/api/json/accounts/login/: {'userna
 me': 'deva', 'password': '**'}
 Unable to access http://x.x.x.x/api/json/accounts/login/. The host pa
 th may be invalid
 urlopen error [Errno 10054] An existing connection was forcibly
 closed by the r
 emote host

 Can anyone please help me ?

 Regards,
 Deva

 --
 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

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

-- 
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: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-23 Thread Timothy Pinet
Thanks for the suggestion. I downloaded rbtools 0.2 stable from
http://github.com/reviewboard/rbtools, made my changes to
postreview.py, and reinstalled the package using python setup.py
install at the rbtool root.

In postreview.py I changed the svn diff --diff-cmd=diff to svn diff
--diff-cmd=C:\\diffForceText.bat and recompiled. I create the wrapper
script diffForceText.bat to simply include:


@echo off
diff %* --text


Which forwards all calls into GNU diff but forces the --text param.
This overcomes my binary file being text diffed and returning exit
code 2 to SVN diff.

I now have a fully working ReviewBoard + Windows + TFS repo solution
for my org and it is blowing minds and melting faces. I will post how
I got TFS to work as it was not very straight forward.

Tim

-- 
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: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-20 Thread Timothy Pinet
In which file does reviewboard invoke the svn diff --diff-cmd=diff
command? I have been looking but can not find it. I then thought that
you probably call the pysvn lib for this. I checked the pysvn
documentation (http://pysvn.tigris.org/docs/
pysvn_prog_ref.html#pysvn_client_diff) and there is a flag
ignore_content_type for the pysvn.Client.diff command. I assume
that reviewboard calls this method.

Is there a way I can manually include this flag?

Are there any other suggestions on how I can tell subversion, gnu
diff, or pysvn to ignore the content type so binary files do not block
reviews?

Thanks for all of your help!

I really want to get this operating to show my organization how
awesome this tool is. In my opinion it beats out SmartBear
CodeCollaborator and Atlassian Crucible suites.

Thanks,
Tim

-- 
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: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-20 Thread Chris Clark

Timothy Pinet wrote:

In which file does reviewboard invoke the svn diff --diff-cmd=diff
command? I have been looking but can not find it. I then thought that
you probably call the pysvn lib for this. I checked the pysvn
documentation (http://pysvn.tigris.org/docs/
pysvn_prog_ref.html#pysvn_client_diff) and there is a flag
ignore_content_type for the pysvn.Client.diff command. I assume
that reviewboard calls this method.
  


It actually isn't in ReviewBoard, it is in postreview.py :-)

Jut fork postreview locally and you can then customize it for your 
environment. This is very common, I've done it for my team as we needed 
a bunch of customizations which really aren't appropriate for anyone else.


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


Re: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-20 Thread Timothy Pinet
I wanted to check post-review however since I am on Windows my
Python26\Scripts\post-review.exe is a compiled executable and not a
python file. Would it be enough to grab the post-review.py from the
repo and delete the exe?

Thanks,
Tim

On Aug 20, 12:54 pm, Chris Clark chris.cl...@ingres.com wrote:

 It actually isn't in ReviewBoard, it is in postreview.py :-)

 Jut fork postreview locally and you can then customize it for your
 environment. This is very common, I've done it for my team as we needed
 a bunch of customizations which really aren't appropriate for anyone else.

 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


Re: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-20 Thread Chris Clark

Timothy Pinet wrote:

I wanted to check post-review however since I am on Windows my
Python26\Scripts\post-review.exe is a compiled executable and not a
python file. Would it be enough to grab the post-review.py from the
repo and delete the exe?
  


Not with the current version. There are a few dependencies (not many), 
e.g. a version module. So you should take the whole thing. You can of 
course make an exe from it very easily. See end of mail for the custom 
setup script I use.


Chris



Issue:

NOTE using Python 2.4, this results in an exe about 4Mb in size.
NOTE using Python 2.6, this results in an exe about 5.5Mb in size.

   c:\python24\python ingres_setup.py py2exe
   setup.py py2exe

Quick-N-Dirty create win32 binaries and zip file script.
Zero error checking.

TODO inject 'py2exe' into sys.argv?

import os
import glob
import shutil

from distutils.core import setup

import py2exe


# Clean temp Python/Jython files
delete_list=glob.glob('simplejson/*.pyc')+glob.glob('simplejson/*$py.class')
for x in delete_list:
   os.remove(x)

try:
   shutil.rmtree('dist')
except WindowsError, info:
   # assume directory does not exist
   pass

# disable optimization- we _may_ need docs strings, specifically copyright
setup(
   options = {py2exe: {
   #includes: [decimal],
   optimize: 1,  ## 1 and NOT 2 because I use 
the __doc__ string as the usage string. 2 optimises out the doc strings

   'bundle_files': 1,
   ## options to reduce size of final exe
   #~ 'ascii': True,  # Exclude encodings
   'excludes':[
   '_ssl',  # Exclude _ssl
   'pyreadline', #'difflib',
   'doctest', #'locale',
   #'optparse',
   'pickle', #'calendar',# Exclude 
standard library

   #'re',
   ],  
   }},
   zipfile = None, ## try and make a single exe, if do not want this 
loose this and the 'bundle_files' option

   console=['postreview.py']
   )

zipfilename='distribute_me.zip'
zipfilelist=['ingres_readme.txt', 'postreview.py', ] + 
glob.glob('simplejson/*')+ glob.glob('dist/*')


import zipfile
z = zipfile.ZipFile(zipfilename, 'w')
for x in zipfilelist:
   z.write(x)
z.close()

print 'Created:', zipfilename

--
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: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-19 Thread Timothy Pinet
Alternatively, the diffutils docs (http://www.gnu.org/software/hello/
manual/diff.html#Binary) state:

Differing binary files are considered to cause trouble because the
resulting diff output does not capture all the differences. This
trouble causes diff to exit with status 2. However, this trouble
cannot occur with the --a or --text option, or with the -q or --brief
option, as these options both cause diff to treat binary files like
text files.

Can post-review call svn diff with the -a option? Is there a way that
I can tell it to do that in config?

I believe the reason is that the svn:mime-type property is not set
correctly on the binary file and svn diff is trying to text diff it.

Thanks,
Tim

-- 
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: post-review error with svn and gnu diffutils (new review with an added binary file)

2010-08-19 Thread Timothy Pinet
In which file would I look to remove the param? I am using a Windows
environment.

 

Thanks,

Tim

 

From: reviewboard@googlegroups.com [mailto:reviewbo...@googlegroups.com] On
Behalf Of Christian Hammond
Sent: August-19-10 8:43 PM
To: reviewboard@googlegroups.com
Subject: Re: post-review error with svn and gnu diffutils (new review with
an added binary file)

 

There's no way to pass additional arguments right now. Personally I don't
like that we even use --diff-cmd, but it works around some other problems
we've hit in the past (I'd have to check the commit logs to remember exactly
what they were).

It may be worth modifying your copy for now to remove that parameter, if
it's blocking you.

Christian

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



On Thu, Aug 19, 2010 at 5:42 AM, Timothy Pinet tim.pi...@gmail.com wrote:

Alternatively, the diffutils docs (http://www.gnu.org/software/hello/
http://www.gnu.org/software/hello/%0Amanual/diff.html#Binary 
manual/diff.html#Binary) state:

Differing binary files are considered to cause trouble because the
resulting diff output does not capture all the differences. This
trouble causes diff to exit with status 2. However, this trouble
cannot occur with the --a or --text option, or with the -q or --brief
option, as these options both cause diff to treat binary files like
text files.

Can post-review call svn diff with the -a option? Is there a way that
I can tell it to do that in config?

I believe the reason is that the svn:mime-type property is not set
correctly on the binary file and svn diff is trying to text diff it.


Thanks,
Tim

--
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
mailto: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

-- 
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: Post-Review: Error updating requests

2010-04-16 Thread Daniel
I'm sorry. I missed the part about nightly build. No, it's just from
1.0.6. Is using nightly the only way to go about it now?

Thanks,
Daniel


On Apr 15, 5:36 pm, Christian Hammond chip...@chipx86.com wrote:
 Hmm, and this is the nightly? It's hard to tell from the debug output.

 Christian

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



 On Wed, Apr 14, 2010 at 11:43 AM, Daniel dyoon@gmail.com wrote:
  It only happens you submit the request twice.

  $ post-review 186598 (first time submit)
  ...

  Review request # 5 posted.

 http:///reviews/r/6

  $ post-review 186598 (same change #)
  Traceback (most recent call last):
   File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
  module
     load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
  review')()
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2774, in main
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2423, in tempt_fate
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
  in new_review_request
  UnboundLocalError: local variable 'rsp' referenced before assignment

  On Apr 14, 11:28 am, Daniel dyoon@gmail.com wrote:
   I am still seeing it.

   $ post-review 186593
   Traceback (most recent call last):
     File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
   module
       load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
   review')()
     File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
   2774, in main
     File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
   2423, in tempt_fate
     File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
   in new_review_requestUnboundLocalError: local variable 'rsp' referenced
  before assignment

   On Feb 24, 8:54 pm, Christian Hammond chip...@chipx86.com wrote:

There's a fix for this going in tonight. You'll want to grab tomorrow's
nightly.

Christian

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

On Wed, Feb 24, 2010 at 6:43 AM, Florian Föbel
arenaweb...@googlemail.comwrote:

 Hi,

 Found an error in post-review from RBTools 0.2rc1:
 Traceback (most recent call last):
   File /usr/local/bin/post-review, line 10, in module
     load_entry_point('RBTools==0.2rc1', 'console_scripts',
  'post-review')()
   File

  /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 2774, in main
   File

  /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 2423, in tempt_fate
   File

  /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 427, in new_review_request
UnboundLocalError: local variable 'rsp' referenced before assignment

 It was introduced with the following commit:

 http://github.com/reviewboard/rbtools/commit/5968f84456d1e349bd174c2b...

 I tried to manually add rsp, = e.args but that won't work. I guess
  I
 currently lack the python skills and understanding of post-review to
  fix
 this without some time to spare.

 Anyone having a simple fix?

 Regards,
 Florian

 --
 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...@googlegroups.com
  reviewboard%2bunsubscr...@googlegroups.comreviewboard%252bunsubscr...@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 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 at
 http://groups.google.com/group/reviewboard?hl=en

  To unsubscribe, reply using remove me as the subject.

 --
 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/

Re: Post-Review: Error updating requests

2010-04-16 Thread Christian Hammond
For the time-being, yes. It's safe. I'm probably doing a release off the
current nightly (maybe another fix or two) this weekend.

Christian

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


On Fri, Apr 16, 2010 at 10:40 AM, Daniel dyoon@gmail.com wrote:

 I'm sorry. I missed the part about nightly build. No, it's just from
 1.0.6. Is using nightly the only way to go about it now?

 Thanks,
 Daniel


 On Apr 15, 5:36 pm, Christian Hammond chip...@chipx86.com wrote:
  Hmm, and this is the nightly? It's hard to tell from the debug output.
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.reviewboard.org
  VMware, Inc. -http://www.vmware.com
 
 
 
  On Wed, Apr 14, 2010 at 11:43 AM, Daniel dyoon@gmail.com wrote:
   It only happens you submit the request twice.
 
   $ post-review 186598 (first time submit)
   ...
 
   Review request # 5 posted.
 
  http:///reviews/r/6
 
   $ post-review 186598 (same change #)
   Traceback (most recent call last):
File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
   module
  load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
   review')()
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
   2774, in main
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
   2423, in tempt_fate
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
   in new_review_request
   UnboundLocalError: local variable 'rsp' referenced before assignment
 
   On Apr 14, 11:28 am, Daniel dyoon@gmail.com wrote:
I am still seeing it.
 
$ post-review 186593
Traceback (most recent call last):
  File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
module
load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
review')()
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
2774, in main
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
2423, in tempt_fate
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
 427,
in new_review_requestUnboundLocalError: local variable 'rsp'
 referenced
   before assignment
 
On Feb 24, 8:54 pm, Christian Hammond chip...@chipx86.com wrote:
 
 There's a fix for this going in tonight. You'll want to grab
 tomorrow's
 nightly.
 
 Christian
 
 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com
 
 On Wed, Feb 24, 2010 at 6:43 AM, Florian Föbel
 arenaweb...@googlemail.comwrote:
 
  Hi,
 
  Found an error in post-review from RBTools 0.2rc1:
  Traceback (most recent call last):
File /usr/local/bin/post-review, line 10, in module
  load_entry_point('RBTools==0.2rc1', 'console_scripts',
   'post-review')()
File
 
  
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
  line 2774, in main
File
 
  
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
  line 2423, in tempt_fate
File
 
  
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
  line 427, in new_review_request
 UnboundLocalError: local variable 'rsp' referenced before
 assignment
 
  It was introduced with the following commit:
 
  http://github.com/reviewboard/rbtools/commit/5968f84456d1e349bd174c2b.
 ..
 
  I tried to manually add rsp, = e.args but that won't work. I
 guess
   I
  currently lack the python skills and understanding of post-review
 to
   fix
  this without some time to spare.
 
  Anyone having a simple fix?
 
  Regards,
  Florian
 
  --
  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...@googlegroups.com
 reviewboard%2bunsubscr...@googlegroups.comreviewboard%252bunsubscr...@googlegroups.com
 
   reviewboard%2bunsubscr...@googlegroups.comreviewboard%252bunsubscr...@googlegroups.com
 reviewboard%252bunsubscr...@googlegroups.comreviewboard%25252bunsubscr...@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 athttp://www.reviewboard.org/users/
   -~--~~~~--~~--~--~---
   To unsubscribe from this group, send email to
   reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
 

Re: Post-Review: Error updating requests

2010-04-15 Thread Christian Hammond
Hmm, and this is the nightly? It's hard to tell from the debug output.

Christian

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


On Wed, Apr 14, 2010 at 11:43 AM, Daniel dyoon@gmail.com wrote:

 It only happens you submit the request twice.

 $ post-review 186598 (first time submit)
 ...

 Review request # 5 posted.

 http:///reviews/r/6


 $ post-review 186598 (same change #)
 Traceback (most recent call last):
  File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
 module
load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
 review')()
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
 2774, in main
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
 2423, in tempt_fate
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
 in new_review_request
 UnboundLocalError: local variable 'rsp' referenced before assignment



 On Apr 14, 11:28 am, Daniel dyoon@gmail.com wrote:
  I am still seeing it.
 
  $ post-review 186593
  Traceback (most recent call last):
File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
  module
  load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
  review')()
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2774, in main
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2423, in tempt_fate
File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
  in new_review_requestUnboundLocalError: local variable 'rsp' referenced
 before assignment
 
  On Feb 24, 8:54 pm, Christian Hammond chip...@chipx86.com wrote:
 
   There's a fix for this going in tonight. You'll want to grab tomorrow's
   nightly.
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.reviewboard.org
   VMware, Inc. -http://www.vmware.com
 
   On Wed, Feb 24, 2010 at 6:43 AM, Florian Föbel
   arenaweb...@googlemail.comwrote:
 
Hi,
 
Found an error in post-review from RBTools 0.2rc1:
Traceback (most recent call last):
  File /usr/local/bin/post-review, line 10, in module
load_entry_point('RBTools==0.2rc1', 'console_scripts',
 'post-review')()
  File
   
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
line 2774, in main
  File
   
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
line 2423, in tempt_fate
  File
   
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
line 427, in new_review_request
   UnboundLocalError: local variable 'rsp' referenced before assignment
 
It was introduced with the following commit:
 
   
 http://github.com/reviewboard/rbtools/commit/5968f84456d1e349bd174c2b...
 
I tried to manually add rsp, = e.args but that won't work. I guess
 I
currently lack the python skills and understanding of post-review to
 fix
this without some time to spare.
 
Anyone having a simple fix?
 
Regards,
Florian
 
--
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...@googlegroups.com
 reviewboard%2bunsubscr...@googlegroups.comreviewboard%252bunsubscr...@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.comreviewboard%2bunsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en

 To unsubscribe, reply using remove me as the subject.


-- 
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: Post-Review: Error updating requests

2010-04-14 Thread Daniel
It only happens you submit the request twice.

$ post-review 186598 (first time submit)
...

Review request # 5 posted.

http:///reviews/r/6


$ post-review 186598 (same change #)
Traceback (most recent call last):
  File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
module
load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
review')()
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
2774, in main
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
2423, in tempt_fate
  File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
in new_review_request
UnboundLocalError: local variable 'rsp' referenced before assignment



On Apr 14, 11:28 am, Daniel dyoon@gmail.com wrote:
 I am still seeing it.

 $ post-review 186593
 Traceback (most recent call last):
   File /net/adc2100488/scratch/opt/bin/post-review, line 8, in
 module
     load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-
 review')()
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
 2774, in main
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
 2423, in tempt_fate
   File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line 427,
 in new_review_requestUnboundLocalError: local variable 'rsp' referenced 
 before assignment

 On Feb 24, 8:54 pm, Christian Hammond chip...@chipx86.com wrote:

  There's a fix for this going in tonight. You'll want to grab tomorrow's
  nightly.

  Christian

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

  On Wed, Feb 24, 2010 at 6:43 AM, Florian Föbel
  arenaweb...@googlemail.comwrote:

   Hi,

   Found an error in post-review from RBTools 0.2rc1:
   Traceback (most recent call last):
     File /usr/local/bin/post-review, line 10, in module
       load_entry_point('RBTools==0.2rc1', 'console_scripts', 
   'post-review')()
     File
   /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
   line 2774, in main
     File
   /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
   line 2423, in tempt_fate
     File
   /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
   line 427, in new_review_request
  UnboundLocalError: local variable 'rsp' referenced before assignment

   It was introduced with the following commit:

  http://github.com/reviewboard/rbtools/commit/5968f84456d1e349bd174c2b...

   I tried to manually add rsp, = e.args but that won't work. I guess I
   currently lack the python skills and understanding of post-review to fix
   this without some time to spare.

   Anyone having a simple fix?

   Regards,
   Florian

   --
   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...@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

To unsubscribe, reply using remove me as the subject.


Re: Post-Review: Error updating requests

2010-02-24 Thread Christian Hammond
There's a fix for this going in tonight. You'll want to grab tomorrow's
nightly.

Christian

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


On Wed, Feb 24, 2010 at 6:43 AM, Florian Föbel
arenaweb...@googlemail.comwrote:

 Hi,

 Found an error in post-review from RBTools 0.2rc1:
 Traceback (most recent call last):
   File /usr/local/bin/post-review, line 10, in module
 load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post-review')()
   File
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 2774, in main
   File
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 2423, in tempt_fate
   File
 /usr/local/lib/python2.5/site-packages/RBTools-0.2rc1-py2.5.egg/rbtools/postreview.py,
 line 427, in new_review_request
 UnboundLocalError: local variable 'rsp' referenced before assignment

 It was introduced with the following commit:

 http://github.com/reviewboard/rbtools/commit/5968f84456d1e349bd174c2b79d348de77321cd8

 I tried to manually add rsp, = e.args but that won't work. I guess I
 currently lack the python skills and understanding of post-review to fix
 this without some time to spare.

 Anyone having a simple fix?

 Regards,
 Florian

 --
 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: post-review error: WindowsError: [Error 2] The system cannot find the file specified

2009-11-07 Thread Pv

I ran in to the same thing and opened a bug to make a missing diff
binary a friendlier message.

Pv

On Nov 6, 10:24 pm, Akhilesh akhileshjo...@gmail.com wrote:
 After a lot of troubleshooting I found the problem: diff.exe was not
 present in the PATH.

 On Nov 6, 2:57 pm, Akhilesh akhileshjo...@gmail.com wrote:



  I have setup Reviewboard on Windows and now setting up post-review on
  client machine so that I can start creating review requests.

  I'm getting following error:

  C:\p4post-review  25159 -d p4 info
   repository info: Path: ssb-sc.mimosasystems.com:1666, Base path: None, 
   Supports changesets: True
   p4 counters
   Generating diff for changenum 25159
   p4 describe -s 25159
   Processing edit of 
   //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/Ind
exDisposer.cs
   Writing 
   //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/In
dexDisposer.cs#4 to c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3-
   p4 print -o c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3- -q 
   //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/Ind
exDisposer.cs#4
   Writing 
   //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/In
dexDisposer.cs#5 to c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh
   p4 print -o c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh -q 
   //mimosa/dev/dev-EWS/Products/FDA/Components/MsiIndexer/DocumentIndexer/Ind
exDisposer.cs#5
   diff -urNp c:\docume~1\admini~1.jam\locals~1\temp\tmp2gtu3- 
   c:\docume~1\admini~1.jam\locals~1\temp\tmpxqneeh

  Traceback (most recent call last):
    File C:\Python25\Scripts\post-review-script.py, line 8, in
  module
      load_entry_point('RBTools==0.2beta2', 'console_scripts', 'post-
  review')()
    File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2511, in main
    File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  1358, in diff
    File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  1609, in _changenum_diff
    File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  1639, in _do_diff
    File build/bdist.linux-x86_64/egg/rbtools/postreview.py, line
  2112, in execute
    File C:\Python25\lib\subprocess.py, line 593, in __init__
      errread, errwrite)
    File C:\Python25\lib\subprocess.py, line 793, in _execute_child
      startupinfo)
  WindowsError: [Error 2] The system cannot find the file specified

  C:\p4

  Any ideas?
--~--~-~--~~~---~--~~
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: post-review error

2009-09-26 Thread Christian Hammond
Is the domain really codereview.com? Because I could understand a 404, since
while your site here is codereview.com, you will most likely hit the actual
codereview.com which doesn't host Review Board.

Christian

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


On Fri, Sep 25, 2009 at 7:04 PM, lotin lotin2...@gmail.com wrote:


 The return code is 405
 VirtualHost 168.3.63.233:81
ServerName codereview.com
DocumentRoot e:/rb-site/htdocs

# Error handlers
ErrorDocument 500 /errordocs/500.html

# Serve django pages
Location /review/
PythonPath ['e:/rb-site/conf'] + sys.path
SetEnv DJANGO_SETTINGS_MODULE reviewboard.settings
SetEnv PYTHON_EGG_CACHE e:/rb-site/tmp/egg_cache
SetHandler mod_python
PythonHandler django.core.handlers.modpython
PythonAutoReload Off
PythonDebug Off
# Used to run multiple mod_python sites in the same apache
PythonInterpreter reviewboard_rb-site
/Location

# Serve static media without running it through mod_python
# (overrides the above)
Location /review/media
SetHandler None
/Location
Location /review/errordocs
SetHandler None
/Location

Directory e:/rb-site/htdocs
AllowOverride All
Allow from all
/Directory

# Alias static media requests to filesystem
Alias /review/media e:/rb-site/htdocs/media
Alias /review/errordocs e:/rb-site/htdocs/errordocs
 /VirtualHost


 On Sep 25, 12:12 pm, Christian Hammond chip...@chipx86.com wrote:
  Where's the Review Board configuration?
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.review-board.org
  VMware, Inc. -http://www.vmware.com
 
 
 
  On Thu, Sep 24, 2009 at 7:15 PM, lotin lotin2...@gmail.com wrote:
 
   #
   # This is the main Apache HTTP server configuration file.  It contains
   the
   # configuration directives that give the server its instructions.
   # See URL:http://httpd.apache.org/docs/2.2/ for detailed
   information.
   # In particular, see
   # URL:http://httpd.apache.org/docs/2.2/mod/directives.html
   # for a discussion of each configuration directive.
   #
   # Do NOT simply read the instructions in here without understanding
   # what they do.  They're here only as hints or reminders.  If you are
   unsure
   # consult the online docs. You have been warned.
   #
   # Configuration and logfile names: If the filenames you specify for
   many
   # of the server's control files begin with / (or drive:/ for
   Win32), the
   # server will use that explicit path.  If the filenames do *not* begin
   # with /, the value of ServerRoot is prepended -- so logs/foo.log
   # with ServerRoot set to C:/Program Files/Apache Software Foundation/
   Apache2.2 will be interpreted by the
   # server as C:/Program Files/Apache Software Foundation/Apache2.2/
   logs/foo.log.
   #
   # NOTE: Where filenames are specified, you must use forward slashes
   # instead of backslashes (e.g., c:/apache instead of c:\apache).
   # If a drive letter is omitted, the drive on which Apache.exe is
   located
   # will be used by default.  It is recommended that you always supply
   # an explicit drive letter in absolute paths, however, to avoid
   # confusion.
   #
 
   # ThreadsPerChild: constant number of worker threads in the server
   process
   # MaxRequestsPerChild: maximum  number of requests a server process
   serves
   ThreadsPerChild 250
   MaxRequestsPerChild  0
 
   #
   # ServerRoot: The top of the directory tree under which the server's
   # configuration, error, and log files are kept.
   #
   # Do not add a slash at the end of the directory path.  If you point
   # ServerRoot at a non-local disk, be sure to point the LockFile
   directive
   # at a local disk.  If you wish to share the same ServerRoot for
   multiple
   # httpd daemons, you will need to change at least LockFile and
   PidFile.
   #
   ServerRoot C:/Program Files/Apache Software Foundation/Apache2.2
 
   #
   # Listen: Allows you to bind Apache to specific IP addresses and/or
   # ports, instead of the default. See also the VirtualHost
   # directive.
   #
   # Change this to Listen on specific IP addresses as shown below to
   # prevent Apache from glomming onto all bound IP addresses (0.0.0.0)
   #
   #Listen 127.0.0.1:80
 
   Listen 80
 
   #
   # Dynamic Shared Object (DSO) Support
   #
   # To be able to use the functionality of a module which was built as a
   DSO you
   # have to place corresponding `LoadModule' lines at this location so
   the
   # directives contained in it are actually available _before_ they are
   used.
   # Statically compiled modules (those listed by `httpd -l') do not need
   # to be 

Re: post-review error

2009-09-26 Thread lotin



I did'nt discribe the error well.

when I use the url by IE http://168.3.63.233:81/review/;  is  OK.

and i can post the review request by WEB UI. So I think I created the
rb-site success.

but I can't use the post-review ,when I use it ,it show Unable to
access http://x.x.x.23:81/review/api/json/accounts/login/.
Th
e host path may be invalid
urlopen error (10061, 'Connection refused') .
then i try to use the http://x.x.x.23:81/review/api/json/accounts/login/
by IE  ,it return 405.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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: post-review error

2009-09-24 Thread lotin

#
# This is the main Apache HTTP server configuration file.  It contains
the
# configuration directives that give the server its instructions.
# See URL:http://httpd.apache.org/docs/2.2/ for detailed
information.
# In particular, see
# URL:http://httpd.apache.org/docs/2.2/mod/directives.html
# for a discussion of each configuration directive.
#
# Do NOT simply read the instructions in here without understanding
# what they do.  They're here only as hints or reminders.  If you are
unsure
# consult the online docs. You have been warned.
#
# Configuration and logfile names: If the filenames you specify for
many
# of the server's control files begin with / (or drive:/ for
Win32), the
# server will use that explicit path.  If the filenames do *not* begin
# with /, the value of ServerRoot is prepended -- so logs/foo.log
# with ServerRoot set to C:/Program Files/Apache Software Foundation/
Apache2.2 will be interpreted by the
# server as C:/Program Files/Apache Software Foundation/Apache2.2/
logs/foo.log.
#
# NOTE: Where filenames are specified, you must use forward slashes
# instead of backslashes (e.g., c:/apache instead of c:\apache).
# If a drive letter is omitted, the drive on which Apache.exe is
located
# will be used by default.  It is recommended that you always supply
# an explicit drive letter in absolute paths, however, to avoid
# confusion.
#

# ThreadsPerChild: constant number of worker threads in the server
process
# MaxRequestsPerChild: maximum  number of requests a server process
serves
ThreadsPerChild 250
MaxRequestsPerChild  0

#
# ServerRoot: The top of the directory tree under which the server's
# configuration, error, and log files are kept.
#
# Do not add a slash at the end of the directory path.  If you point
# ServerRoot at a non-local disk, be sure to point the LockFile
directive
# at a local disk.  If you wish to share the same ServerRoot for
multiple
# httpd daemons, you will need to change at least LockFile and
PidFile.
#
ServerRoot C:/Program Files/Apache Software Foundation/Apache2.2

#
# Listen: Allows you to bind Apache to specific IP addresses and/or
# ports, instead of the default. See also the VirtualHost
# directive.
#
# Change this to Listen on specific IP addresses as shown below to
# prevent Apache from glomming onto all bound IP addresses (0.0.0.0)
#
#Listen 127.0.0.1:80

Listen 80

#
# Dynamic Shared Object (DSO) Support
#
# To be able to use the functionality of a module which was built as a
DSO you
# have to place corresponding `LoadModule' lines at this location so
the
# directives contained in it are actually available _before_ they are
used.
# Statically compiled modules (those listed by `httpd -l') do not need
# to be loaded here.
#
# Example:
# LoadModule foo_module modules/mod_foo.so
#
LoadModule actions_module modules/mod_actions.so
LoadModule alias_module modules/mod_alias.so
LoadModule asis_module modules/mod_asis.so
LoadModule auth_basic_module modules/mod_auth_basic.so
#LoadModule auth_digest_module modules/mod_auth_digest.so
#LoadModule authn_anon_module modules/mod_authn_anon.so
#LoadModule authn_dbm_module modules/mod_authn_dbm.so
LoadModule authn_default_module modules/mod_authn_default.so
LoadModule authn_file_module modules/mod_authn_file.so
#LoadModule authz_dbm_module modules/mod_authz_dbm.so
LoadModule authz_default_module modules/mod_authz_default.so
LoadModule authz_groupfile_module modules/mod_authz_groupfile.so
LoadModule authz_host_module modules/mod_authz_host.so
LoadModule authz_user_module modules/mod_authz_user.so
LoadModule autoindex_module modules/mod_autoindex.so
#LoadModule cern_meta_module modules/mod_cern_meta.so
LoadModule cgi_module modules/mod_cgi.so
#LoadModule dav_module modules/mod_dav.so
#LoadModule dav_fs_module modules/mod_dav_fs.so
#LoadModule deflate_module modules/mod_deflate.so
LoadModule dir_module modules/mod_dir.so
LoadModule env_module modules/mod_env.so
#LoadModule expires_module modules/mod_expires.so
#LoadModule file_cache_module modules/mod_file_cache.so
#LoadModule headers_module modules/mod_headers.so
LoadModule imagemap_module modules/mod_imagemap.so
LoadModule include_module modules/mod_include.so
#LoadModule info_module modules/mod_info.so
LoadModule isapi_module modules/mod_isapi.so
LoadModule log_config_module modules/mod_log_config.so
LoadModule mime_module modules/mod_mime.so
#LoadModule mime_magic_module modules/mod_mime_magic.so
#LoadModule proxy_module modules/mod_proxy.so
#LoadModule proxy_ajp_module modules/mod_proxy_ajp.so
#LoadModule proxy_balancer_module modules/mod_proxy_balancer.so
#LoadModule proxy_connect_module modules/mod_proxy_connect.so
#LoadModule proxy_http_module modules/mod_proxy_http.so
#LoadModule proxy_ftp_module modules/mod_proxy_ftp.so
LoadModule negotiation_module modules/mod_negotiation.so
#LoadModule rewrite_module modules/mod_rewrite.so
LoadModule setenvif_module modules/mod_setenvif.so
#LoadModule speling_module modules/mod_speling.so
#LoadModule status_module 

Re: post-review error

2009-09-24 Thread Christian Hammond
Where's the Review Board configuration?

Christian

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


On Thu, Sep 24, 2009 at 7:15 PM, lotin lotin2...@gmail.com wrote:


 #
 # This is the main Apache HTTP server configuration file.  It contains
 the
 # configuration directives that give the server its instructions.
 # See URL:http://httpd.apache.org/docs/2.2/ for detailed
 information.
 # In particular, see
 # URL:http://httpd.apache.org/docs/2.2/mod/directives.html
 # for a discussion of each configuration directive.
 #
 # Do NOT simply read the instructions in here without understanding
 # what they do.  They're here only as hints or reminders.  If you are
 unsure
 # consult the online docs. You have been warned.
 #
 # Configuration and logfile names: If the filenames you specify for
 many
 # of the server's control files begin with / (or drive:/ for
 Win32), the
 # server will use that explicit path.  If the filenames do *not* begin
 # with /, the value of ServerRoot is prepended -- so logs/foo.log
 # with ServerRoot set to C:/Program Files/Apache Software Foundation/
 Apache2.2 will be interpreted by the
 # server as C:/Program Files/Apache Software Foundation/Apache2.2/
 logs/foo.log.
 #
 # NOTE: Where filenames are specified, you must use forward slashes
 # instead of backslashes (e.g., c:/apache instead of c:\apache).
 # If a drive letter is omitted, the drive on which Apache.exe is
 located
 # will be used by default.  It is recommended that you always supply
 # an explicit drive letter in absolute paths, however, to avoid
 # confusion.
 #

 # ThreadsPerChild: constant number of worker threads in the server
 process
 # MaxRequestsPerChild: maximum  number of requests a server process
 serves
 ThreadsPerChild 250
 MaxRequestsPerChild  0

 #
 # ServerRoot: The top of the directory tree under which the server's
 # configuration, error, and log files are kept.
 #
 # Do not add a slash at the end of the directory path.  If you point
 # ServerRoot at a non-local disk, be sure to point the LockFile
 directive
 # at a local disk.  If you wish to share the same ServerRoot for
 multiple
 # httpd daemons, you will need to change at least LockFile and
 PidFile.
 #
 ServerRoot C:/Program Files/Apache Software Foundation/Apache2.2

 #
 # Listen: Allows you to bind Apache to specific IP addresses and/or
 # ports, instead of the default. See also the VirtualHost
 # directive.
 #
 # Change this to Listen on specific IP addresses as shown below to
 # prevent Apache from glomming onto all bound IP addresses (0.0.0.0)
 #
 #Listen 127.0.0.1:80

 Listen 80

 #
 # Dynamic Shared Object (DSO) Support
 #
 # To be able to use the functionality of a module which was built as a
 DSO you
 # have to place corresponding `LoadModule' lines at this location so
 the
 # directives contained in it are actually available _before_ they are
 used.
 # Statically compiled modules (those listed by `httpd -l') do not need
 # to be loaded here.
 #
 # Example:
 # LoadModule foo_module modules/mod_foo.so
 #
 LoadModule actions_module modules/mod_actions.so
 LoadModule alias_module modules/mod_alias.so
 LoadModule asis_module modules/mod_asis.so
 LoadModule auth_basic_module modules/mod_auth_basic.so
 #LoadModule auth_digest_module modules/mod_auth_digest.so
 #LoadModule authn_anon_module modules/mod_authn_anon.so
 #LoadModule authn_dbm_module modules/mod_authn_dbm.so
 LoadModule authn_default_module modules/mod_authn_default.so
 LoadModule authn_file_module modules/mod_authn_file.so
 #LoadModule authz_dbm_module modules/mod_authz_dbm.so
 LoadModule authz_default_module modules/mod_authz_default.so
 LoadModule authz_groupfile_module modules/mod_authz_groupfile.so
 LoadModule authz_host_module modules/mod_authz_host.so
 LoadModule authz_user_module modules/mod_authz_user.so
 LoadModule autoindex_module modules/mod_autoindex.so
 #LoadModule cern_meta_module modules/mod_cern_meta.so
 LoadModule cgi_module modules/mod_cgi.so
 #LoadModule dav_module modules/mod_dav.so
 #LoadModule dav_fs_module modules/mod_dav_fs.so
 #LoadModule deflate_module modules/mod_deflate.so
 LoadModule dir_module modules/mod_dir.so
 LoadModule env_module modules/mod_env.so
 #LoadModule expires_module modules/mod_expires.so
 #LoadModule file_cache_module modules/mod_file_cache.so
 #LoadModule headers_module modules/mod_headers.so
 LoadModule imagemap_module modules/mod_imagemap.so
 LoadModule include_module modules/mod_include.so
 #LoadModule info_module modules/mod_info.so
 LoadModule isapi_module modules/mod_isapi.so
 LoadModule log_config_module modules/mod_log_config.so
 LoadModule mime_module modules/mod_mime.so
 #LoadModule mime_magic_module modules/mod_mime_magic.so
 #LoadModule proxy_module modules/mod_proxy.so
 #LoadModule proxy_ajp_module modules/mod_proxy_ajp.so
 #LoadModule proxy_balancer_module modules/mod_proxy_balancer.so
 #LoadModule proxy_connect_module modules/mod_proxy_connect.so
 

Re: post-review error

2009-09-22 Thread Christian Hammond
This sounds like a server or proxy error rather than a Review Board error.

Do you need to be behind a proxy server to connect to Review Board?

Is this Windows or Linux?

As for the 404, it sounds like your web server isn't actually serving up
Review Board at the right URL. Which web server are you using, and can you
show the web server configuration?

Christian

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


On Tue, Sep 22, 2009 at 12:51 AM, lotin lotin2...@gmail.com wrote:


 When i try to post from client machine using 'post-review' tool, it
 shows below mess.

 Unable to access http://x.x.x.23:81/review/api/json/accounts/login/.
 Th
 e host path may be invalid
 urlopen error (10061, 'Connection refused')

 i set the 'REVIEWBOARD_URL' = http://x.x.x.23:81/review/

 i try use the url by the IE browser return 404



 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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: post-review error

2009-09-22 Thread Pubudu Rathnayake
Hello ,

I set the IP address of  the server machine as review board URL  like this
and it worked for me.

REVIEWBOARD_URL = http://192.168.1.107;
pubudu.

On Tue, Sep 22, 2009 at 1:21 PM, lotin lotin2...@gmail.com wrote:


 When i try to post from client machine using 'post-review' tool, it
 shows below mess.

 Unable to access http://x.x.x.23:81/review/api/json/accounts/login/.
 Th
 e host path may be invalid
 urlopen error (10061, 'Connection refused')

 i set the 'REVIEWBOARD_URL' = http://x.x.x.23:81/review/

 i try use the url by the IE browser return 404



 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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: post-review error

2009-09-22 Thread lotin

I use the Apache2.2,and my system is windows.
and the returncode was 405 when I use the URL by IE.

On 9月22日, 下午3时56分, Christian Hammond chip...@chipx86.com wrote:
 This sounds like a server or proxy error rather than a Review Board error.

 Do you need to be behind a proxy server to connect to Review Board?

 Is this Windows or Linux?

 As for the 404, it sounds like your web server isn't actually serving up
 Review Board at the right URL. Which web server are you using, and can you
 show the web server configuration?

 Christian

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



 On Tue, Sep 22, 2009 at 12:51 AM, lotin lotin2...@gmail.com wrote:

  When i try to post from client machine using 'post-review' tool, it
  shows below mess.

  Unable to accesshttp://x.x.x.23:81/review/api/json/accounts/login/.
  Th
  e host path may be invalid
  urlopen error (10061, 'Connection refused')

  i set the 'REVIEWBOARD_URL' =http://x.x.x.23:81/review/

  i try use the url by the IE browser return 404- 隐藏被引用文字 -

 - 显示引用的文字 -
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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: post-review error

2009-08-20 Thread Agile

Thanks Christian. It works now. I can create a post-commit
review for a particular release.

However, it failed in creating a post-review for the very last
revision.

Also, unless it is the first review, it always shows two successive
revisions side by side. Is there any way of having reviewers look
at a single revision on the screen?

Thanks again.

Kyle

On Aug 19, 6:57 pm, Christian Hammond chip...@chipx86.com wrote:
 You should put the URL shown by 'svn info' in your checkout as the Mirror
 Path. Hopefully it doesn't have auth info in the URL.

 Christian

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



 On Wed, Aug 19, 2009 at 2:31 PM, Code Review review.c...@gmail.com wrote:
  Thanks.

  My checkout is using:

  svn cohttp://myrepository.mydomain.edu/svn/myproject

  The authentication method used is svn+htaccess.

  So, what should I write under the Mirror Path?

  Kyle

  On Wed, Aug 19, 2009 at 3:38 PM, Christian Hammond 
  chip...@chipx86.comwrote:

  Sounds like your repository as reported by svn info doesn't match the
  Repository entry in Review Board.

  The likely cause is that your checkout is using https or svn+ssh, or
  something, whereas your Repository entry is using http.

  You should continue using http for the Path in the Repository, but then
  set Mirror Path to be whatever your checkout is using.

  Christian

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

  On Wed, Aug 19, 2009 at 6:37 AM, Agile review.c...@gmail.com wrote:

  Just a quick correction, I forgot to say that I had put a dot ('.') at
  the end while setting:

   svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu.

  I think I forgot to mention that. I find this useful to add so that
  people
  will not think that I made a mistake there.

  Thanks in advance.

  Kyle

  On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
   Thanks Christian. I grabbed and installed the latest tgz file:
   RBTools-0.2beta2.dev-20090819.tar.gz
 http://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009..
  .

   and gave a sudo python setup.py. I think everything went
   smoothly.

   Now, I am having problem with setting the repository configuration.
   I would appreciate if you could help me on this. Here is
   what I do:

   1. Get the most recent working copy with

svn cohttp://myrepository.mydomain.edu/svn/myproject

   2. Change directory into the top directory of my project

cd myproject

   3. I want to do a post-commit review. So, I issue:

post-review --revision-range 122
Unable to find a Review Board server for this source code tree.

   OK. Then, I follow the instructions here:

  http://www.review-board.org/docs/manual/dev/users/tools/post-review/

   Particularly, I look under the configuration section. While I am in
   the top directory of my project (myproject), I issue:

svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
property 'reviewboard:url' set on '.'

   Note that this address is the address of the front page of my
  reviewboard
   site.

   I again issue:

post-review --revision-range 122

   Error creating review request: The repository path specified is not in
  the
   list of known repositories (code 206)

   However, I had indeed created this repository in my Review Board site.

   Then, I see the next line in the configuration section ofhttp://
 www.review-board.org/docs/manual/dev/users/tools/post-review/
   It says:

   You can set the repository URL in .reviewboardrc file in any
  directory.
   This must be a directory in the user’s checkout path to work.

   So, without changing anything I also create a .reviewboardrc
   (under the top directory, myproject)

vi .reviewboardrc

   As a single line, I enter

   REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;

   I try again:

post-review --revision-range 122
Error creating review request: The repository path specified is not
  in the

   list of known repositories (code 206)

   Thank you very much in advance.

   Kyle

   On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond 
  chip...@chipx86.comwrote:

Hi Kyle,

Can you grab the latest nightly from
   http://downloads.review-board.org/nightlies/andgive it a try? This
  may
be fixed.

Christian

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

On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com
  wrote:

hello -

I would appreciate your help on this problem.

I receive the following error when I issue:

post-review

(pre or post commit)

*

Traceback (most recent call last):
 File /usr/local/bin/post-review, line 5, in module
   pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
 File 

Re: post-review error

2009-08-20 Thread Christian Hammond
The side-by-side diffs? That's part of the review process, showing the diff
of what changed. Not really sure what you mean by unless it is the first
review?

Christian

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


On Thu, Aug 20, 2009 at 6:15 AM, Agile review.c...@gmail.com wrote:


 Thanks Christian. It works now. I can create a post-commit
 review for a particular release.

 However, it failed in creating a post-review for the very last
 revision.

 Also, unless it is the first review, it always shows two successive
 revisions side by side. Is there any way of having reviewers look
 at a single revision on the screen?

 Thanks again.

 Kyle

 On Aug 19, 6:57 pm, Christian Hammond chip...@chipx86.com wrote:
  You should put the URL shown by 'svn info' in your checkout as the Mirror
  Path. Hopefully it doesn't have auth info in the URL.
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.review-board.org
  VMware, Inc. -http://www.vmware.com
 
 
 
  On Wed, Aug 19, 2009 at 2:31 PM, Code Review review.c...@gmail.com
 wrote:
   Thanks.
 
   My checkout is using:
 
   svn cohttp://myrepository.mydomain.edu/svn/myproject
 
   The authentication method used is svn+htaccess.
 
   So, what should I write under the Mirror Path?
 
   Kyle
 
   On Wed, Aug 19, 2009 at 3:38 PM, Christian Hammond 
 chip...@chipx86.comwrote:
 
   Sounds like your repository as reported by svn info doesn't match the
   Repository entry in Review Board.
 
   The likely cause is that your checkout is using https or svn+ssh, or
   something, whereas your Repository entry is using http.
 
   You should continue using http for the Path in the Repository, but
 then
   set Mirror Path to be whatever your checkout is using.
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.review-board.org
   VMware, Inc. -http://www.vmware.com
 
   On Wed, Aug 19, 2009 at 6:37 AM, Agile review.c...@gmail.com wrote:
 
   Just a quick correction, I forgot to say that I had put a dot ('.')
 at
   the end while setting:
 
svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu.
 
   I think I forgot to mention that. I find this useful to add so that
   people
   will not think that I made a mistake there.
 
   Thanks in advance.
 
   Kyle
 
   On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
Thanks Christian. I grabbed and installed the latest tgz file:
RBTools-0.2beta2.dev-20090819.tar.gz
  http://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009
 ..
   .
 
and gave a sudo python setup.py. I think everything went
smoothly.
 
Now, I am having problem with setting the repository configuration.
I would appreciate if you could help me on this. Here is
what I do:
 
1. Get the most recent working copy with
 
 svn cohttp://myrepository.mydomain.edu/svn/myproject
 
2. Change directory into the top directory of my project
 
 cd myproject
 
3. I want to do a post-commit review. So, I issue:
 
 post-review --revision-range 122
 Unable to find a Review Board server for this source code tree.
 
OK. Then, I follow the instructions here:
 
   
 http://www.review-board.org/docs/manual/dev/users/tools/post-review/
 
Particularly, I look under the configuration section. While I am in
the top directory of my project (myproject), I issue:
 
 svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
 property 'reviewboard:url' set on '.'
 
Note that this address is the address of the front page of my
   reviewboard
site.
 
I again issue:
 
 post-review --revision-range 122
 
Error creating review request: The repository path specified is not
 in
   the
list of known repositories (code 206)
 
However, I had indeed created this repository in my Review Board
 site.
 
Then, I see the next line in the configuration section ofhttp://
  www.review-board.org/docs/manual/dev/users/tools/post-review/
It says:
 
You can set the repository URL in .reviewboardrc file in any
   directory.
This must be a directory in the user’s checkout path to work.
 
So, without changing anything I also create a .reviewboardrc
(under the top directory, myproject)
 
 vi .reviewboardrc
 
As a single line, I enter
 
REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;
 
I try again:
 
 post-review --revision-range 122
 Error creating review request: The repository path specified is
 not
   in the
 
list of known repositories (code 206)
 
Thank you very much in advance.
 
Kyle
 
On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond 
   chip...@chipx86.comwrote:
 
 Hi Kyle,
 
 Can you grab the latest nightly from
http://downloads.review-board.org/nightlies/andgive it a try?
 This
   may
 be fixed.
 
 Christian
 
 --
 Christian 

Re: post-review error

2009-08-20 Thread Agile

Would it be possible show a single
snapshot of the system in one screen
and ask them to enter
comments?

(I don't want to show two revisions side
by side.)

On Aug 20, 5:45 pm, Christian Hammond chip...@chipx86.com wrote:
 The side-by-side diffs? That's part of the review process, showing the diff
 of what changed. Not really sure what you mean by unless it is the first
 review?

 Christian

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



 On Thu, Aug 20, 2009 at 6:15 AM, Agile review.c...@gmail.com wrote:

  Thanks Christian. It works now. I can create a post-commit
  review for a particular release.

  However, it failed in creating a post-review for the very last
  revision.

  Also, unless it is the first review, it always shows two successive
  revisions side by side. Is there any way of having reviewers look
  at a single revision on the screen?

  Thanks again.

  Kyle

  On Aug 19, 6:57 pm, Christian Hammond chip...@chipx86.com wrote:
   You should put the URL shown by 'svn info' in your checkout as the Mirror
   Path. Hopefully it doesn't have auth info in the URL.

   Christian

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

   On Wed, Aug 19, 2009 at 2:31 PM, Code Review review.c...@gmail.com
  wrote:
Thanks.

My checkout is using:

svn cohttp://myrepository.mydomain.edu/svn/myproject

The authentication method used is svn+htaccess.

So, what should I write under the Mirror Path?

Kyle

On Wed, Aug 19, 2009 at 3:38 PM, Christian Hammond 
  chip...@chipx86.comwrote:

Sounds like your repository as reported by svn info doesn't match the
Repository entry in Review Board.

The likely cause is that your checkout is using https or svn+ssh, or
something, whereas your Repository entry is using http.

You should continue using http for the Path in the Repository, but
  then
set Mirror Path to be whatever your checkout is using.

Christian

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

On Wed, Aug 19, 2009 at 6:37 AM, Agile review.c...@gmail.com wrote:

Just a quick correction, I forgot to say that I had put a dot ('.')
  at
the end while setting:

 svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu.

I think I forgot to mention that. I find this useful to add so that
people
will not think that I made a mistake there.

Thanks in advance.

Kyle

On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
 Thanks Christian. I grabbed and installed the latest tgz file:
 RBTools-0.2beta2.dev-20090819.tar.gz
   http://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009
  ..
.

 and gave a sudo python setup.py. I think everything went
 smoothly.

 Now, I am having problem with setting the repository configuration.
 I would appreciate if you could help me on this. Here is
 what I do:

 1. Get the most recent working copy with

  svn cohttp://myrepository.mydomain.edu/svn/myproject

 2. Change directory into the top directory of my project

  cd myproject

 3. I want to do a post-commit review. So, I issue:

  post-review --revision-range 122
  Unable to find a Review Board server for this source code tree.

 OK. Then, I follow the instructions here:

 http://www.review-board.org/docs/manual/dev/users/tools/post-review/

 Particularly, I look under the configuration section. While I am in
 the top directory of my project (myproject), I issue:

  svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
  property 'reviewboard:url' set on '.'

 Note that this address is the address of the front page of my
reviewboard
 site.

 I again issue:

  post-review --revision-range 122

 Error creating review request: The repository path specified is not
  in
the
 list of known repositories (code 206)

 However, I had indeed created this repository in my Review Board
  site.

 Then, I see the next line in the configuration section ofhttp://
   www.review-board.org/docs/manual/dev/users/tools/post-review/
 It says:

 You can set the repository URL in .reviewboardrc file in any
directory.
 This must be a directory in the user’s checkout path to work.

 So, without changing anything I also create a .reviewboardrc
 (under the top directory, myproject)

  vi .reviewboardrc

 As a single line, I enter

 REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;

 I try again:

  post-review --revision-range 122
  Error creating review request: The repository path specified is
  not
in the

 list of known repositories (code 206)

 Thank you very much in advance.

 Kyle

 On 

Re: post-review error

2009-08-19 Thread Agile

Just a quick correction, I forgot to say that I had put a dot ('.') at
the end while setting:

 svn propset reviewboard:url http://myreviewboard.mydomain.edu .

I think I forgot to mention that. I find this useful to add so that
people
will not think that I made a mistake there.

Thanks in advance.

Kyle

On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
 Thanks Christian. I grabbed and installed the latest tgz file:
 RBTools-0.2beta2.dev-20090819.tar.gzhttp://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009...

 and gave a sudo python setup.py. I think everything went
 smoothly.

 Now, I am having problem with setting the repository configuration.
 I would appreciate if you could help me on this. Here is
 what I do:

 1. Get the most recent working copy with

  svn cohttp://myrepository.mydomain.edu/svn/myproject

 2. Change directory into the top directory of my project

  cd myproject

 3. I want to do a post-commit review. So, I issue:

  post-review --revision-range 122
  Unable to find a Review Board server for this source code tree.

 OK. Then, I follow the instructions here:

 http://www.review-board.org/docs/manual/dev/users/tools/post-review/

 Particularly, I look under the configuration section. While I am in
 the top directory of my project (myproject), I issue:

  svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
  property 'reviewboard:url' set on '.'

 Note that this address is the address of the front page of my reviewboard
 site.

 I again issue:

  post-review --revision-range 122

 Error creating review request: The repository path specified is not in the
 list of known repositories (code 206)

 However, I had indeed created this repository in my Review Board site.

 Then, I see the next line in the configuration section 
 ofhttp://www.review-board.org/docs/manual/dev/users/tools/post-review/
 It says:

 You can set the repository URL in .reviewboardrc file in any directory.
 This must be a directory in the user’s checkout path to work.

 So, without changing anything I also create a .reviewboardrc
 (under the top directory, myproject)

  vi .reviewboardrc

 As a single line, I enter

 REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;

 I try again:

  post-review --revision-range 122
  Error creating review request: The repository path specified is not in the

 list of known repositories (code 206)

 Thank you very much in advance.

 Kyle

 On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond 
 chip...@chipx86.comwrote:



  Hi Kyle,

  Can you grab the latest nightly from
 http://downloads.review-board.org/nightlies/and give it a try? This may
  be fixed.

  Christian

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

  On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com wrote:

  hello -

  I would appreciate your help on this problem.

  I receive the following error when I issue:

  post-review

  (pre or post commit)

  *

  Traceback (most recent call last):
   File /usr/local/bin/post-review, line 5, in module
     pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
   File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 448,
  in run_script
     self.require(requires)[0].run_script(script_name, ns)
   File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 1173,
  in run_script
     exec script_code in namespace, namespace
   File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
  py2.6.egg/EGG-INFO/scripts/post-review, line 2313, in module

   File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
  py2.6.egg/EGG-INFO/scripts/post-review, line 2295, in main
   File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
  py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
   File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
  py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
  
  
   (repeats the last line many times)
  
  
  
  
   File /usr/lib/python2.6/urllib2.py, line 124, in urlopen
     return _opener.open(url, data, timeout)
   File /usr/lib/python2.6/urllib2.py, line 383, in open
     response = self._open(req, data)
   File /usr/lib/python2.6/urllib2.py, line 401, in _open
     '_open', req)
   File /usr/lib/python2.6/urllib2.py, line 361, in _call_chain
     result = func(*args)
   File /usr/lib/python2.6/urllib2.py, line 1130, in http_open
     return self.do_open(httplib.HTTPConnection, req)
   File /usr/lib/python2.6/urllib2.py, line 1102, in do_open
     h.request(req.get_method(), req.get_selector(), req.data, headers)
   File /usr/lib/python2.6/httplib.py, line 874, in request
     self._send_request(method, url, body, headers)
   File /usr/lib/python2.6/httplib.py, line 911, in _send_request
     self.endheaders()
   File /usr/lib/python2.6/httplib.py, line 868, in endheaders
     

Re: post-review error

2009-08-19 Thread Code Review
Thanks Christian. I grabbed and installed the latest tgz file:
RBTools-0.2beta2.dev-20090819.tar.gzhttp://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-20090819.tar.gz

and gave a sudo python setup.py. I think everything went
smoothly.

Now, I am having problem with setting the repository configuration.
I would appreciate if you could help me on this. Here is
what I do:

1. Get the most recent working copy with

 svn co http://myrepository.mydomain.edu/svn/myproject

2. Change directory into the top directory of my project

 cd myproject

3. I want to do a post-commit review. So, I issue:

 post-review --revision-range 122
 Unable to find a Review Board server for this source code tree.

OK. Then, I follow the instructions here:

http://www.review-board.org/docs/manual/dev/users/tools/post-review/

Particularly, I look under the configuration section. While I am in
the top directory of my project (myproject), I issue:

 svn propset reviewboard:url http://myreviewboard.mydomain.edu
 property 'reviewboard:url' set on '.'

Note that this address is the address of the front page of my reviewboard
site.

I again issue:

 post-review --revision-range 122
Error creating review request: The repository path specified is not in the
list of known repositories (code 206)

However, I had indeed created this repository in my Review Board site.

Then, I see the next line in the configuration section of
http://www.review-board.org/docs/manual/dev/users/tools/post-review/
It says:

You can set the repository URL in .reviewboardrc file in any directory.
This must be a directory in the user’s checkout path to work.

So, without changing anything I also create a .reviewboardrc
(under the top directory, myproject)

 vi .reviewboardrc

As a single line, I enter

REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;

I try again:

 post-review --revision-range 122
 Error creating review request: The repository path specified is not in the
list of known repositories (code 206)

Thank you very much in advance.

Kyle








On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond chip...@chipx86.comwrote:

 Hi Kyle,

 Can you grab the latest nightly from
 http://downloads.review-board.org/nightlies/ and give it a try? This may
 be fixed.

 Christian

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



 On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com wrote:


 hello -

 I would appreciate your help on this problem.

 I receive the following error when I issue:

 post-review

 (pre or post commit)


 *

 Traceback (most recent call last):
  File /usr/local/bin/post-review, line 5, in module
pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
  File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 448,
 in run_script
self.require(requires)[0].run_script(script_name, ns)
  File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 1173,
 in run_script
exec script_code in namespace, namespace
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2313, in module

  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2295, in main
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
 
 
  (repeats the last line many times)
 
 
 
 
  File /usr/lib/python2.6/urllib2.py, line 124, in urlopen
return _opener.open(url, data, timeout)
  File /usr/lib/python2.6/urllib2.py, line 383, in open
response = self._open(req, data)
  File /usr/lib/python2.6/urllib2.py, line 401, in _open
'_open', req)
  File /usr/lib/python2.6/urllib2.py, line 361, in _call_chain
result = func(*args)
  File /usr/lib/python2.6/urllib2.py, line 1130, in http_open
return self.do_open(httplib.HTTPConnection, req)
  File /usr/lib/python2.6/urllib2.py, line 1102, in do_open
h.request(req.get_method(), req.get_selector(), req.data, headers)
  File /usr/lib/python2.6/httplib.py, line 874, in request
self._send_request(method, url, body, headers)
  File /usr/lib/python2.6/httplib.py, line 911, in _send_request
self.endheaders()
  File /usr/lib/python2.6/httplib.py, line 868, in endheaders
self._send_output()
  File /usr/lib/python2.6/httplib.py, line 740, in _send_output
self.send(msg)
  File /usr/lib/python2.6/httplib.py, line 699, in send
self.connect()
  File /usr/lib/python2.6/httplib.py, line 683, in connect
self.timeout)
  File /usr/lib/python2.6/socket.py, line 502, in create_connection
sock = socket(af, socktype, proto)
  File /usr/lib/python2.6/socket.py, line 182, in __init__
_sock = _realsocket(family, type, proto)
 

Re: post-review error

2009-08-19 Thread Christian Hammond
Sounds like your repository as reported by svn info doesn't match the
Repository entry in Review Board.

The likely cause is that your checkout is using https or svn+ssh, or
something, whereas your Repository entry is using http.

You should continue using http for the Path in the Repository, but then set
Mirror Path to be whatever your checkout is using.

Christian

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


On Wed, Aug 19, 2009 at 6:37 AM, Agile review.c...@gmail.com wrote:


 Just a quick correction, I forgot to say that I had put a dot ('.') at
 the end while setting:

  svn propset reviewboard:url http://myreviewboard.mydomain.edu .

 I think I forgot to mention that. I find this useful to add so that
 people
 will not think that I made a mistake there.

 Thanks in advance.

 Kyle

 On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
  Thanks Christian. I grabbed and installed the latest tgz file:
  RBTools-0.2beta2.dev-20090819.tar.gz
 http://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009...
 
  and gave a sudo python setup.py. I think everything went
  smoothly.
 
  Now, I am having problem with setting the repository configuration.
  I would appreciate if you could help me on this. Here is
  what I do:
 
  1. Get the most recent working copy with
 
   svn cohttp://myrepository.mydomain.edu/svn/myproject
 
  2. Change directory into the top directory of my project
 
   cd myproject
 
  3. I want to do a post-commit review. So, I issue:
 
   post-review --revision-range 122
   Unable to find a Review Board server for this source code tree.
 
  OK. Then, I follow the instructions here:
 
  http://www.review-board.org/docs/manual/dev/users/tools/post-review/
 
  Particularly, I look under the configuration section. While I am in
  the top directory of my project (myproject), I issue:
 
   svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
   property 'reviewboard:url' set on '.'
 
  Note that this address is the address of the front page of my reviewboard
  site.
 
  I again issue:
 
   post-review --revision-range 122
 
  Error creating review request: The repository path specified is not in
 the
  list of known repositories (code 206)
 
  However, I had indeed created this repository in my Review Board site.
 
  Then, I see the next line in the configuration section ofhttp://
 www.review-board.org/docs/manual/dev/users/tools/post-review/
  It says:
 
  You can set the repository URL in .reviewboardrc file in any directory.
  This must be a directory in the user’s checkout path to work.
 
  So, without changing anything I also create a .reviewboardrc
  (under the top directory, myproject)
 
   vi .reviewboardrc
 
  As a single line, I enter
 
  REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;
 
  I try again:
 
   post-review --revision-range 122
   Error creating review request: The repository path specified is not in
 the
 
  list of known repositories (code 206)
 
  Thank you very much in advance.
 
  Kyle
 
  On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond chip...@chipx86.com
 wrote:
 
 
 
   Hi Kyle,
 
   Can you grab the latest nightly from
  http://downloads.review-board.org/nightlies/and give it a try? This may
   be fixed.
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.review-board.org
   VMware, Inc. -http://www.vmware.com
 
   On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com wrote:
 
   hello -
 
   I would appreciate your help on this problem.
 
   I receive the following error when I issue:
 
   post-review
 
   (pre or post commit)
 
   *
 
   Traceback (most recent call last):
File /usr/local/bin/post-review, line 5, in module
  pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 448,
   in run_script
  self.require(requires)[0].run_script(script_name, ns)
File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 1173,
   in run_script
  exec script_code in namespace, namespace
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2313, in module
 
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2295, in main
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
   
   
    (repeats the last line many times)
   
   
   
   
File /usr/lib/python2.6/urllib2.py, line 124, in urlopen
  return _opener.open(url, data, timeout)
File /usr/lib/python2.6/urllib2.py, line 383, in open
  response = 

Re: post-review error

2009-08-19 Thread Code Review
Thanks.

My checkout is using:

svn co http://myrepository.mydomain.edu/svn/myproject

The authentication method used is svn+htaccess.

So, what should I write under the Mirror Path?

Kyle

On Wed, Aug 19, 2009 at 3:38 PM, Christian Hammond chip...@chipx86.comwrote:

 Sounds like your repository as reported by svn info doesn't match the
 Repository entry in Review Board.

 The likely cause is that your checkout is using https or svn+ssh, or
 something, whereas your Repository entry is using http.

 You should continue using http for the Path in the Repository, but then set
 Mirror Path to be whatever your checkout is using.

 Christian

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


 On Wed, Aug 19, 2009 at 6:37 AM, Agile review.c...@gmail.com wrote:


 Just a quick correction, I forgot to say that I had put a dot ('.') at
 the end while setting:

  svn propset reviewboard:url http://myreviewboard.mydomain.edu .

 I think I forgot to mention that. I find this useful to add so that
 people
 will not think that I made a mistake there.

 Thanks in advance.

 Kyle

 On Aug 19, 9:30 am, Code Review review.c...@gmail.com wrote:
  Thanks Christian. I grabbed and installed the latest tgz file:
  RBTools-0.2beta2.dev-20090819.tar.gz
 http://downloads.review-board.org/nightlies/RBTools-0.2beta2.dev-2009...
 
  and gave a sudo python setup.py. I think everything went
  smoothly.
 
  Now, I am having problem with setting the repository configuration.
  I would appreciate if you could help me on this. Here is
  what I do:
 
  1. Get the most recent working copy with
 
   svn cohttp://myrepository.mydomain.edu/svn/myproject
 
  2. Change directory into the top directory of my project
 
   cd myproject
 
  3. I want to do a post-commit review. So, I issue:
 
   post-review --revision-range 122
   Unable to find a Review Board server for this source code tree.
 
  OK. Then, I follow the instructions here:
 
  http://www.review-board.org/docs/manual/dev/users/tools/post-review/
 
  Particularly, I look under the configuration section. While I am in
  the top directory of my project (myproject), I issue:
 
   svn propset reviewboard:urlhttp://myreviewboard.mydomain.edu
   property 'reviewboard:url' set on '.'
 
  Note that this address is the address of the front page of my
 reviewboard
  site.
 
  I again issue:
 
   post-review --revision-range 122
 
  Error creating review request: The repository path specified is not in
 the
  list of known repositories (code 206)
 
  However, I had indeed created this repository in my Review Board site.
 
  Then, I see the next line in the configuration section ofhttp://
 www.review-board.org/docs/manual/dev/users/tools/post-review/
  It says:
 
  You can set the repository URL in .reviewboardrc file in any directory.
  This must be a directory in the user’s checkout path to work.
 
  So, without changing anything I also create a .reviewboardrc
  (under the top directory, myproject)
 
   vi .reviewboardrc
 
  As a single line, I enter
 
  REVIEWBOARD_URL = http://myreviewboard.mydomain.edu;
 
  I try again:
 
   post-review --revision-range 122
   Error creating review request: The repository path specified is not in
 the
 
  list of known repositories (code 206)
 
  Thank you very much in advance.
 
  Kyle
 
  On Tue, Aug 18, 2009 at 10:49 PM, Christian Hammond 
 chip...@chipx86.comwrote:
 
 
 
   Hi Kyle,
 
   Can you grab the latest nightly from
  http://downloads.review-board.org/nightlies/and give it a try? This
 may
   be fixed.
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.review-board.org
   VMware, Inc. -http://www.vmware.com
 
   On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com wrote:
 
   hello -
 
   I would appreciate your help on this problem.
 
   I receive the following error when I issue:
 
   post-review
 
   (pre or post commit)
 
   *
 
   Traceback (most recent call last):
File /usr/local/bin/post-review, line 5, in module
  pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 448,
   in run_script
  self.require(requires)[0].run_script(script_name, ns)
File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 1173,
   in run_script
  exec script_code in namespace, namespace
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2313, in module
 
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2295, in main
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
   py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
   
   
   

Re: post-review error

2009-08-18 Thread Christian Hammond
Hi Kyle,

Can you grab the latest nightly from
http://downloads.review-board.org/nightlies/ and give it a try? This may be
fixed.

Christian

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


On Tue, Aug 18, 2009 at 11:44 AM, Kyle review.c...@gmail.com wrote:


 hello -

 I would appreciate your help on this problem.

 I receive the following error when I issue:

 post-review

 (pre or post commit)


 *

 Traceback (most recent call last):
  File /usr/local/bin/post-review, line 5, in module
pkg_resources.run_script('RBTools==0.2beta1', 'post-review')
  File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 448,
 in run_script
self.require(requires)[0].run_script(script_name, ns)
  File /usr/lib/python2.6/dist-packages/pkg_resources.py, line 1173,
 in run_script
exec script_code in namespace, namespace
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2313, in module

  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2295, in main
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
  File /usr/local/lib/python2.6/dist-packages/RBTools-0.2beta1-
 py2.6.egg/EGG-INFO/scripts/post-review, line 2009, in tempt_fate
 
 
  (repeats the last line many times)
 
 
 
 
  File /usr/lib/python2.6/urllib2.py, line 124, in urlopen
return _opener.open(url, data, timeout)
  File /usr/lib/python2.6/urllib2.py, line 383, in open
response = self._open(req, data)
  File /usr/lib/python2.6/urllib2.py, line 401, in _open
'_open', req)
  File /usr/lib/python2.6/urllib2.py, line 361, in _call_chain
result = func(*args)
  File /usr/lib/python2.6/urllib2.py, line 1130, in http_open
return self.do_open(httplib.HTTPConnection, req)
  File /usr/lib/python2.6/urllib2.py, line 1102, in do_open
h.request(req.get_method(), req.get_selector(), req.data, headers)
  File /usr/lib/python2.6/httplib.py, line 874, in request
self._send_request(method, url, body, headers)
  File /usr/lib/python2.6/httplib.py, line 911, in _send_request
self.endheaders()
  File /usr/lib/python2.6/httplib.py, line 868, in endheaders
self._send_output()
  File /usr/lib/python2.6/httplib.py, line 740, in _send_output
self.send(msg)
  File /usr/lib/python2.6/httplib.py, line 699, in send
self.connect()
  File /usr/lib/python2.6/httplib.py, line 683, in connect
self.timeout)
  File /usr/lib/python2.6/socket.py, line 502, in create_connection
sock = socket(af, socktype, proto)
  File /usr/lib/python2.6/socket.py, line 182, in __init__
_sock = _realsocket(family, type, proto)
 RuntimeError: maximum recursion depth exceeded while calling a Python
 object






 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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: post-review error Unable to parse diff header:

2008-10-29 Thread tom

Hi Christian,


Thanks for the tip - the 500 is because of an exception to do with a
licensing issue with p4:

Traceback:
File /usr/lib/python2.5/site-packages/django/core/handlers/base.py
in get_response
  86. response = callback(request, *callback_args,
**callback_kwargs)
File /usr/lib/python2.5/site-packages/django/views/decorators/
cache.py in _wrapped_view_func
  44. response = view_func(request, *args, **kwargs)
...
...
  35. changeset =
repository.get_scmtool().get_changeset(changenum)
File /home/guest/dist/reviewboard/scmtools/perforce.py in
get_changeset
  57. changeset = self.p4.run_describe(#39;-s#39;,
str(changesetid))
File /usr/lib/python2.5/site-packages/P4.py in lt;lambdagt;
  215. f = lambda *args: self.run(cmd, *args)
File /usr/lib/python2.5/site-packages/P4.py in run
  260. return P4API.P4Adapter.run(self, *self.__flatten(args))

Exception Type: P4Exception at /api/json/reviewrequests/new/
Exception Value: [P4#run] Errors during command execution( quot;p4
describe -s 23710quot; )

[Error]: Can#39;t create a new user - over license quota.
Try deleting old users with #39;user -d#39;.
License count: 55 users used of 55 licensed.

The reason for this was that I needed to source ~/.p4c from the
shell, so that it could set the valid $P4USER, before then running ./
manage.py runserver


On Oct 28, 7:46 pm, Christian Hammond [EMAIL PROTECTED] wrote:
 If you run with --debug, you should be able to see more error info.

 Christian

 --
 Christian Hammond - [EMAIL PROTECTED]
 VMware, Inc.

 On Tue, Oct 28, 2008 at 5:00 AM, tom [EMAIL PROTECTED] wrote:

  Hi,

  Has this been resolved?

  If not, this is to do with the fact that the newline character(s) are
  in the name of the local file extracted:

  here's a diff of what I did to fix the issue as a quick solution:

   #
   # Repository URL.
  @@ -963,6 +963,7 @@
              else:
                  die(Unknown change type '%s' for %s % (changetype,
  depot_path))

  +            new_file = new_file.rstrip('\r\n');
               diff_cmd = [diff, -urNp, old_file, new_file]
               # Diff returns 1 if differences were found.

  However, I got an internal server error when I tried a test post:

  Unable to accesshttp://localhost:8000/api/json/reviewrequests/new/.
  The host path may be invalid
  HTTP Error 500: INTERNAL SERVER ERROR

  logs:

  [28/Oct/2008 05:59:48] POST /api/json/reviewrequests/new/ HTTP/1.1
  500 96260

  On Oct 16, 8:54 am, Florian Föbel [EMAIL PROTECTED]
  wrote:
   Hi,

   I just got the same error after upgrading post-review. What I found out
  is
   that the subprocess.stdout that is read for the diff command contains a
   newline (\n) right after the second filename in the diff header. Kind of
   weird...

   No fix or explanation found yet.

   Regards,
   Florian

   2008/10/16 Joshua Slominski [EMAIL PROTECTED]

Can you post the debug output?

Sent from my iPhone

On Oct 15, 2008, at 6:00 PM, AndyP [EMAIL PROTECTED] wrote:

 Well I have a .reviewboardrc with valid stuff in it.  I modified
  (just
 now) the script too with the same data but it didn't work.

 I really suspect that that the 'execute' command isn't working right.

 Note that it gets past that if I change those Popen calls from 'True'
 back to 'False'.

 If they are 'False' it can't find the repository, and if it is 'True'
 it can't execute the diff (per the original posts in this thread).

 Thanks,

 On Oct 15, 2:01 pm, Joshua Slominski [EMAIL PROTECTED] wrote:
 I seen that error when i forgot to put in my review board URL in
 post-review.  Also be sure that post-review is using the correct
 SCM tool.
 It defaults to SVN if it can't find anything else.

 On Wed, Oct 15, 2008 at 4:48 PM, AndyP [EMAIL PROTECTED]
  wrote:

 That didn't work for me out of the box.  The two Popen() statements
 are different than the version that I had (v0.7 that worked), but
 when
 I changed it to True I got a different error:

 The current directory does not contain a checkout from a supported
 source code repository.

 Looking at this further, it appears that the only real difference
  is
 the way the execute command is created:

 v0.7
  data = execute('p4 info', ignore_errors=True)

 v0.8
  data = execute([p4, info], ignore_errors=True)

 On Oct 15, 12:45 pm, Joshua Slominski [EMAIL PROTECTED]
 wrote:- Hide quoted text -

 - Show quoted text -
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---



Re: post-review error Unable to parse diff header:

2008-10-28 Thread 13Strider

 Repository info 'Path: salmon.test.local:1999, Base path: None, Supports 
 changesets: True'
 Generating diff for changenum 121473
 p4 describe -s 121473
 Processing edit of //data/cmn/ini/common.ini
 Writing //data/cmn/ini/common.ini#1 to 
 c:\users\sam1\appdata\local\temp\tmpliutrf
 p4 print -q //data/cmn/ini/common.ini#1
 p4 where //data/cmn/ini/common.ini
 diff -urNp c:\users\sam1\appdata\local\temp\tmpliutrf 
 c:\data\cmn\ini\common.ini

Unable to parse diff header: : Invalid argument

On Oct 28, 4:46 pm, Christian Hammond [EMAIL PROTECTED] wrote:
 If you run with --debug, you should be able to see more error info.

 Christian

 --
 Christian Hammond - [EMAIL PROTECTED]
 VMware, Inc.

 On Tue, Oct 28, 2008 at 5:00 AM, tom [EMAIL PROTECTED] wrote:

  Hi,

  Has this been resolved?

  If not, this is to do with the fact that the newline character(s) are
  in the name of the local file extracted:

  here's a diff of what I did to fix the issue as a quick solution:

   #
   # Repository URL.
  @@ -963,6 +963,7 @@
              else:
                  die(Unknown change type '%s' for %s % (changetype,
  depot_path))

  +            new_file = new_file.rstrip('\r\n');
               diff_cmd = [diff, -urNp, old_file, new_file]
               # Diff returns 1 if differences were found.

  However, I got an internal server error when I tried a test post:

  Unable to accesshttp://localhost:8000/api/json/reviewrequests/new/.
  The host path may be invalid
  HTTP Error 500: INTERNAL SERVER ERROR

  logs:

  [28/Oct/2008 05:59:48] POST /api/json/reviewrequests/new/ HTTP/1.1
  500 96260

  On Oct 16, 8:54 am, Florian Föbel [EMAIL PROTECTED]
  wrote:
   Hi,

   I just got the same error after upgrading post-review. What I found out
  is
   that the subprocess.stdout that is read for the diff command contains a
   newline (\n) right after the second filename in the diff header. Kind of
   weird...

   No fix or explanation found yet.

   Regards,
   Florian

   2008/10/16 Joshua Slominski [EMAIL PROTECTED]

Can you post the debug output?

Sent from my iPhone

On Oct 15, 2008, at 6:00 PM, AndyP [EMAIL PROTECTED] wrote:

 Well I have a .reviewboardrc with valid stuff in it.  I modified
  (just
 now) the script too with the same data but it didn't work.

 I really suspect that that the 'execute' command isn't working right.

 Note that it gets past that if I change those Popen calls from 'True'
 back to 'False'.

 If they are 'False' it can't find the repository, and if it is 'True'
 it can't execute the diff (per the original posts in this thread).

 Thanks,

 On Oct 15, 2:01 pm, Joshua Slominski [EMAIL PROTECTED] wrote:
 I seen that error when i forgot to put in my review board URL in
 post-review.  Also be sure that post-review is using the correct
 SCM tool.
 It defaults to SVN if it can't find anything else.

 On Wed, Oct 15, 2008 at 4:48 PM, AndyP [EMAIL PROTECTED]
  wrote:

 That didn't work for me out of the box.  The two Popen() statements
 are different than the version that I had (v0.7 that worked), but
 when
 I changed it to True I got a different error:

 The current directory does not contain a checkout from a supported
 source code repository.

 Looking at this further, it appears that the only real difference
  is
 the way the execute command is created:

 v0.7
  data = execute('p4 info', ignore_errors=True)

 v0.8
  data = execute([p4, info], ignore_errors=True)

 On Oct 15, 12:45 pm, Joshua Slominski [EMAIL PROTECTED]
 wrote:- Hide quoted text -

 - Show quoted text -
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---



Re: post-review error Unable to parse diff header:

2008-10-16 Thread Florian Föbel
Hi,

I just got the same error after upgrading post-review. What I found out is
that the subprocess.stdout that is read for the diff command contains a
newline (\n) right after the second filename in the diff header. Kind of
weird...

No fix or explanation found yet.

Regards,
Florian

2008/10/16 Joshua Slominski [EMAIL PROTECTED]


 Can you post the debug output?

 Sent from my iPhone

 On Oct 15, 2008, at 6:00 PM, AndyP [EMAIL PROTECTED] wrote:

 
  Well I have a .reviewboardrc with valid stuff in it.  I modified (just
  now) the script too with the same data but it didn't work.
 
  I really suspect that that the 'execute' command isn't working right.
 
  Note that it gets past that if I change those Popen calls from 'True'
  back to 'False'.
 
  If they are 'False' it can't find the repository, and if it is 'True'
  it can't execute the diff (per the original posts in this thread).
 
  Thanks,
 
 
 
 
  On Oct 15, 2:01 pm, Joshua Slominski [EMAIL PROTECTED] wrote:
  I seen that error when i forgot to put in my review board URL in
  post-review.  Also be sure that post-review is using the correct
  SCM tool.
  It defaults to SVN if it can't find anything else.
 
 
 
  On Wed, Oct 15, 2008 at 4:48 PM, AndyP [EMAIL PROTECTED] wrote:
 
  That didn't work for me out of the box.  The two Popen() statements
  are different than the version that I had (v0.7 that worked), but
  when
  I changed it to True I got a different error:
 
  The current directory does not contain a checkout from a supported
  source code repository.
 
  Looking at this further, it appears that the only real difference is
  the way the execute command is created:
 
  v0.7
   data = execute('p4 info', ignore_errors=True)
 
  v0.8
   data = execute([p4, info], ignore_errors=True)
 
  On Oct 15, 12:45 pm, Joshua Slominski [EMAIL PROTECTED]
  wrote:- Hide quoted text -
 
  - Show quoted text -
  

 


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



Re: post-review error Unable to parse diff header:

2008-10-15 Thread Joshua Slominski
I fixed mine (sort of) in r1512 on line 1274, the shell option in
subprocess.Popen() was changed from True to False.  When I changed it back
to True it started working agian.  I don't know what the intent was so i
don't know if i fixed it properly.  Does anyone have any insigts on why this
would fix my problem?

On Wed, Oct 15, 2008 at 2:50 PM, Joshua Slominski [EMAIL PROTECTED]wrote:

  I am also getting this error my exact error is Unable to parse diff
 header: : No such file or directory

 I put in some more debug functions and this is what i get:

  Repository info 'Path: isis.hoem.ad.harman.com:1666, Base path: None,
 Supports changesets: True'
  Generating diff for changenum 163531
  p4 describe -s 163531
  making tempfile
  fd == 6
  tmpfile == d:\docume~1\jslomi~1\locals~1\temp\tmpodiyp_
  making tempfile
  fd == 6
  tmpfile == d:\docume~1\jslomi~1\locals~1\temp\tmpnfuhoi
  making tempfile
  fd == 6
  tmpfile == d:\docume~1\jslomi~1\locals~1\temp\tmpahrqhd
  Processing edit of
 //DC_Refresh/Development/Main/HMIController/cocos/hmi/common/app/private/CHMIAppImpl.cpp
  Writing
 //DC_Refresh/Development/Main/HMIController/cocos/hmi/common/app/private/CHMIAppImpl.cpp#13
 to d:\docume~1\jslomi~1\locals~1\temp\tmpnfuhoi
  p4 print -q
 //DC_Refresh/Development/Main/HMIController/cocos/hmi/common/app/private/CHMIAppImpl.cpp#13
  f.name = d:\docume~1\jslomi~1\locals~1\temp\tmpnfuhoi
  p4 where
 //DC_Refresh/Development/Main/HMIController/cocos/hmi/common/app/private/CHMIAppImpl.cpp
  diff -urNpbBw d:\docume~1\jslomi~1\locals~1\temp\tmpnfuhoi
 D:/NTG4/Refresh\cocos\hmi\common\app\private\CHMIAppImpl.cpp
 I think the problem is that the temp file is never being created but i
 don't know why.

   On Tue, Oct 14, 2008 at 7:52 PM, AndyP [EMAIL PROTECTED] wrote:


 Nope.  all the python stuff is under cygwin - doesn't run (without
 some serious setup issues) under dos.  Note that this used to work
 with the previous version of post-review (v0.7)... under cygwin.
  



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



Re: post-review error Unable to parse diff header:

2008-10-15 Thread AndyP

That didn't work for me out of the box.  The two Popen() statements
are different than the version that I had (v0.7 that worked), but when
I changed it to True I got a different error:

The current directory does not contain a checkout from a supported
source code repository.


Looking at this further, it appears that the only real difference is
the way the execute command is created:

v0.7
 data = execute('p4 info', ignore_errors=True)

v0.8
 data = execute([p4, info], ignore_errors=True)

On Oct 15, 12:45 pm, Joshua Slominski [EMAIL PROTECTED] wrote:

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



Re: post-review error Unable to parse diff header:

2008-10-13 Thread AndyP

I am attempting to upgrade to the latest-and-greatest and am now
running into this exact same problem.

post-review under cygwin (under DOS).

It looks like the p4 where command is returning a DOS style path,
and then if you look in a redirected differences output file, all the
letters are scrunched together with the '\' characters missing.

--- /cygdrive/c/DOCUME~1/APETER~1/LOCALS~1/Temp/Device.cpp  2008-08-12
10:52:53.549697400 -0700
+++ c:companyproductcoresrcwin32desktopDevice.cpp   1969-12-31
16:00:00.0 -0800


... and of course, the +++ file does not exist.




On Oct 9, 5:55 pm, Christian Hammond [EMAIL PROTECTED] wrote:
 Interesting. It could be that there's assumptions being made that confuse
 post-review when running inside the cygwin shell instead of a DOS prompt on
 Windows. The file paths look very inconsistent. One file is UNIX-style
 (/foo/bar), whereas the other is a mix of Windows (C:\...) and then partly
 UNIX. It's probably due to where we're getting these paths and how we're
 concatenating them.

 Can you run the same command in a DOS box and see what happens?

 Christian

 --
 Christian Hammond - [EMAIL PROTECTED]
 VMware, Inc.



 On Thu, Oct 9, 2008 at 5:45 PM, Amit Mokal [EMAIL PROTECTED] wrote:

  Hi

  I would like to thank you for developing review board.

  I am using review board with  perforce.
  When I use the post-review script I get the error
  Unable to parse diff header:

  Running the script with debug set to True gives

  ---­--
  $ python post-review.py 52004
   svn info
   p4 info
   repository info: Path: milan.engdlp.symantec.com:1666, Base path:
  None, Supports changesets: True
   Repository info 'Path: milan.engdlp.symantec.com:1666, Base path:
  None, Supports changesets: True'
   Generating diff for changenum 52004
   p4 describe -s 52004
   Processing edit of
  //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphSt­atePool.cpp
   Writing
  //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphS­tatePool.cpp#2
  to /c/DOCUME~1/ADMINI~1/LOCALS~1/Temp/tmpkWJo
  L
   p4 print -q
  //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphSt­atePool.cpp#2
   p4 where
  //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphSt­atePool.cpp
   diff -urNp /c/DOCUME~1/ADMINI~1/LOCALS~1/Temp/tmpkWJoCL
  C:/dev/sources\Vontu9\dev\native\src\endpoint\Agent\DetectionCore\Execution­GraphStatePool
  cpp

  +++ C:/dev/sources\Vontu9\dev\native\src\endpoint\Agent\DetectionCore
  \ExecutionGraphStatePool.cpp

  Unable to parse diff header: +++ C:/dev/sources\Vontu9\dev\native\src
  \endpoint\Agent\DetectionCore\ExecutionGraphStatePool.cpp

  ---­-

  My environment is cygwin. I am using
  python 2.5.1
  diffutils 2.8.7

  If I copy paste the diff command to the command line and save the
  output to a file I actually see a timestamp. However when the script
  executes the command the time stamp is missing.

  Thanks
  Amit- Hide quoted text -

 - Show quoted text -
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---



Re: post-review error Unable to parse diff header:

2008-10-09 Thread Christian Hammond
Interesting. It could be that there's assumptions being made that confuse
post-review when running inside the cygwin shell instead of a DOS prompt on
Windows. The file paths look very inconsistent. One file is UNIX-style
(/foo/bar), whereas the other is a mix of Windows (C:\...) and then partly
UNIX. It's probably due to where we're getting these paths and how we're
concatenating them.

Can you run the same command in a DOS box and see what happens?

Christian

-- 
Christian Hammond - [EMAIL PROTECTED]
VMware, Inc.


On Thu, Oct 9, 2008 at 5:45 PM, Amit Mokal [EMAIL PROTECTED] wrote:


 Hi

 I would like to thank you for developing review board.

 I am using review board with  perforce.
 When I use the post-review script I get the error
 Unable to parse diff header:

 Running the script with debug set to True gives

 -
 $ python post-review.py 52004
  svn info
  p4 info
  repository info: Path: milan.engdlp.symantec.com:1666, Base path:
 None, Supports changesets: True
  Repository info 'Path: milan.engdlp.symantec.com:1666, Base path:
 None, Supports changesets: True'
  Generating diff for changenum 52004
  p4 describe -s 52004
  Processing edit of
 //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphStatePool.cpp
  Writing
 //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphStatePool.cpp#2
 to /c/DOCUME~1/ADMINI~1/LOCALS~1/Temp/tmpkWJo
 L
  p4 print -q
 //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphStatePool.cpp#2
  p4 where
 //depot/Vontu9/dev/native/src/endpoint/Agent/DetectionCore/ExecutionGraphStatePool.cpp
  diff -urNp /c/DOCUME~1/ADMINI~1/LOCALS~1/Temp/tmpkWJoCL
 C:/dev/sources\Vontu9\dev\native\src\endpoint\Agent\DetectionCore\ExecutionGraphStatePool
 cpp

 +++ C:/dev/sources\Vontu9\dev\native\src\endpoint\Agent\DetectionCore
 \ExecutionGraphStatePool.cpp

 Unable to parse diff header: +++ C:/dev/sources\Vontu9\dev\native\src
 \endpoint\Agent\DetectionCore\ExecutionGraphStatePool.cpp

 

 My environment is cygwin. I am using
 python 2.5.1
 diffutils 2.8.7

 If I copy paste the diff command to the command line and save the
 output to a file I actually see a timestamp. However when the script
 executes the command the time stamp is missing.

 Thanks
 Amit
 


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



Re: post-review error

2008-10-08 Thread Christian Hammond
Sounds like you don't have patch.exe in the path. I advise getting the
Cygwin version.

Christian

-- 
Christian Hammond - [EMAIL PROTECTED]
VMware, Inc.


On Wed, Oct 8, 2008 at 1:02 AM, Jay [EMAIL PROTECTED] wrote:


 The following is error message:
 Error uploading diff:One or more fields had errors (105)
 Your review request still exists, but the diff is not attached.

 I have installed reviewboard on WindowsXP.

 And I set the my svn repository to svn://localhost.

 If I use the pre-review by web, I will get the following error message
 after clicking the view diff link.

 [Error 2] The system cannot find the file specified

 Traceback (most recent call last):
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\views.py, line 86, in view_diff
files = get_diff_files(diffset, None, interdiffset, highlighting)
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 651, in get_diff_files
enable_syntax_highlighting)
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 567, in generate_files
lambda: get_chunks(filediff.diffset,
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \djblets\util\misc.py, line 55, in cache_memoize
data = lookup_callable()
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 570, in lambda
enable_syntax_highlighting))
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 338, in get_chunks
new = get_patched_file(old, filediff)
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 254, in get_patched_file
return patch(filediff.diff, buffer, filediff.dest_file)
  File D:\reviewboard\reviewboard\reviewboard-read-only\reviewboard
 \diffviewer\diffutils.py, line 105, in patch
stderr=subprocess.STDOUT)
  File D:\Python\lib\subprocess.py, line 593, in __init__
errread, errwrite)
  File D:\Python\lib\subprocess.py, line 815, in _execute_child
startupinfo)
 WindowsError: [Error 2] The system cannot find the file specified

 Could anyone help me?

 


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