problem with post-review and clearcase
Hey,
I'm trying to run ReviewBoard with clearcase, and I'm having trouble
uploading a diff to the website. I'm running reviewboard 1.7.14 and RBtools
0.5.2. Here is the message I'm getting when running post-review in debug
mode (pathnames and addresses replaced by Zs and hostname):
---
DEBUG:root:Running: cleartool describe -fmt %On
/view/my_view/Zs/Zs/Zs/src/Zs/Zs/FilenameZ@@/main/my_stream/3
DEBUG:root:Running: cleartool describe -fmt %On
/view/my_view/Zs/Zs/Zs/src/Zs/Zs/FilenameZ
DEBUG:root:Running: cleartool lsvob -long /Zs/Zs/Zs
DEBUG:root:Repository's /Zs/Zs/Zs uuid is
'c59494a2.4fc211db.8468.00:0d:60:14:bb:cc'
Error uploading diff
RBTools 0.5.2
Python 2.6.6 (r266:84292, May 1 2012, 13:52:17)
[GCC 4.4.6 20110731 (Red Hat 4.4.6-3)]
Running on Linux-2.6.32-279.el6.x86_64-x86_64-with-redhat-6.3-Santiago
Home = /myhomedir/
Current Directory = /view/my_view/Zs/Zs/Zs
Checking the repository type. Errors shown below are mostly harmless.
Finished checking the repository type.
HTTP GETting api/info/
HTTP GETting api/
HTTP GETting http://hostname.com/api/info/
Using the new web API
HTTP GETting http://hostname.com/api/repositories/
HTTP GETting http://hostname.com/api/repositories/3/
HTTP GETting http://hostname.com/api/repositories/3/info/
Got API Error 210 (HTTP code 500): There was an error fetching extended
information for this repository
Error data: {u'stat': u'fail', u'err': {u'msg': u'There was an error
fetching extended information for this repository', u'code': 210}}
Attempting to create review request on my_view for None
HTTP POSTing to http://hostname.com/api/review-requests/:
{'repository': 'my_view'}
Review request created
Uploading diff, size: 2387
HTTP POSTing to http://hostname.com/api/review-requests/35/diffs/:
{'basedir': '/view/my_view/Zs/Zs/Zs'}
Got API Error 105 (HTTP code 400): One or more fields had errors
Error data: {u'fields': {u'path': [u'cleartool: Error: Unable to
determine VOB for pathname ..\n']}, 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.
=
The server logs the following error:
2013-09-23 15:51:31,534 - DEBUG - - DiffParser.parse: Beginning parse of
diff, size = 2387
2013-09-23 15:51:31,620 - ERROR - None - my_user_name -
/api/review-requests/35/diffs/ - Error uploading new diff: cleartool:
Error: Unable to determine VOB for pathname ..
Traceback (most recent call last):
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/webapi/resources.py,
line 2056, in create
request.FILES.get('parent_diff_path'))
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/reviews/forms.py,
line 309, in create
history)
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/forms.py,
line 67, in create
request=self.request)
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/managers.py,
line 82, in create_from_upload
save=save)
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/managers.py,
line 103, in create_from_data
check_existence=(not parent_diff_file_contents)))
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/managers.py,
line 190, in _process_files
for f in parser.parse():
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/parser.py,
line 51, in parse
next_linenum, new_file = self.parse_change_header(i)
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/diffviewer/parser.py,
line 83, in parse_change_header
linenum = self.parse_diff_header(linenum, info)
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/scmtools/clearcase.py,
line 311, in parse_diff_header
info['origFile'] = self._oid2filename(m.group(1))
File
/usr/lib/python2.6/site-packages/ReviewBoard-1.7.14-py2.6.egg/reviewboard/scmtools/clearcase.py,
line 345, in _oid2filename
raise SCMError(error)
SCMError: cleartool: Error: Unable to determine VOB for pathname ..
Any help is appreciated.
Thanks,
Gabriela
--
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
Support post-review in Clearcase snapshot view
Hi, submited a review request for rbtool to support Clearcase snapshot post-review, see below link: http://reviews.reviewboard.org/r/3822/ Please check if it has some problem, I would like to see this feature being formally supported, as it will be useful for clearcase snapshot users. Regards, Tao -- 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 --- You received this message because you are subscribed to the Google Groups reviewboard group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
Re: post-review with clearcase
Hi,
I found the exact location where it is hanging. It hangs
while
uploading a diff at line:
r = urllib2.Request(url, body, headers) (in file postreview.py).
I read about urllib2.Request, and it says that it requires 'https' to
work. So does this mean that i have to create a https webui for
reviewboard?
Regards,
Shambhu.
On Aug 11, 4:10 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi Dan,
I went through pervious post with same issue, but still i am
stuck. It would be great if you could provide me location of patches
which would be needed to make it working.
Regards,
Shambhu.
On Aug 10, 10:09 pm, Dan Savilonis d...@n-cube.org wrote:
Can you check the mailing lists and reviews.reviewboard.org? This
sounds like something that came up before. Unfortunately post-review
and Clearcase support are not in particularly good shape in the
released version. It really needs to have some newer patches merged in
to get things working more sensibly.
Dan
On Aug 9, 11:06 am, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53 DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53 DEBUG DiffParser.parse: Finished parsing diff.
20:24:41 DEBUG DiffParser.parse: Beginning parse of diff, size =
620
20:24:41 DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log
Re: post-review with clearcase
I just replied to this in the other thread. There's a couple things that
could be causing this. Your request only appears to be hanging, because the
server is never responding, due to it being stuck, likely on the cat
process.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Tue, Aug 17, 2010 at 3:03 AM, Shambhu shambhu.1...@gmail.com wrote:
Hi,
I found the exact location where it is hanging. It hangs
while
uploading a diff at line:
r = urllib2.Request(url, body, headers) (in file postreview.py).
I read about urllib2.Request, and it says that it requires 'https' to
work. So does this mean that i have to create a https webui for
reviewboard?
Regards,
Shambhu.
On Aug 11, 4:10 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi Dan,
I went through pervious post with same issue, but still i am
stuck. It would be great if you could provide me location of patches
which would be needed to make it working.
Regards,
Shambhu.
On Aug 10, 10:09 pm, Dan Savilonis d...@n-cube.org wrote:
Can you check the mailing lists and reviews.reviewboard.org? This
sounds like something that came up before. Unfortunately post-review
and Clearcase support are not in particularly good shape in the
released version. It really needs to have some newer patches merged in
to get things working more sensibly.
Dan
On Aug 9, 11:06 am, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to
upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing tohttp://
bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_pathhttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:%7B%27repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing tohttp://
bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedirhttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:%7B%27basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53DEBUG DiffParser.parse: Beginning parse of diff,
size =
621
20:23:53DEBUG DiffParser.parse: Finished parsing diff.
20:24:41DEBUG DiffParser.parse: Beginning parse of diff,
size = 620
20:24:41DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is
post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script.
Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path
Re: post-review with clearcase
Can you check the mailing lists and reviews.reviewboard.org? This
sounds like something that came up before. Unfortunately post-review
and Clearcase support are not in particularly good shape in the
released version. It really needs to have some newer patches merged in
to get things working more sensibly.
Dan
On Aug 9, 11:06 am, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53 DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53 DEBUG DiffParser.parse: Finished parsing diff.
20:24:41 DEBUG DiffParser.parse: Beginning parse of diff, size = 620
20:24:41 DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access is denied: 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
After looking into postreview.py, i found that it is failing at
os.unlink(tf) line. While executing os.unlink(tf), it is adding
additional backslash \ with the pathname and which is causing the
problem. I am able to access file 'c:\docume~1\shambhu\locals~1\temp
Re: post-review with clearcase
Hi Guys,
After debugging postreview.py, i found that diff upload
is hanging at line r = urllib2.Request(url, body, headers) under
function http_post. Here is value of 'headers', 'url', 'body'
value of 'r' from debug print:
HEADERS: {'Content-Length': '868', 'Content-Type': 'multipart/form-data;
boundary=10.201.50.195.1.5328.1281448897.188.2'}
URL: http://review.mywebpage/api/json/reviewrequests/31/diff/new/
BODY: --10.201.50.195.1.5328.1281448897.188.2
Content-Disposition: form-data; name=basedir
K:/
--10.201.50.195.1.5328.1281448897.188.2
Content-Disposition: form-data; name=path; filename=diff
DIFF CONTENT DELETED
--10.201.50.195.1.5328.1281448897.188.2--
VALUE OF R: urllib2.Request instance at 0x00D1F058
But still no clue to make it working. Please let me know how to
resolve it, or do i need to re-install it again ::((
Regards,
Shambhu.
On Aug 9, 8:06 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53 DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53 DEBUG DiffParser.parse: Finished parsing diff.
20:24:41 DEBUG DiffParser.parse: Beginning parse of diff, size = 620
20:24:41 DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
Re: post-review with clearcase
Hi Everyone,
Please help. Any suggestion to resolve this
problem?
Regards,
Shambhu.
On Aug 9, 8:06 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53 DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53 DEBUG DiffParser.parse: Finished parsing diff.
20:24:41 DEBUG DiffParser.parse: Beginning parse of diff, size = 620
20:24:41 DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access is denied: 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
After looking into postreview.py, i found that it is failing at
os.unlink(tf) line. While executing os.unlink(tf), it is adding
additional backslash \ with the pathname and which is causing the
problem. I am able to access file 'c:\docume~1\shambhu\locals~1\temp
\4e0c6ccee4b4ee199cc18529c0bbf8c4' but not 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
Please suggest how can i fix above issue. I tried a sample script
having
Re: post-review with clearcase
Hi Dan,
I went through pervious post with same issue, but still i am
stuck. It would be great if you could provide me location of patches
which would be needed to make it working.
Regards,
Shambhu.
On Aug 10, 10:09 pm, Dan Savilonis d...@n-cube.org wrote:
Can you check the mailing lists and reviews.reviewboard.org? This
sounds like something that came up before. Unfortunately post-review
and Clearcase support are not in particularly good shape in the
released version. It really needs to have some newer patches merged in
to get things working more sensibly.
Dan
On Aug 9, 11:06 am, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:{'repository_path':
'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing
tohttp://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:{'basedir':
'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53 DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53 DEBUG DiffParser.parse: Finished parsing diff.
20:24:41 DEBUG DiffParser.parse: Beginning parse of diff, size =
620
20:24:41 DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access
Re: post-review with clearcase
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access is denied: 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
After looking into postreview.py, i found that it is failing at
os.unlink(tf) line. While executing os.unlink(tf), it is adding
additional backslash \ with the pathname and which is causing the
problem. I am able to access file 'c:\docume~1\shambhu\locals~1\temp
\4e0c6ccee4b4ee199cc18529c0bbf8c4' but not 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
Please suggest how can i fix above issue. I tried a sample script
having os.unlink with Python3.1 and even with that i am getting same
issue of additional backslash \.
Regards,
Shambhu.
On Aug 6, 2:00 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
I am new to ReviewBoard. I installed reviewboard verion 1.5rc1
rbtools 0.2.1alpha0.dev in windows machine. I am using clearcase as
version control. When i use command $post-review -d file.name, i am
getting file not found error, but the file is actually there. I am
using dynamic view and the file mentioned below is checked-in. Here is
log which i am getting:
K:\reviewboard_view\test_vob1\test_dirpost-review -d hello.c cleartool
pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
cleartool desc -pre K:\reviewboard_view\test_vob1\test_dir\hello.c
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
ERROR: FILE NOT FOUND : K:\reviewboard_view\test_vob1\@@\main
\2\test_dir\@@\main\3\hello.c@@\@@\main\4
If i just run $post-review -d then i am getting following error:
cleartool pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2982, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1035, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 899, in
get_extended_namespace
File build\bdist.win32\egg\rbtools\postreview.py, line 890, in
get_previous_version
Re: post-review with clearcase
Hi,
After making some minor modification to resolve below error,
post-review script is able to proceed further. But now it hangs at
diff upload. It is not showing any error or warning. I tried to upload
diff generated by post-review through webui, but it hangs there also
without any error msg, it just shows 'Loading' and nothing happens.
Here is the debug message where it stucks:
Looking for 'reviewboard.test.com /' cookie in C:\Documents and
Settings\shambhu\Application Data\.post-review-cookies.txt
Loaded valid cookie -- no login required
Attempting to create review request on K for None
HTTP POSTing to http://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/new/:
{'repository_path': 'K'}
Review request created
Uploading diff, size: 620
HTTP POSTing to
http://bnrpcwl0203.bnr.st.com/api/json/reviewrequests/16/diff/new/:
{'basedir': 'K'}
And nothing happens after this. I checked server log and there also
nothing is mentioned. Here it is:
20:23:53DEBUG DiffParser.parse: Beginning parse of diff, size =
621
20:23:53DEBUG DiffParser.parse: Finished parsing diff.
20:24:41DEBUG DiffParser.parse: Beginning parse of diff, size = 620
20:24:41DEBUG DiffParser.parse: Finished parsing diff.
I didn't find any error or info in apache log also. Just to verify
whether i have correct directory permission of media directory, i
tried to upload screenshot, and that worked fine.
Please suggest how to fix this and let me know if more info needed. I
think i am just one step away from using it, and it annoying me. :(
Regards,
Shambhu.
On Aug 9, 12:41 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
After checking with python-list i found that i was
misinterpreting the output error message. The actual problem is post-
review script is not able to access file c:\docume~1\shambhu
\locals~1\temp\4e0c6ccee4b4ee199cc18529c0bbf8c4 if this file is
already present if there is failure in previous run.
Now i am stuck again in function do_diff in post-review script. Here
is piece of code:
for filenam in (onam, mnam) :
if cpath.exists(filenam) and self.viewtype ==
dynamic:
debug(Dynamic view Type)
do_rem = False
fn = filenam
elif len(filenam) 254 or self.viewtype ==
snapshot:
debug(Snapshot View Type)
fn = self.get_filename_hash(filenam)
debug(FileName_Hash: %s % fn)
fn = cpath.join(tempfile.gettempdir(), fn)
debug(TempFileTempdir: %s % fn)
do_rem = True
fd = open(cpath.normpath(fn))
fdata = fd.readlines()
fd.close()
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
os.remove(filenam)
Here i am getting error at last line os.remove(filenam), because
filenam contains version extended path name and hence it is not able
to access/remove windows command.
WindowsError: [Error 206] The filename or extension is too long: 'K:\
\SHAMBHU_XXX\\YY\\@@\\main\\2\\abcabcabc\\@@\\main\
\abcabcabc\\abcabcabc_r1\\xyzxyz\\1\\hello\\@@\\main\\abcabcabc\
\abcabcabc_r1\\xyzxyz\\1\\hello.c@@\\@@\\main\\abcabcabc\\abcabcabc_r1\
\xyzxyz\\xyzxyzxyz_1\\0'
Regards,
Shambhu.
On Aug 7, 3:43 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access is denied: 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
After looking into postreview.py, i found that it is failing at
os.unlink(tf) line. While executing os.unlink(tf), it is adding
additional backslash \ with the pathname and which is causing the
problem. I am able to access file 'c:\docume~1\shambhu\locals~1\temp
\4e0c6ccee4b4ee199cc18529c0bbf8c4' but not 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
Please suggest how can i fix above issue. I tried a sample script
having os.unlink with Python3.1 and even with that i am getting same
issue of additional backslash \.
Regards,
Shambhu.
On Aug 6, 2:00 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
I am new to ReviewBoard. I installed reviewboard verion 1.5rc1
Re: post-review with clearcase
Hi,
This time I tried post-review with checkedout file. I ran
command $post-review -d checkedout_Filename
But even with checkedout file it is failing. Here is trace log:
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2993, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1041, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 999, in
get_extended_namespace
WindowsError: [Error 5] Access is denied: 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
After looking into postreview.py, i found that it is failing at
os.unlink(tf) line. While executing os.unlink(tf), it is adding
additional backslash \ with the pathname and which is causing the
problem. I am able to access file 'c:\docume~1\shambhu\locals~1\temp
\4e0c6ccee4b4ee199cc18529c0bbf8c4' but not 'c:\\docume~1\\shambhu\
\locals~1\\temp\\4e0c6ccee4b4ee199cc18529c0bbf8c4'
Please suggest how can i fix above issue. I tried a sample script
having os.unlink with Python3.1 and even with that i am getting same
issue of additional backslash \.
Regards,
Shambhu.
On Aug 6, 2:00 pm, Shambhu shambhu.1...@gmail.com wrote:
Hi,
I am new to ReviewBoard. I installed reviewboard verion 1.5rc1
rbtools 0.2.1alpha0.dev in windows machine. I am using clearcase as
version control. When i use command $post-review -d file.name, i am
getting file not found error, but the file is actually there. I am
using dynamic view and the file mentioned below is checked-in. Here is
log which i am getting:
K:\reviewboard_view\test_vob1\test_dirpost-review -d hello.c cleartool
pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
cleartool desc -pre K:\reviewboard_view\test_vob1\test_dir\hello.c
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
ERROR: FILE NOT FOUND : K:\reviewboard_view\test_vob1\@@\main
\2\test_dir\@@\main\3\hello.c@@\@@\main\4
If i just run $post-review -d then i am getting following error: cleartool
pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2982, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1035, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 899, in
get_extended_namespace
File build\bdist.win32\egg\rbtools\postreview.py, line 890, in
get_previous_version
IndexError: list index out of range
Please let me know how to fix this issue or i have done some mistake
in configuration.
Regards,
Shambhu.
--
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 with clearcase
Hi,
I am new to ReviewBoard. I installed reviewboard verion 1.5rc1
rbtools 0.2.1alpha0.dev in windows machine. I am using clearcase as
version control. When i use command $post-review -d file.name, i am
getting file not found error, but the file is actually there. I am
using dynamic view and the file mentioned below is checked-in. Here is
log which i am getting:
K:\reviewboard_view\test_vob1\test_dirpost-review -d hello.c
cleartool pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
cleartool desc -pre K:\reviewboard_view\test_vob1\test_dir\hello.c
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
cleartool desc -fmt %Vn K:\
cleartool desc -fmt %Vn K:\reviewboard_view
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1
cleartool desc -fmt %Vn K:\reviewboard_view\test_vob1\test_dir
ERROR: FILE NOT FOUND : K:\reviewboard_view\test_vob1\@@\main
\2\test_dir\@@\main\3\hello.c@@\@@\main\4
If i just run $post-review -d then i am getting following error:
cleartool pwv -short
repository info: Path: K, Base path: K, Supports changesets: False
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('rbtools==0.2.1alpha0.dev', 'console_scripts',
'post-review')()
File build\bdist.win32\egg\rbtools\postreview.py, line 2982, in
main
File build\bdist.win32\egg\rbtools\postreview.py, line 1035, in
diff
File build\bdist.win32\egg\rbtools\postreview.py, line 899, in
get_extended_namespace
File build\bdist.win32\egg\rbtools\postreview.py, line 890, in
get_previous_version
IndexError: list index out of range
Please let me know how to fix this issue or i have done some mistake
in configuration.
Regards,
Shambhu.
--
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: Access is denied using post-review with ClearCase under Cygwin
Maybe someone want to know. Actually I not pretty sure which change
made that all start working. I just describe my configuration.
I use Vobs which location is somewhere in company's network on
clearcase servers. I use Python 2.5.4 and made my changes in C:
\Python25\Lib\site-packages\RBTools-0.2beta2-py2.5\postreview.py. One
tip. Default RBTools Python egg is compressed to zip file. But there
is switch in easy_install which may change that behaviour:
easy_install --always-unzip -U RBTools
So my post-review is straight python text file in standard folder.
Next I patch postreview.py using patch from this issue:
http://code.google.com/p/reviewboard/issues/detail?id=1423
After that I change my ClearCaseClient class. My ClearCaseClient class
now looks like:
ccroot_path = Z:\SCM_tools
viewinfo =
viewtype = dynamic
I'am not sure is this should be Z:\ or Z:/ but post-review generate
diff properly. Unfortunately I'am now in home and my VPN doesn't work
again :/ So I can't check anything else for 100%. I just try remember.
The first thing is that ccroot_path must be the same as Path field in
Repository form. This is something what I discover in my battles with
ClearCase on ReviewBoard. I ran post-review from cygwin like this:
post-review -d --server http://localhost/ file1 file2 file3
So Yes I must pass filenames as parameters. xargs is also good option
to automate this :) But I can't remember that I made this on checkin
files or checkedout files... But this is something what should be easy
to check. Option -n in post-review help me very much because I find
that confguration (checkin/checkout files) where all start working.
That is all what i remember.
On Jan 26, 2:12 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
After use dynamic view and made changes in ClearCaseClient class in
postreview I stop getting this problem.
On Jan 25, 5:14 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
Hi!
On the beginning I want to say that ReviewBoard is great tool. The
best I have ever seen. Excellent job!
I'am trying to use ReviewBoard in ClearCase under Windows. I was
install whole stuff (ReviewBoard and RBTools) using easy_install under
Windows cmd. I made new snapshot etc... Then I installed cygwin (I
don't know is the order is correct) and run there:
$ post-review index.wsgi --server localhost
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('RBTools==0.2beta2', 'console_scripts', 'post-
review')()
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 2520, in main
diff, parent_diff = tool.diff(args)
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 951, in diff
return self.do_diff(self.get_extended_namespace(files))
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 912, in get_extended_namespace
os.unlink(tf)
WindowsError: [Error 5] Access is denied: 'c:\\temp\
\62b637b6048cb59c9bf5e3ab0e37aa83'
I know only that c:\temp\62b637b6048cb59c9bf5e3ab0e37aa83 is read-only
and this is a problem but I don't know what fix environ to save this
normally. Any sugestions? After delete this file I get:
$ post-review index.wsgi --server localhost
Failed to execute command: ['cleartool', 'get', '-to', 'c:\\temp\
\81c147b7ee258f26c9cc2854623b769c', 'C:\\Snapshots\\jkoprows_view\
\SCM_tools\\scripts\\ReleaseRobot\\index.wsgi@@\\main\\3\n']
cleartool: Error: C:\Snapshots\jkoprows_view\SCM_tools\scripts
\ReleaseRobot\index.wsgi@@\main\3 does not exist, or is not a
ClearCase version, or is a checked out version which was removed.
C:\\Snapshots\\jkoprows_view\\SCM_tools\\scripts\\ReleaseRobot\
\index.wsgi file is checkedin on 100% so I don't know why this is
wrong.
--
Greetings from Poland
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: Access is denied using post-review with ClearCase under Cygwin
After use dynamic view and made changes in ClearCaseClient class in
postreview I stop getting this problem.
On Jan 25, 5:14 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
Hi!
On the beginning I want to say that ReviewBoard is great tool. The
best I have ever seen. Excellent job!
I'am trying to use ReviewBoard in ClearCase under Windows. I was
install whole stuff (ReviewBoard and RBTools) using easy_install under
Windows cmd. I made new snapshot etc... Then I installed cygwin (I
don't know is the order is correct) and run there:
$ post-review index.wsgi --server localhost
Traceback (most recent call last):
File C:\Python25\Scripts\post-review-script.py, line 8, in
module
load_entry_point('RBTools==0.2beta2', 'console_scripts', 'post-
review')()
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 2520, in main
diff, parent_diff = tool.diff(args)
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 951, in diff
return self.do_diff(self.get_extended_namespace(files))
File C:\Python25\lib\site-packages\rbtools-0.2beta2-py2.5.egg
\rbtools\postreview.py, line 912, in get_extended_namespace
os.unlink(tf)
WindowsError: [Error 5] Access is denied: 'c:\\temp\
\62b637b6048cb59c9bf5e3ab0e37aa83'
I know only that c:\temp\62b637b6048cb59c9bf5e3ab0e37aa83 is read-only
and this is a problem but I don't know what fix environ to save this
normally. Any sugestions? After delete this file I get:
$ post-review index.wsgi --server localhost
Failed to execute command: ['cleartool', 'get', '-to', 'c:\\temp\
\81c147b7ee258f26c9cc2854623b769c', 'C:\\Snapshots\\jkoprows_view\
\SCM_tools\\scripts\\ReleaseRobot\\index.wsgi@@\\main\\3\n']
cleartool: Error: C:\Snapshots\jkoprows_view\SCM_tools\scripts
\ReleaseRobot\index.wsgi@@\main\3 does not exist, or is not a
ClearCase version, or is a checked out version which was removed.
C:\\Snapshots\\jkoprows_view\\SCM_tools\\scripts\\ReleaseRobot\
\index.wsgi file is checkedin on 100% so I don't know why this is
wrong.
--
Greetings from Poland
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 under clearcase / HOW TO ?
Can you submit this to http://reviews.review-board.org/? All patches should go through there for review. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Mon, May 25, 2009 at 3:54 AM, Bartlomiej Celary bartlomiej.cel...@gmail.com wrote: I had this problem as well few months ago. Following patch to post-review should resolve it (attached). I had to fix some permissions in order to get rid of temp files + the '\n' (or \r?) was not stripped from the argument and also split_lines option was needed in a execute call. I don't remember if I reached the point of actually posting a review though as I had not much time to play with clearcase+reviewboard :) Regards, Bartek Celary 2009/5/25 mahesh forvai...@gmail.com: post-review under clearcase / HOW TO ? Hi, I have set up a django based review-board (i.e. without apache / mod_python) and it came up well. I would like to post-review from another client where I've installed RBTools (latest version_ I am in clearcase dynamic view and in vob path .. example /vobs/src/tmp_vob/tools and checked out build.ksh and I am issuing following command to make it diff from predecessor version and CHECKEDOUT version to RB as if posted by an admin user. Command - post-review --server=http://mvaidya-inl:8000 --username=admin -- password=admin build.ksh Please advise me if command usage is correct and I am getting this errors for now File /home/mvaidya/EGG-INFO/scripts/post-review, line 938, in diff return self.do_diff(self.get_extended_namespace(files)) File /home/mvaidya/EGG-INFO/scripts/post-review, line 802, in get_extended_namespace versions = self.get_previous_version(files) File /home/mvaidya/EGG-INFO/scripts/post-review, line 781, in get_previous_version [cleartool, desc, -pre, elem_path]) ValueError: too many values to unpack --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard group. To post to this group, send email to reviewboard@googlegroups.com 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 under clearcase / HOW TO ?
I had this problem as well few months ago. Following patch to
post-review should resolve it (attached).
I had to fix some permissions in order to get rid of temp files + the
'\n' (or \r?) was not stripped from the argument and also split_lines
option was needed in a execute call. I don't remember if I reached the
point of actually posting a review though as I had not much time to
play with clearcase+reviewboard :)
Regards,
Bartek Celary
2009/5/25 mahesh forvai...@gmail.com:
post-review under clearcase / HOW TO ?
Hi, I have set up a django based review-board (i.e. without apache /
mod_python) and it came up well. I would like to post-review from
another client where I've installed RBTools (latest version_
I am in clearcase dynamic view and in vob path .. example
/vobs/src/tmp_vob/tools and checked out build.ksh
and I am issuing following command to make it diff from predecessor
version and CHECKEDOUT version to RB as if posted by an admin user.
Command -
post-review --server=http://mvaidya-inl:8000 --username=admin --
password=admin build.ksh
Please advise me if command usage is correct and I am getting this
errors for now
File /home/mvaidya/EGG-INFO/scripts/post-review, line 938, in diff
return self.do_diff(self.get_extended_namespace(files))
File /home/mvaidya/EGG-INFO/scripts/post-review, line 802, in
get_extended_namespace
versions = self.get_previous_version(files)
File /home/mvaidya/EGG-INFO/scripts/post-review, line 781, in
get_previous_version
[cleartool, desc, -pre, elem_path])
ValueError: too many values to unpack
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
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
-~--~~~~--~~--~--~---
Index: post-review
===
--- post-review (revision 1980)
+++ post-review (working copy)
@@ -1,4 +1,5 @@
#!/usr/bin/env python
+import stat
import cookielib
import difflib
import getpass
@@ -781,7 +782,7 @@
# Call cleartool to get this version and the previous version
# of the element.
curr_version, pre_version = execute(
-[cleartool, desc, -pre, elem_path])
+[cleartool, desc, -pre, elem_path], split_lines=True)
curr_version = cpath.normpath(curr_version)
pre_version = pre_version.split(':')[1].strip()
@@ -899,8 +900,9 @@
tf = cpath.normpath(cpath.join(td, name))
if cpath.exists(tf):
debug(WARNING: FILE EXISTS)
-os.unlink(tf)
-execute([cleartool, get, -to, tf, normkey])
+os.chmod(tf, stat.S_IWRITE | stat.S_IREAD)
+os.remove(tf)
+execute([cleartool, get, -to, tf, normkey.strip(\n)])
else:
die(ERROR: FILE NOT FOUND : %s % epstr)
@@ -995,7 +997,8 @@
file_data.append(fdata)
# If the file was temp, it should be removed.
if do_rem:
-os.remove(filenam)
+os.chmod(fn, stat.S_IWRITE | stat.S_IREAD)
+os.remove(fn)
modi = file_data.pop()
orig = file_data.pop()
