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 <>
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 > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Wed, Jul 11, 2012 at 1:45 PM, superakuma > > 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 >>> Review Board - http://www.reviewboard.org >>> VMware, Inc. - http://www.vmware.com >>> >>> >>> On Wed, Jul 11, 2012 at 9:45 AM, superakuma >>> > 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 >>>>> Review Board - http://www.reviewboard.org >>>>> VMware, Inc. - http://www.vmware.com >>>>> >>>>> >>>>> On Tue, Jul 10, 2012 at 10:29 PM, superakuma >>>>> >>>>> > 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 >>>>>> For more options, visit this group at http://groups.google.com/** >>>>>> group**/reviewboard?hl=en<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 > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Tue, Oct 16, 2012 at 3:06 AM, Peer >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 >>> 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 >>> 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 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 >>>
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 > 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 > 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 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"
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 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 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 > > 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 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 > > 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
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 > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Fri, Oct 12, 2012 at 3:18 AM, Peer >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 >> 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 >> 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 >> 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
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 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 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 > > VMware, Inc. > > > > > > On Tue, Aug 5, 2008 at 4:56 AM, Tarun > > 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 > &g
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 wrote: > > Hi, > > > > After some integration, i get below error warning in post review, > > > > Could someone help > > > > Z:\docs\ccadmin\test>post-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=en<>
Re: Need Help with Integrating Clearcase with Review Board
Hi, After some integration, i get below error warning in post review, Could someone help Z:\docs\ccadmin\test>post-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
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
