Re: ClearCase and ReviewBoard setup : Any pointer, best practice or setup instruction ?
Thanks you. Yes I did fix it for the first error. For the permission I did follow the technote before and also others setting. As I was using the apache user (added to the correct group) I have to chmod everything, which I did. I finally found that I was missing one chmod in one folder. It's now working. Creating review with base CC is working fine, now I'll check with UCM. Thanks again. On Monday, August 19, 2019 at 6:32:25 PM UTC+2, Laurent M. wrote: > > Good morning, > > I've setup RB with Git with no issue, but when trying with ClearCase, that > did not went well. > I understand the current integration is done by the community. > Looking at old messages (both in forum, review and bugtracking) did not > really help. > > > I was wondering if there are any kind of best practice or setup > instructions somewhere or made by current CC user ? > > I'm on CentOS, CC9.0.1.7, Apache, RB 3.0.15, RBT 1.0.2 > > The first issue I got was during a rbt post where it keep failing with > missing attribute. > > if parse_version(server.rb_version) >= parse_version('1.5.3'): > File "/usr/lib/python2.7/site-packages/rbtools/api/resource.py", line > 348, in __getattr__ > % (self.__class__.__name__, name)) > AttributeError: This RootResource does not have an attribute "rb_version". > > > After some time, I found out that it cannot grab the uuid and by renaming > the repository to the CC vobtag, it could process. > > Which is where I'm stuck now. I "know" the issue, it's with access > permissions. But until now whatever I changed did not help. > > When trying to post from rbt > > > >>> Error data: {u'stat': u'fail', u'err': {u'msg': u'There was an error > fetching extended information for this repository.', u'code': 210}} > Traceback (most recent call last): > File "/usr/bin/rbt", line 11, in > sys.exit(main()) > File "/usr/lib/python2.7/site-packages/rbtools/commands/main.py", line > 120, in main > command.run_from_argv([RB_MAIN, command_name] + args) > File "/usr/lib/python2.7/site-packages/rbtools/commands/__init__.py", > line 725, in run_from_argv > exit_code = self.main(*args) or 0 > File "/usr/lib/python2.7/site-packages/rbtools/commands/post.py", line > 770, in main > repository_info = repository_info.find_server_repository_info(api_root > ) > File "/usr/lib/python2.7/site-packages/rbtools/clients/clearcase.py", > line 1252, in find_server_repository_info > raise SCMError('You do not have permission to access ' > rbtools.clients.errors.SCMError: You do not have permission to access this > repository. > > > > When doing it from the web > > > File > "/usr/lib64/python2.7/site-packages/reviewboard/scmtools/clearcase.py", > line 58, in __init__ > self.viewtype = self._get_view_type(self.repopath) > File > "/usr/lib64/python2.7/site-packages/reviewboard/scmtools/clearcase.py", > line 156, in _get_view_type > shell=_popen_shell) > File "/usr/lib64/python2.7/subprocess.py", line 711, in __init__ > errread, errwrite) > File "/usr/lib64/python2.7/subprocess.py", line 1327, in _execute_child > raise child_exception > OSError: [Errno 13] Permission denied: '/view/rbview/vobs/stdvob' > > From what I understand, it cannot access the view/vob to grab/extract the > file content. > The process is running under the apache/apache user > I have added the apache group to the VOB group list and reprotected (using > protect and protectvob (so 'others' also have access) > Also made apache user as a member of the VOB group > I tried with a view from one user, also with a view created by the apache > user. > > So for people that actually run it, do you run it with the generic apache > user or a more "CC" ones (like running an httpd instance as vob_owner) ? > I'm kind of surpized that, even if there are only a few CC user, there is > not a lot of info on the "how-to". > > Regards. > > > > > > -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "Review Board Community" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/reviewboard/a775d7d2-1b44-4009-a46c-f30849f180a9%40googlegroups.com.
Re: ClearCase and ReviewBoard setup : Any pointer, best practice or setup instruction ?
Hi Laurent, Yeah, it's been a while since the community has made improvements to the ClearCase implementation. We've considered completely redoing it, but haven't been clear on how many people are still using it, and I'm not sure how those people who still are are using it in production. I know the first issue is just legacy code that needs to be fixed to no longer check an ancient variable, and we'll get that check fixed in the next version (it's easy to just remove manually for now, if you want to dig into that clearcase.py file in your installed copy of RBTools). The latter two exceptions are definitely the same root cause of the permissions issue. IBM has this Technote on the situation: https://www-01.ibm.com/support/docview.wss?uid=swg21230157 Given that, I think either you want to just make Apache a user in your group, or just have the permissions be 775 instead of 770. It's going to be dependent on your organization's needs, I think. The error could use some serious improving, and I'll look at that. Christian On Mon, Aug 19, 2019 at 9:32 AM Laurent M. wrote: > Good morning, > > I've setup RB with Git with no issue, but when trying with ClearCase, that > did not went well. > I understand the current integration is done by the community. > Looking at old messages (both in forum, review and bugtracking) did not > really help. > > > I was wondering if there are any kind of best practice or setup > instructions somewhere or made by current CC user ? > > I'm on CentOS, CC9.0.1.7, Apache, RB 3.0.15, RBT 1.0.2 > > The first issue I got was during a rbt post where it keep failing with > missing attribute. > > if parse_version(server.rb_version) >= parse_version('1.5.3'): > File "/usr/lib/python2.7/site-packages/rbtools/api/resource.py", line > 348, in __getattr__ > % (self.__class__.__name__, name)) > AttributeError: This RootResource does not have an attribute "rb_version". > > > After some time, I found out that it cannot grab the uuid and by renaming > the repository to the CC vobtag, it could process. > > Which is where I'm stuck now. I "know" the issue, it's with access > permissions. But until now whatever I changed did not help. > > When trying to post from rbt > > > >>> Error data: {u'stat': u'fail', u'err': {u'msg': u'There was an error > fetching extended information for this repository.', u'code': 210}} > Traceback (most recent call last): > File "/usr/bin/rbt", line 11, in > sys.exit(main()) > File "/usr/lib/python2.7/site-packages/rbtools/commands/main.py", line > 120, in main > command.run_from_argv([RB_MAIN, command_name] + args) > File "/usr/lib/python2.7/site-packages/rbtools/commands/__init__.py", > line 725, in run_from_argv > exit_code = self.main(*args) or 0 > File "/usr/lib/python2.7/site-packages/rbtools/commands/post.py", line > 770, in main > repository_info = repository_info.find_server_repository_info(api_root > ) > File "/usr/lib/python2.7/site-packages/rbtools/clients/clearcase.py", > line 1252, in find_server_repository_info > raise SCMError('You do not have permission to access ' > rbtools.clients.errors.SCMError: You do not have permission to access this > repository. > > > > When doing it from the web > > > File > "/usr/lib64/python2.7/site-packages/reviewboard/scmtools/clearcase.py", > line 58, in __init__ > self.viewtype = self._get_view_type(self.repopath) > File > "/usr/lib64/python2.7/site-packages/reviewboard/scmtools/clearcase.py", > line 156, in _get_view_type > shell=_popen_shell) > File "/usr/lib64/python2.7/subprocess.py", line 711, in __init__ > errread, errwrite) > File "/usr/lib64/python2.7/subprocess.py", line 1327, in _execute_child > raise child_exception > OSError: [Errno 13] Permission denied: '/view/rbview/vobs/stdvob' > > From what I understand, it cannot access the view/vob to grab/extract the > file content. > The process is running under the apache/apache user > I have added the apache group to the VOB group list and reprotected (using > protect and protectvob (so 'others' also have access) > Also made apache user as a member of the VOB group > I tried with a view from one user, also with a view created by the apache > user. > > So for people that actually run it, do you run it with the generic apache > user or a more "CC" ones (like running an httpd instance as vob_owner) ? > I'm kind of surpized that, even if there are only a few CC user, there is > not a lot of info on the "how-to". > > Regards. > > > > > > -- > Supercharge your Review Board with Power Pack: > https://www.reviewboard.org/powerpack/ > Want us to host Review Board for you? Check out RBCommons: > https://rbcommons.com/ > Happy user? Let us know! https://www.reviewboard.org/users/ > --- > You received this message because you are subscribed to the Google Groups > "Review Board Community" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to
Re: ClearCase and Reviewboard setup Continues...
Hi Karthik, I am facing the same upload diff hanging issue with post-review and webUI both. Any idea how to resolve this? My repository type is ClearCase and path is K:/ (clearcase drive). Please let me know how to resolve this issue. Regards, Shambhu. On Aug 11 2009, 11:15 am, Karthikeyan S karthikeyan1...@gmail.com wrote: I think I got it from here, http://sourceforge.net/projects/gnuwin32/files/patch/ Karthik On Thu, Aug 6, 2009 at 8:37 AM, jianquan bianjianq...@gmail.com wrote: hi Karthik where can i download the GNU-Patch.exe file? I meet the same issue as you. On Jul 22, 10:43 pm, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik -- 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: ClearCase and Reviewboard setup Continues...
Hi Karthik, I followed the link, but didn't get it working. I have put K:/ as drive letter. Regards, Shambhu. On Tue, Aug 10, 2010 at 1:16 PM, Karthikeyan S karthikeyan1...@gmail.comwrote: Hi Shambhu, Please look at the last mail sent by Dan Savilonis in the thread link given below, http://groups.google.com/group/reviewboard/browse_thread/thread/1a6124f46fd926a/5cc7b4f94af09873?lnk=gstq=karthik#5cc7b4f94af09873 Hope it helps. Thanks, Karthik On Tue, Aug 10, 2010 at 12:57 PM, Shambhu shambhu.1...@gmail.com wrote: Hi Karthik, I am facing the same upload diff hanging issue with post-review and webUI both. Any idea how to resolve this? My repository type is ClearCase and path is K:/ (clearcase drive). Please let me know how to resolve this issue. Regards, Shambhu. On Aug 11 2009, 11:15 am, Karthikeyan S karthikeyan1...@gmail.com wrote: I think I got it from here, http://sourceforge.net/projects/gnuwin32/files/patch/ Karthik On Thu, Aug 6, 2009 at 8:37 AM, jianquan bianjianq...@gmail.com wrote: hi Karthik where can i download the GNU-Patch.exe file? I meet the same issue as you. On Jul 22, 10:43 pm, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik -- If linux doesn't have a solution, then u have a wrong problem. Shambhu Kumar Sharma 91-98864 91913 -- 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: ClearCase and Reviewboard setup Continues...
I think I got it from here, http://sourceforge.net/projects/gnuwin32/files/patch/ Karthik On Thu, Aug 6, 2009 at 8:37 AM, jianquan bianjianq...@gmail.com wrote: hi Karthik where can i download the GNU-Patch.exe file? I meet the same issue as you. On Jul 22, 10:43 pm, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup Continues...
Jianquan, You can follow the link in the windows installation instructions here: http://review-board.org/docs/manual/dev/admin/installation/windows/ Dan On Aug 5, 11:07 pm, jianquan bianjianq...@gmail.com wrote: hi Karthik where can i download the GNU-Patch.exe file? I meet the same issue as you. On Jul 22, 10:43 pm, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup Continues...
I've only deployed RB with Clearcase for local testing, but the base path should just point to the server's view. It doesn't matter locally that that drive may or may not exist. Clearcase doesn't really fit into the model of the other supported scms where you can query it to find out the 'remote' repository. We could add an option to post- review to accept this path on the command line so it doesn't need to be hard-coded. I don't know if there's any way it could actually be stored as metadata in your view, like we do with other scms, though. Another thought -- maybe you could fill in the 'mirror path' in your configuration with a bogus name that's more user-friendly than 'y:/', like 'cc_vob_foo' and then specify that with post-review instead of 'y:/'. Dan On Jul 23, 1:47 am, Karthik karthikeyan1...@gmail.com wrote: Hi Dan, We have given the view drive (Y:/) as a base path directory in the post-review script and in the review board admin location. Does it hold good if we are going to run the common centralized review board server which will be used by group of people? What is the base path we can give in that situation? Because the drive letter is just the mapping or mounting of the view for a single user. Please let me know if you have any thoughts on this. Thanks, Karthik On Jul 23, 10:43 am, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup Continues...
Hi Dan, We have given the view drive (Y:/) as a base path directory in the post-review script and in the review board admin location. Does it hold good if we are going to run the common centralized review board server which will be used by group of people? What is the base path we can give in that situation? Because the drive letter is just the mapping or mounting of the view for a single user. Please let me know if you have any thoughts on this. Thanks, Karthik On Jul 23, 10:43 am, Karthik karthikeyan1...@gmail.com wrote: ( I was not able to post it in the same thread, it always error encountered). Hi, I could post the diff into the review board server. It is no more hanging after considering Dan's suggestion(Changing the repository path to the drive). I was facing another issue after this. When I click to see the view-diff in the review request, I was getting following error. [Errno 2] No such file or directory: 'd:\\temp\\reviewboard.7px08a\ \tmpgjsi_l-new' Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\views.py, line 152, in view_diff interdiffset, highlighting, True) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 623, in get_diff_files large_data=True) File c:\python25\lib\site-packages\Djblets-0.5-py2.5.egg\djblets \util\misc.py, line 143, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 622, in lambda enable_syntax_highlighting), File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 345, in get_chunks new = get_patched_file(old, filediff) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 261, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File c:\python25\lib\site-packages\ReviewBoard-1.0-py2.5.egg \reviewboard\diffviewer\diffutils.py, line 131, in patch f = open(newfile, r) IOError: [Errno 2] No such file or directory: 'd:\\temp\\reviewboard. 7px08a\\tmpgjsi_l-ne I had to install the GNU-Patch.exe and put it under apache webserver bin path to solve this issue. As of now, everything seems to be going fine till now!! :) Thanks, Karthik --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup.
Hi Dan, I have done the changes to make it work with Clearcase. I have followed all the steps you have mentioned. I am using dynamic clearcase view only. When I run the post-review script, it hangs when it tries to post the diff content. But I can see the diff content if I use output-diff option for post-review. Following are the output taken from post-review script. karthik-wx ct lsco -cview -me -short -rec | sed s//\//g | xargs python C:/Python25/Scripts/post-review --server=http://localhost: 8082 --submit-as=karthik --password=mywork –d C:\Python25\lib\site-packages\rbtools-0.2beta1-py2.5.egg\EGG-INFO \scripts\post-review cleartool pwv -short repository info: Path: /sw0, Base path: /sw0, Supports changesets: False cleartool desc -pre K:\sw0\software\code\include\techMgr.h Looking for 'localhost /' cookie in C:\Documents and Settings\karthik\Local Settings\Application Data\.post-review-cookies.txt Cookie file loaded, but no cookie for this server == Review Board Login Required Enter username and password for Review Board at http://localhost:8082/ Logging in with username karthik HTTP POSTing to http://localhost:8082/api/json/accounts/login/: {'username': 'karthik', 'password': '**'} Logged in. Attempting to create review request for None Submitting the review request as karthik HTTP POSTing to http://localhost:8082/api/json/reviewrequests/new/: {'repository_path': '/sw0', 'submit_as': 'karthik'} Review request created Attempting to set field 'description' to 'karthik_view_9.0BugFix VIEWTYPE: dynamic ' for review request '6' HTTP POSTing to http://localhost:8082/api/json/reviewrequests/6/draft/set/: {'description': 'karthik_view_9.0BugFix\nVIEWTYPE: dynamic\n'} HTTP POSTing to http://localhost:8082/api/json/reviewrequests/6/draft/save/: {} Review request draft saved Uploading diff, size: 548 HTTP POSTing to http://localhost:8082/api/json/reviewrequests/6/diff/new/: {'basedir': '/sw0'} It hangs after this line --- Please let me know If I miss any configuration or something. Thanks, Karthik On Jul 18, 6:05 am, Dan Savilonis d...@n-cube.org wrote: The requirements to use post-review with clearcase are the same as with any other version control system. The only difference is you pass a list of filenames as arguments to the script that you want to review: python post-review --server=http://example.com/file1.c file2.c file3.c If post-review doesn't work, easy_install rbtools should install the python dependencies. Dan On Jul 17, 8:11 pm, 吴开春 wukaic...@gmail.com wrote: 请教一下,你装起来用了没有? On Jul 17, 10:23 pm, Dan Savilonis d...@n-cube.org wrote: It is not required. post-review simply includes support for cygwin (i.e. path translation since cleartool does not understand cygwin paths). My example used sed and xargs to make it easier, so that's the only reason I suggested having the tools in your path. This feature can eventually be added to post-review to automatically collect checked out files. Dan On Jul 17, 6:50 am, 夏勇杰 notox...@gmail.com wrote: Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard withClearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add aClearCaserepository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcaseextended path. I assumed you could only use that notation within a dynamic view, but I'm probably
Re: ClearCase and Reviewboard setup.
Well, it looks like you are having the same problem as in another thread, however both have Clearcase in common. Everything is working okay in the post-review half of things, so my guess is something is going wrong trying to access the base files from Clearcase on the server side. It looks like you are running Windows, but your basedir and repository_path are '/sw0' which looks unix-like. Are you sure you have the repository path configured correctly on the server? If it's a dynamic view, it ought to just be a root drive letter like X:/. Dan On Jul 20, 6:37 am, Karthik karthikeyan1...@gmail.com wrote: Hi Dan, I have done the changes to make it work with Clearcase. I have followed all the steps you have mentioned. I am using dynamic clearcase view only. When I run the post-review script, it hangs when it tries to post the diff content. But I can see the diff content if I use output-diff option for post-review. Following are the output taken from post-review script. karthik-wx ct lsco -cview -me -short -rec | sed s//\//g | xargs python C:/Python25/Scripts/post-review --server=http://localhost: 8082 --submit-as=karthik --password=mywork –d C:\Python25\lib\site-packages\rbtools-0.2beta1-py2.5.egg\EGG-INFO \scripts\post-review cleartool pwv -short repository info: Path: /sw0, Base path: /sw0, Supports changesets: False cleartool desc -pre K:\sw0\software\code\include\techMgr.h Looking for 'localhost /' cookie in C:\Documents and Settings\karthik\Local Settings\Application Data\.post-review-cookies.txt Cookie file loaded, but no cookie for this server == Review Board Login Required Enter username and password for Review Board athttp://localhost:8082/ Logging in with username karthik HTTP POSTing tohttp://localhost:8082/api/json/accounts/login/:{'username': 'karthik', 'password': '**'} Logged in. Attempting to create review request for None Submitting the review request as karthik HTTP POSTing tohttp://localhost:8082/api/json/reviewrequests/new/:{'repository_path': '/sw0', 'submit_as': 'karthik'} Review request created Attempting to set field 'description' to 'karthik_view_9.0BugFix VIEWTYPE: dynamic ' for review request '6' HTTP POSTing tohttp://localhost:8082/api/json/reviewrequests/6/draft/set/:{'description': 'karthik_view_9.0BugFix\nVIEWTYPE: dynamic\n'} HTTP POSTing tohttp://localhost:8082/api/json/reviewrequests/6/draft/save/:{} Review request draft saved Uploading diff, size: 548 HTTP POSTing tohttp://localhost:8082/api/json/reviewrequests/6/diff/new/:{'basedir': '/sw0'} It hangs after this line --- Please let me know If I miss any configuration or something. Thanks, Karthik On Jul 18, 6:05 am, Dan Savilonis d...@n-cube.org wrote: The requirements to use post-review with clearcase are the same as with any other version control system. The only difference is you pass a list of filenames as arguments to the script that you want to review: python post-review --server=http://example.com/file1.cfile2.c file3.c If post-review doesn't work, easy_install rbtools should install the python dependencies. Dan On Jul 17, 8:11 pm, 吴开春 wukaic...@gmail.com wrote: 请教一下,你装起来用了没有? On Jul 17, 10:23 pm, Dan Savilonis d...@n-cube.org wrote: It is not required. post-review simply includes support for cygwin (i.e. path translation since cleartool does not understand cygwin paths). My example used sed and xargs to make it easier, so that's the only reason I suggested having the tools in your path. This feature can eventually be added to post-review to automatically collect checked out files. Dan On Jul 17, 6:50 am, 夏勇杰 notox...@gmail.com wrote: Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard withClearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add aClearCaserepository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic
Re: ClearCase and Reviewboard setup.
Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard with Clearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add a ClearCase repository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/to your post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcase extended path. I assumed you could only use that notation within a dynamic view, but I'm probably wrong. Dan On Jul 10, 5:38 pm, Dan Savilonis d...@n-cube.org wrote: I will try to write up my experience as I can. (Un)fortunately, we're not using RB with Clearcase right now, as I run a rogue Clearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get the Clearcase SCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have no ClearCase support or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes, Clearcase is in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul 10, 1:24 am, Oliver Jones orjo...@gmail.com wrote: According to the ReviewBoard 1.0 release notes ReviewBoard is supposed to support ClearCase out of the box. However once I had ReviewBoard 1.0 installed on my Windows server the config section's Scmtools area didn't mention ClearCase. After a little Googling I found that you need to add the ClearCase SCMTool manually into the DB. That I have done. Now I assume I need to add a repository. However I have no idea what to enter into the Path configuration item when adding the repo. This page contains info on determining the path with SVN, Git, etc, but not ClearCase: http://www.review-board.org/docs/manual/dev/admin/management/reposito... Is ClearCase actually supported?? Reading the post-review script it also appears that the clearcase code assumes you're using Cygwin. We do not have Cygwin installed here. And getting it installed could be difficult. It also seems to hard code the vob location. Can someone who has actually gotten ClearCase and ReviewBoard to play nice with each other in a Windows (non-Cygwin) environment help me out here. Thanks
Re: ClearCase and Reviewboard setup.
It is not required. post-review simply includes support for cygwin (i.e. path translation since cleartool does not understand cygwin paths). My example used sed and xargs to make it easier, so that's the only reason I suggested having the tools in your path. This feature can eventually be added to post-review to automatically collect checked out files. Dan On Jul 17, 6:50 am, 夏勇杰 notox...@gmail.com wrote: Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard with Clearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add a ClearCase repository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcase extended path. I assumed you could only use that notation within a dynamic view, but I'm probably wrong. Dan On Jul 10, 5:38 pm, Dan Savilonis d...@n-cube.org wrote: I will try to write up my experience as I can. (Un)fortunately, we're not using RB with Clearcase right now, as I run a rogue Clearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get the Clearcase SCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have no ClearCase support or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes, Clearcase is in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul 10, 1:24 am, Oliver Jones orjo...@gmail.com wrote: According to the ReviewBoard 1.0 release notes ReviewBoard is supposed to support ClearCase out of the box. However once I had ReviewBoard 1.0 installed on my Windows server the config section's Scmtools area didn't mention ClearCase. After a little Googling I found that you need to add the ClearCase SCMTool manually into the DB. That I have done. Now I assume I need to add a repository. However I have no idea what to enter into the Path configuration item when adding the repo. This page contains info on determining the path with SVN, Git, etc, but not ClearCase:
Re: ClearCase and Reviewboard setup.
请教一下,你装起来用了没有? On Jul 17, 10:23 pm, Dan Savilonis d...@n-cube.org wrote: It is not required. post-review simply includes support for cygwin (i.e. path translation since cleartool does not understand cygwin paths). My example used sed and xargs to make it easier, so that's the only reason I suggested having the tools in your path. This feature can eventually be added to post-review to automatically collect checked out files. Dan On Jul 17, 6:50 am, 夏勇杰 notox...@gmail.com wrote: Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard withClearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add aClearCaserepository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcaseextended path. I assumed you could only use that notation within a dynamic view, but I'm probably wrong. Dan On Jul 10, 5:38 pm, Dan Savilonis d...@n-cube.org wrote: I will try to write up my experience as I can. (Un)fortunately, we're not using RB withClearcaseright now, as I run a rogueClearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get theClearcaseSCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have noClearCasesupport or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes,Clearcaseis in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul 10, 1:24 am, Oliver Jones orjo...@gmail.com wrote: According to the ReviewBoard 1.0 release notes ReviewBoard is supposed to supportClearCaseout of the box. However once I had ReviewBoard 1.0 installed on my Windows server the config section's Scmtools area didn't mentionClearCase. After a little Googling I found that you need to add theClearCase SCMTool manually into the DB. That I have done. Now I assume I need to add a repository. However I have no idea what
Re: ClearCase and Reviewboard setup.
The requirements to use post-review with clearcase are the same as with any other version control system. The only difference is you pass a list of filenames as arguments to the script that you want to review: python post-review --server=http://example.com/ file1.c file2.c file3.c If post-review doesn't work, easy_install rbtools should install the python dependencies. Dan On Jul 17, 8:11 pm, 吴开春 wukaic...@gmail.com wrote: 请教一下,你装起来用了没有? On Jul 17, 10:23 pm, Dan Savilonis d...@n-cube.org wrote: It is not required. post-review simply includes support for cygwin (i.e. path translation since cleartool does not understand cygwin paths). My example used sed and xargs to make it easier, so that's the only reason I suggested having the tools in your path. This feature can eventually be added to post-review to automatically collect checked out files. Dan On Jul 17, 6:50 am, 夏勇杰 notox...@gmail.com wrote: Hi Dan, I have some questions about post review. Should we install cygwin for post-review in order to make it work with Clearcase? On Jul 17, 9:32 am, Dan Savilonis d...@n-cube.org wrote: I went through a fresh installation of reviewboard withClearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add aClearCaserepository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch fromhttp://reviews.review-board.org/r/906/toyour post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcaseextended path. I assumed you could only use that notation within a dynamic view, but I'm probably wrong. Dan On Jul 10, 5:38 pm, Dan Savilonis d...@n-cube.org wrote: I will try to write up my experience as I can. (Un)fortunately, we're not using RB withClearcaseright now, as I run a rogueClearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get theClearcaseSCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have noClearCasesupport or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes,Clearcaseis in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul
Re: ClearCase and Reviewboard setup.
I went through a fresh installation of reviewboard with Clearcase today and recorded the steps. Here's what you need to do to get up and running. For the purpose of this test, I used a dynamic view and was running the server on the same machine as I had the view and did a review of checked out files. 1) Add reviewboard.scmtools.clearcase.ClearCaseTool in the admin interface (Home Scmtools Tools), if using ReviewBoard 1.0. This step is not necessary if you're running svn revision = r2041. 2) Add a ClearCase repository via the admin interface, and specify a path to your view (I used the top of the view, not the vob). 3) Apply the patch from http://reviews.review-board.org/r/906/ to your post-review script. 4) Edit post-review ClearCaseClient class. Change cc_rootpath to be identical to what you typed into the admin interface, change viewtype to 'dynamic' (if you're trying this with a dynamic view, of course) 4) Check out a file in the view and make a change 4) Run post-review in your view and pass a list of the files that you've checked out after the options. Alternatively if you're lazy like me, you can do something like (on windows, with cygwin, msys, etc, in your vob directory): cleartool lsco -cview -me -recurse -short | sed s//\//g | xargs python post-review --server=http://example.com/ I tried using a snapshot view but it didn't work immediately for me and I haven't debugged it. It looked like it was trying to use a clearcase extended path. I assumed you could only use that notation within a dynamic view, but I'm probably wrong. Dan On Jul 10, 5:38 pm, Dan Savilonis d...@n-cube.org wrote: I will try to write up my experience as I can. (Un)fortunately, we're not using RB with Clearcase right now, as I run a rogue Clearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get the Clearcase SCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have no ClearCase support or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes, Clearcase is in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul 10, 1:24 am, Oliver Jones orjo...@gmail.com wrote: According to the ReviewBoard 1.0 release notes ReviewBoard is supposed to support ClearCase out of the box. However once I had ReviewBoard 1.0 installed on my Windows server the config section's Scmtools area didn't mention ClearCase. After a little Googling I found that you need to add the ClearCase SCMTool manually into the DB. That I have done. Now I assume I need to add a repository. However I have no idea what to enter into the Path configuration item when adding the repo. This page contains info on determining the path with SVN, Git, etc, but not ClearCase: http://www.review-board.org/docs/manual/dev/admin/management/reposito... Is ClearCase actually supported?? Reading the post-review script it also appears that the clearcase code assumes you're using Cygwin. We do not have Cygwin installed here. And getting it installed could be difficult. It also seems to hard code the vob location. Can someone who has actually gotten ClearCase and ReviewBoard to play nice with each other in a Windows (non-Cygwin) environment help me out here. Thanks --~--~-~--~~~---~--~~ 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
Re: ClearCase and Reviewboard setup.
Hi Dan, We did have to modify one line of code in post-review to get it to run Could you check if those changes are not similar to mine (http://reviews.review-board.org/r/906/)? I know I had difficulties submitting a diff for a clearcase repo and those changes fixed it. I think that we should also make a hard-coded part configurable. Thanks, Bartek --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup.
Yes, split_lines=True is what I needed to add to get it to work. I'm less sure about the other changes. They seem like they might be a problem with your version of python on Windows. I'll add comments to the review. Dan On Jul 10, 11:22 am, Bartlomiej Celary bartlomiej.cel...@gmail.com wrote: Hi Dan, We did have to modify one line of code in post-review to get it to run Could you check if those changes are not similar to mine (http://reviews.review-board.org/r/906/)?I know I had difficulties submitting a diff for a clearcase repo and those changes fixed it. I think that we should also make a hard-coded part configurable. Thanks, Bartek --~--~-~--~~~---~--~~ 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: ClearCase and Reviewboard setup.
I will try to write up my experience as I can. (Un)fortunately, we're not using RB with Clearcase right now, as I run a rogue Clearcase-to- git system, and use RB with git :) At minimum right now, we need to get the the line from Bartek's patch to add split_line=TRUE committed or post-review will just fail for end users. If you can get the Clearcase SCMTool to be added to the list of scmtools by default, that would be great, too. I have no idea how to do that, though. Dan On Jul 10, 3:29 pm, Christian Hammond chip...@chipx86.com wrote: Hi djs, Since David and I have no ClearCase support or really test infrastructure, would I be able to convince you to write some entries for our docs on setting it up and using it? Nothing major, but it would be good if people were able to get it working easier than they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board -http://www.review-board.org VMware, Inc. -http://www.vmware.com On Fri, Jul 10, 2009 at 7:22 AM, djs d...@n-cube.org wrote: Hi Oliver, Yes, Clearcase is in fact supported. We have tested out setting it up running on Windows here and were able to get it to working. The path should be the location of a dynamic view and you don't need cygwin to use it. There is code in post-review to support using cygwin or the regular cmd.exe on Windows. We did have to modify one line of code in post-review to get it to run successfully, at which point we were able to run post-review and pass a list of files are arguments. This uploaded a diff of the checked out files against their base revisions. I believe you also need to modify the hard-coded location in the post-review script for it to work. I will try to submit a patch in a few days to fix post-review. Let me know if you have any luck setting it up. Dan On Jul 10, 1:24 am, Oliver Jones orjo...@gmail.com wrote: According to the ReviewBoard 1.0 release notes ReviewBoard is supposed to support ClearCase out of the box. However once I had ReviewBoard 1.0 installed on my Windows server the config section's Scmtools area didn't mention ClearCase. After a little Googling I found that you need to add the ClearCase SCMTool manually into the DB. That I have done. Now I assume I need to add a repository. However I have no idea what to enter into the Path configuration item when adding the repo. This page contains info on determining the path with SVN, Git, etc, but not ClearCase: http://www.review-board.org/docs/manual/dev/admin/management/reposito... Is ClearCase actually supported?? Reading the post-review script it also appears that the clearcase code assumes you're using Cygwin. We do not have Cygwin installed here. And getting it installed could be difficult. It also seems to hard code the vob location. Can someone who has actually gotten ClearCase and ReviewBoard to play nice with each other in a Windows (non-Cygwin) environment help me out here. Thanks --~--~-~--~~~---~--~~ 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 -~--~~~~--~~--~--~---