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 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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://xxxx.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 > > [email protected] > > 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 [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
