Re: post-review diff upload error on windows XP

2010-08-23 Thread Chetan
Guys
In the current error msg the file path it is taking to process for the
clearcase file is wrong

Processed path: M:/etan.d_RB_Integration_int/RB_CVOB/
Actual path: M:/chetan.d_RB_Integration_int/RB_CVOB/

Below is the error msg can someone help to get this rectified.


  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/scmtools/core.py, line 68, in file_exists
self.get_file(path, revision)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/scmtools/clearcase.py, line 45, in get_file
return self.client.cat_file(self.adjust_path(path), revision)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/scmtools/clearcase.py, line 151, in cat_file
raise SCMError(errmsg)
SCMError: cat: M:/etan.d_RB_Integration_int/RB_CVOB/@@/main/
RB_Integration_main/2/Documents/@@/main/RB_Integration_main/2/
Doc1.txt@@/@@/main/RB_Integration_main/2: No such file or directory

As I am not into python coding i am not sure what to change in
clearcase.py script to handle this issue.

Any help appreciated


On Aug 23, 9:01 am, Chetan chetan@gmail.com wrote:
 Below are my RB and ClearCase configurations.

 RB is setup on Ubuntu
 post-review and clearcase is on windows XP client from wer i am
 generating the request.

 Thnks for the info i will do few trail and errors and see if i can get
 it working.

 On Aug 19, 2:01 am, Dan Savilonis d...@n-cube.org wrote:



  Is your server running Linux? I've only tested Clearcase support using
  a Windows RB server. Take a look at line 83 of 
  clearcase.py:http://github.com/reviewboard/reviewboard/blob/master/reviewboard/scm...

  You're likely going to need to do some editing here. I don't know
  anything about how Clearcase works on Linux, but I'm guessing the
  assumption that all vobs are mounted under vobs/ might not be true...

  Dan

  On Aug 10, 6:47 am, Chetan chetan@gmail.com wrote:

   As suggested by you i enabled logging to debug clearcase error. I
   given the log information when i execute post-review for clearcase
   below.

   ###

   2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
   diff, size = 1070
   2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
   diff.
   2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
   not found
   Traceback (most recent call last):
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/webapi/json.py, line 1169, in new_diff
       request.FILES.get('parent_diff_path'))
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/reviews/forms.py, line 234, in create
       history)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py, line 63, in create
       diff_file, basedir, check_existance=(not parent_diff_file)))
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py, line 149, in _process
   _files
       not tool.file_exists(filename, revision))):
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/core.py, line 68, in file_exists
       self.get_file(path, revision)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/clearcase.py, line 45, in get_fil
   e
       return self.client.cat_file(self.adjust_path(path), revision)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/clearcase.py, line 83, in adjust_
   path
       elem_path = elem_path[elem_path.rindex(vobs/)+5:]
   ValueError: substring not found
   ###


   On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:

I wanted execute svn diff recursively but found could not generate
one. But now I found if i run svn diff from the svn root directory it
handles all changes recursively.

On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:

 I'm not sure what you mean exactly. Is it not generating a diff with 
 all
 files recursively, or is there some reason why doing it recursively 
 isn't
 good enough?...

 read more »

 Are you using the post-review tool?

 Christian

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

 On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
  I got reviewboard working for subversion including https.
  In svn diff we cannot generate recursive diff coz of which we may 
  have
  to generate folder specific diffs. Is there a way around this?

  Thanks for the support.

  On Aug 5, 11:55 am, 

Re: post-review diff upload error on windows XP

2010-08-23 Thread Chetan

I did few trials on line 83 of clearcase.py. IF i change the vobs/
to M:/ it gives the element path in the error which otherwise would
state substring not found.

I am not sure what exactly shud i do after this.can you guys
suggest me some ways to figure out this.

when its is vobs/ in clearcase.py script.

 Uploading diff, size: 907
 HTTP POSTing to 
 http://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/176/diff/new/: 
 {'basedir': 'M:/'}
 Got API Error 105 (HTTP code 200): One or more fields had errors
 Error data: {u'fields': {u'path': [u'substring not found']}, u'stat': 
 u'fail', u'err': {u'msg': u'One or more fields
 had errors', u'code': 105}, u'deprecated': {u'in_version': u'1.5'}}

When its is changes to M:/:

 Uploading diff, size: 907
 HTTP POSTing to 
 http://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/175/diff/new/: 
 {'basedir': 'M:/'}
 Got API Error 105 (HTTP code 200): One or more fields had errors
 Error data: {u'fields': {u'path': [u'cat: 
 M:/chetan.d_RB_Integration_int/RB_CVOB/@@/main/RB_Integration_main/2/Docum
ents/@@/main/RB_Integration_main/2/Doc1.txt@@/@@/main/
RB_Integration_main/2: No such file or directory\n']}, u'stat': u'
fail', u'err': {u'msg': u'One or more fields had errors', u'code':
105}, u'deprecated': {u'in_version': u'1.5'}}







On Aug 23, 9:01 am, Chetan chetan@gmail.com wrote:
 Below are my RB and ClearCase configurations.

 RB is setup on Ubuntu
 post-review and clearcase is on windows XP client from wer i am
 generating the request.

 Thnks for the info i will do few trail and errors and see if i can get
 it working.

 On Aug 19, 2:01 am, Dan Savilonis d...@n-cube.org wrote:



  Is your server running Linux? I've only tested Clearcase support using
  a Windows RB server. Take a look at line 83 of 
  clearcase.py:http://github.com/reviewboard/reviewboard/blob/master/reviewboard/scm...

  You're likely going to need to do some editing here. I don't know
  anything about how Clearcase works on Linux, but I'm guessing the
  assumption that all vobs are mounted under vobs/ might not be true...

  Dan

  On Aug 10, 6:47 am, Chetan chetan@gmail.com wrote:

   As suggested by you i enabled logging to debug clearcase error. I
   given the log information when i execute post-review for clearcase
   below.

   ###

   2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
   diff, size = 1070
   2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
   diff.
   2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
   not found
   Traceback (most recent call last):
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/webapi/json.py, line 1169, in new_diff
       request.FILES.get('parent_diff_path'))
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/reviews/forms.py, line 234, in create
       history)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py, line 63, in create
       diff_file, basedir, check_existance=(not parent_diff_file)))
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py, line 149, in _process
   _files
       not tool.file_exists(filename, revision))):
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/core.py, line 68, in file_exists
       self.get_file(path, revision)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/clearcase.py, line 45, in get_fil
   e
       return self.client.cat_file(self.adjust_path(path), revision)
     File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/clearcase.py, line 83, in adjust_
   path
       elem_path = elem_path[elem_path.rindex(vobs/)+5:]
   ValueError: substring not found
   ###


   On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:

I wanted execute svn diff recursively but found could not generate
one. But now I found if i run svn diff from the svn root directory it
handles all changes recursively.

On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:

 I'm not sure what you mean exactly. Is it not generating a diff with 
 all
 files recursively, or is there some reason why doing it recursively 
 isn't
 good enough?...

 read more »

 Are you using the post-review tool?

 Christian

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

 On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
  I got reviewboard working for subversion including https.
 

Re: post-review diff upload error on windows XP

2010-08-22 Thread Chetan
Below are my RB and ClearCase configurations.

RB is setup on Ubuntu
post-review and clearcase is on windows XP client from wer i am
generating the request.

Thnks for the info i will do few trail and errors and see if i can get
it working.



On Aug 19, 2:01 am, Dan Savilonis d...@n-cube.org wrote:
 Is your server running Linux? I've only tested Clearcase support using
 a Windows RB server. Take a look at line 83 of 
 clearcase.py:http://github.com/reviewboard/reviewboard/blob/master/reviewboard/scm...

 You're likely going to need to do some editing here. I don't know
 anything about how Clearcase works on Linux, but I'm guessing the
 assumption that all vobs are mounted under vobs/ might not be true...

 Dan

 On Aug 10, 6:47 am, Chetan chetan@gmail.com wrote:



  As suggested by you i enabled logging to debug clearcase error. I
  given the log information when i execute post-review for clearcase
  below.

  ### 
  
  2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
  diff, size = 1070
  2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
  diff.
  2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
  not found
  Traceback (most recent call last):
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/webapi/json.py, line 1169, in new_diff
      request.FILES.get('parent_diff_path'))
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/reviews/forms.py, line 234, in create
      history)
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/diffviewer/forms.py, line 63, in create
      diff_file, basedir, check_existance=(not parent_diff_file)))
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/diffviewer/forms.py, line 149, in _process
  _files
      not tool.file_exists(filename, revision))):
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/scmtools/core.py, line 68, in file_exists
      self.get_file(path, revision)
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/scmtools/clearcase.py, line 45, in get_fil
  e
      return self.client.cat_file(self.adjust_path(path), revision)
    File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/scmtools/clearcase.py, line 83, in adjust_
  path
      elem_path = elem_path[elem_path.rindex(vobs/)+5:]
  ValueError: substring not found
  ### 
  

  On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:

   I wanted execute svn diff recursively but found could not generate
   one. But now I found if i run svn diff from the svn root directory it
   handles all changes recursively.

   On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:

