Re: Post-review error when uploading the diff
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
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
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
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
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
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
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
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
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
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
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
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
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
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/
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/
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/
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/
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/
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
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
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
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.
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
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)
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)
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)
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)
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)
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)
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)
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
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
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
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
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
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
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
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
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
# # 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
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
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
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
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
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
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
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
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
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
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
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
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:
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:
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:
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:
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:
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:
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/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- 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:
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
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 -~--~~~~--~~--~--~---