Re: post-review tool
FWIW - I posted a review request yesterday that contains a different
approach to fixing this:
http://reviews.review-board.org/r/815/
(The same changes are also posted to issue 840 as a patch file:
http://code.google.com/p/reviewboard/issues/detail?id=840)
-- Steven
On Apr 14, 10:54 pm, roshan pius roshanpiustho...@gmail.com wrote:
Hi Christian,
I am not too good at python programming, but i have got it working now.The
issue was definitely beacause of the spaces in the path in the local file.
This was the culprit - last_line.split(' ')[2] in:
def _depot_to_local(self, depot_path):
Given a path in the depot return the path on the local filesystem to
the same file.
# $ p4 where //user/bvanzant/main/testing
# //user/bvanzant/main/testing //bvanzant:test05:home/testing
/home/bvanzant/home-versioned/testing
where_output = execute([p4, where, depot_path], split_lines=True)
# Take only the last line from the where command. If you have a
# multi-line view mapping with exclusions, Perforce will display
# the exclusions in order, with the last line showing the actual
# location.
last_line = where_output[-1]
# XXX: This breaks on filenames with spaces.
return last_line.split(' ')[2]
In my case the last_line variable was :
//depot/branch/src-dir-path/Assisted File Transfer/AftMain.cpp
//worksapce/branch/src-dir-path/Assisted File Transfer/AftMain.cpp
worksapce-src-dir-path\Assisted File Transfer\AftMain.cpp
So last_line.split(' ')[2] returned :
Transfer/AftMain.cpp.
It should have been:
worksapce-src-dir-path\Assisted File Transfer\AftMain.cpp
So I've made some changes(very crude ones, but it's working :-) )
The New Function Definition:
def _depot_to_local(self, depot_path):
Given a path in the depot return the path on the local filesystem to
the same file.
# $ p4 where //user/bvanzant/main/testing
# //user/bvanzant/main/testing //bvanzant:test05:home/testing
/home/bvanzant/home-versioned/testing
where_output = execute([p4, where, depot_path], split_lines=True)
# Take only the last line from the where command. If you have a
# multi-line view mapping with exclusions, Perforce will display
# the exclusions in order, with the last line showing the actual
# location.
debug(Local Path Info From Perforce: %s % (where_output))
last_line = where_output[-1].strip()
debug(Last Line Of the Info: %s % (last_line))
local_file_name=last_line.split('//')[1].rstrip().split('/')[-1].rstrip()
debug(File Name: %s % (local_file_name))
file_path_dir=last_line.split(local_file_name)[-2].rstrip().lstrip()
debug(Directory Name: %s % (file_path_dir))
file_path_name=string.join([file_path_dir,local_file_name],'')
debug(The File Path: %s %(file_path_name))
return file_path_name
Maybe you can make it better and submit a fix for the issue.
Roshan Pius
On Wed, Apr 15, 2009 at 2:15 AM, Christian Hammond chip...@chipx86.com
wrote:
Yeah, it's definitely due to spaces in the filenames. Eris Huss did some
work that should lead to a fix for this, I believe, but someone
encountering
this problem needs to find where it's breaking and update the code to make
this work.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board -http://www.review-board.org
VMware, Inc. -http://www.vmware.com
On Tue, Apr 14, 2009 at 6:37 AM, roshan pius roshanpiustho...@gmail.com
wrote:
The complete output of post-review -d :
p4 info
repository info: Path: :1666, Base path: None, Supports
changesets: True
Generating diff for changenum 10020
p4 describe -s 10020
Processing edit of //branch/host/src/Assisted File
Transfer/AftMain.cpp
Writing //branch/host/src/Assisted File Transfer/AftMain.cpp#1 to
c:\docume~1\rpius\locals~1\temp\tmptkche3
p4 print -q //branch/host/src/Assisted File
Transfer/AftMain.cpp#1 p4 where //branch/host/src/Assisted File
Transfer/AftMain.cpp
diff -urNp c:\docume~1\rpius\locals~1\temp\tmptkche3
Transfer/AftMain.cpp
Processing edit of //branch/host/src/Assisted File
Transfer/ConfigReader.cpp
Writing //branch/host/src/Assisted File
Transfer/ConfigReader.cpp#1 to
c:\docume~1\rpius\locals~1\temp\tmptkche3 p4 print -q
//branch/host/src/Assisted File
Transfer/ConfigReader.cpp#1
p4 where //branch/host/src/Assisted File
Transfer/ConfigReader.cpp diff -urNp
c:\docume~1\rpius\locals~1\temp\tmptkche3
Transfer/ConfigReader.cpp
Processing edit of //branch/host/src/Assisted File
Transfer/StdAfx.h
Writing //branch/host/src/Assisted File Transfer/StdAfx.h#1 to
c:\docume~1\rpius\locals~1\temp\tmptkche3
p4 print -q //branch/host/src/Assisted File Transfer/StdAfx.h#1
p4 where //branch/host/src/Assisted File
Re: Poll: Now that IE8 is arriving...
The engineers I work with use a variety of browsers (chrome, firefox, IE, safari even...) Our IT department has tried to get everyone to stop using IE, but that hasn't occurred since other web apps we use require the use of IE. I'm not really certain if they support newer versions of the browser unfortunately. --~--~-~--~~~---~--~~ 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: easy_install --upgrade hangs
So I attempted to upgrade by downloading the eggs and doing an upgrade on them, but now I have two copies of ReviewBoard and Djiblits egg files... I assume this is a bad thing? see log below: [r...@home src]# wget http://downloads.review-board.org/releases/ReviewBoard-1.0beta2-py2.4.egg --12:09:55-- http://downloads.review-board.org/releases/ReviewBoard-1.0beta2-py2.4.egg Resolving downloads.review-board.org... 208.78.244.88 Connecting to downloads.review-board.org|208.78.244.88|:80... connected. HTTP request sent, awaiting response... 200 OK Length: 1091447 (1.0M) [text/plain] Saving to: `ReviewBoard-1.0beta2-py2.4.egg' 100%[===] 1,091,447220K/s in 5.6s 12:10:02 (191 KB/s) - `ReviewBoard-1.0beta2-py2.4.egg' saved [1091447/1091447] [r...@home src]# wget http://downloads.review-board.org/releases/Djblets-0.5beta1-py2.4.egg --12:15:29-- http://downloads.review-board.org/releases/Djblets-0.5beta1-py2.4.egg Resolving downloads.review-board.org... 208.78.244.88 Connecting to downloads.review-board.org|208.78.244.88|:80... connected. HTTP request sent, awaiting response... 200 OK Length: 451839 (441K) [text/plain] Saving to: `Djblets-0.5beta1-py2.4.egg' 100%[===] 451,839 150K/s in 2.9s 12:15:33 (150 KB/s) - `Djblets-0.5beta1-py2.4.egg' saved [451839/451839] [r...@home src]# easy_install --upgrade /usr/local/src/ Djblets-0.5beta1-py2.4.egg Processing Djblets-0.5beta1-py2.4.egg creating /usr/lib/python2.4/site-packages/Djblets-0.5beta1-py2.4.egg Extracting Djblets-0.5beta1-py2.4.egg to /usr/lib/python2.4/site- packages Removing Djblets 0.5alpha3 from easy-install.pth file Adding Djblets 0.5beta1 to easy-install.pth file Installed /usr/lib/python2.4/site-packages/Djblets-0.5beta1-py2.4.egg Processing dependencies for Djblets==0.5beta1 Finished processing dependencies for Djblets==0.5beta1 [r...@home src]# easy_install --upgrade /usr/local/src/ ReviewBoard-1.0beta2-py2.4.egg Processing ReviewBoard-1.0beta2-py2.4.egg removing '/usr/lib/python2.4/site-packages/ReviewBoard-1.0beta2- py2.4.egg' (and everything under it) creating /usr/lib/python2.4/site-packages/ReviewBoard-1.0beta2- py2.4.egg Extracting ReviewBoard-1.0beta2-py2.4.egg to /usr/lib/python2.4/site- packages ReviewBoard 1.0beta2 is already the active version in easy- install.pth Installing rb-site script to /usr/bin Installed /usr/lib/python2.4/site-packages/ReviewBoard-1.0beta2- py2.4.egg Processing dependencies for ReviewBoard==1.0beta2 Finished processing dependencies for ReviewBoard==1.0beta2 [r...@home src]# find / -iname ReviewBoard /var/lib/mysql/reviewboard /usr/lib/python2.4/site-packages/ReviewBoard-1.0alpha4-py2.4.egg/ reviewboard /usr/lib/python2.4/site-packages/ReviewBoard-1.0beta2-py2.4.egg/ reviewboard [r...@home src]# find / -iname Djblets /var/www/rb/htdocs/media/djblets /usr/lib/python2.4/site-packages/Djblets-0.5beta1-py2.4.egg/djblets /usr/lib/python2.4/site-packages/Djblets-0.5alpha3-py2.4.egg/djblets --~--~-~--~~~---~--~~ 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: easy_install --upgrade hangs
one more thing... if I go into the admin area of reviewboard, i see this: Review Board version1.0beta2 but I haven't run the rb-site upgrade command yet (by the way, the reviewboard website is all 500 errors) so, after running rb-site upgrade, it appears like things are ok... except that the old .egg files are still in my python directories - is it safe to delete them or what should I be doing? [r...@home src]# rb-site upgrade /var/www/rb/ Rebuilding directory structure Updating database Upgrading Review Board from 1.0alpha4 to 1.0beta2 Installing json fixture 'initial_data' from '/usr/lib/python2.4/site- packages/ReviewBoard-1.0beta2-py2.4.egg/reviewboard/scmtools/ fixtures'. Installed 6 object(s) from 1 fixture(s) No evolution required . --~--~-~--~~~---~--~~ 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: easy_install --upgrade hangs
here I am answering my own questions, perhaps someone else will have the same problem as me... when you're done upgrading, you can safely delete the old .egg files the easy_install system keeps these old egg files around, you can read all about it here: http://peak.telecommunity.com/DevCenter/EasyInstall most notably this part: Uninstalling Packages If you have replaced a package with another version, then you can just delete the package(s) you don't need by deleting the PackageName- versioninfo.egg file or directory (found in the installation directory) --~--~-~--~~~---~--~~ 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: easy_install --upgrade hangs
Yeah, it's sort of confusing. I'd like down the road to have something a bit more custom for keeping servers installed and auto-upgrading sites, which could delete the old eggs. Still, I suppose there's reasons you may want old eggs around, but as you noticed, the newest version will be used by default even if your site is not upgraded. I should at least update the docs to let users know they can remove the old eggs. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.review-board.org VMware, Inc. - http://www.vmware.com On Wed, Apr 15, 2009 at 9:30 AM, Anverc anv...@gmail.com wrote: here I am answering my own questions, perhaps someone else will have the same problem as me... when you're done upgrading, you can safely delete the old .egg files the easy_install system keeps these old egg files around, you can read all about it here: http://peak.telecommunity.com/DevCenter/EasyInstall most notably this part: Uninstalling Packages If you have replaced a package with another version, then you can just delete the package(s) you don't need by deleting the PackageName- versioninfo.egg file or directory (found in the installation directory) --~--~-~--~~~---~--~~ 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 -~--~~~~--~~--~--~---
Reviewboard compared to Crucible
Hi all, We just finished a comparison of reviewboard with Crucible and CodeCollaborator. Overall everyone was impressed with ReviewBoard as it certainly holds its own against these commercial tools. However, the users noted a few areas that could be improved. Here are the comments from the team: The Dashboard doesn't keep track of reviews that you have completed. Instead it shows all the reviews in a large list. It's quite difficult to know if you have completed a review or not by just looking at the list. I believe it just colorizes the Posted and Last Updated columns, but I believe this doesn't directly indicate the status of the review, and doesn't indicate which reviewers haven't completed. It's probably just triggered by time from when the review was created. Comments are hard to follow. I find that the way the list of comments gets rendered that it's hard to follow. I realize they're sequential, but I just find it confusing at times. Having to click publish is a bit of a clunky workflow. Sometimes I forget that the green strip is there with a publish on it. I just want to say OK to a comment and have it publish. Having to click publish is an extra step I feel the tool could do without. There's no indication as to who has completed the review, and who has yet to complete it. This would be helpful It can't roll up revisions. Though this is because this tool is for pre-checkins, it still makes it harder for the review to stay relevant once you have made some further changes. You would have to start another review with the new patch file, and that just makes the review dashboard messier than it already is. We'll continue to watch ReviewBoard closely -- it seems like it's on its way to being a solid tool. Darren. --~--~-~--~~~---~--~~ 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 -~--~~~~--~~--~--~---
Issue 998 in reviewboard: post-review GUI for Windows Explorer
Comment #2 on issue 998 by go2cxg: post-review GUI for Windows Explorer http://code.google.com/p/reviewboard/issues/detail?id=998 I am sorry to ask a question from where I download the run.exe? thanks very much~~ -- 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 998 in reviewboard: post-review GUI for Windows Explorer
Comment #3 on issue 998 by go2cxg: post-review GUI for Windows Explorer http://code.google.com/p/reviewboard/issues/detail?id=998 Can you give me the URL? or mail to me ? go2...@gmail.com thanks~~ -- 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 1043 in reviewboard: register page redirects to login page
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1043 by kofemann: register page redirects to login page http://code.google.com/p/reviewboard/issues/detail?id=1043 If a user tries to register himself he got redirected to login page again. review board rev: 1904 (running on RHEL5 with mysql ) browser: Firefox 3.0.8, 3.1 beta3 site: rb.dcache.org -- 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 1016 in reviewboard: Demo not running (for unregistered users?)
Updates: Status: Fixed Comment #2 on issue 1016 by chipx86: Demo not running (for unregistered users?) http://code.google.com/p/reviewboard/issues/detail?id=1016 (No comment was entered for this change.) -- 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 1033 in reviewboard: Registration link redirects back to login
Comment #2 on issue 1033 by chipx86: Registration link redirects back to login http://code.google.com/p/reviewboard/issues/detail?id=1033 Issue 1043 has been merged into this issue. -- 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 1043 in reviewboard: register page redirects to login page
Comment #2 on issue 1043 by kofemann: register page redirects to login page http://code.google.com/p/reviewboard/issues/detail?id=1043 I updated my installation to revision r1905 and it works as expected! Thanks -- 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 1043 in reviewboard: register page redirects to login page
Updates: Status: Duplicate Mergedinto: 1033 Comment #1 on issue 1043 by chipx86: register page redirects to login page http://code.google.com/p/reviewboard/issues/detail?id=1043 (No comment was entered for this change.) -- 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 839 in reviewboard: Option to convert tabs in the diff to (n) spaces
Comment #2 on issue 839 by lee.winder: Option to convert tabs in the diff to (n) spaces http://code.google.com/p/reviewboard/issues/detail?id=839 Personally I see this as quite an important features. As much as people recommend against using tabs to format text documents, it happens a lot. We do all our development in Visual Studio and most people use tabs. This pretty much makes all our side-by-side diffs harder to read than they should be since a tab in VS is 4 spaces, and it seems like the diff viewer sees them as 9 spaces. -- 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 1044 in reviewboard: Your website is DOWN!
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1044 by kharifcrop: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 http://review-board.org is down. Here is the cut/pasted error page. 500 - Internal Server Error Something broke on the page. Please report this and describe exactly what happened when you hit this problem. Let's get you back to a working page. * Home * News * Developer Blogs * Screenshots * Press and Media * Downloads * Mailing Lists * Happy Users -- 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 1045 in reviewboard: Website down 500 error
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1045 by steve.hebert: Website down 500 error http://code.google.com/p/reviewboard/issues/detail?id=1045 *NOTE: Do not post confidential information in this bug report.* What's the URL of the page containing the problem? http://review-board.org What steps will reproduce the problem? 1. Go to http://review-board.org with browser 2. Get a 500 internal server error What is the expected output? What do you see instead? Expected the review board home page. Received a 500 error What operating system are you using? What browser? Windows Vista + Firefox Please provide any additional information below. -- 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 1047 in reviewboard: 500 - Internal Server Error
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1047 by a...@arh.ca: 500 - Internal Server Error http://code.google.com/p/reviewboard/issues/detail?id=1047 www.review-board.org -- 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 1016 in reviewboard: Demo not running (for unregistered users?)
Comment #3 on issue 1016 by neha.g.jain: Demo not running (for unregistered users?) http://code.google.com/p/reviewboard/issues/detail?id=1016 issue resolved Neha -- 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 1050 in reviewboard: Screenshot Comment location issue with Chrome
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1050 by anverc: Screenshot Comment location issue with Chrome http://code.google.com/p/reviewboard/issues/detail?id=1050 *NOTE: Do not post confidential information in this bug report.* What's the URL of the page containing the problem? http://reviewboard.site.com/r/revision-number/s/screenshot-number/ What steps will reproduce the problem? 1. In Chrome, create a screenshot comment 2. Save your review 3. Look at the comment in any other browser, or in a different window size of chrome What is the expected output? What do you see instead? The screenshot location should be based on the image itself, currently it looks like the location is purely based on where the box was in the browser window itself when it was created - thus those viewing the screenshot comment will not see it in the correct location. In addition, if you create a screenshot comment in, say, FireFox and then look at it in chrome, you'll notice that the same problem occurs (the comment isn't where it is supposed to be) What operating system are you using? What browser? WinXP - Chrome - This appears to be a problem with Chrome and ReviewBoard. Please provide any additional information below. Djblets-0.5beta2.dev_20090415-py2.4.egg ReviewBoard-1.0beta3.dev_20090415-py2.4.egg -- 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 1051 in reviewboard: Got 500 error on review-board.org page.
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1051 by wlegge...@cleversafe.com: Got 500 error on review-board.org page. http://code.google.com/p/reviewboard/issues/detail?id=1051 *NOTE: Do not post confidential information in this bug report.* What's the URL of the page containing the problem? What steps will reproduce the problem? 1. 2. 3. What is the expected output? What do you see instead? What operating system are you using? What browser? Please provide any additional information below. -- 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 1044 in reviewboard: Your website is DOWN!
Comment #1 on issue 1044 by trowbrds: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Issue 1045 has been merged into this issue. -- 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 1044 in reviewboard: Your website is DOWN!
Comment #3 on issue 1044 by trowbrds: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Issue 1047 has been merged into this issue. -- 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 1047 in reviewboard: 500 - Internal Server Error
Updates: Status: Duplicate Mergedinto: 1044 Comment #1 on issue 1047 by trowbrds: 500 - Internal Server Error http://code.google.com/p/reviewboard/issues/detail?id=1047 (No comment was entered for this change.) -- 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 1048 in reviewboard: Every page on website giving a 500
Updates: Status: Duplicate Mergedinto: 1044 Comment #1 on issue 1048 by trowbrds: Every page on website giving a 500 http://code.google.com/p/reviewboard/issues/detail?id=1048 (No comment was entered for this change.) -- 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 1044 in reviewboard: Your website is DOWN!
Comment #4 on issue 1044 by trowbrds: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Issue 1048 has been merged into this issue. -- 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 1044 in reviewboard: Your website is DOWN!
Comment #5 on issue 1044 by trowbrds: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Issue 1049 has been merged into this issue. -- 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 1044 in reviewboard: Your website is DOWN!
Comment #6 on issue 1044 by trowbrds: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Issue 1051 has been merged into this issue. -- 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 1051 in reviewboard: Got 500 error on review-board.org page.
Updates: Status: Duplicate Mergedinto: 1044 Comment #1 on issue 1051 by trowbrds: Got 500 error on review-board.org page. http://code.google.com/p/reviewboard/issues/detail?id=1051 (No comment was entered for this change.) -- 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 1044 in reviewboard: Your website is DOWN!
Updates: Status: Fixed Comment #7 on issue 1044 by chipx86: Your website is DOWN! http://code.google.com/p/reviewboard/issues/detail?id=1044 Looks like the server ran out of space. Fixing this and will be upgrading storage later. -- 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 -~--~~~~--~~--~--~---