I'm not sure what you mean exactly. Is it not generating a diff with all
files recursively, or is there some reason why doing it recursively 
isn't
good enough?...

read more »

Are you using the post-review tool?

Christian

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

On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
 I got reviewboard working for subversion including https.
 In svn diff we cannot generate recursive diff coz of which we may have
 to generate folder specific diffs. Is there a way around this?

 Thanks for the support.

 On Aug 5, 11:55 am, Christian Hammond chip...@chipx86.com wrote:
  As long as you have the $sitedir/data directory writable by the web
 server's
  user, you should be able to get https working. If you already had 
  the
  repository entry in there from RB 1.0.x, go back to that repository 
  and
 save
  it again. It should attempt to verify the certificate and then ask 
  you
 for
  confirmation. If that data directory is writable, it will store it 
  there
 and
  it should just work from then on

  read more »

  Christian

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

  On Wed, Aug 4, 2010 at 10:28 PM, Chetan chetan@gmail.com 
  wrote:
   Thank a ton.
   Review board with svn is working for me. svn project is using just
   local protocol authentication like svn://prj and not on https. I 
   still
   get errors for https projects.

   What are the steps to configure reviewboard to use with https
   configured svn projects?
   Once i have completed svn configiration will start to work on
   clearcase issue.

   You asked me to enable 

Re: post-review diff upload error on windows XP

2010-08-18 Thread Dan Savilonis
Is your server running Linux? I've only tested Clearcase support using
a Windows RB server. Take a look at line 83 of clearcase.py:
http://github.com/reviewboard/reviewboard/blob/master/reviewboard/scmtools/clearcase.py

You're likely going to need to do some editing here. I don't know
anything about how Clearcase works on Linux, but I'm guessing the
assumption that all vobs are mounted under vobs/ might not be true...

Dan

On Aug 10, 6:47 am, Chetan chetan@gmail.com wrote:
 As suggested by you i enabled logging to debug clearcase error. I
 given the log information when i execute post-review for clearcase
 below.

 ### 
 
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
 diff, size = 1070
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
 not found
 Traceback (most recent call last):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/webapi/json.py, line 1169, in new_diff
     request.FILES.get('parent_diff_path'))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py, line 234, in create
     history)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 63, in create
     diff_file, basedir, check_existance=(not parent_diff_file)))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 149, in _process
 _files
     not tool.file_exists(filename, revision))):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/core.py, line 68, in file_exists
     self.get_file(path, revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 45, in get_fil
 e
     return self.client.cat_file(self.adjust_path(path), revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 83, in adjust_
 path
     elem_path = elem_path[elem_path.rindex(vobs/)+5:]
 ValueError: substring not found
 ### 
 

 On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:







  I wanted execute svn diff recursively but found could not generate
  one. But now I found if i run svn diff from the svn root directory it
  handles all changes recursively.

  On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:

   I'm not sure what you mean exactly. Is it not generating a diff with all
   files recursively, or is there some reason why doing it recursively isn't
   good enough?...

   read more »

   Are you using the post-review tool?

   Christian

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

   On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
I got reviewboard working for subversion including https.
In svn diff we cannot generate recursive diff coz of which we may have
to generate folder specific diffs. Is there a way around this?

Thanks for the support.

On Aug 5, 11:55 am, Christian Hammond chip...@chipx86.com wrote:
 As long as you have the $sitedir/data directory writable by the web
server's
 user, you should be able to get https working. If you already had the
 repository entry in there from RB 1.0.x, go back to that repository 
 and
save
 it again. It should attempt to verify the certificate and then ask you
for
 confirmation. If that data directory is writable, it will store it 
 there
and
 it should just work from then on

 read more »

 Christian

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

 On Wed, Aug 4, 2010 at 10:28 PM, Chetan chetan@gmail.com wrote:
  Thank a ton.
  Review board with svn is working for me. svn project is using just
  local protocol authentication like svn://prj and not on https. I 
  still
  get errors for https projects.

  What are the steps to configure reviewboard to use with https
  configured svn projects?
  Once i have completed svn configiration will start to work on
  clearcase issue.

  You asked me to enable logging to debug clearcase issue, wer shud i
  enable it. Is it the same one under reviewboard- settinges- 
  logging -
   enable logging option?

  On Aug 4, 9:33 am, Christian Hammond chip...@chipx86.com wrote:
   You'll need to use svn diff. TortoiseSVN's diffs aren't
compatible

   read more »

   I strongly recommend using our post-review tool (part of RBTools) 
 

Re: post-review diff upload error on windows XP

2010-08-17 Thread Christian Hammond
It looks like it's trying to find vbos/ in the path in the diff, but it
doesn't exist.

I don't have access to ClearCase, and don't really know how it all works. We
are the moment a very low-budget project and can't afford a ClearCase
license. Our ClearCase support is heavily volunteer-based, so hopefully
someone with more knowledge can debug it.

Christian

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


On Tue, Aug 10, 2010 at 3:47 AM, Chetan chetan@gmail.com wrote:

 As suggested by you i enabled logging to debug clearcase error. I
 given the log information when i execute post-review for clearcase
 below.


 ###
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
 diff, size = 1070
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
 not found
 Traceback (most recent call last):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/webapi/json.py, line 1169, in new_diff
 request.FILES.get('parent_diff_path'))
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py, line 234, in create
 history)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 63, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 149, in _process
 _files
