Negative values in Starred reviews count
Hi Christian, We are having issue with Starred reviews count in dash board, getting negative values as in attached screenshot could you pls let us know how to avoid this, and get/reset to actual correct count https://lh4.googleusercontent.com/-nonbhXP1tW8/UPzrSubhzPI/AU4/JtklfCxH1GE/s1600/rb.JPG Thanks Peer -- 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 attachment: rb.JPG
Re: Email setup help
Hi Christian, mail notify for close submitted, action is not working. is this notification not active by default. if its by default. please let me know if i can enable a mail notify for submitted action . Thanks, Peer On Thursday, July 12, 2012 2:26:56 AM UTC+5:30, Christian Hammond wrote: Right, it's only on publish, reviews, and replies. We're told we're too noisy as it is, so doing it on drafts would probably not be in our best interest ;) Also, it's just not information most users would care about. Christian -- Christian Hammond - chi...@chipx86.com javascript: Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed, Jul 11, 2012 at 1:45 PM, superakuma hwo...@gmail.comjavascript: wrote: Christian, Email works now. We use zimbra for mail and it allows relays. Have also configured exim locally and did a telnet localhost 25 to test it. It works so I set this: Mail Server: localhost port: 25 No username and password After I publish a review, I do see an email. I was hoping that I get email notifications after I create a draft review via post-review but it seems like emails are only sent when you make a publish action. Thanks, Howard On Wednesday, July 11, 2012 11:28:57 AM UTC-7, Christian Hammond wrote: We show when there's an error, but not when a mail was sent. Can you describe your setup more? What SMTP server are you using, and is it on the same machine? What action are you performing where you expect e-mails to be sent? Are the recipients on mailing lists, or individuals? Gmail/Yahoo/etc, or company e-mail? Christian -- Christian Hammond - chi...@chipx86.com javascript: Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed, Jul 11, 2012 at 9:45 AM, superakuma hwo...@gmail.comjavascript: wrote: Thanks for the quick response. I don't see anything going to our mail server and nothing in the RB log. Is there a way to see something in the log that shows it tried to send the emails? Some debugging message would help. Thanks, Howard On Tuesday, July 10, 2012 10:46:49 PM UTC-7, Christian Hammond wrote: Hi Howard, I'm not aware of any bug. Plenty of people are using 1.6.9 with mail servers, including us. Did you check the Review Board log or the mail server log? Might be worth seeing what the mail server says. Christian -- Christian Hammond - chi...@chipx86.com javascript: Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Tue, Jul 10, 2012 at 10:29 PM, superakuma hwo...@gmail.comjavascript: wrote: I have RB 1.6.9 installed and got everything working but the email setup. I input a valid SMTP server and even tried it out using telnet test. That works. I checked the log and nothing shows up. Is there anyway for me to debug this to see what's wrong or there is a bug in this release? Thanks, Howard -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/**don**ate/http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/**use**rs/http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard...@**google **groups.com javascript: For more options, visit this group at http://groups.google.com/** group**/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/**donate/http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/**users/http://www.reviewboard.org/users/ -~--~~~~--**~~--~--~--- To unsubscribe from this group, send email to reviewboard...@** googlegroups.com javascript: For more options, visit this group at http://groups.google.com/** group/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en -- 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...@googlegroups.com javascript: For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
post-review error : Got API Error 105 (HTTP code 400): One or more fields had errors
HI Christian, I get post-review error : Got API Error 105 (HTTP code 400): One or more fields had errors, [u'cleartool: Error: Not a vob object: a75177da.2f9344e8.bea8.0e:a7:60:3a:cd:d1.\r\n'] the above generates diff but fails after HTTP POSTing to http we have multiple individual vobs, if i give post-review under the vob which has a repository in RB and .reviewboardrc file it works fine, if give for a file not under the vob path (but in diff vob path) i get the above eror, but there is repository for other vob and a .reviewboardrc file. is there any way workaround so i can run post-review in a songle vob and get all files changed in to one review? pls help how to change post-review python script to not chk this and ignore above error thanks Peer -- 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: create diff problem
HI Nandor, I am trying to setup RB with windows clearcase similar to yours, could you help on how to upload diff manually or using post-review for changes made in branches. ex: file.cc@@/main/pvt_br/5 i'm facing issue while uploading the err No such file or directory i thin its because of config-spec between RB repository config-spec (main/LATEST) and user view config-spec(private branch) how to over come this issue. please share how you made it work for branch based development. any help will be really useful. thanks, Peer On Friday, July 27, 2012 4:39:47 PM UTC+5:30, Nandor Nagy'inverz wrote: Hi there, We are using Reviewboard 1.6.4.1 with ClearCase 7.1.2 and python 2.5.4 on a windows server 2008 for code review purposes, and it's really useful and saves a lot of time and effort :) Some days ago we encountered a strange issue with reviewboard. We are courrently working on a big project, using postreview script automated and it fails when processing long (cearcase)paths. For example: * drive:\project_vob\.@@\main\version_numbers_and_branches\...\more_branch_info\...\filename.ext\@@\developer_branch\2 * #circa 500 char long After debugging it seems that a file or directory does not exist exception was thrown, when postreview is checking the given path. (for shorter than ~255char paths it works fine every time) We have tried to create the diff manually with reviewboard's GnuWin32 diff tool (diff -uN long_path1 longpath2) but same result: file or directory does not exist. Kdiff3 and clearcase diff tool can handle the extreme long paths, but they can't produce unified diff format output. So the question is: -Is there any workaround to get the diff tool working with longer paths or maybe another diff tool, which is compatible with reviewboard? thank you very much in advance, Nandor -- 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: Review Board 1.6.12 Issue with view diff
Hi Christian, can we install an older version of RB 1.6.11 its dependencies using easy_install with net connectivity. if yes please let me know the proc for it. Thanks. Peer On Tuesday, October 16, 2012 4:33:50 PM UTC+5:30, Christian Hammond wrote: You can't. We require easy_install. You can download every dependency, and easy_install them manually, but we don't provide any list of what you'd need. We're dropping support for installing on Windows, as it tends to be a support nightmare, isn't as efficient, and not all of our dependencies work well on it. We'll require Linux going forward. If you're a Windows shop, you can try installing in a Linux VM, or look into hosting at http://rbcommons.com/ Christian -- Christian Hammond - chi...@chipx86.com javascript: Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Tue, Oct 16, 2012 at 3:06 AM, Peer peerm...@gmail.com javascript:wrote: Also Christian pls let me know how to install RB in windows without easy-install / offline Thanks Peer On Tuesday, October 16, 2012 1:18:24 PM UTC+5:30, Peer wrote: the above traceback err was for the diff uploaded wich was generated manually by diff -uN, below trace back is for diff uploaded by post-review, and yes i use dynamic views and the oid is ther for the file and starts with The file 'M:/reviews/sw0/triggers.txt@@**/main/2' (r\main\2) could not be found in the repository Traceback (most recent call last): File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\views.py, line 254, in view_diff populate_diff_chunks(temp_**files, highlighting) File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 1182, in populate_diff_chunks large_data=True) File c:\python25\lib\site-**packages\Djblets-0.6.23-py2.5.**egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 1181, in lambda enable_syntax_highlighting)), File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 588, in get_chunks old = get_original_file(filediff) File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 368, in get_original_file large_data=True)[0] File c:\python25\lib\site-**packages\Djblets-0.6.23-py2.5.**egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 367, in lambda data = cache_memoize(key, lambda: [fetch_file(file, revision)], File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**diffviewer\diffutils.py, line 347, in fetch_file data = repository.get_file(file, revision) File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**scmtools\models.py, line 155, in get_file return self.get_scmtool().get_file(**path, revision) File c:\python25\lib\site-**packages\ReviewBoard-1.6.12-** py2.5.egg\reviewboard\**scmtools\clearcase.py, line 235, in get_file raise FileNotFoundError(extended_**path, revision) FileNotFoundError: The file 'M:/reviews/sw0/triggers.txt@@**/main/2' (r\main\2) could not be found in the repository On Monday, October 15, 2012 10:06:29 PM UTC+5:30, Tomasz Moń wrote: On Mon, Oct 15, 2012 at 4:06 PM, Peer peerm...@gmail.com wrote: i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time) If you submitted patch using post-review, the extended_path should include version information (the fabulous '@@'). In your traceback it appears to be missing. In that case, for Dynamic Clear Case Views, it will try to apply the patch on current version of the file, so it breaks when both: a) file version changes (for example, review gets shipped) b) the old file version disappears from cache When using postreview the patches should include oids, and then the ReviewBoard should output correct version and extended paths. You can check if your patches contain the oids, by downloading those and checking for lines starting/ending with four equal signs (). -- 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...@googlegroups.com javascript: For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -- Want to help the Review
Re: Review Board 1.6.12 Issue with view diff
the above traceback err was for the diff uploaded wich was generated manually by diff -uN, below trace back is for diff uploaded by post-review, and yes i use dynamic views and the oid is ther for the file and starts with The file 'M:/reviews/sw0/triggers.txt@@/main/2' (r\main\2) could not be found in the repository Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\views.py, line 254, in view_diff populate_diff_chunks(temp_files, highlighting) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1182, in populate_diff_chunks large_data=True) File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1181, in lambda enable_syntax_highlighting)), File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 588, in get_chunks old = get_original_file(filediff) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 368, in get_original_file large_data=True)[0] File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 367, in lambda data = cache_memoize(key, lambda: [fetch_file(file, revision)], File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 347, in fetch_file data = repository.get_file(file, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\models.py, line 155, in get_file return self.get_scmtool().get_file(path, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\clearcase.py, line 235, in get_file raise FileNotFoundError(extended_path, revision) FileNotFoundError: The file 'M:/reviews/sw0/triggers.txt@@/main/2' (r\main\2) could not be found in the repository On Monday, October 15, 2012 10:06:29 PM UTC+5:30, Tomasz Moń wrote: On Mon, Oct 15, 2012 at 4:06 PM, Peer peerm...@gmail.com javascript: wrote: i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time) If you submitted patch using post-review, the extended_path should include version information (the fabulous '@@'). In your traceback it appears to be missing. In that case, for Dynamic Clear Case Views, it will try to apply the patch on current version of the file, so it breaks when both: a) file version changes (for example, review gets shipped) b) the old file version disappears from cache When using postreview the patches should include oids, and then the ReviewBoard should output correct version and extended paths. You can check if your patches contain the oids, by downloading those and checking for lines starting/ending with four equal signs (). -- 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: Review Board 1.6.12 Issue with view diff
Also Christian pls let me know how to install RB in windows without easy-install / offline Thanks Peer On Tuesday, October 16, 2012 1:18:24 PM UTC+5:30, Peer wrote: the above traceback err was for the diff uploaded wich was generated manually by diff -uN, below trace back is for diff uploaded by post-review, and yes i use dynamic views and the oid is ther for the file and starts with The file 'M:/reviews/sw0/triggers.txt@@/main/2' (r\main\2) could not be found in the repository Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\views.py, line 254, in view_diff populate_diff_chunks(temp_files, highlighting) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1182, in populate_diff_chunks large_data=True) File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1181, in lambda enable_syntax_highlighting)), File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 588, in get_chunks old = get_original_file(filediff) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 368, in get_original_file large_data=True)[0] File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 367, in lambda data = cache_memoize(key, lambda: [fetch_file(file, revision)], File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 347, in fetch_file data = repository.get_file(file, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\models.py, line 155, in get_file return self.get_scmtool().get_file(path, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\clearcase.py, line 235, in get_file raise FileNotFoundError(extended_path, revision) FileNotFoundError: The file 'M:/reviews/sw0/triggers.txt@@/main/2' (r\main\2) could not be found in the repository On Monday, October 15, 2012 10:06:29 PM UTC+5:30, Tomasz Moń wrote: On Mon, Oct 15, 2012 at 4:06 PM, Peer peerm...@gmail.com wrote: i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time) If you submitted patch using post-review, the extended_path should include version information (the fabulous '@@'). In your traceback it appears to be missing. In that case, for Dynamic Clear Case Views, it will try to apply the patch on current version of the file, so it breaks when both: a) file version changes (for example, review gets shipped) b) the old file version disappears from cache When using postreview the patches should include oids, and then the ReviewBoard should output correct version and extended paths. You can check if your patches contain the oids, by downloading those and checking for lines starting/ending with four equal signs (). -- 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: Review Board 1.6.12 Issue with view diff
Hi Christian, the diff uploaded with post-review also show err with similar scenario (i.e view diff works fine for a while and get the err after some time) could you please help assist in resolving this error. Thanks. Peer On Friday, October 12, 2012 10:38:33 PM UTC+5:30, Christian Hammond wrote: Review Board can't work with diffs generated manually with a standard 'diff -c'. You'd need to use our post-review tool for Clear Case diffs. Christian -- Christian Hammond - chi...@chipx86.com javascript: Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Oct 12, 2012 at 3:18 AM, Peer peerm...@gmail.com javascript:wrote: *Could someone help with below issue. What version are you running?* Review Board 1.6.12 *What's the URL of the page containing the problem?* http://junose-wxp15.englab/reviews/r/9/diff/#index_header *What steps will reproduce the problem?* 1. Generate diff using diff -c and upload through web UI, diff seems to be fine and publish goes fine. 2. view diff works fine 3. after some time. if click view diff for that review get this err *What is the expected output? What do you see instead?* *What operating system are you using? What browser?* ie 8/ mozilla 15.0.1 *Please provide any additional information below.* I'm using RB setup in windows machine with clearcase repo. Get below err The file 'M:/pmohamed_remote/sw0/host/tools/clearcase/utils/mountsnapvols.pl' could not be found in the repository Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\views.py, line 254, in view_diff populate_diff_chunks(temp_files, highlighting) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1182, in populate_diff_chunks large_data=True) File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1181, in lambda enable_syntax_highlighting)), File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 588, in get_chunks old = get_original_file(filediff) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 368, in get_original_file large_data=True)[0] File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 367, in lambda data = cache_memoize(key, lambda: [fetch_file(file, revision)], File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 347, in fetch_file data = repository.get_file(file, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\models.py, line 155, in get_file return self.get_scmtool().get_file(path, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\clearcase.py, line 235, in get_file raise FileNotFoundError(extended_path, revision) FileNotFoundError: The file 'M:/pmohamed_remote/sw0/host/tools/clearcase/utils/mountsnapvols.pl' could not be found in the repository Pls help on the above issue. -- 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...@googlegroups.com javascript: For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -- 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: Review Board 1.6.12 Issue with view diff
Hi Tomasz, The file was not renamed/moved after the review submitted. the diff view simply breaks after some time with the above error. i tried using uploading diff generated by diff -uN/diff -c (or) post-review (submit diff of checked-out files), all of them diff get broken after some time) Thanks Peer On Monday, October 15, 2012 7:15:30 PM UTC+5:30, Tomasz Moń wrote: On Mon, Oct 15, 2012 at 3:22 PM, Peer peerm...@gmail.com javascript: wrote: the diff uploaded with post-review also show err with similar scenario (i.e view diff works fine for a while and get the err after some time) could you please help assist in resolving this error. Was the file renamed or removed before the view diff broke? -- 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
Review Board 1.6.12 Issue with view diff
*Could someone help with below issue. What version are you running?* Review Board 1.6.12 *What's the URL of the page containing the problem?* http://junose-wxp15.englab/reviews/r/9/diff/#index_header *What steps will reproduce the problem?* 1. Generate diff using diff -c and upload through web UI, diff seems to be fine and publish goes fine. 2. view diff works fine 3. after some time. if click view diff for that review get this err *What is the expected output? What do you see instead?* *What operating system are you using? What browser?* ie 8/ mozilla 15.0.1 *Please provide any additional information below.* I'm using RB setup in windows machine with clearcase repo. Get below err The file 'M:/pmohamed_remote/sw0/host/tools/clearcase/utils/mountsnapvols.pl' could not be found in the repository Traceback (most recent call last): File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\views.py, line 254, in view_diff populate_diff_chunks(temp_files, highlighting) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1182, in populate_diff_chunks large_data=True) File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 1181, in lambda enable_syntax_highlighting)), File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 588, in get_chunks old = get_original_file(filediff) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 368, in get_original_file large_data=True)[0] File c:\python25\lib\site-packages\Djblets-0.6.23-py2.5.egg\djblets\util\misc.py, line 156, in cache_memoize data = lookup_callable() File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 367, in lambda data = cache_memoize(key, lambda: [fetch_file(file, revision)], File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\diffviewer\diffutils.py, line 347, in fetch_file data = repository.get_file(file, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\models.py, line 155, in get_file return self.get_scmtool().get_file(path, revision) File c:\python25\lib\site-packages\ReviewBoard-1.6.12-py2.5.egg\reviewboard\scmtools\clearcase.py, line 235, in get_file raise FileNotFoundError(extended_path, revision) FileNotFoundError: The file 'M:/pmohamed_remote/sw0/host/tools/clearcase/utils/mountsnapvols.pl' could not be found in the repository Pls help on the above issue. -- 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 directory diffs
Hi Tomasz, Need help for installing RB in windows for clearcase, sorry to reply via this post. I have trouble with installing RB in windows machine and integrating with windows clearcase. Could not find much help in RB site also for the same. Could you please help me with the steps procedure for installing RB in windows and setting up with clearcase repository with dynamic or snapshot views. Thanks, Peer On Wednesday, September 5, 2012 6:07:40 PM UTC+5:30, Tomasz Moń wrote: Hi everyone, The post-review ClearCase client is supposed to be working both with dynamic and snapshot views. Although, in case of snapshot views it fails when review contains new files. The problem is related to the diff_directories() function which doesn't work with snapshot views. Additionally, if one manages to create diff file with directory diff, it does not work on Review Board side when the special case (denoted by .@@) extended path is used [1]. I have not managed to figure out a way to list directory contents in case of special case extended path neither in dynamic nor snapshot view. Therefore, why does the ClearCase client even pass the directory changes to review? Can this get removed? Both new and deleted files are already properly handled by Review Board, so why include semi-file which contains directory listing? Regards, Tomasz [1] http://publib.boulder.ibm.com/infocenter/cchelp/v7r0m0/index.jsp?topic=/com.ibm.rational.clearcase.cc_ref.doc/topics/pathnames_ccase.htm -- 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+Solaris
Hi Tarun, Would it possible if you can share the script to convert clearcase diff to unified diff, that can be parsed by RB. Also will the genrated format be able to upload via WEB GUI RB. Thanks Peer On Wednesday, August 6, 2008 10:38:05 AM UTC+5:30, Tarun wrote: I hadn't used post review before.I used to post new reviews the hard way.by hand! I guess I will start using post review now and see how i can make a diff for the ClearCase file. About the ClearCase SCMTool , I have a custom tool that I wrote up. It is similar to the CVS SCMTool; since I generate my diffs in a certain format my Clearcase SCMTool is able to parse that format and use it as a normal diff file. -Tarun Christian Hammond wrote: That's one of the reasons post-review exists. If you look at the Perforce code, you'll see we build our own diff. In your case, you should probably use the serialized diff and go over and process it to turn it into a unified diff, or just generate your own completely. As for the repository type, I don't understand fully. We don't have a ClearCase SCMTool, so what are you using? Something custom? Christian -- Christian Hammond - chi...@chipx86.com javascript: VMware, Inc. On Tue, Aug 5, 2008 at 4:56 AM, Tarun taru...@gmail.com javascript: wrote: What do you use for the repository type on Review Board? I didnt quite understand this question, I use a clearcase repository on my Review Board setup. I am working on the post review aspect to see how I can implement it for Clearcase. About the clearcase diff issue. Clearcase gives a command to present a diff in a format where the predecessor file and the current file can be shown in the diff file. This is called the serial format, but the problem with this format is that it cannot be identified by the patch command. For example, I have the diff over here for a file named avm- mib-nointervention.tcl Diff: Serial format file 1: /vobs/tetware/systems/avm-mib-nointervention.tcl@@/main/ tware_avan_r3_br/test_infra_br/0 file 2: avm-mib-nointervention.tcl -[1 changed to 1-2]- #!/usr/bin/expect --- #!/usr/bin/tcl package require Expect -[775 changed to 776]- source $env(TET_ROOT)/lib/expect/expect.tcm.dat --- source $env(TET_ROOT)/lib/tcl/tcl.tcm.dat Now file 1 is the predecessor file(previous revision) and file 2 is the current file(working copy) . To mention changed lines, the diff presents it verbosely -[775 changed to 776]- it is this line that creates a conflict. Now the other clearcase diff format (the unified diff format option) presents the output in the following format 1c1,2 #!/usr/bin/expect --- #!/usr/bin/tcl package require Expect 775c776 source $env(TET_ROOT)/lib/expect/expect.tcm.dat --- source $env(TET_ROOT)/lib/tcl/tcl.tcm.dat The diff in this format can be understood by the patch command, but with this option I cannot get the predecessor and the current file to be mentioned in the diff file. Thats why I had to write a script to meet my needs. -Tarun Christian Hammond wrote: What do you use for the repository type on Review Board? It would be good to integrate a lot of this into post-review. post-review already does stuff for other repository types to mold the diffs into having the required data we need. I don't know if I fully understand the diff issue, but post-review could generate the multiple diffs and concatenate them into a single diff automatically. Christian -- Christian Hammond - chi...@chipx86.com javascript: VMware, Inc. On Mon, Aug 4, 2008 at 11:50 PM, Tarun taru...@gmail.comjavascript: wrote: Hi Akshay, I have reviewboard setup with clearcase(i would not call it a proper integration thoughi shall explain it). This is on RHEL4. My setup has an intermediary step. This involves a user to run a script that generates diff for multiple files present in the clearcase repository in the unified diff format. Another reason I have the users run this script because i can then get the revision number for a particular file. (If you know of a way to generate multiple diffs in a single fileplease let me know. I do not use the clearcase cleartool diff serial format command to generate the diffs as they are not in the diff format that can be recognised by the patch command and the clearcase cleartool diff diff format command does not show the revision number though it is the format
Re: Need Help with Integrating Clearcase with Review Board
Thanks Michal, Replacing with ; did work, Could you please also help with GUI, uploading a diff for clearcase, I get below error page when i try to create a new review request in GUI Err: Something broke! (Error 500) If i try to upoad-diff on an already available i get the attached error. Pls let me know if you can help on this. my PATH in repo config is like Z:/docs is this OK? Thanks Peer On Tuesday, August 7, 2012 1:52:51 PM UTC+5:30, Michal Muszynski wrote: Hi, You should probably use ';' as a separator between files in --revision-range. On 7 August 2012 08:43, Peer peermoh...@gmail.com wrote: Hi, After some integration, i get below error warning in post review, Could someone help Z:\docs\ccadmin\testpost-review --revision-range=Z:\docs\ccadmin\test\test.txt@@\main\13@@Z:\docs\ ccadmin\test\test.txt@@\main\14 There don't seem to be any diffs! Thanks, Peer On Monday, August 6, 2012 1:13:00 PM UTC+5:30, Peer wrote: Hi, I'm new to Review Board, i have installed RB 1.6.11 in windows machine after long struggle. I'm now facing issue with Integrating Clearcase with Review Board. and upload diffs. If someone could help me with step by step of the same would be really helpful, Thanks, Peer -- 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 -- 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=enattachment: err.JPG
Need Help with Integrating Clearcase with Review Board
Hi, I'm new to Review Board, i have installed RB 1.6.11 in windows machine after long struggle. I'm now facing issue with Integrating Clearcase with Review Board. and upload diffs. If someone could help me with step by step of the same would be really helpful, Thanks, Peer -- 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