Re: backup and restore SQLite DB
Hi Jeff, Fixed it. It wasn't just the file/folder permissions. I had to run rb- site manage env syncdb, which i didn't know about. Sigh. After a little reading, I realised it! Thanks a lot man. Pravin On Jul 17, 10:39 am, Jeff Andros j...@bigredtj.com wrote: It's not just the file, the folder it resides in as well. That should be a 7 for the user that the web server runs under --Jeff -Original Message- From: Pravin Nadarajoo ramp...@gmail.com Date: Thu, 16 Jul 2009 19:04:30 To: reviewboardreviewboard@googlegroups.com Subject: Re: backup and restore SQLite DB Thanks for your reply Jeff. I left the permissions of the DB file as 644. Will just try 777 for now and see what happens. Will write here again. Thanks Pravin On Jul 17, 2:21 am, Jeff Andros j...@bigredtj.com wrote: not to be a pain, but does the web user also have write access to the directory that the SQLite file resides in? it needs to create a couple of temp files in that directory for indices and such. --Jeff 2009/7/16 Pravin Nadarajoo ramp...@gmail.com Hi there, I've just installed the new stable release of ReviewBoard (ver 1.0) on a new machine and i'm trying to move my Sqlite DB (.db) from an old one with ReviewBoard (rc2) to the new machine. From your site, I learnt that all I need to do is point ReviewBoard to the location of the old DB file and it should work. But i'm getting thrown with an Internal Server Error, even after changing the ownership of it to the web user. Is there a proper way in which i can backup my SQLite DB from the old ReviewBoard (old machine) and then properly restore it to my new machine's ReviewBoard? Please point me in the right direction. Thanks Truly Pravin -- Jeff O|||O --~--~-~--~~~---~--~~ 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: backup and restore SQLite DB
Hi Jeff, Fixed it. It wasn't just the file/folder permissions. I had to run rb- site manage env syncdb, which i didn't know about. Sigh. After a little reading, I realised it! Thanks a lot man. Pravin On Jul 17, 10:39 am, Jeff Andros j...@bigredtj.com wrote: It's not just the file, the folder it resides in as well. That should be a 7 for the user that the web server runs under --Jeff -Original Message- From: Pravin Nadarajoo ramp...@gmail.com Date: Thu, 16 Jul 2009 19:04:30 To: reviewboardreviewboard@googlegroups.com Subject: Re: backup and restore SQLite DB Thanks for your reply Jeff. I left the permissions of the DB file as 644. Will just try 777 for now and see what happens. Will write here again. Thanks Pravin On Jul 17, 2:21 am, Jeff Andros j...@bigredtj.com wrote: not to be a pain, but does the web user also have write access to the directory that the SQLite file resides in? it needs to create a couple of temp files in that directory for indices and such. --Jeff 2009/7/16 Pravin Nadarajoo ramp...@gmail.com Hi there, I've just installed the new stable release of ReviewBoard (ver 1.0) on a new machine and i'm trying to move my Sqlite DB (.db) from an old one with ReviewBoard (rc2) to the new machine. From your site, I learnt that all I need to do is point ReviewBoard to the location of the old DB file and it should work. But i'm getting thrown with an Internal Server Error, even after changing the ownership of it to the web user. Is there a proper way in which i can backup my SQLite DB from the old ReviewBoard (old machine) and then properly restore it to my new machine's ReviewBoard? Please point me in the right direction. Thanks Truly Pravin -- Jeff O|||O --~--~-~--~~~---~--~~ 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: backup and restore SQLite DB
Oh, if this was a moved install, you'd want to run rb-site upgrade /path/to/site. syncdb is handled automatically, as is an evolution. Make sure you still run upgrade, as more operations are performed than that, and they may be important. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Fri, Jul 17, 2009 at 12:57 AM, Pravin Nadarajoo ramp...@gmail.comwrote: Hi Jeff, Fixed it. It wasn't just the file/folder permissions. I had to run rb- site manage env syncdb, which i didn't know about. Sigh. After a little reading, I realised it! Thanks a lot man. Pravin On Jul 17, 10:39 am, Jeff Andros j...@bigredtj.com wrote: It's not just the file, the folder it resides in as well. That should be a 7 for the user that the web server runs under --Jeff -Original Message- From: Pravin Nadarajoo ramp...@gmail.com Date: Thu, 16 Jul 2009 19:04:30 To: reviewboardreviewboard@googlegroups.com Subject: Re: backup and restore SQLite DB Thanks for your reply Jeff. I left the permissions of the DB file as 644. Will just try 777 for now and see what happens. Will write here again. Thanks Pravin On Jul 17, 2:21 am, Jeff Andros j...@bigredtj.com wrote: not to be a pain, but does the web user also have write access to the directory that the SQLite file resides in? it needs to create a couple of temp files in that directory for indices and such. --Jeff 2009/7/16 Pravin Nadarajoo ramp...@gmail.com Hi there, I've just installed the new stable release of ReviewBoard (ver 1.0) on a new machine and i'm trying to move my Sqlite DB (.db) from an old one with ReviewBoard (rc2) to the new machine. From your site, I learnt that all I need to do is point ReviewBoard to the location of the old DB file and it should work. But i'm getting thrown with an Internal Server Error, even after changing the ownership of it to the web user. Is there a proper way in which i can backup my SQLite DB from the old ReviewBoard (old machine) and then properly restore it to my new machine's ReviewBoard? Please point me in the right direction. Thanks Truly Pravin -- Jeff O|||O --~--~-~--~~~---~--~~ 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: Do clients nee pysvn in order to use reviewboard with an svn repo?
Hi, Only the server needs pysvn. Clients using post-review only need svn in their path, which I presume they have anyway. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Fri, Jul 17, 2009 at 1:21 AM, Kaan kaan...@gmail.com wrote: Hi, I have recently installed reviewboard (manually) on my own pc for testing purposes. After testing, I will setup reviewboard for an intranet environment which is absolutely isolated ( no proxy etc. ). I wondered whether the clients, in addition to server, need pysvn in order to use the system wih subversion. I am planning to ease things up for client machines, so this would be a good to know :). Thanks and Best, Kaan --~--~-~--~~~---~--~~ 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: Do clients nee pysvn in order to use reviewboard with an svn repo?
Thanks a lot Kaan --~--~-~--~~~---~--~~ 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 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
Search Error: DLL load failed: The specified module could not be found
Hi, I have installed pyLucene successfully ( at least, I thought I did ), here is the related debug information: OS: win xp -- Environment: Request Method: GET Request URL: http://localhost:8082/reviewboard/r/search/ Django Version: 1.0.2 final Python Version: 2.5.4 Installed Applications: ['django.contrib.admin', 'django.contrib.auth', 'django.contrib.contenttypes', 'django.contrib.markup', 'django.contrib.sites', 'django.contrib.sessions', 'djblets.datagrid', 'djblets.feedview', 'djblets.siteconfig', 'djblets.util', 'djblets.webapi', 'reviewboard.accounts', 'reviewboard.admin', 'reviewboard.changedescs', 'reviewboard.diffviewer', 'reviewboard.iphone', 'reviewboard.reports', 'reviewboard.reviews', 'reviewboard.scmtools', 'reviewboard.webapi', 'django_evolution'] Installed Middleware: ('django.middleware.gzip.GZipMiddleware', 'django.middleware.common.CommonMiddleware', 'django.middleware.doc.XViewMiddleware', 'django.middleware.http.ConditionalGetMiddleware', 'django.middleware.locale.LocaleMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', 'django.contrib.auth.middleware.AuthenticationMiddleware', 'djblets.siteconfig.middleware.SettingsMiddleware', 'reviewboard.admin.middleware.LoadSettingsMiddleware', 'djblets.log.middleware.LoggingMiddleware', 'reviewboard.admin.middleware.CheckUpdatesRequiredMiddleware') Traceback: File c:\python25\lib\site-packages\django-1.0.2_final-py2.5.egg\django \core\handlers\base.py in get_response 86. response = callback(request, *callback_args, **callback_kwargs) File c:\python25\lib\site-packages\reviewboard-1.0-py2.5.egg \reviewboard\reviews\views.py in search 729. import lucene File c:\python25\lib\site-packages\lucene-2.4.1-py2.5-win32.egg\lucene \__init__.py in module 5. import jcc, _lucene File c:\python25\lib\site-packages\JCC-2.3-py2.5-win32.egg\jcc \__init__.py in module 29. from _jcc import initVM Exception Type: ImportError at /reviewboard/r/search/ Exception Value: DLL load failed: The specified module could not be found. - I had the similiar problem in importing initvm while installing pylucene but it was solved after i added jre/bin and jre/bin/client to the path variable and installation was successful Any ideas on why the proble has re-occurred? Best, Kaan --~--~-~--~~~---~--~~ 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: Upgrade issues
Hi there, Upgrading from an svn install I'm also hitting that one ... How did you solved it ? manually editing the database ? Regards, Benoit On Mon, Dec 8, 2008 at 20:12, Chris Clarkchris.cl...@ingres.com wrote: Christian Hammond wrote: This was due to a bug I'm currently fixing. I should have it in in about 15 mins. I'm not sure which this referred to. I've pulled (new) headrevs - I still get the problem with htdocs and an existing review not viewable anymore. BUT I know how to deal with htdocs (I'll start using the new rb-site script). RE the review not visible anymore I now know what the problem is! If you have any reviews in Draft, i.e not yet published, and you are pre change description on review update, at the time of the evolve. With sqlite the evolve appears to use the string 'changedesc_id' for the reviews_reviewrequestdraft.changedesc_id column (supposed to be integer or null). For the draft reviews I've simple updated all the reviews_reviewrequestdraft.changedesc_id columns to NULL if they == 'changedesc_id' - I've not looked any further into this or checked other database engines. Summary of the problem: upgrade from release of RB that does not have the change description field on updated reviews using sqlite have an existing review, that is in draft (with some updated diffs) you are logged in as the review owner you evolve the database view review Error traceback (from debug on, so result go to web browser) TemplateSyntaxError at /r/62/ Caught an exception while rendering: invalid literal for int(): changedesc_id Original Traceback (most recent call last): File C:\svn\django\django\template\debug.py, line 71, in render_node result = node.render(context) File C:\svn\django\django\template\debug.py, line 87, in render output = force_unicode(self.filter_expression.resolve(context)) File C:\svn\django\django\template\__init__.py, line 535, in resolve obj = self.var.resolve(context) File C:\svn\django\django\template\__init__.py, line 676, in resolve value = self._resolve_lookup(context) File C:\svn\django\django\template\__init__.py, line 705, in _resolve_lookup current = getattr(current, bit) File C:\svn\django\django\db\models\fields\related.py, line 248, in __get__ rel_obj = QuerySet(self.field.rel.to).get(**params) File C:\svn\django\django\db\models\query.py, line 297, in get clone = self.filter(*args, **kwargs) File C:\svn\django\django\db\models\query.py, line 483, in filter return self._filter_or_exclude(False, *args, **kwargs) File C:\svn\django\django\db\models\query.py, line 501, in _filter_or_exclude clone.query.add_q(Q(*args, **kwargs)) File C:\svn\django\django\db\models\sql\query.py, line 1224, in add_q can_reuse=used_aliases) File C:\svn\django\django\db\models\sql\query.py, line 1167, in add_filter self.where.add((alias, col, field, lookup_type, value), connector) File C:\svn\django\django\db\models\sql\where.py, line 48, in add params = field.get_db_prep_lookup(lookup_type, value) File C:\svn\django\django\db\models\fields\__init__.py, line 202, in get_db_prep_lookup return [self.get_db_prep_value(value)] File C:\svn\django\django\db\models\fields\__init__.py, line 353, in get_db_prep_value return int(value) ValueError: invalid literal for int(): changedesc_id Request Method: GET Request URL: http://clach04-745.ingres.prv:8000/r/62/ Exception Type: TemplateSyntaxError Exception Value: Caught an exception while rendering: invalid literal for int(): changedesc_id Original Traceback (most recent call last): File C:\svn\django\django\template\debug.py, line 71, in render_node result = node.render(context) File C:\svn\django\django\template\debug.py, line 87, in render output = force_unicode(self.filter_expression.resolve(context)) File C:\svn\django\django\template\__init__.py, line 535, in resolve obj = self.var.resolve(context) File C:\svn\django\django\template\__init__.py, line 676, in resolve value = self._resolve_lookup(context) File C:\svn\django\django\template\__init__.py, line 705, in _resolve_lookup current = getattr(current, bit) File C:\svn\django\django\db\models\fields\related.py, line 248, in __get__ rel_obj = QuerySet(self.field.rel.to).get(**params) File C:\svn\django\django\db\models\query.py, line 297, in get clone = self.filter(*args, **kwargs) File C:\svn\django\django\db\models\query.py, line 483, in filter return self._filter_or_exclude(False, *args, **kwargs) File C:\svn\django\django\db\models\query.py, line 501, in _filter_or_exclude clone.query.add_q(Q(*args, **kwargs)) File C:\svn\django\django\db\models\sql\query.py, line 1224, in add_q can_reuse=used_aliases) File C:\svn\django\django\db\models\sql\query.py, line 1167, in add_filter self.where.add((alias, col, field, lookup_type, value),
Re: Upgrade issues
I went to the admin part and looked for something that looks like 'reviews_reviewrequestdraft.changedesc_id' visited the right page (http://server/admin/db/reviews/reviewrequestdraft/), did nothing, saved, and it's gone ... thanks anyway for pointing the right database ... On Fri, Jul 17, 2009 at 15:38, Benal.ya...@gmail.com wrote: Hi there, Upgrading from an svn install I'm also hitting that one ... How did you solved it ? manually editing the database ? Regards, Benoit On Mon, Dec 8, 2008 at 20:12, Chris Clarkchris.cl...@ingres.com wrote: Christian Hammond wrote: This was due to a bug I'm currently fixing. I should have it in in about 15 mins. I'm not sure which this referred to. I've pulled (new) headrevs - I still get the problem with htdocs and an existing review not viewable anymore. BUT I know how to deal with htdocs (I'll start using the new rb-site script). RE the review not visible anymore I now know what the problem is! If you have any reviews in Draft, i.e not yet published, and you are pre change description on review update, at the time of the evolve. With sqlite the evolve appears to use the string 'changedesc_id' for the reviews_reviewrequestdraft.changedesc_id column (supposed to be integer or null). For the draft reviews I've simple updated all the reviews_reviewrequestdraft.changedesc_id columns to NULL if they == 'changedesc_id' - I've not looked any further into this or checked other database engines. Summary of the problem: upgrade from release of RB that does not have the change description field on updated reviews using sqlite have an existing review, that is in draft (with some updated diffs) you are logged in as the review owner you evolve the database view review Error traceback (from debug on, so result go to web browser) TemplateSyntaxError at /r/62/ Caught an exception while rendering: invalid literal for int(): changedesc_id Original Traceback (most recent call last): File C:\svn\django\django\template\debug.py, line 71, in render_node result = node.render(context) File C:\svn\django\django\template\debug.py, line 87, in render output = force_unicode(self.filter_expression.resolve(context)) File C:\svn\django\django\template\__init__.py, line 535, in resolve obj = self.var.resolve(context) File C:\svn\django\django\template\__init__.py, line 676, in resolve value = self._resolve_lookup(context) File C:\svn\django\django\template\__init__.py, line 705, in _resolve_lookup current = getattr(current, bit) File C:\svn\django\django\db\models\fields\related.py, line 248, in __get__ rel_obj = QuerySet(self.field.rel.to).get(**params) File C:\svn\django\django\db\models\query.py, line 297, in get clone = self.filter(*args, **kwargs) File C:\svn\django\django\db\models\query.py, line 483, in filter return self._filter_or_exclude(False, *args, **kwargs) File C:\svn\django\django\db\models\query.py, line 501, in _filter_or_exclude clone.query.add_q(Q(*args, **kwargs)) File C:\svn\django\django\db\models\sql\query.py, line 1224, in add_q can_reuse=used_aliases) File C:\svn\django\django\db\models\sql\query.py, line 1167, in add_filter self.where.add((alias, col, field, lookup_type, value), connector) File C:\svn\django\django\db\models\sql\where.py, line 48, in add params = field.get_db_prep_lookup(lookup_type, value) File C:\svn\django\django\db\models\fields\__init__.py, line 202, in get_db_prep_lookup return [self.get_db_prep_value(value)] File C:\svn\django\django\db\models\fields\__init__.py, line 353, in get_db_prep_value return int(value) ValueError: invalid literal for int(): changedesc_id Request Method: GET Request URL: http://clach04-745.ingres.prv:8000/r/62/ Exception Type: TemplateSyntaxError Exception Value: Caught an exception while rendering: invalid literal for int(): changedesc_id Original Traceback (most recent call last): File C:\svn\django\django\template\debug.py, line 71, in render_node result = node.render(context) File C:\svn\django\django\template\debug.py, line 87, in render output = force_unicode(self.filter_expression.resolve(context)) File C:\svn\django\django\template\__init__.py, line 535, in resolve obj = self.var.resolve(context) File C:\svn\django\django\template\__init__.py, line 676, in resolve value = self._resolve_lookup(context) File C:\svn\django\django\template\__init__.py, line 705, in _resolve_lookup current = getattr(current, bit) File C:\svn\django\django\db\models\fields\related.py, line 248, in __get__ rel_obj = QuerySet(self.field.rel.to).get(**params) File C:\svn\django\django\db\models\query.py, line 297, in get clone = self.filter(*args, **kwargs) File C:\svn\django\django\db\models\query.py, line 483, in filter return self._filter_or_exclude(False, *args, **kwargs) File
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
Issue 1227 in reviewboard: Data lost when resizing comment box in IE8
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1227 by nathan.auch: Data lost when resizing comment box in IE8 http://code.google.com/p/reviewboard/issues/detail?id=1227 What version are you running? Review Board 1.0 RC3 What's the URL of the page containing the problem? http://hostname/r/xxx/diff/ What steps will reproduce the problem? 1. Click on a line number to bring up the comment box 2. Try to resize the comment box by grabbing the edge of the box What is the expected output? What do you see instead? It is expected that the comment box will be resized. Instead, the comment box closes and the text that was already entered is lost. What operating system are you using? What browser? OS: Windows XP SP3 Browser: Internet Explorer 8 The bug only seems to exist in IE8. The behaviour is correct under Firefox. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-issues@googlegroups.com To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en -~--~~~~--~~--~--~---
Issue 1226 in reviewboard: Add group for Derivative Solutions
Updates: Status: NotABug Comment #1 on issue 1226 by chipx86: Add group for Derivative Solutions http://code.google.com/p/reviewboard/issues/detail?id=1226 You'd have to install this some place in your company. Review Board isn't a centralized hosting service. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-issues@googlegroups.com To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en -~--~~~~--~~--~--~---