not tool.file_exists(filename, revision))):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/core.py, line 68, in file_exists
self.get_file(path, revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 45, in get_fil
 e
return self.client.cat_file(self.adjust_path(path), revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 83, in adjust_
 path
elem_path = elem_path[elem_path.rindex(vobs/)+5:]
 ValueError: substring not found

 ###

 On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:
  I wanted execute svn diff recursively but found could not generate
  one. But now I found if i run svn diff from the svn root directory it
  handles all changes recursively.
 
  On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:
 
 
 
   I'm not sure what you mean exactly. Is it not generating a diff with
 all
   files recursively, or is there some reason why doing it recursively
 isn't
   good enough?...
 
   read more »
 
   Are you using the post-review tool?
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.reviewboard.org
   VMware, Inc. -http://www.vmware.com
 
   On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
I got reviewboard working for subversion including https.
In svn diff we cannot generate recursive diff coz of which we may
 have
to generate folder specific diffs. Is there a way around this?
 
Thanks for the support.
 
On Aug 5, 11:55 am, Christian Hammond chip...@chipx86.com wrote:
 As long as you have the $sitedir/data directory writable by the web
server's
 user, you should be able to get https working. If you already had
 the
 repository entry in there from RB 1.0.x, go back to that repository
 and
save
 it again. It should attempt to verify the certificate and then ask
 you
for
 confirmation. If that data directory is writable, it will store it
 there
and
 it should just work from then on
 
 read more »
 
 Christian
 
 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com
 
 On Wed, Aug 4, 2010 at 10:28 PM, Chetan chetan@gmail.com
 wrote:
  Thank a ton.
  Review board with svn is working for me. svn project is using
 just
  local protocol authentication like svn://prj and not on https. I
 still
  get errors for https projects.
 
  What are the steps to configure reviewboard to use with https
  configured svn projects?
  Once i have completed svn configiration will start to work on
  clearcase issue.
 
  You asked me to enable logging to debug clearcase issue, wer shud
 i
  enable it. Is it the same one under reviewboard- settinges-
 logging -
   enable logging option?
 
  On Aug 4, 9:33 am, Christian Hammond chip...@chipx86.com
 wrote:
   You'll need to use svn diff. TortoiseSVN's diffs aren't
compatible
 
   read more »
 
   I 

Re: post-review diff upload error on windows XP

2010-08-16 Thread Chetan
Is there any other way i can get reviewboard to work with clearcase.
I am not able to find oout more from the Logging errors.




On Aug 10, 3:47 pm, Chetan chetan@gmail.com wrote:
 As suggested by you i enabled logging to debug clearcase error. I
 given the log information when i execute post-review for clearcase
 below.

 ###­
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Beginning parse of
 diff, size = 1070
 2010-08-10 03:39:32,064 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2010-08-10 03:39:32,065 - ERROR - Error uploading new diff: substring
 not found
 Traceback (most recent call last):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/webapi/json.py, line 1169, in new_diff
     request.FILES.get('parent_diff_path'))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py, line 234, in create
     history)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 63, in create
     diff_file, basedir, check_existance=(not parent_diff_file)))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py, line 149, in _process
 _files
     not tool.file_exists(filename, revision))):
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/core.py, line 68, in file_exists
     self.get_file(path, revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 45, in get_fil
 e
     return self.client.cat_file(self.adjust_path(path), revision)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/clearcase.py, line 83, in adjust_
 path
     elem_path = elem_path[elem_path.rindex(vobs/)+5:]
 ValueError: substring not found
 ###­

 On Aug 10, 11:15 am, Chetan chetan@gmail.com wrote:



  I wanted execute svn diff recursively but found could not generate
  one. But now I found if i run svn diff from the svn root directory it
  handles all changes recursively.

  On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:

   I'm not sure what you mean exactly. Is it not generating a diff with all
   files recursively, or is there some reason why doing it recursively isn't
   good enough?...

   read more »

   Are you using the post-review tool?

   Christian

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

   On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
I got reviewboard working for subversion including https.
In svn diff we cannot generate recursive diff coz of which we may have
to generate folder specific diffs. Is there a way around this?

Thanks for the support.

On Aug 5, 11:55 am, Christian Hammond chip...@chipx86.com wrote:
 As long as you have the $sitedir/data directory writable by the web
server's
 user, you should be able to get https working. If you already had the
 repository entry in there from RB 1.0.x, go back to that repository 
 and
save
 it again. It should attempt to verify the certificate and then ask you
for
 confirmation. If that data directory is writable, it will store it 
 there
and
 it should just work from then on

 read more »

 Christian

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

 On Wed, Aug 4, 2010 at 10:28 PM, Chetan chetan@gmail.com wrote:
  Thank a ton.
  Review board with svn is working for me. svn project is using just
  local protocol authentication like svn://prj and not on https. I 
  still
  get errors for https projects.

  What are the steps to configure reviewboard to use with https
  configured svn projects?
  Once i have completed svn configiration will start to work on
  clearcase issue.

  You asked me to enable logging to debug clearcase issue, wer shud i
  enable it. Is it the same one under reviewboard- settinges- 
  logging -
   enable logging option?

  On Aug 4, 9:33 am, Christian Hammond chip...@chipx86.com wrote:
   You'll need to use svn diff. TortoiseSVN's diffs aren't
compatible

   read more »

   I strongly recommend using our post-review tool (part of RBTools) 
   to
  create
   and update review requests, as it works around some problems in
standard
  SVN
   diffs as well.

   Christian

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

 

Re: post-review diff upload error on windows XP

2010-08-10 Thread Chetan

I wanted execute svn diff recursively but found could not generate
one. But now I found if i run svn diff from the svn root directory it
handles all changes recursively.




On Aug 9, 12:24 pm, Christian Hammond chip...@chipx86.com wrote:
 I'm not sure what you mean exactly. Is it not generating a diff with all
 files recursively, or is there some reason why doing it recursively isn't
 good enough?...

 read more »

 Are you using the post-review tool?

 Christian

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



 On Mon, Aug 9, 2010 at 12:21 AM, Chetan chetan@gmail.com wrote:
  I got reviewboard working for subversion including https.
  In svn diff we cannot generate recursive diff coz of which we may have
  to generate folder specific diffs. Is there a way around this?

  Thanks for the support.

  On Aug 5, 11:55 am, Christian Hammond chip...@chipx86.com wrote:
   As long as you have the $sitedir/data directory writable by the web
  server's
   user, you should be able to get https working. If you already had the
   repository entry in there from RB 1.0.x, go back to that repository and
  save
   it again. It should attempt to verify the certificate and then ask you
  for
   confirmation. If that data directory is writable, it will store it there
  and
   it should just work from then on

   read more »

   Christian

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

   On Wed, Aug 4, 2010 at 10:28 PM, Chetan chetan@gmail.com wrote:
Thank a ton.
Review board with svn is working for me. svn project is using just
local protocol authentication like svn://prj and not on https. I still
get errors for https projects.

What are the steps to configure reviewboard to use with https
configured svn projects?
Once i have completed svn configiration will start to work on
clearcase issue.

You asked me to enable logging to debug clearcase issue, wer shud i
enable it. Is it the same one under reviewboard- settinges- logging -
 enable logging option?

On Aug 4, 9:33 am, Christian Hammond chip...@chipx86.com wrote:
 You'll need to use svn diff. TortoiseSVN's diffs aren't
  compatible

 read more »

 I strongly recommend using our post-review tool (part of RBTools) to
create
 and update review requests, as it works around some problems in
  standard
SVN
 diffs as well.

 Christian

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

 On Tue, Aug 3, 2010 at 9:31 PM, Chetan chetan@gmail.com wrote:
  Is it a problem with the diff format? The date and time in diff
  output
  file is in Tue Jul 27 12:34:43 2010 format , I changed it to
  2010-07-27 12:34:43 2010 format and uploaded but has not helped
  in
  sovling the issue.

  On Aug 2, 4:18 pm, Chetan chetan@gmail.com wrote:
   I have used TortoiseSVN diff and selected create unified diff
option
   to create the diff in unified format.

   On Aug 2, 1:44 pm, Christian Hammond chip...@chipx86.com
  wrote:

How did you generate the diff? It looks like the diff contains
  a
date
  string
instead of revision information

read more »

Christian

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

On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com
wrote:
 I have changed the Traceback Switch to copy-and-paste view
  mode
and
 pasting the errors.

 Environment:

 Request Method: POST
 Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
 Django Version: 1.2.1
 Python Version: 2.5.2
 Installed Applications:
 ['django.contrib.admin',
  'django.contrib.auth',
  'django.contrib.contenttypes',
  'django.contrib.markup',
  'django.contrib.sites',
  'django.contrib.sessions',
  'djblets.datagrid',
  'djblets.feedview',
  'djblets.log',
  'djblets.siteconfig',
  'djblets.util',
  'djblets.webapi',
  'reviewboard.accounts',
  'reviewboard.admin',
  'reviewboard.changedescs',
  'reviewboard.diffviewer',
  'reviewboard.iphone',
  'reviewboard.notifications',
  'reviewboard.reports',
  'reviewboard.reviews',
  'reviewboard.scmtools',
  'reviewboard.webapi',
  'django_evolution']
 Installed Middleware:
 ('django.middleware.gzip.GZipMiddleware',
  'django.middleware.common.CommonMiddleware',
  'django.middleware.doc.XViewMiddleware',
  

Re: post-review diff upload error on windows XP

2010-08-04 Thread Chetan
Thank a ton.
Review board with svn is working for me. svn project is using just
local protocol authentication like svn://prj and not on https. I still
get errors for https projects.


What are the steps to configure reviewboard to use with https
configured svn projects?
Once i have completed svn configiration will start to work on
clearcase issue.

You asked me to enable logging to debug clearcase issue, wer shud i
enable it. Is it the same one under reviewboard- settinges- logging -
 enable logging option?




On Aug 4, 9:33 am, Christian Hammond chip...@chipx86.com wrote:
 You'll need to use svn diff. TortoiseSVN's diffs aren't compatible

 read more »

 I strongly recommend using our post-review tool (part of RBTools) to create
 and update review requests, as it works around some problems in standard SVN
 diffs as well.

 Christian

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



 On Tue, Aug 3, 2010 at 9:31 PM, Chetan chetan@gmail.com wrote:
  Is it a problem with the diff format? The date and time in diff output
  file is in Tue Jul 27 12:34:43 2010 format , I changed it to
  2010-07-27 12:34:43 2010 format and uploaded but has not helped in
  sovling the issue.

  On Aug 2, 4:18 pm, Chetan chetan@gmail.com wrote:
   I have used TortoiseSVN diff and selected create unified diff option
   to create the diff in unified format.

   On Aug 2, 1:44 pm, Christian Hammond chip...@chipx86.com wrote:

How did you generate the diff? It looks like the diff contains a date
  string
instead of revision information

read more »

Christian

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

On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com wrote:
 I have changed the Traceback Switch to copy-and-paste view mode and
 pasting the errors.

 Environment:

 Request Method: POST
 Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
 Django Version: 1.2.1
 Python Version: 2.5.2
 Installed Applications:
 ['django.contrib.admin',
  'django.contrib.auth',
  'django.contrib.contenttypes',
  'django.contrib.markup',
  'django.contrib.sites',
  'django.contrib.sessions',
  'djblets.datagrid',
  'djblets.feedview',
  'djblets.log',
  'djblets.siteconfig',
  'djblets.util',
  'djblets.webapi',
  'reviewboard.accounts',
  'reviewboard.admin',
  'reviewboard.changedescs',
  'reviewboard.diffviewer',
  'reviewboard.iphone',
  'reviewboard.notifications',
  'reviewboard.reports',
  'reviewboard.reviews',
  'reviewboard.scmtools',
  'reviewboard.webapi',
  'django_evolution']
 Installed Middleware:
 ('django.middleware.gzip.GZipMiddleware',
  'django.middleware.common.CommonMiddleware',
  'django.middleware.doc.XViewMiddleware',
  'django.middleware.http.ConditionalGetMiddleware',
  'django.middleware.locale.LocaleMiddleware',
  'django.contrib.sessions.middleware.SessionMiddleware',
  'django.contrib.auth.middleware.AuthenticationMiddleware',
  'djblets.siteconfig.middleware.SettingsMiddleware',
  'reviewboard.admin.middleware.LoadSettingsMiddleware',
  'djblets.log.middleware.LoggingMiddleware',
  'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware',
  'reviewboard.admin.middleware.X509AuthMiddleware')

 Traceback:
 File /usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/
 core/handlers/base.py in get_response
   100.                     response = callback(request,
 *callback_args, **callback_kwargs)
 File
  /usr/lib/python2.5/site-packages/Djblets-0.6.3-py2.5.egg/djblets/
 auth/util.py in _checklogin
   46.             return view_func(request, *args, **kwargs)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/views.py in new_review_request
   68.
 parent_diff_file=request.FILES.get('parent_diff_path'))
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py in create
   182.                              attach_to_history=True)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py in create
   234.                                                      history)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py in create
   63.             diff_file, basedir, check_existance=(not
 parent_diff_file)))
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py in _process_files
   137.             f2, revision =
  tool.parse_diff_revision(f.origFile,
 f.origInfo)
 File 

