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 i

Post-review error when uploading the diff

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 permissi

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

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 Sa

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

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 16

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 --filename=" For your existing On Mon, Aug 19, 2013 at 4:35 AM, Akshaya Samantaray < akshaya.samanta...@gmail.com> wrote: > Hi David , > > My ob

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

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 kn

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

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

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 r

post-review error : Got API Error 105 (HTTP code 400): One or more fields had errors

HI Christian, I get post-review error : Got API Error 105 (HTTP code 400): One or more fields had errors, [u'cleartool: Error: Not a vob object: "a75177da.2f9344e8.bea8.0e:a7:60:3a:cd:d1".\r\n'] the above generates diff but fails after "HTTP POSTing to http" we

URGENT: post-review: Error uploading diff

Dear All, I have seen this message many times now, but i don't know how to fix this. After few days some user reports this issues, I have got the solution from chris to place p4 in /usr/bin/ sometime back but it seems now its not working can you please help how to fix this issue, what could the is

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 abov

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

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

post-review: Error uploading diff

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 c

Re: post-review error Unable to find root /api/

Hi Chrisitan, C:\Python25\Scripts>post-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:\Use

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 wrote: > Hi, > > I sent you details with debug options > > On Dec 10, 12:56 am, Christian Hammond wrote: >> Can

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 wrote: > If I am using jigars02v > C:\Python25\Scripts>post-review -d --server=http://jigars02v/reviews-- > summary=533900 --description=533900 --username=admin --password=admin > --p4-cl > ient=j

Re: post-review error Unable to find root /api/

Hi, I sent you details with debug options On Dec 10, 12:56 am, Christian Hammond 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

Re: post-review error Unable to find root /api/

If I am using jigars02v C:\Python25\Scripts>post-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 >>> R

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 wrote: > I am running post-review and receiving

post-review error Unable to find root /api/

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

Re: post-review error

On Thu, Nov 10, 2011 at 4:16 PM, Krles 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-

post-review error

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 c

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 wrote: > clutch@fx09:~/workspace/example$ post-review --debug RBTools 0.3.3 Home = /home/clutch svn info diff --ver

post-review error: Unable to find a Review Board server for this source code tree.

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

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

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 wrote: > Hi Deva, > > Are you behind a proxy? > > Christian > > On Thursday, August 18, 2011, Deva wrote: > > Hi, > > A

Re: post-review error

Hi Deva, Are you behind a proxy? Christian On Thursday, August 18, 2011, Deva 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 > closed by the r > emote host>' > > C:\

post-review error

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 ' C:\workspace\BDBJE>python C:\rbtools\rbtools\postreview.py --debug >>> svn info >>> repository info: Path: http://x.x.x.x/wds, Base path

Re: post-review error "The current directory does not contain a checkout from a supported source code repository."

Paresh wrote: I'm running reviewboard 1.0.9 on an ubuntu server with RBTools v0.2 on windows XP clients and subversion (TortoiseSVN v1.6.12) and python v2.6.6 I have one machine that is able to post reviews successfully and one machine that reports the error when I try to run post-review. Both

post-review error "The current directory does not contain a checkout from a supported source code repository."

I'm running reviewboard 1.0.9 on an ubuntu server with RBTools v0.2 on windows XP clients and subversion (TortoiseSVN v1.6.12) and python v2.6.6 I have one machine that is able to post reviews successfully and one machine that reports the error when I try to run post-review. Both machines have us

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

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 d

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 wrote: > > It actually isn'

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_clie

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

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

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 th

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 troubl

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

Good day! I am encountering an error when using post-review with a changeset range in a very particular scenario. I am using Subversion as my repo. There were a group of new files (one was a binary image) svn added to the repository in one of the changesets. When svn attempts its diff all of the t

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

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 wrote: > Hmm, and this is the nightly? It's hard to tell from the debug output. > > Christian > > -- > Christian Ham

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 wrote: > It only happens you submit the request

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

Re: Post-Review: Error updating requests

I am still seeing it. $ post-review 186593 Traceback (most recent call last): File "/net/adc2100488/scratch/opt/bin/post-review", line 8, in load_entry_point('RBTools==0.2rc1', 'console_scripts', 'post- review')() File "build/bdist.linux-x86_64/egg/rbtools/postreview.py", line 2774, in ma

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 wrote: > Hi, > > Found an error i

Post-Review: Error updating requests

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

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 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 wrote: > > > > > I have setup Rev

Re: post-review error

My problem was solved , thank you all the same. I found the client set a proxy. On Sep 27, 12:55 pm, Christian Hammond wrote: > This sounds like it might be a proxy issue. I didn't see an answer to > my earlier question. Do you go through a proxy server for outside > sites? If so, Python may be

Re: post-review error

This sounds like it might be a proxy issue. I didn't see an answer to my earlier question. Do you go through a proxy server for outside sites? If so, Python may be attempting to use the proxy server to access this site. Christian On Saturday, September 26, 2009, lotin wrote: > > > > I did'nt di

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

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

Re: post-review error

The return code is 405 ServerName codereview.com DocumentRoot "e:/rb-site/htdocs" # Error handlers ErrorDocument 500 /errordocs/500.html # Serve django pages PythonPath "['e:/rb-site/conf'] + sys.path" SetEnv DJANG

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 wrote: > > # > # This is the main Apache HTTP server configuration file. It con

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 http://httpd.apache.org/docs/2.2/> for detailed information. # In particular, see # http://httpd.apache.org/docs/2.2/mod/directives.html> # for a dis

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

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 wrote: > > When i try to post from client machine using 'post-review' tool, it > shows below mess. >

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 yo

post-review error

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 i set the 'REVIEWBOARD_URL' = http://x.x.x.23:81/review/ i try use the url by the IE browser return 404 --

Re: post-review error

Well, that would be a large change that, quite frankly, I'm not sure I see much value in for the most part. It would bump up the maintenance, testing, and caching burdens quite a bit. Maybe you can help me understand why this would work better for your organization. Is there a particular case wher

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 wrote: > The side-by-side diffs? That's part of the review process, showing the diff > of what changed

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

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 lo

Re: post-review error

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,

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 wrote: > Sounds like your repository as reported by svn i

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 t

Re: post-review error

Thanks Christian. I grabbed and installed the latest tgz file: 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 repo

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 advanc

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 A

post-review error

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 pkg_resources.run_script('RBTools==0.2beta1', 'post-review') Fil

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

Have you fixed this? On Oct 28, 9: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 adiffof what I did to fix the issue as a quick solution: > >  #

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

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

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

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 char

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

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("Unknow

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 J

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

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

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

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

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

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 b

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 f

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 fi

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

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 changen

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

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 sub

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

Maybe I missed this part in your response, but did you try running post-review in a standard DOS box with command.com? Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Tue, Oct 14, 2008 at 2:14 PM, AndyP <[EMAIL PROTECTED]> wrote: > > I'm stuck. > > I did a difference betwee

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

I'm stuck. I did a difference between the 0.7 version I was using (a patched version from a while ago) and the new pristine 0.8 version. The diff command line is different, so I figured I'd just use the older version. The execute subroutine is pretty significantly different too though, so I was

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 scrun

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 UNI

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

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

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 h

post-review error

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