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