Re: post-review diff upload error on windows XP

2010-08-03 Thread Chetan
Is it a problem with the diff format? The date and time in diff output
file is in Tue Jul 27 12:34:43 2010 format , I changed it to
2010-07-27 12:34:43 2010 format and uploaded but has not helped in
sovling the issue.



On Aug 2, 4:18 pm, Chetan chetan@gmail.com wrote:
 I have used TortoiseSVN diff and selected create unified diff option
 to create the diff in unified format.

 On Aug 2, 1:44 pm, Christian Hammond chip...@chipx86.com wrote:



  How did you generate the diff? It looks like the diff contains a date string
  instead of revision information

  read more »

  Christian

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

  On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com wrote:
   I have changed the Traceback Switch to copy-and-paste view mode and
   pasting the errors.

   Environment:

   Request Method: POST
   Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
   Django Version: 1.2.1
   Python Version: 2.5.2
   Installed Applications:
   ['django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.markup',
    'django.contrib.sites',
    'django.contrib.sessions',
    'djblets.datagrid',
    'djblets.feedview',
    'djblets.log',
    'djblets.siteconfig',
    'djblets.util',
    'djblets.webapi',
    'reviewboard.accounts',
    'reviewboard.admin',
    'reviewboard.changedescs',
    'reviewboard.diffviewer',
    'reviewboard.iphone',
    'reviewboard.notifications',
    'reviewboard.reports',
    'reviewboard.reviews',
    'reviewboard.scmtools',
    'reviewboard.webapi',
    'django_evolution']
   Installed Middleware:
   ('django.middleware.gzip.GZipMiddleware',
    'django.middleware.common.CommonMiddleware',
    'django.middleware.doc.XViewMiddleware',
    'django.middleware.http.ConditionalGetMiddleware',
    'django.middleware.locale.LocaleMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
    'django.contrib.auth.middleware.AuthenticationMiddleware',
    'djblets.siteconfig.middleware.SettingsMiddleware',
    'reviewboard.admin.middleware.LoadSettingsMiddleware',
    'djblets.log.middleware.LoggingMiddleware',
    'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware',
    'reviewboard.admin.middleware.X509AuthMiddleware')

   Traceback:
   File /usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/
   core/handlers/base.py in get_response
     100.                     response = callback(request,
   *callback_args, **callback_kwargs)
   File /usr/lib/python2.5/site-packages/Djblets-0.6.3-py2.5.egg/djblets/
   auth/util.py in _checklogin
     46.             return view_func(request, *args, **kwargs)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/reviews/views.py in new_review_request
     68.
   parent_diff_file=request.FILES.get('parent_diff_path'))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/reviews/forms.py in create
     182.                              attach_to_history=True)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/reviews/forms.py in create
     234.                                                      history)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py in create
     63.             diff_file, basedir, check_existance=(not
   parent_diff_file)))
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/diffviewer/forms.py in _process_files
     137.             f2, revision = tool.parse_diff_revision(f.origFile,
   f.origInfo)
   File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
   reviewboard/scmtools/svn.py in parse_diff_revision
     189.                            revision_str)

   Exception Type: SCMError at /reviewboard_lgsi/r/new/
   Exception Value: Unable to parse diff revision header 'Mon Aug  2
   10:23:04 2010'

   On Aug 2, 12:59 pm, Chetan chetan@gmail.com wrote:
I am using 1.09. Now I upgraded it to 1.5 RC1 but i am getting error
when i create new review request. ERROR 500

There is an other thread on facing a similar problem after upgrade.
You had mentioned to enable DEBUG. I have enabled DEBUG and i get the
below error when i try creating the new request.

The error msg is huge and i have pasted just a part of it. Let me know
if i can upload the complete error msg.

   ###­­­
SCMError at /reviewboard_lgsi/r/new/
Unable to parse diff revision header 'Mon Aug  2 10:23:04 2010'Request
Method: POST
Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
Django Version: 1.2.1
Exception Type: SCMError
Exception Value: Unable to parse diff revision header 'Mon Aug  2
10:23:04 2010'

Re: post-review diff upload error on windows XP

2010-08-03 Thread Christian Hammond
You'll need to use svn diff. TortoiseSVN's diffs aren't compatible.

I strongly recommend using our post-review tool (part of RBTools) to create
and update review requests, as it works around some problems in standard SVN
diffs as well.

Christian

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


On Tue, Aug 3, 2010 at 9:31 PM, Chetan chetan@gmail.com wrote:

 Is it a problem with the diff format? The date and time in diff output
 file is in Tue Jul 27 12:34:43 2010 format , I changed it to
 2010-07-27 12:34:43 2010 format and uploaded but has not helped in
 sovling the issue.



 On Aug 2, 4:18 pm, Chetan chetan@gmail.com wrote:
  I have used TortoiseSVN diff and selected create unified diff option
  to create the diff in unified format.
 
  On Aug 2, 1:44 pm, Christian Hammond chip...@chipx86.com wrote:
 
 
 
   How did you generate the diff? It looks like the diff contains a date
 string
   instead of revision information
 
   read more »
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.reviewboard.org
   VMware, Inc. -http://www.vmware.com
 
   On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com wrote:
I have changed the Traceback Switch to copy-and-paste view mode and
pasting the errors.
 
Environment:
 
Request Method: POST
Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
Django Version: 1.2.1
Python Version: 2.5.2
Installed Applications:
['django.contrib.admin',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.markup',
 'django.contrib.sites',
 'django.contrib.sessions',
 'djblets.datagrid',
 'djblets.feedview',
 'djblets.log',
 'djblets.siteconfig',
 'djblets.util',
 'djblets.webapi',
 'reviewboard.accounts',
 'reviewboard.admin',
 'reviewboard.changedescs',
 'reviewboard.diffviewer',
 'reviewboard.iphone',
 'reviewboard.notifications',
 'reviewboard.reports',
 'reviewboard.reviews',
 'reviewboard.scmtools',
 'reviewboard.webapi',
 'django_evolution']
