i've got a bit further with this...it looks like the
tool.file_exists(filename, revision) on the server side fails
It tries to parse a file name form the clearcase extended name i.e.
the one with the versions and @@ char sequences
included in the path. e.g Y:/wran_cm_cms/@@/main/22/src/@@/main/3/com/
@@/main/1/......etc

There is a comment on the post-review.py
class ClearCaseClient(SCMClient):
    """
    A wrapper around the clearcase tool that fetches repository
    information and generates compatible diffs.
    This client assumes that cygwin is installed on windows.
    """

I'm not sure then if the post-review worked as expected .
What exactly is  cygwin's role in this process.
I do'nt have it installed ...do I need it.

I think I'm almost there ..!


Thanks
T






On Feb 12, 12:19 pm, eeiths <eei...@gmail.com> wrote:
> I'm trying the post-review tool on windows env but i get the following
> error ...wondering is this a clearcase issue or reviewboard.
>
> is the a command to create the diff in isolation ..so I can see or
> upload it via the GUI.
> How can i debug the server side ..?
>
> Any help appreciated .
>
> C:\python25\Scripts>cleartool lsco  -me -cview -recurse -short  X:\ |
> sed s/\\/\//g | xargs post-review  -d
>  --server=http://ev001f29a331be/>>> cleartool pwv -short
> >>> repository info: Path: X:/, Base path: X:/, Supports changesets: False
> >>> cleartool desc -pre X:\wran_cm_cms\PlannedAreaUpdator.java
> >>> cleartool desc -fmt %Vn X:\
> >>> cleartool desc -fmt %Vn X:\wran_cm_cms
>
> :
> :
>
> >>> cleartool get -to 
> >>> c:\users\etomhar\appdata\local\temp\ae893516429051478f79222898e00140 
> >>> X:\wran_cm_cms\PlannedAreaUpdator.java@@\main\at_ossrc_dev\at_ossrc_r7_inc70.26\4
> >>> Looking for 'ev001f29a331be /reviews/' cookie in C:\Users\etomhar\Local 
> >>> Settings\Application Data\.post-review-cookies.txt
> >>> Loaded valid cookie -- no login required
> >>> Attempting to create review request for None
> >>> HTTP POSTing 
> >>> tohttp://ev001f29a331be/reviews/api/json/reviewrequests/new/:{'repository_path':
> >>>  'X:/'}
> >>> Review request created
> >>> Attempting to set field 'description' to '** NONE **
>
> VIEWTYPE: dynamic
> ' for review request '6'>>> HTTP POSTing 
> tohttp://ev001f29a331be/reviews/api/json/reviewrequests/6/draft/set/:{'description':
>  '** NONE **\nVIEWTYPE: dynamic\n'}
> >>> HTTP POSTing 
> >>> tohttp://ev001f29a331be/reviews/api/json/reviewrequests/6/draft/save/:{}
> >>> Review request draft saved
> >>> Uploading diff, size: 29
> >>> HTTP POSTing 
> >>> tohttp://ev001f29a331be/reviews/api/json/reviewrequests/6/diff/new/:{'basedir':
> >>>  'X:/'}
>
> Error uploading diff: One or more fields had errors (105)>>> {'fields': 
> {'path': ['The diff file is empty']}, 'stat': 'fail', 'err': {'msg': 'One or 
> more fields had errors', 'code': 105}}
>
> Your review request still exists, but the diff is not attached.

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

Reply via email to