Installed Middleware:
('django.middleware.gzip.GZipMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.doc.XViewMiddleware',
 'django.middleware.http.ConditionalGetMiddleware',
 'django.middleware.locale.LocaleMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'djblets.siteconfig.middleware.SettingsMiddleware',
 'reviewboard.admin.middleware.LoadSettingsMiddleware',
 'djblets.log.middleware.LoggingMiddleware',
 'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware',
 'reviewboard.admin.middleware.X509AuthMiddleware')
 
Traceback:
File /usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/
core/handlers/base.py in get_response
  100. response = callback(request,
*callback_args, **callback_kwargs)
File
 /usr/lib/python2.5/site-packages/Djblets-0.6.3-py2.5.egg/djblets/
auth/util.py in _checklogin
  46. return view_func(request, *args, **kwargs)
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/reviews/views.py in new_review_request
  68.
parent_diff_file=request.FILES.get('parent_diff_path'))
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/reviews/forms.py in create
  182.  attach_to_history=True)
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/reviews/forms.py in create
  234.  history)
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/diffviewer/forms.py in create
  63. diff_file, basedir, check_existance=(not
parent_diff_file)))
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/diffviewer/forms.py in _process_files
  137. f2, revision =
 tool.parse_diff_revision(f.origFile,
f.origInfo)
File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
reviewboard/scmtools/svn.py in parse_diff_revision
  189.revision_str)
 
Exception Type: SCMError at /reviewboard_lgsi/r/new/
Exception Value: Unable to parse diff revision header 'Mon Aug  2
10:23:04 2010'
 
On Aug 2, 12:59 pm, Chetan chetan@gmail.com wrote:
 I am using 1.09. Now I upgraded it to 1.5 RC1 but i am getting
 error
 when i create new review request. ERROR 500
 
 There is an other thread on facing a similar problem after upgrade.
 You had mentioned to enable DEBUG. I have enabled DEBUG and i get
 the
 below error when i try creating the new request.
 
 The 

Re: post-review diff upload error on windows XP

2010-08-02 Thread Chetan
I am using 1.09. Now I upgraded it to 1.5 RC1 but i am getting error
when i create new review request. ERROR 500

There is an other thread on facing a similar problem after upgrade.
You had mentioned to enable DEBUG. I have enabled DEBUG and i get the
below error when i try creating the new request.

The error msg is huge and i have pasted just a part of it. Let me know
if i can upload the complete error msg.


###
SCMError at /reviewboard_lgsi/r/new/
Unable to parse diff revision header 'Mon Aug  2 10:23:04 2010'Request
Method: POST
Request URL: http://172.24.1.107/reviewboard_lgsi/r/new/
Django Version: 1.2.1
Exception Type: SCMError
Exception Value: Unable to parse diff revision header 'Mon Aug  2
10:23:04 2010'
Exception Location: /usr/lib/python2.5/site-packages/
ReviewBoard-1.5rc1-py2.5.egg/reviewboard/scmtools/svn.py in
parse_diff_revision, line 189
Python Executable: /usr/bin/python
Python Version: 2.5.2
Python Path: ['/var/www/reviewboard_lgsi/conf', '/usr/lib/python2.5/
site-packages/MySQL_python-1.2.3c1-py2.5-linux-i686.egg', '/usr/lib/
python2.5/site-packages/python_memcached-1.45-py2.5.egg', '/usr/lib/
python2.5/site-packages/pytz-2010h-py2.5.egg', '/usr/lib/python2.5/
site-packages/flup-1.0.3.dev_20100525-py2.5.egg', '/usr/lib/python2.5/
site-packages/Pygments-1.3.1-py2.5.egg', '/usr/lib/python2.5/site-
packages/Django-1.2.1-py2.5.egg', '/usr/lib/python2.5/site-packages/
RBTools-0.2-py2.5.egg', '/usr/lib/python2.5/site-packages/
simplejson-2.1.1-py2.5.egg', '/usr/lib/python2.5/site-packages/
setuptools-0.6c11-py2.5.egg', '/usr/lib/python2.5/site-packages/
Djblets-0.6.3-py2.5.egg', '/usr/lib/python2.5/site-packages/
django_evolution-0.5-py2.5.egg', '/usr/lib/python2.5/site-packages/
ReviewBoard-1.5rc1-py2.5.egg', '/usr/lib/python2.5/site-packages/
recaptcha_client-1.0.5-py2.5.egg', '/usr/lib/python2.5/site-packages/
python_dateutil-1.5-py2.5.egg', '/usr/lib/python2.5/site-packages/
paramiko-1.7.6-py2.5.egg', '/usr/lib/python2.5/site-packages/
pycrypto-2.1.0-py2.5-linux-i686.egg', '/usr/lib/python25.zip', '/usr/
lib/python2.5', '/usr/lib/python2.5/plat-linux2', '/usr/lib/python2.5/
lib-tk', '/usr/lib/python2.5/lib-dynload', '/usr/local/lib/python2.5/
site-packages', '/usr/lib/python2.5/site-packages', '/usr/lib/
python2.5/site-packages/PIL', '/var/lib/python-support/python2.5', '/
var/lib/python-support/python2.5/gtk-2.0', '/usr/lib/python2.5/site-
packages/wx-2.6-gtk2-unicode']
Server time: Sun, 1 Aug 2010 22:35:28 -0700

Traceback Switch to copy-and-paste view
/usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/core/
handlers/base.py in get_response
# Apply view middlewarefor
middleware_method in self._view_middleware:
response = middleware_method(request, callback, callback_args,
callback_kwargs)if
response:return response
try:response = callback(request, *callback_args,
**callback_kwargs) ...except Exception,
e:# If the view raised an exception, run it
through exception# middleware, and if the
exception middleware returns a# response, use
that. Otherwise, reraise the exception.for
middleware_method in
self._exception_middleware:response =
middleware_method(request, e)▶ Local vars
Variable Value
callback function new_review_request at 0x8d0780c
callback_args ()
callback_kwargs {}
e SCMError(Unable to parse diff revision header 'Mon Aug 2 10:23:04
2010',)
exc_info (class 'reviewboard.scmtools.errors.SCMError',
SCMError(Unable to parse diff revision header 'Mon Aug 2 10:23:04
2010',), traceback object at 0x9207e3c)
exceptions module 'django.core.exceptions' from '/usr/lib/python2.5/
site-packages/Django-1.2.1-py2.5.egg/django/core/exceptions.pyc'
middleware_method bound method LoggingMiddleware.process_exception of
djblets.log.middleware.LoggingMiddleware object at 0x8e5392c
receivers [(function _rollback_on_exception at 0x8712374, None)]
request ModPythonRequest path:/reviewboard_lgsi/r/new/,
GET:QueryDict: {}, POST:QueryDict: {u'changenum': [u''],
u'basedir': [u'F:\\Review_svn\\'], u'repository': [u'3']}, COOKIES:
{'rbsessionid': 'ae9f14ec4e59c8e37131b69f0589b8b5'}, META:
{'AUTH_TYPE': None, 'CONTENT_LENGTH': '723', 'CONTENT_TYPE':
'multipart/form-data;
boundary=---7da246141103d2',
'GATEWAY_INTERFACE': 'CGI/1.1', 'HTTP_ACCEPT': 'application/x-
shockwave-flash, image/gif, image/x-xbitmap, image/jpeg, image/pjpeg,
application/vnd.ms-excel, application/vnd.ms-powerpoint, application/
msword, application/x-ms-application, application/x-ms-xbap,
application/vnd.ms-xpsdocument, application/xaml+xml, */*',
'HTTP_ACCEPT_ENCODING': 'gzip, deflate', 'HTTP_ACCEPT_LANGUAGE': 'en-
us', 'HTTP_CACHE_CONTROL': 'max-age=259200', 'HTTP_CONNECTION': 'keep-
alive', 

Re: post-review diff upload error on windows XP

2010-08-02 Thread Christian Hammond
How did you generate the diff? It looks like the diff contains a date string
instead of revision information.

Christian

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


On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com wrote:

 I have changed the Traceback Switch to copy-and-paste view mode and
 pasting the errors.

 Environment:

 Request Method: POST
 Request URL: http://172.24.1.107/reviewboard_lgsi/r/new/
 Django Version: 1.2.1
 Python Version: 2.5.2
 Installed Applications:
 ['django.contrib.admin',
  'django.contrib.auth',
  'django.contrib.contenttypes',
  'django.contrib.markup',
  'django.contrib.sites',
  'django.contrib.sessions',
  'djblets.datagrid',
  'djblets.feedview',
  'djblets.log',
  'djblets.siteconfig',
  'djblets.util',
  'djblets.webapi',
  'reviewboard.accounts',
  'reviewboard.admin',
  'reviewboard.changedescs',
  'reviewboard.diffviewer',
  'reviewboard.iphone',
  'reviewboard.notifications',
  'reviewboard.reports',
  'reviewboard.reviews',
  'reviewboard.scmtools',
  'reviewboard.webapi',
  'django_evolution']
 Installed Middleware:
 ('django.middleware.gzip.GZipMiddleware',
  'django.middleware.common.CommonMiddleware',
  'django.middleware.doc.XViewMiddleware',
  'django.middleware.http.ConditionalGetMiddleware',
  'django.middleware.locale.LocaleMiddleware',
  'django.contrib.sessions.middleware.SessionMiddleware',
  'django.contrib.auth.middleware.AuthenticationMiddleware',
  'djblets.siteconfig.middleware.SettingsMiddleware',
  'reviewboard.admin.middleware.LoadSettingsMiddleware',
  'djblets.log.middleware.LoggingMiddleware',
  'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware',
  'reviewboard.admin.middleware.X509AuthMiddleware')


 Traceback:
 File /usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/
 core/handlers/base.py in get_response
   100. response = callback(request,
 *callback_args, **callback_kwargs)
 File /usr/lib/python2.5/site-packages/Djblets-0.6.3-py2.5.egg/djblets/
 auth/util.py in _checklogin
   46. return view_func(request, *args, **kwargs)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/views.py in new_review_request
   68.
 parent_diff_file=request.FILES.get('parent_diff_path'))
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py in create
   182.  attach_to_history=True)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/reviews/forms.py in create
   234.  history)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py in create
   63. diff_file, basedir, check_existance=(not
 parent_diff_file)))
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/diffviewer/forms.py in _process_files
   137. f2, revision = tool.parse_diff_revision(f.origFile,
 f.origInfo)
 File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
 reviewboard/scmtools/svn.py in parse_diff_revision
   189.revision_str)

 Exception Type: SCMError at /reviewboard_lgsi/r/new/
 Exception Value: Unable to parse diff revision header 'Mon Aug  2
 10:23:04 2010'







 On Aug 2, 12:59 pm, Chetan chetan@gmail.com wrote:
  I am using 1.09. Now I upgraded it to 1.5 RC1 but i am getting error
  when i create new review request. ERROR 500
 
  There is an other thread on facing a similar problem after upgrade.
  You had mentioned to enable DEBUG. I have enabled DEBUG and i get the
  below error when i try creating the new request.
 
  The error msg is huge and i have pasted just a part of it. Let me know
  if i can upload the complete error msg.
 
 
 ###­
  SCMError at /reviewboard_lgsi/r/new/
  Unable to parse diff revision header 'Mon Aug  2 10:23:04 2010'Request
  Method: POST
  Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
  Django Version: 1.2.1
  Exception Type: SCMError
  Exception Value: Unable to parse diff revision header 'Mon Aug  2
  10:23:04 2010'
  Exception Location: /usr/lib/python2.5/site-packages/
  ReviewBoard-1.5rc1-py2.5.egg/reviewboard/scmtools/svn.py in
  parse_diff_revision, line 189
  Python Executable: /usr/bin/python
  Python Version: 2.5.2
  Python Path: ['/var/www/reviewboard_lgsi/conf', '/usr/lib/python2.5/
  site-packages/MySQL_python-1.2.3c1-py2.5-linux-i686.egg', '/usr/lib/
  python2.5/site-packages/python_memcached-1.45-py2.5.egg', '/usr/lib/
  python2.5/site-packages/pytz-2010h-py2.5.egg', '/usr/lib/python2.5/
  site-packages/flup-1.0.3.dev_20100525-py2.5.egg', '/usr/lib/python2.5/
  site-packages/Pygments-1.3.1-py2.5.egg', '/usr/lib/python2.5/site-
  

Re: post-review diff upload error on windows XP

2010-08-02 Thread Chetan
I have used TortoiseSVN diff and selected create unified diff option
to create the diff in unified format.



On Aug 2, 1:44 pm, Christian Hammond chip...@chipx86.com wrote:
 How did you generate the diff? It looks like the diff contains a date string
 instead of revision information

 read more »

 Christian

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



 On Mon, Aug 2, 2010 at 1:11 AM, Chetan chetan@gmail.com wrote:
  I have changed the Traceback Switch to copy-and-paste view mode and
  pasting the errors.

  Environment:

  Request Method: POST
  Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
  Django Version: 1.2.1
  Python Version: 2.5.2
  Installed Applications:
  ['django.contrib.admin',
   'django.contrib.auth',
   'django.contrib.contenttypes',
   'django.contrib.markup',
   'django.contrib.sites',
   'django.contrib.sessions',
   'djblets.datagrid',
   'djblets.feedview',
   'djblets.log',
   'djblets.siteconfig',
   'djblets.util',
   'djblets.webapi',
   'reviewboard.accounts',
   'reviewboard.admin',
   'reviewboard.changedescs',
   'reviewboard.diffviewer',
   'reviewboard.iphone',
   'reviewboard.notifications',
   'reviewboard.reports',
   'reviewboard.reviews',
   'reviewboard.scmtools',
   'reviewboard.webapi',
   'django_evolution']
  Installed Middleware:
  ('django.middleware.gzip.GZipMiddleware',
   'django.middleware.common.CommonMiddleware',
   'django.middleware.doc.XViewMiddleware',
   'django.middleware.http.ConditionalGetMiddleware',
   'django.middleware.locale.LocaleMiddleware',
   'django.contrib.sessions.middleware.SessionMiddleware',
   'django.contrib.auth.middleware.AuthenticationMiddleware',
   'djblets.siteconfig.middleware.SettingsMiddleware',
   'reviewboard.admin.middleware.LoadSettingsMiddleware',
   'djblets.log.middleware.LoggingMiddleware',
   'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware',
   'reviewboard.admin.middleware.X509AuthMiddleware')

  Traceback:
  File /usr/lib/python2.5/site-packages/Django-1.2.1-py2.5.egg/django/
  core/handlers/base.py in get_response
    100.                     response = callback(request,
  *callback_args, **callback_kwargs)
  File /usr/lib/python2.5/site-packages/Djblets-0.6.3-py2.5.egg/djblets/
  auth/util.py in _checklogin
    46.             return view_func(request, *args, **kwargs)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/reviews/views.py in new_review_request
    68.
  parent_diff_file=request.FILES.get('parent_diff_path'))
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/reviews/forms.py in create
    182.                              attach_to_history=True)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/reviews/forms.py in create
    234.                                                      history)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/diffviewer/forms.py in create
    63.             diff_file, basedir, check_existance=(not
  parent_diff_file)))
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/diffviewer/forms.py in _process_files
    137.             f2, revision = tool.parse_diff_revision(f.origFile,
  f.origInfo)
  File /usr/lib/python2.5/site-packages/ReviewBoard-1.5rc1-py2.5.egg/
  reviewboard/scmtools/svn.py in parse_diff_revision
    189.                            revision_str)

  Exception Type: SCMError at /reviewboard_lgsi/r/new/
  Exception Value: Unable to parse diff revision header 'Mon Aug  2
  10:23:04 2010'

  On Aug 2, 12:59 pm, Chetan chetan@gmail.com wrote:
   I am using 1.09. Now I upgraded it to 1.5 RC1 but i am getting error
   when i create new review request. ERROR 500

   There is an other thread on facing a similar problem after upgrade.
   You had mentioned to enable DEBUG. I have enabled DEBUG and i get the
   below error when i try creating the new request.

   The error msg is huge and i have pasted just a part of it. Let me know
   if i can upload the complete error msg.

  ###­­
   SCMError at /reviewboard_lgsi/r/new/
   Unable to parse diff revision header 'Mon Aug  2 10:23:04 2010'Request
   Method: POST
   Request URL:http://172.24.1.107/reviewboard_lgsi/r/new/
   Django Version: 1.2.1
   Exception Type: SCMError
   Exception Value: Unable to parse diff revision header 'Mon Aug  2
   10:23:04 2010'
   Exception Location: /usr/lib/python2.5/site-packages/
   ReviewBoard-1.5rc1-py2.5.egg/reviewboard/scmtools/svn.py in
   parse_diff_revision, line 189
   Python Executable: /usr/bin/python
   Python Version: 2.5.2
   Python Path: ['/var/www/reviewboard_lgsi/conf', '/usr/lib/python2.5/
   site-packages/MySQL_python-1.2.3c1-py2.5-linux-i686.egg', '/usr/lib/
   

Re: post-review diff upload error on windows XP

2010-08-01 Thread Christian Hammond
Which version are you running?

HTTPS certificates for Subversion are supported in Review Board 1.5 RC1.

As for the substring error, we'd need to do some debugging to see what the
problem actually is. In 1.5 RC1, if you can enable logging, it may provide a
stack trace that shows what exactly failed.

Christian

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


On Thu, Jul 29, 2010 at 12:35 AM, Chetan chetan@gmail.com wrote:

 I have provided the permission for www-data to upload directory. But
 still it doesnt work.
 For me the error doesnt say about permissions. It is as below.

 Uploading diff, size: 1017
 HTTP POSTing to
 http://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...
 {'basedir': 'M:/'}
 Got API Error 105 (HTTP code 200): One or more fields had errors
 Error data: {u'fields': {u'path': [u'substring not found']}, u'stat':
 u'fail', u'err': {u'msg': u'One or more fields had errors', u'code':
 105}}
 Your review request still exists, but the diff is not attached.
 Error uploading diff

 Reviewboard is not working with clearcase nor with subversion at my
 setup. Clearcase prompts me the substring error and for svn i am stuck
 with the hhtps certificate error.

 I have configured RB on ubuntu.


 
 
 
  On Tue, Jul 27, 2010 at 8:01 AM, Chetan chetan@gmail.com wrote:
   I have tried both 1 and 2. I am stuck in the second step. I have
   checkedout the files made changes and then executed post-review...here
   a new request gets created but fails to upload the diff. When i output
   the diff to the console i can see the changes that are done. My
   current problem is with attaching the diff to the review request. I
   have tried manual upload as well as post-review. And both seems to
   give the same error.
 
   On Jul 26, 10:50 am, Jan Koprowski jan.koprow...@gmail.com wrote:
   I mean we implement --revision-range for ClearCase and we use this
   with full success. You probably use ReviewBoard in pre-review. We also
   try to use it and we have problems too. But in our company all
   checkins is saved in ClearQuest so we get them from ClearQuest using
   CQPERL script and send to --revision-range of post-review. This works.
 
   Before You look to review.reviewboard.org try do things:
   1) Checkout file make change checking and checkout again - then try
   run post-review this-file
   2) Now make change (without checkin) and run post-review this-file
   one more time
   I mean - try very simple scenarios. I remember I try hard and this
   work at least.
 
   On Mon, Jul 26, 2010 at 6:16 AM, Chetan chetan@gmail.com wrote:
 
Do you mean i have to use --label option in postreview for
 generating
diff for clearcase?
 
I have tried adding diff file manual in GUI by using a sample file
 and
also using the output from postreview as an diff file. But there is
error stating Substring not found.
 
Have i misssed anything in the basic setup?
 
On Jul 20, 3:05 pm, Dan Savilonis d...@n-cube.org wrote:
The documentation and clearcase support in rbtools is rather
 outdated.
I believe Jan has an improvement so you don't need to use my
 cleartool
lsco hack among other things.
 
There are a large number of patches in the queue on
reviews.reviewboard.org but they don't appear to be going
 anywhere.
Nonetheless it would be better to start with these changes as a
baseline rather than debugging the existing broken code. Jan will
 need
to comment if he's interested in doing this. I can't do it easily
myself since RB doesn't upload full git patches right now...
 
Dan
 
On Jul 19, 8:06 am, Chetan chetan@gmail.com wrote:
 
 Hello..
 
 Do anyone have a hint on this issue? i have searched the forum
 but
 could not find any solution.
 
 On Jul 15, 11:44 am, Chetan chetan@gmail.com wrote:
 
  Hello All,
  I have facing problem in uploading the clearcase diff in
 review
  request using post-review script.
  command used: cleartool lsco -cview -me -recurse -short | sed
 s//
  \//g | xargs post-review -d --diff-only
 --target-people=chetan.d --
  summary=test2_by_chetan --description test2_by_chetan --submit-
  as=admin --server=http://.xxx.net/xxx/;
 
  When i use -n option i get the diff on the prompt.
 
  Error are as below:
 
   HTTP POSTing tohttp://
 172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...{'descriptionhttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...%7B%27description':
 'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: dynamic\ntest2_by_chetan'}
   Uploading diff, size: 1017
   HTTP POSTing tohttp://
 172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...{'basedirhttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...%7B%27basedir':
 'M:/'}
   Got API Error 105 (HTTP code 200): One or more 

Re: post-review diff upload error on windows XP

2010-07-27 Thread Jan Koprowski
So if upload is problem maybe this is permission error.
Look here: 
http://www.reviewboard.org/docs/manual/dev/admin/sites/creating-sites/#changing-permissions.
Meybe this is problem Apache server doesn't have write access to
upload directory.

On Tue, Jul 27, 2010 at 8:01 AM, Chetan chetan@gmail.com wrote:
 I have tried both 1 and 2. I am stuck in the second step. I have
 checkedout the files made changes and then executed post-review...here
 a new request gets created but fails to upload the diff. When i output
 the diff to the console i can see the changes that are done. My
 current problem is with attaching the diff to the review request. I
 have tried manual upload as well as post-review. And both seems to
 give the same error.




 On Jul 26, 10:50 am, Jan Koprowski jan.koprow...@gmail.com wrote:
 I mean we implement --revision-range for ClearCase and we use this
 with full success. You probably use ReviewBoard in pre-review. We also
 try to use it and we have problems too. But in our company all
 checkins is saved in ClearQuest so we get them from ClearQuest using
 CQPERL script and send to --revision-range of post-review. This works.

 Before You look to review.reviewboard.org try do things:
 1) Checkout file make change checking and checkout again - then try
 run post-review this-file
 2) Now make change (without checkin) and run post-review this-file
 one more time
 I mean - try very simple scenarios. I remember I try hard and this
 work at least.





 On Mon, Jul 26, 2010 at 6:16 AM, Chetan chetan@gmail.com wrote:

  Do you mean i have to use --label option in postreview for generating
  diff for clearcase?

  I have tried adding diff file manual in GUI by using a sample file and
  also using the output from postreview as an diff file. But there is
  error stating Substring not found.

  Have i misssed anything in the basic setup?

  On Jul 20, 3:05 pm, Dan Savilonis d...@n-cube.org wrote:
  The documentation and clearcase support in rbtools is rather outdated.
  I believe Jan has an improvement so you don't need to use my cleartool
  lsco hack among other things.

  There are a large number of patches in the queue on
  reviews.reviewboard.org but they don't appear to be going anywhere.
  Nonetheless it would be better to start with these changes as a
  baseline rather than debugging the existing broken code. Jan will need
  to comment if he's interested in doing this. I can't do it easily
  myself since RB doesn't upload full git patches right now...

  Dan

  On Jul 19, 8:06 am, Chetan chetan@gmail.com wrote:

   Hello..

   Do anyone have a hint on this issue? i have searched the forum but
   could not find any solution.

   On Jul 15, 11:44 am, Chetan chetan@gmail.com wrote:

Hello All,
    I have facing problem in uploading the clearcase diff in review
request using post-review script.
command used: cleartool lsco -cview -me -recurse -short | sed 
s//
\//g | xargs post-review -d --diff-only --target-people=chetan.d --
summary=test2_by_chetan --description test2_by_chetan --submit-
as=admin --server=http://.xxx.net/xxx/;

When i use -n option i get the diff on the prompt.

Error are as below:

 HTTP POSTing 
 tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...{'description':
  'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: dynamic\ntest2_by_chetan'}
 Uploading diff, size: 1017
 HTTP POSTing 
 tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...{'basedir':
  'M:/'}
 Got API Error 105 (HTTP code 200): One or more fields had errors
 Error data: {u'fields': {u'path': [u'substring not found']}, 
 u'stat': u'fail', u'err': {u'msg': u'One or more fields had 
 errors', u'code': 105}}

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

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.

Request some help in getting this error fixed.

Thnks- Hide quoted text -

  - Show quoted text -

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

 --



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

Re: post-review diff upload error on windows XP

2010-07-25 Thread Jan Koprowski
I mean we implement --revision-range for ClearCase and we use this
with full success. You probably use ReviewBoard in pre-review. We also
try to use it and we have problems too. But in our company all
checkins is saved in ClearQuest so we get them from ClearQuest using
CQPERL script and send to --revision-range of post-review. This works.

Before You look to review.reviewboard.org try do things:
1) Checkout file make change checking and checkout again - then try
run post-review this-file
2) Now make change (without checkin) and run post-review this-file
one more time
I mean - try very simple scenarios. I remember I try hard and this
work at least.

On Mon, Jul 26, 2010 at 6:16 AM, Chetan chetan@gmail.com wrote:

 Do you mean i have to use --label option in postreview for generating
 diff for clearcase?

 I have tried adding diff file manual in GUI by using a sample file and
 also using the output from postreview as an diff file. But there is
 error stating Substring not found.

 Have i misssed anything in the basic setup?


 On Jul 20, 3:05 pm, Dan Savilonis d...@n-cube.org wrote:
 The documentation and clearcase support in rbtools is rather outdated.
 I believe Jan has an improvement so you don't need to use my cleartool
 lsco hack among other things.

 There are a large number of patches in the queue on
 reviews.reviewboard.org but they don't appear to be going anywhere.
 Nonetheless it would be better to start with these changes as a
 baseline rather than debugging the existing broken code. Jan will need
 to comment if he's interested in doing this. I can't do it easily
 myself since RB doesn't upload full git patches right now...

 Dan

 On Jul 19, 8:06 am, Chetan chetan@gmail.com wrote:



  Hello..

  Do anyone have a hint on this issue? i have searched the forum but
  could not find any solution.

  On Jul 15, 11:44 am, Chetan chetan@gmail.com wrote:

   Hello All,
       I have facing problem in uploading the clearcase diff in review
   request using post-review script.
   command used: cleartool lsco -cview -me -recurse -short | sed s//
   \//g | xargs post-review -d --diff-only --target-people=chetan.d --
   summary=test2_by_chetan --description test2_by_chetan --submit-
   as=admin --server=http://.xxx.net/xxx/;

   When i use -n option i get the diff on the prompt.

   Error are as below:

HTTP POSTing 
tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...{'description':
 'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: dynamic\ntest2_by_chetan'}
Uploading diff, size: 1017
HTTP POSTing 
tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...{'basedir':
 'M:/'}
Got API Error 105 (HTTP code 200): One or more fields had errors
Error data: {u'fields': {u'path': [u'substring not found']}, 
u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors', 
u'code': 105}}

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

   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.

   Request some help in getting this error fixed.

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



-- 
 Jan Koprowski

-- 
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 diff upload error on windows XP

2010-07-20 Thread Dan Savilonis
The documentation and clearcase support in rbtools is rather outdated.
I believe Jan has an improvement so you don't need to use my cleartool
lsco hack among other things.

There are a large number of patches in the queue on
reviews.reviewboard.org but they don't appear to be going anywhere.
Nonetheless it would be better to start with these changes as a
baseline rather than debugging the existing broken code. Jan will need
to comment if he's interested in doing this. I can't do it easily
myself since RB doesn't upload full git patches right now...

Dan

On Jul 19, 8:06 am, Chetan chetan@gmail.com wrote:
 Hello..

 Do anyone have a hint on this issue? i have searched the forum but
 could not find any solution.

 On Jul 15, 11:44 am, Chetan chetan@gmail.com wrote:



  Hello All,
      I have facing problem in uploading the clearcase diff in review
  request using post-review script.
  command used: cleartool lsco -cview -me -recurse -short | sed s//
  \//g | xargs post-review -d --diff-only --target-people=chetan.d --
  summary=test2_by_chetan --description test2_by_chetan --submit-
  as=admin --server=http://.xxx.net/xxx/;

  When i use -n option i get the diff on the prompt.

  Error are as below:

   HTTP POSTing 
   tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...{'description':
'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: dynamic\ntest2_by_chetan'}
   Uploading diff, size: 1017
   HTTP POSTing 
   tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...{'basedir':
'M:/'}
   Got API Error 105 (HTTP code 200): One or more fields had errors
   Error data: {u'fields': {u'path': [u'substring not found']}, u'stat': 
   u'fail', u'err': {u'msg': u'One or more fields had errors', u'code': 
   105}}

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

  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.

  Request some help in getting this error fixed.

  Thnks

-- 
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 diff upload error on windows XP

2010-07-20 Thread Jan Koprowski
Is not my intention to rewrite Clear Case implementation. We use Clear
Case in company and my manager ask me I can do something to switch
from Code Striker to Review Board - so I do it. Now in company I have
other tasks. From time to time I write some useful (i believe useful)
patches but in my company we use very untypical, post review workflow
for Clear Case (actually Clear Quest) so my patches may looks
strange because most people think Review Board used in pre review :)

On Tue, Jul 20, 2010 at 12:05 PM, Dan Savilonis d...@n-cube.org wrote:
 The documentation and clearcase support in rbtools is rather outdated.
 I believe Jan has an improvement so you don't need to use my cleartool
 lsco hack among other things.

 There are a large number of patches in the queue on
 reviews.reviewboard.org but they don't appear to be going anywhere.
 Nonetheless it would be better to start with these changes as a
 baseline rather than debugging the existing broken code. Jan will need
 to comment if he's interested in doing this. I can't do it easily
 myself since RB doesn't upload full git patches right now...

 Dan

 On Jul 19, 8:06 am, Chetan chetan@gmail.com wrote:
 Hello..

 Do anyone have a hint on this issue? i have searched the forum but
 could not find any solution.

 On Jul 15, 11:44 am, Chetan chetan@gmail.com wrote:



  Hello All,
      I have facing problem in uploading the clearcase diff in review
  request using post-review script.
  command used: cleartool lsco -cview -me -recurse -short | sed s//
  \//g | xargs post-review -d --diff-only --target-people=chetan.d --
  summary=test2_by_chetan --description test2_by_chetan --submit-
  as=admin --server=http://.xxx.net/xxx/;

  When i use -n option i get the diff on the prompt.

  Error are as below:

   HTTP POSTing 
   tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft...{'description':
'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: dynamic\ntest2_by_chetan'}
   Uploading diff, size: 1017
   HTTP POSTing 
   tohttp://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/...{'basedir':
'M:/'}
   Got API Error 105 (HTTP code 200): One or more fields had errors
   Error data: {u'fields': {u'path': [u'substring not found']}, u'stat': 
   u'fail', u'err': {u'msg': u'One or more fields had errors', u'code': 
   105}}

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

  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.

  Request some help in getting this error fixed.

  Thnks

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



-- 
 Jan Koprowski

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


post-review diff upload error on windows XP

2010-07-15 Thread Chetan
Hello All,
I have facing problem in uploading the clearcase diff in review
request using post-review script.
command used: cleartool lsco -cview -me -recurse -short | sed s//
\//g | xargs post-review -d --diff-only --target-people=chetan.d --
summary=test2_by_chetan --description test2_by_chetan --submit-
as=admin --server=http://.xxx.net/xxx/;

When i use -n option i get the diff on the prompt.

Error are as below:

 HTTP POSTing to 
 http://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/draft/set/: 
 {'description': 'VIEW: chetan.d_sdaf_2_int\nVIEWTYPE: 
 dynamic\ntest2_by_chetan'}
 Uploading diff, size: 1017
 HTTP POSTing to 
 http://172.24.1.107/reviewboard_lgsi/api/json/reviewrequests/16/diff/new/: 
 {'basedir': 'M:/'}
 Got API Error 105 (HTTP code 200): One or more fields had errors
 Error data: {u'fields': {u'path': [u'substring not found']}, u'stat': 
 u'fail', u'err': {u'msg': u'One or more fields had errors', u'code': 105}}
Your review request still exists, but the diff is not attached.

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.


Request some help in getting this error fixed.


Thnks

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