Issue 1384 in reviewboard: expand diff not working in 1.0.5
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1384 by papacino: expand diff not working in 1.0.5 http://code.google.com/p/reviewboard/issues/detail?id=1384 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.0.5.1 What's the URL of the page containing the problem? the diff viewer What steps will reproduce the problem? 1. click on `85 lines hidden [expand]` 2. browser shows spinning/waiting 3. diff is not expanded when browser What is the expected output? What do you see instead? same diff as was started with What operating system are you using? What browser? firefox 3.5.4 Please provide any additional information below. open a file for review, and collapsed areas will no longer be expanded on demand. Cliff Young -- 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 -~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Comment #6 on issue 1382 by robert.munteanu: Inline review comments are not
saved with Firefox 3.6
http://code.google.com/p/reviewboard/issues/detail?id=1382
Firebug does not catch any errors, and I got the following XMLHttpRequest
message:
Ship It
/xxxfile
(Diff revision 5)
Sample
comment.
var editor = $("#diff-comment-191").reviewFormCommentEditor({
path: getDiffAPIPath(5,
312,
null, null,
22),
data: {
action: "set",
num_lines: 1
}
});
$("#edit-diff-comment-191").click(function() {
editor.inlineEditor("startEdit");
});
queueLoadDiffFragment("review_draft_diff_comments", "191",
"312"
);
--
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
-~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Comment #5 on issue 1382 by robert.munteanu: Inline review comments are not
saved with Firefox 3.6
http://code.google.com/p/reviewboard/issues/detail?id=1382
Firebug does not catch any errors, and I got the following XMLHttpRequest
message:
Ship It
/xxxfile
(Diff revision 5)
Sample
comment.
var editor = $("#diff-comment-191").reviewFormCommentEditor({
path: getDiffAPIPath(5,
312,
null, null,
22),
data: {
action: "set",
num_lines: 1
}
});
$("#edit-diff-comment-191").click(function() {
editor.inlineEditor("startEdit");
});
queueLoadDiffFragment("review_draft_diff_comments", "191",
"312"
);
--
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
-~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Comment #4 on issue 1382 by robert.munteanu: Inline review comments are not saved with Firefox 3.6 http://code.google.com/p/reviewboard/issues/detail?id=1382 This is how the comments section of the review dialog appears. The comment line ( 22 here ) is attached, with the proper contents, but the comment text ( 'Sample comment' ) is instead empty. Attachments: review-board-missing-comment.jpg 14.1 KB -- 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 -~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Comment #3 on issue 1382 by robert.munteanu: Inline review comments are not saved with Firefox 3.6 http://code.google.com/p/reviewboard/issues/detail?id=1382 1. It is the big review dialog. The comments are present in the diff viewer, even if I go to the "View reviews" page and then come back to the diff viewer. But once I go to 'Edit review' to publish the review, the inline comments are empty, but the lines they are attached to are listed as expected. 2.Shift-reload does not change anything. This was working properly on Firefox 3.5 FWIW. -- 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 -~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Updates: Status: NeedInfo Comment #2 on issue 1382 by chipx86: Inline review comments are not saved with Firefox 3.6 http://code.google.com/p/reviewboard/issues/detail?id=1382 Is this in the big review dialog, or comment flags in the diff viewer? Does shift-reload fix anything? -- 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 1383 in reviewboard: RBTools easy_install on py26 fails when trying to install simplejson
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1383 by p...@swooby.com: RBTools easy_install on py26 fails when trying to install simplejson http://code.google.com/p/reviewboard/issues/detail?id=1383 *NOTE: Do not post confidential information in this bug report.* What version are you running? post-review-script.py 0.8 What's the URL of the page containing the problem? n/a What steps will reproduce the problem? 1. Install py26 2. Install setuptools for py26 3. easy_install -U RBtools What is the expected output? What do you see instead? Install should be successful. Instead, install fails when trying to install the simplejson egg. This initially makes sense, because simplejson has no py26 installer. However, according to the simplejson site, simplejson is built in to py26. The net effect is that easy_install RBTool should be be installing simplejson in py26. What operating system are you using? What browser? XP/Vista Please provide any additional information below. Per Christian Hammond: "We just need to turn off the sqlite dependency when building Python 2.6 packages." -- 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 -~--~~~~--~~--~--~---
Re: Issue 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Comment #1 on issue 1382 by robert.munteanu: Inline review comments are not saved with Firefox 3.6 http://code.google.com/p/reviewboard/issues/detail?id=1382 Just noticed that if I save the comments ( draft review ) with FF 3.6 and then edit the review e.g. with Chromium, the comments are visible. If I edit the review with FF 3.6 they are not visible. -- 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 1382 in reviewboard: Inline review comments are not saved with Firefox 3.6
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1382 by robert.munteanu: Inline review comments are not saved with Firefox 3.6 http://code.google.com/p/reviewboard/issues/detail?id=1382 What version are you running? Review board 1.0.4 What steps will reproduce the problem? 1. Enter a review comment with Firefox 3.6 2. Submit the review What is the expected output? What do you see instead? The review comments are saved as empty text, but the code positions - file and line number - are saved. What operating system are you using? What browser? Linux/ Firefox 3.6 beta 1 Please provide any additional information below. Existing inline comments are loaded. Comments are said to be saved when reviewing the diff, but appear as empty after publishing. -- 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 1381 in reviewboard: diff update doesn't work in web GUI and in post-review utility
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1381 by ostocek: diff update doesn't work in web GUI and in
post-review utility
http://code.google.com/p/reviewboard/issues/detail?id=1381
*NOTE: Do not post confidential information in this bug report.*
What version are you running?
1.0.5.1
What's the URL of the page containing the problem?
https://ourdevelserver/rb/r/187/diff/
What steps will reproduce the problem?
1. Create a request with diff (works fine)
2. Try to update diff by "Upload diff" form
What is the expected output? What do you see instead?
The error is displayed with this detail message
{"fields": {"path": ["The diff file is empty"]}, "stat": "fail", "err":
{"msg": "One or more fields had errors", "code": 105}}
when I try post-review in the terminal, problem is following:
$ post-review -p -r 187 --description "some message"
==> HTTP Authentication Required
Enter username and password for "" at ourdevelserver
Username: user
Password:
Error uploading diff: One or more fields had errors (105)
Your review request still exists, but the diff is not attached.
What operating system are you using? What browser?
Firefox 3.5.1, Opera 10.0.1
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
-~--~~~~--~~--~--~---
Re: Issue 1360 in reviewboard: Can not see diff when the diff was uploaded with parent option on post-review
Comment #1 on issue 1360 by owen.yamauchi: Can not see diff when the diff was uploaded with parent option on post-review http://code.google.com/p/reviewboard/issues/detail?id=1360 The -t option doesn't actually do the right thing - you should get a reversed diff with that flag on. The problem seems to be that the file RB is trying to patch has already been patched. "Previously applied patch detected" is correct. -- 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 -~--~~~~--~~--~--~---
Re: Issue 514 in reviewboard: Comments don't show between multiple versions of a review
Comment #5 on issue 514 by chipx86: Comments don't show between multiple versions of a review http://code.google.com/p/reviewboard/issues/detail?id=514 Issue 1379 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 -~--~~~~--~~--~--~---
Re: Issue 1379 in reviewboard: Display comment flags for previous diff revisions
Updates: Status: Duplicate Mergedinto: 514 Comment #1 on issue 1379 by chipx86: Display comment flags for previous diff revisions http://code.google.com/p/reviewboard/issues/detail?id=1379 (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 1379 in reviewboard: Display comment flags for previous diff revisions
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 1379 by nathan.auch: Display comment flags for previous diff revisions http://code.google.com/p/reviewboard/issues/detail?id=1379 What version are you running? ReviewBoard 1.0 alpha 4 What's the URL of the page this enhancement relates to, if any? Diffviewer - http://hostname/r/xxx/diff/ Describe the enhancement and the motivation for it: Comment flags are only displayed for the revision of the diff currently being viewed. It would be nice if these were displayed regardless of the diff revision selected. If a reviewer makes a comment on a specific line of code and the submitter uploads a new diff (that does not address the comment) it is difficult to tell when viewing the diff that a comment for this line has already been made. This confusion is further increased when multiple reviewers are reviewing a request at different times and would like to avoid making duplicate comments. -- 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #36 on issue 1072 by chipx86: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 At this point, I recommend taking this to the mailing list and referencing this bug. The original bug is fixed in 1.1, and we don't support HTTPS in any official capacity for 1.0. You would have to see if other people have workarounds they've used. -- 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 -~--~~~~--~~--~--~---
Re: Issue 953 in reviewboard: Hg SCMTool doesn't support old mercurial installs
Comment #4 on issue 953 by deuresti79: Hg SCMTool doesn't support old mercurial installs http://code.google.com/p/reviewboard/issues/detail?id=953 I think there are two bugs here. #1 Not supporting old mercurial installs like sandeep says. #2 Not supporting ssh HG repositories (even in newer versions of mercurial sshrepository doesn't have a changectx) -- 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #35 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 I have tried anyway i can do, still no help on this issue, i will work on the issue until it works, and i still hope someone could help me to workaround it, many 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #34 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 I have tried anyway i can do, still no help on this issue, i will work on the issue until it works, and i still wish someone could help me to workaround it, many 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #33 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 detailed apache log Attachments: error.log 4.0 KB -- 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #32 on issue 1072 by yunshan.di: HTTPS Subversion repositories
don't work with Review Board
http://code.google.com/p/reviewboard/issues/detail?id=1072
I followed the step:
2) Figure out what the home directory is for Apache (I don't know on
Windows, but you
may be able to figure it out by turning on logging and temporarily
modifying, say,
reviewboard/__init__.py to add:
logging.info("Home directory = %s" % os.path.expanduser("~"))
After restart apache, check the apache log, and found a few errors
Attachments:
apache-log.txt 15.6 KB
--
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
-~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #31 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 yes，I did, see attached screenshot. I still got the error "HTTPS certificate not accepted. Please ensure that the proper certificate exists in C:/Documents and Settings/yunshandi/.subversion/auth for the user that reviewboard is running as." But the certificate does exist under the .subversion/auth. Attachments: env.png 16.1 KB failed.png 16.4 KB -- 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #30 on issue 1072 by chipx86: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 Did my recommendations not work out for setting those environment variables? -- 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 -~--~~~~--~~--~--~---
Re: Issue 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #29 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 How is going on? -- 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 -~--~~~~--~~--~--~---
Comment on ReviewBoardAPI in reviewboard
Comment by p...@swooby.com:
Example Python code:
{{{
import simplejson
import sys
import urllib
class ReviewBoard:
def __init__(self, server, username, password):
self.server = server
self.username = username
self.password = password
def Run(self, command, params=None, throw=True):
if params:
params = urllib.urlencode(params)
url = 'http://%s/api/json/%s/' % (self.server, command)
f = urllib.urlopen(url, params)
# TODO: Request Error handling...
s = f.read()
response = simplejson.loads(s)
self.OK(response, throw)
return response
def OK(self, response, throw=False):
ok = response['stat'] == 'ok'
if not ok and throw:
raise Exception('ReviewBoard response stat not "ok"')
return ok
def Login(self, username=None, password=None):
if username is None:
username = self.username
if password is None:
password = self.password
rb.Run('accounts/login', {'username': username, 'password': password})
if __name__ == '__main__':
reviewers = ['larry', 'moe', 'curly']
rb = ReviewBoard('10.100.120.86', 'username', 'password')
rb.Login()
response = rb.Run('reviewrequests/11/reviews')
reviews = response['reviews']
for review in reviews:
reviewer = review['user']['username']
ship_it = review['ship_it']
if reviewer in reviewers and ship_it:
print '"%s" SAYS SHIP IT!' % reviewer.upper()
}}}
For more information:
http://code.google.com/p/reviewboard/wiki/ReviewBoardAPI
--~--~-~--~~~---~--~~
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
-~--~~~~--~~--~--~---
Comment on ReviewBoardAPI in reviewboard
Comment by p...@swooby.com:
Example Python code:
{{{
import simplejson
import sys
import urllib
class ReviewBoard:
def __init__(self, server, username, password):
self.server = server
self.username = username
self.password = password
def Run(self, command, params=None, throw=True):
if params:
params = urllib.urlencode(params)
url = 'http://%s/api/json/%s/' % (self.server, command)
f = urllib.urlopen(url, params)
# TODO:(pv) Request Error handling...
s = f.read()
response = simplejson.loads(s)
self.OK(response, throw)
return response
def OK(self, response, throw=False):
ok = response['stat'] == 'ok'
if not ok and throw:
raise Exception('ReviewBoard response stat not "ok"')
return ok
def Login(self, username=None, password=None):
if username is None:
username = self.username
if password is None:
password = self.password
rb.Run('accounts/login', {'username': username, 'password': password})
if __name__ == '__main__':
reviewers = ['pv', 'lconley']
rb = ReviewBoard('10.100.120.86', 'pv', 'g00gl3')
rb.Login()
response = rb.Run('reviewrequests/11/reviews')
reviews = response['reviews']
for review in reviews:
reviewer = review['user']['username']
ship_it = review['ship_it']
if reviewer in reviewers and ship_it:
print '"%s" SAYS SHIP IT!' % reviewer.upper()
}}}
For more information:
http://code.google.com/p/reviewboard/wiki/ReviewBoardAPI
--~--~-~--~~~---~--~~
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
-~--~~~~--~~--~--~---
Comment on ReviewBoardAPI in reviewboard
Comment by p...@swooby.com:
Python example:
import simplejson
import sys
import urllib
class ReviewBoard:
def __init__(self, server, username, password):
self.server = server
self.username = username
self.password = password
def Run(self, command, params=None, throw=True):
if params:
params = urllib.urlencode(params)
url = 'http://%s/api/json/%s/' % (self.server, command)
f = urllib.urlopen(url, params)
# TODO: Request Error handling...
s = f.read()
response = simplejson.loads(s)
self.OK(response, throw)
return response
def OK(self, response, throw=False):
ok = response['stat'] == 'ok'
if not ok and throw:
raise Exception('ReviewBoard response stat not "ok"')
return ok
def Login(self, username=None, password=None):
if username is None:
username = self.username
if password is None:
password = self.password
rb.Run('accounts/login', {'username': username, 'password': password})
if __name__ == '__main__':
reviewers = ['larry', 'moe', 'curly']
rb = ReviewBoard('10.100.120.86', 'username', 'password')
rb.Login()
response = rb.Run('reviewrequests/11/reviews')
reviews = response['reviews']
for review in reviews:
reviewer = review['user']['username']
ship_it = review['ship_it']
if reviewer in reviewers and ship_it:
print '"%s" SAYS SHIP IT!' % reviewer.upper()
For more information:
http://code.google.com/p/reviewboard/wiki/ReviewBoardAPI
--~--~-~--~~~---~--~~
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 1376 in reviewboard: post-review: --repository-url pointing to a file doesn't work
Comment #2 on issue 1376 by niko...@prokoschenko.de: post-review: --repository-url pointing to a file doesn't work http://code.google.com/p/reviewboard/issues/detail?id=1376 It makes perfect sense to me. A little explanation might be in order :) Use case: I've got a developer who'd like to have his committed changes to a file file.txt reviewed, let's say he started at revision 100 and finished at revision 150. There have been five commits, so his commits are probably at 100, 110, 120, 130, 140 and 150 -- but this doesn't really matter, it matters that there were no other changes to that file in the meantime, he's been the only developer. Now I've been asked how to create a review for that. Normally, "svn diff -r 100:150 file.txt" would generate a proper diff, however post-review doesn't support filenames for revision ranges. So I looked into documentation and notices -- repository-url parameter. Since in SVN every URL inside a repository may be checked out independently, I tried to use http://svn.server/svn/path/to/file.txt as repository URL. "post-review -d -n --revision-range=100:150 --repository- url=http://svn.server/svn/path/to/file.txt"; ran without problems and has produced proper diff for that single file for revisions 100 to 150. It's only when post- review tried to submit that diff to Review Board that problems began. post-review replaced the "Info:" field in the diff with "http://svn.server/svn/path/to/file.txt/file.txt"; (previously just "file.txt") assuming that filename in the diff should be attached to the base path given as -- repository-url. This is not really correct -- if the URL points to a file, that filename should be removed from the path between diffing and uploading to Review Board. In that case, filename gets attached properly and RB accepts the diff. That said, I fully understand that I might be bending RB's strain features quite a bit and would prefer doing this all properly. For that, I'd need a way to review changes to single files over several commits -- again, I know developer discipline is probably the Right Way (tm) to go, but my fellow developers don't even grasp the necessity for proper commit messages, not to mention proper atomic bundling of changes. I hope this explanation made it a bit clearer to you. -- 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 1378 in reviewboard: RBTools: More friendly handling of error when diff utility not in path
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1378 by p...@swooby.com: RBTools: More friendly handling of error when diff utility not in path http://code.google.com/p/reviewboard/issues/detail?id=1378 *NOTE: Do not post confidential information in this bug report.* Opening this bug per Christian Hammond. What version are you running? RBTools 0.2beta2 What's the URL of the page containing the problem? n/a What steps will reproduce the problem? 1. Don't have diff[.exe] in the path 2. Run a post-review What is the expected output? More friendly error reporting of the missing diff utility. It should be pretty easy to detect "svn: Can't start process 'diff': The system cannot find the file specified." and highlight that in the output (and ideally a quick "Check your PATH" hint). What do you see instead? >>> svn info >>> repository info: Path: svn://devsrvsea01/svnroot, Base path: /trunk/StopProcess, Supports changesets: False >>> svn propget reviewboard:url C:\src\trunk\StopProcess >>> svn propget reviewboard:url C:\src\trunk\ >>> svn diff --diff-cmd=diff -r 24506 Failed to execute command: ['svn', 'diff', '--diff-cmd=diff', '-r', '24506'] ['Index: StopProcess/obj/Debug/StopProcess.exe\n', '=== \n', 'Cannot display: file marked as a binary type.\n', 'svn:mime-type = application/octet-stream\n', 'Index: StopProcess/obj/Debug/ StopProcess.pdb\n', '=== \n', 'Cannot display: file marked as a binary type.\n', 'svn:mime-type = application/octet-stream\n', 'Index: StopProcess/obj/Debug/StopProcess.csproj.FileListAbsolute.txt\n', ' ===\n', "svn: Can't start process 'diff': The system cannot find the file specified. \n"] What operating system are you using? What browser? n/a Please provide any additional information below. n/a -- 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 1374 in reviewboard: Installation/configuration issue
Updates: Status: Invalid Comment #3 on issue 1374 by chipx86: Installation/configuration issue http://code.google.com/p/reviewboard/issues/detail?id=1374 These are support issues, and should be on the mailing list instead of the bug tracker. Can you post to the mailing list (reviewbo...@googlegroups.com) and post your Apache configuration file? -- 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 1376 in reviewboard: post-review: --repository-url pointing to a file doesn't work
Updates: Status: NeedInfo Comment #1 on issue 1376 by chipx86: post-review: --repository-url pointing to a file doesn't work http://code.google.com/p/reviewboard/issues/detail?id=1376 I don't understand. There's no way we could support passing a filename to --repository-url, because that doesn't make sense. --repository-url must point to the head of a repository, so that we can look it up on the server. You should be able to pass that to post-review and then specify the exact files you need as parameters, but I suppose in this case you're wanting to reference files that are on a remote SVN server, instead of locally? -- 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 1376 in reviewboard: post-review: --repository-url pointing to a file doesn't work
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1376 by niko...@prokoschenko.de: post-review: --repository-url pointing to a file doesn't work http://code.google.com/p/reviewboard/issues/detail?id=1376 What version are you running? 1.1a2 What steps will reproduce the problem? 1. Try to create a review with --repository- url=http://server/svn/path/to/file.txt 2. post-review will try to upload a diff with http://server/svn/path/to/file.txt/file.txt as filename, which wouldn't be found 3. --repository-url=http://server/svn/path/to/ performs ok, but possibly includes more files than needed. What is the expected output? What do you see instead? post-review should create a post-commit review for this one file Please provide any additional information below. Would be really nice if diffing single files would be possible without the need for the exact URL! -- 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 1374 in reviewboard: Installation/configuration issue
Comment #2 on issue 1374 by pmuralidharan: Installation/configuration issue http://code.google.com/p/reviewboard/issues/detail?id=1374 Now i got the page to view the following contents. But no one is accessible. I mean if i click Register link then the URL goes to rft-test/account/register but there is no change in my page. And i try to find any folders which named account, groups, users, account/register and account/login under the root dir of the created site. but it is not available. * Log in * - Register * - Documentation * - Bugs * - Report bug Review Board 1.0.5.1 * All review requests * Groups * Submitters The page you were looking for does not exist. If you're pretty sure this page does exist, try logging in and trying again. Thanks Murali -- 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 1374 in reviewboard: Installation/configuration issue
Comment #1 on issue 1374 by pmuralidharan: Installation/configuration issue http://code.google.com/p/reviewboard/issues/detail?id=1374 Now i got the page to view the following contents. But no one is accessible. I mean if i click Register URL goes to rft-test/account/register but there is no change in my page. And i found any folder which named account under the created site. but is not available. * Log in * - Register * - Documentation * - Bugs * - Report bug Review Board 1.0.5.1 * All review requests * Groups * Submitters The page you were looking for does not exist. If you're pretty sure this page does exist, try logging in and trying again. Thanks Murali -- 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 1375 in reviewboard: It does not support main\0 and main\1 diff
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1375 by bianjianquan: It does not support main\0 and main\1 diff http://code.google.com/p/reviewboard/issues/detail?id=1375 *NOTE: Do not post confidential information in this bug report.* What version are you running? review board 1.0.1 What's the URL of the page containing the problem? What steps will reproduce the problem? 1.add a new file to ClearCase. 2.there is only main\0 and main\1 version in ClearCase. get main\0 is empty file 3.use post-review to add request.WindowsError: 4.[Error 3] The system cannot find the path specified: BackUpResultTask.java@@\\@@\\main\\0' 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 1370 in reviewboard: Configuration/installation issue
Updates: Status: NeedInfo Labels: -Type-Defect Type-Support Comment #1 on issue 1370 by chipx86: Configuration/installation issue http://code.google.com/p/reviewboard/issues/detail?id=1370 I'm not seeing enough information to describe what's going on or to figure out why. This happens when going directly to the site as well? Can you describe how you installed Review Board? Is this Apache, or lighttpd? mod_python or fastcgi? Is this an installation into a dedicated subdomain, or an installation into a subdirectory of an existing domain? -- 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 1371 in reviewboard: rb-site install issue on windows: unable to open database file
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1371 by ruijin.ma: rb-site install issue on windows: unable to open database file http://code.google.com/p/reviewboard/issues/detail?id=1371 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.1alpha1 What's the URL of the page containing the problem? N/A What steps will reproduce the problem? 1. Install Python2.5, apache2.2, mod_python, patch, PIL, Crypto 2. download and easy_install ReviewBoard-1.1alpha1-py2.5.egg 3. "rb-site install reviews" in c:\www What is the expected output? What do you see instead? What operating system are you using? What browser? WinXP Please provide any additional information below. Hi, I'm setting up review board on a winxp virtual machine with VirtualBox. But I was always meeting the following error, either with the stable release 1.0.5 or 1.1alpha1. I failed to find help by google. So, I have to ask help here. The following is the message during rb-site install: This will prepare a Review Board site installation in: C:\www\reviews We need to know a few things before we can prepare your site for installation. This will only take a few minutes. * Make sure you have the modules you need. Depending on your installation, you may need certain Python modules and servers that are missing. If you need support for any of the following, you will need to install the necessary modules and restart the install. Databases (optional): * MySQL (MySQLdb) * PostgreSQL (psycopg2) * What's the host name for this site? This should be the fully-qualified host name without the http://, port or path. Domain Name: reviews * What URL path points to Review Board? Typically, Review Board exists at the root of a URL. For example, http://reviews.example.com/. In this case, you would specify "/". However, if you want to listen to, say, http://example.com/reviews/, you can specify "/reviews/". Note that this is the path relative to the domain and should not include the domain name. The default is / Root Path [/]: * What URL will point to the media files? While most installations distribute media files on the same server as the rest of Review Board, some custom installs may instead have a separate server for this purpose. The default is media/ Media URL [media/]: * What database type will you be using? You can type either the name or the number from the list below. (1) sqlite3 Database Type: 1 * Determining database file path. The sqlite database file will be stored in C:\www\reviews\db\reviewboard.db If you are migrating from an existing installation, you can move your existing database there, or edit settings_local.py to point to your old location. * What cache mechanism should be used? memcached is strongly recommended. Use it unless you have a good reason not to. You can type either the name or the number from the list below. (1) memcached (2) file Cache Type: 1 * What memcached connection string should be used? This is generally in the format of memcached://hostname:port/ The default is memcached://localhost:11211/ Memcache Server [memcached://localhost:11211/]: * What web server will you be using? You can type either the name or the number from the list below. (1) apache (2) lighttpd Web Server: 1 * What Python loader module will you be using? Based on our experiences, we recommend using modpython with Review Board. You can type either the name or the number from the list below. (1) modpython (2) fastcgi Python Loader: 1 * Create an administrator account To configure Review Board, you'll need an administrator account. It is advised to have one administrator and then use that account to grant administrator permissions to your personal user account. If you plan to use NIS or LDAP, use an account name other than your NIS/LDAP account so as to prevent conflicts. The default is admin Username [admin]: Password: [!] You must answer this question. Password: E-Mail Address: xxx * Installing the site... Building site directories ... OK Building site configuration files ... OK Creating database ... Creating table django_admin_log Creating table auth_permission Creating table auth_group Creating table auth_user Creating table auth_message Creating table django_content_type Creating table django_site Creating table django_session Creating table siteconfig_siteconfiguration Creating table accounts_reviewrequestvisit Creating table accounts_profile Creating table changedescs_changedescription Creating table diffviewer_filediff Creating table diffviewer_diffset Creating table diffviewer_diffsethistory Creating table reviews_group Creating table reviews_defaultreview
Issue 1370 in reviewboard: Configuration/installation issue
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1370 by pmuralidharan: Configuration/installation issue http://code.google.com/p/reviewboard/issues/detail?id=1370 *NOTE: Do not post confidential information in this bug report.* What version are you running? Review Board 1.0.5.1 What's the URL of the page containing the problem? The page you were looking for does not exist. If you're pretty sure this page does exist, try logging in and trying again. What operating system are you using? What browser? mandriva 2009.1 and firefox3.5 Please provide any additional information below. I couldn't find any pages which refers in link. that is i couldn't find account page, register page or some other. I just installed without any errors. But i couldn't find any pages to configure review form. kindly suggest me. thanks, Murali -- 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 1145 in reviewboard: Description textarea grows 1 line with every character typed while editing it in Chromium for Linux
Comment #17 on issue 1145 by sanfordarmstrong: Description textarea grows 1 line with every character typed while editing it in Chromium for Linux http://code.google.com/p/reviewboard/issues/detail?id=1145 This does not appear to be fixed in 1.0.5.1. Is there any interest in applying this fix for 1.0.x? -- 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 1358 in reviewboard: New review request fails
Comment #1 on issue 1358 by justin.d.teems: New review request fails http://code.google.com/p/reviewboard/issues/detail?id=1358 bump??? -- 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 1367 in reviewboard: Missing double-quotes in reviewboard/cmdline/conf/search-cron.conf.in
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1367 by dmrdavid: Missing double-quotes in reviewboard/cmdline/conf/search-cron.conf.in http://code.google.com/p/reviewboard/issues/detail?id=1367 The second line of search-cron.conf.in is missing a " mark. 0,10,20,30,40,50 * * * * "@rbsite@ manage "@sitedir@" index instead of 0,10,20,30,40,50 * * * * "@rbsite@" manage "@sitedir@" index What version are you running? 1.0.5.1 -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Updates: Status: New Comment #13 on issue 1348 by chipx86: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 Things definitely did change between those versions, and I verified that it was broken. However, I'm no longer able to reproduce this at all since my fix. I'll attempt to reproduce this some more. Perhaps there's another broken case that was missed. -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Comment #12 on issue 1348 by premek.paska: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 Yes, I restarted both apache and memcached (and tried also disabling memcached), reloaded the page multiple times, cleared the browser cache and tested it also in Chromium's incognito mode.. ..just to be sure. And I think there is not a problem with any cache, because it works immediately after downgrading to 1.0.1. What about checking changes in the diff viewer between 1.0.1 and 1.0.3? -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Comment #11 on issue 1348 by chipx86: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 Did you clear both the server-side and client-side caches? -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Comment #10 on issue 1348 by premek.paska: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 Unfortunately, I can't confirm the fix. The issue is still present in version 1.0.5 in our installation (tested with Chromium 4.0.222.4, Firefox 3 and 3.5; all in Linux). The link "Expand" behaves still the same way as in 1.0.4. It is sending wrong requests as described in comments #2 and #5 above. I don't see our configuration obscure in any way; it is the following: Response Headers DateFri, 23 Oct 2009 22:31:23 GMT Server Apache/2.2.9 (Debian) DAV/2 SVN/1.5.1 mod_jk/1.2.26 PHP/5.2.6-1+lenny3 with Suhosin-Patch mod_python/3.3.1 Python/2.5.2 mod_ssl/2.2.9 OpenSSL/0.9.8g mod_perl/2.0.4 Perl/v5.10.0 Content-Length 232 Content-Languageen-us Content-Encodinggzip VaryCookie,Accept-Language,Accept-Encoding Content-Typetext/html; charset=utf-8 Keep-Alive timeout=15, max=100 Connection Keep-Alive Request Headers Host... User-Agent Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3 Accept text/html, */* Accept-Language en,en-us;q=0.8,fr;q=0.5,fr-fr;q=0.3 Accept-Encoding gzip,deflate Accept-Charset UTF-8,* Keep-Alive 300 Connection keep-alive X-Requested-WithXMLHttpRequest Referer https://.../rb/r/171/diff/ Cookie rbsessionid=ae1bc5f72f8e0af9e9f8de5d857676be; ... -- 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 1366 in reviewboard: Group "netster" is not recognized by review board
Updates: Status: Invalid Comment #1 on issue 1366 by chipx86: Group "netster" is not recognized by review board http://code.google.com/p/reviewboard/issues/detail?id=1366 This is the issue tracker for bugs in Review Board, which is an open source project used by many companies. You should contact your system administrator. -- 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 1366 in reviewboard: Group "netster" is not recognized by review board
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1366 by piyush18: Group "netster" is not recognized by review board http://code.google.com/p/reviewboard/issues/detail?id=1366 *NOTE: Do not post confidential information in this bug report.* What version are you running? 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 1364 in reviewboard: submodules no worky
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1364 by anutron: submodules no worky http://code.google.com/p/reviewboard/issues/detail?id=1364 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.04 What's the URL of the page containing the problem? r/new/ What steps will reproduce the problem? 1. create a diff using git in a repository w/ submodules. update a submodule, so that the pointer change is included in the diff. 2. upload the diff 3. breakage What is the expected output? What do you see instead? the diff should be accepted. instead it complains about files not being located in the repository (the submodule pointer). -- 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 1363 in reviewboard: API: /api/json/reviewrequests//reviews/ -- Non-existent reviews return 404 not found
Updates: Status: NotABug Owner: chipx86 Labels: Component-API Comment #1 on issue 1363 by chipx86: API: /api/json/reviewrequests//reviews/ -- Non-existent reviews return 404 not found http://code.google.com/p/reviewboard/issues/detail?id=1363 This is how a proper REST API should behave. In fact, we should be doing this more than we do. 404 means "not found" and in this case, that's exactly the correct response. The idea of REST is that you make use of HTTP's built-in methods and responses as much as possible, since they're universally supported. If you want to get something without any side-effects on the server, you use a GET. If you want to post something, you use a POST. And so on. Likewise, built-in HTTP statuses should be used, since clients all understand them. Now, we don't do nearly as good a job as we should do. We don't use PUT, DELETE, etc., and we don't always use the response codes 100% correctly, but in this case, it is correct. Maybe down the road we could supply a JSON payload along with the 404 (as most error codes can also return page content). This is a nice to have, but won't happen for 1.1. -- 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 1363 in reviewboard: API: /api/json/reviewrequests//reviews/ -- Non-existent reviews return 404 not found
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1363 by cybrocop: API: /api/json/reviewrequests//reviews/ -- Non-existent reviews return 404 not found http://code.google.com/p/reviewboard/issues/detail?id=1363 *NOTE: Do not post confidential information in this bug report.* What version are you running? ReviewBoard-1.1alpha1.dev_r2026 What's the URL of the page containing the problem? http://Reviewboard/api/json/reviewrequests// What steps will reproduce the problem? 1. Create a new review (i.e. Review ID: 123) 2. Delete the review completely. 3. Request the review through the API What is the expected output? What do you see instead? I get a 404 Not Found error. If the review looks like a valid review ID, then instead of 404 I should get a JSON formatted error indicating that the review id was not found. The 404 exception doesn't tell me if the review was not found. It simply tells me there was some server error. Maybe RB is mis-configured. Maybe I'm querying the wrong server. Maybe you will disagree that this is a defect, but I thought I should mention it anyway. The workaround I'm using now is catching 404 exceptions and *assuming* that this means the requested review doesn't exist. What operating system are you using? What browser? N/A 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 1355 in reviewboard: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows
Updates: Status: Fixed Owner: chipx86 Labels: -Milestone-Release1.1 Milestone-Release1.0.x Comment #2 on issue 1355 by chipx86: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows http://code.google.com/p/reviewboard/issues/detail?id=1355 Fixed on release-1.0.x (rd2b18cf) and master. -- 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 1361 in reviewboard: When space is added before a username on the People field I got Error 500
Updates: Status: Duplicate Mergedinto: 728 Comment #1 on issue 1361 by chipx86: When space is added before a username on the People field I got Error 500 http://code.google.com/p/reviewboard/issues/detail?id=1361 1.0 RC2 is pretty ancient. This has since been fixed. -- 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 728 in reviewboard: Whitespace not stripped on username
Comment #3 on issue 728 by chipx86: Whitespace not stripped on username http://code.google.com/p/reviewboard/issues/detail?id=728 Issue 1361 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 1361 in reviewboard: When space is added before a username on the People field I got Error 500
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1361 by frank.x.liu: When space is added before a username on the People field I got Error 500 http://code.google.com/p/reviewboard/issues/detail?id=1361 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.0 RC2 What's the URL of the page containing the problem? Review details What steps will reproduce the problem? 1. put a reviewer's name 2. add space before the name 3. fill in summary and descript 4. publish What is the expected output? What do you see instead? The review details should be saved What operating system are you using? What browser? ubuntu, Firefox Please provide any additional information below. We use LDap authentication What may be happening is the the user with space before it is not found in the LDAP authentication. -- 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 1162 in reviewboard: post-review ignores all but first target file when using --revision-changes
Comment #2 on issue 1162 by cpforbes: post-review ignores all but first target file when using --revision-changes http://code.google.com/p/reviewboard/issues/detail?id=1162 The case where --repository-url=url is not specified is not handled. the do_diff call on line 1137 also needs the files added to the command. -- 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 1196 in reviewboard: Statistics are not available for this backend
Comment #12 on issue 1196 by markbroehm: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 So i tried changing the siteconfig_siteconfiguration table as follows and it works. sweet. OLD: "cache_backend": "locmem:///" NEW: "cache_backend": "memcached://127.0.0.1:11211/" results (after restarting apache and memcached): Server Cache Cache backend: django.core.cache.backends.memcached Statistics 127.0.0.1:11211 Memory usage: 2.9 KB Keys in cache: 1 of 1 Cache hits: 0 of 4: 0% Cache misses: 4 of 4: 100% Cache evictions: 0 Cache traffic: 3.2 KB in, 759 bytes out Uptime: 41 -- 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 1196 in reviewboard: Statistics are not available for this backend
Comment #11 on issue 1196 by markbroehm: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 from the above posts by lonico, i see the siteconfig_siteconfiguration table has "cache_backend": "locmem:///" - is that the problem with my setup? -- 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 1196 in reviewboard: Statistics are not available for this backend
Comment #10 on issue 1196 by markbroehm: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 Realized that i have both installed: net-misc/memcached-1.4.1 dev-python/python-memcached-1.40 dev-lang/python-2.5.4-r2 reviewboard ~ # telnet localhost 11211 Trying 127.0.0.1... Connected to localhost. Escape character is '^]'. stats STAT pid 2263 STAT uptime 1206023 STAT time 1256047646 STAT version 1.4.1 STAT pointer_size 32 STAT rusage_user 6.524407 STAT rusage_system 17.825114 STAT curr_connections 5 STAT total_connections 11595 STAT connection_structures 8 STAT cmd_get 14386 STAT cmd_set 1217 STAT cmd_flush 0 STAT get_hits 2104 STAT get_misses 12282 STAT delete_misses 0 STAT delete_hits 0 STAT incr_misses 0 STAT incr_hits 0 STAT decr_misses 0 STAT decr_hits 0 STAT cas_misses 0 STAT cas_hits 0 STAT cas_badval 0 STAT bytes_read 35665771 STAT bytes_written 65854408 STAT limit_maxbytes 134217728 STAT accepting_conns 1 STAT listen_disabled_num 0 STAT threads 5 STAT conn_yields 0 STAT bytes 34614404 STAT curr_items 1213 STAT total_items 1217 STAT evictions 0 END -- 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 654 in reviewboard: Allow user to reorder the files being diffed
Comment #3 on issue 654 by cybrocop: Allow user to reorder the files being diffed http://code.google.com/p/reviewboard/issues/detail?id=654 Sorting based on file status would be interesting. I personally, would like to see deleted files first. This is because if the author has moved many files around or done re-factoring as a result of which files no longer exist, it would be preferable to see those files first. Then follow the trail and see where the code went. -- 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 1360 in reviewboard: Can not see diff when the diff was uploaded with parent option on post-review
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1360 by TAKAHASHI.Kazuyoshi: Can not see diff when the diff was
uploaded with parent option on post-review
http://code.google.com/p/reviewboard/issues/detail?id=1360
What version are you running?
ReviewBoard 1.0.4, RBTools Beta2
What's the URL of the page containing the problem?
r/xxx/diff/#index_header
What steps will reproduce the problem?
1. "post-review --parent=topicA --summary=testA" on topicB (branch of
topicA).
2. goto the page above.
What is the expected output? What do you see instead?
expected: could see diff.
got:
Traceback (most recent call last):
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/views.py", line 153, in view_diff
interdiffset, highlighting, True)
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py", line 623, in get_diff_files
large_data=True)
File
"/usr/lib/python2.4/site-packages/Djblets-0.5.4-py2.4.egg/djblets/util/misc.py",
line
162, in cache_memoize
data = lookup_callable()
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py", line 622, in
enable_syntax_highlighting),
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py", line 345, in get_chunks
new = get_patched_file(old, filediff)
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py", line 261, in
get_patched_file
return patch(filediff.diff, buffer, filediff.dest_file)
File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py", line 126, in patch
raise Exception(_("The patch to '%s' didn't apply cleanly. The
temporary " +
Exception: The patch to 'adm/Rakefile' didn't apply cleanly. The temporary
files have been left in
'/tmp/reviewboard.xXkaH8' for debugging purposes.
`patch` returned: patching file /tmp/reviewboard.xXkaH8/tmpydvPxC
Reversed (or previously applied) patch detected! Assume -R? [n]
Apply anyway? [n]
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file
/tmp/reviewboard.xXkaH8/tmpydvPxC-new.rej
What operating system are you using? What browser?
OS: Cent OS 5.2
Browser: Safari 4.0.3
Please provide any additional information below.
I've added "-t" option at diffutils.py as below. and it works.
Here it is...
--- /usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils-old.py 2009-10-20
19:08:10.0 +0900
+++ /usr/lib/python2.4/site-packages/ReviewBoard-1.0.4-
py2.4.egg/reviewboard/diffviewer/diffutils.py 2009-10-20 19:08:27.0
+0900
@@ -104,7 +104,7 @@
# XXX: catch exception if Popen fails?
newfile = '%s-new' % oldfile
-p = subprocess.Popen(['patch', '-o', newfile, oldfile],
+p = subprocess.Popen(['patch', '-t', '-o', newfile, oldfile],
stdin=subprocess.PIPE, stdout=subprocess.PIPE,
stderr=subprocess.STDOUT)
p.stdin.write(diff)
--
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 1196 in reviewboard: Statistics are not available for this backend
Comment #9 on issue 1196 by lonico: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 I found a discrepancy between settings_local.py and the siteconfig_siteconfiguration table in MySQL. The table had the file:///tmp/reviewboard_cache entry, but it did not prevent memcached from being in use. I believed the discrepancy is because I first installed Review Board without memcached, and then reinstalled it with memcached. I did not delete the tables, and rb-site does not check for that. I had an error because admin was already defined, but I bypassed the issue by using admin2. After fixing the table, I can see the cache statistics :) -- 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 1196 in reviewboard: Statistics are not available for this backend
Comment #8 on issue 1196 by lonico: Statistics are not available for this
backend
http://code.google.com/p/reviewboard/issues/detail?id=1196
I agree it's a different issue, so maybe it's not the right place to
discuss it
I see it too with 1.4. I added a variable in reviewboard/settings.py
# Load local settings. This can override anything in here, but at the very
# least it needs to define database connectivity.
try:
import settings_local
from settings_local import *
except ImportError:
dependency_error('Unable to read settings_local.py.')
LN_CACHE_BACKEND = CACHE_BACKEND
TEMPLATE_DEBUG = DEBUG
And when I try to print it in reviewboard/admin/views.py
logging.debug(settings.CACHE_BACKEND)
logging.debug("=")
logging.debug(settings.LN_CACHE_BACKEND)
This is what I got:
2009-10-19 16:59:24,116 - DEBUG - file:///tmp/reviewboard_cache
2009-10-19 16:59:24,116 - DEBUG - =
2009-10-19 16:59:24,116 - DEBUG - memcached://127.0.0.1:11211/
So clearly settings.CACHE_BACKEND has been modifed in the meantime, but I
cannot find
where.
--
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 1359 in reviewboard: Diff fails for text file with non-ascii characters
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1359 by pinge...@yahoo.com: Diff fails for text file with non-ascii characters http://code.google.com/p/reviewboard/issues/detail?id=1359 What version are you running? Review board 1.0 on Python 2.4 What's the URL of the page containing the problem? http:///reviews/r/658/diff/#index_header What steps will reproduce the problem? 1. Create a text file containing the byte 0xED (in Windows, Alt+0237) This character corresponds to the "latin small letter i with acute" in the Windows Western encoding. 2. Check this into source control 3. Edit the file to replace this character with a lowercase 'i' 4. Post the change to review board 5. Attempt to view the diff (This actually happened to me today -- I was trying to fix a source file that had this non-ascii, non utf-8 character in a docstring) What is the expected output? What do you see instead? I would expect to see a diff, possibly with invalid characters replaced by hexadecimal representations. Instead I get the following traceback: 'ascii' codec can't decode byte 0xc3 in position 26: ordinal not in range(128) Traceback (most recent call last): File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0-py2.4.egg/reviewboard/diffviewer/views.py", line 152, in view_diff interdiffset, highlighting, True) File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0-py2.4.egg/reviewboard/diffviewer/diffutils.py", line 623, in get_diff_files large_data=True) File "/usr/lib/python2.4/site-packages/Djblets-0.5-py2.4.egg/djblets/util/misc.py", line 143, in cache_memoize data = lookup_callable() File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0-py2.4.egg/reviewboard/diffviewer/diffutils.py", line 622, in enable_syntax_highlighting), File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0-py2.4.egg/reviewboard/diffviewer/diffutils.py", line 434, in get_chunks a[i1:i2], b[j1:j2], oldlines, newlines) File "/usr/lib/python2.4/site-packages/ReviewBoard-1.0-py2.4.egg/reviewboard/diffviewer/diffutils.py", line 268, in diff_line if oldline and newline and oldline != newline: UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 26: ordinal not in range(128) What operating system are you using? What browser? Firefox 3.5.3 on Windows XP SP3 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 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #28 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 I'm still trying to find solution for the issue, unfortunately... I found another webapp(in python), which is similar to Review Board in dealing with access to SVN repository, and it solved the problem about the https authentication support. Attached source code here for further reference. ref: http://transifex.org/browser/transifex/vcs/lib/types/svn.py?rev=946% 3Ae2782e83e8dd Wish it be useful to solved the issue. Attachments: svn.py 7.0 KB -- 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 1125 in reviewboard: server port number is missing in review request email
Comment #7 on issue 1125 by schettino72: server port number is missing in review request email http://code.google.com/p/reviewboard/issues/detail?id=1125 works for me :) thanks a lot! -- 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 1072 in reviewboard: HTTPS Subversion repositories don't work with Review Board
Comment #27 on issue 1072 by yunshan.di: HTTPS Subversion repositories don't work with Review Board http://code.google.com/p/reviewboard/issues/detail?id=1072 I'm still trying to find solution for the issue, unfortunately... I found another webapp(in python), which is similar to Review Board, and it solved the problem about the https authentication support. attached here for reference. ref: http://transifex.org/browser/transifex/vcs/lib/types/svn.py?rev=946% 3Ae2782e83e8dd Attachments: svn.py 7.0 KB -- 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 783 in reviewboard: Could a preference be added to disable animation?
Comment #7 on issue 783 by jamesdlin: Could a preference be added to disable animation? http://code.google.com/p/reviewboard/issues/detail?id=783 I'm using Firefox. A Stylish or Greasemonkey solution would be fine with me. -- 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 242 in reviewboard: OpenID authentication support
Updates: Owner: trowbrds Comment #7 on issue 242 by chipx86: OpenID authentication support http://code.google.com/p/reviewboard/issues/detail?id=242 (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 1299 in reviewboard: Link back to dashboard from admin page
Updates: Labels: -Milestone-Release1.1 Milestone-Release1.5 Comment #2 on issue 1299 by chipx86: Link back to dashboard from admin page http://code.google.com/p/reviewboard/issues/detail?id=1299 I want to rethink the admin UI, and this won't happen for 1.1. There's no place in the current admin UI header area that I really like for this that's obvious. -- 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 783 in reviewboard: Could a preference be added to disable animation?
Updates: Status: NeedInfo Comment #6 on issue 783 by chipx86: Could a preference be added to disable animation? http://code.google.com/p/reviewboard/issues/detail?id=783 James, remind me what browser you're using? Due to server and client-side caching implications, I would much much rather this be implemented as a Stylish or Greasemonkey script, which obviously wouldn't work for IE, but would for Firefox. If we implement this in Review Board, it means changing pretty much all the caching logic to handle this one preference, which 99% of users will probably never need to be honest :/ Doing it browser-side means that we don't have to change all this code and only those users who really need this disabled could disable it. -- 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 489 in reviewboard: [ ] DON'T SHIP IT button
Updates: Labels: -Milestone-Release1.1 Milestone-Release1.5 Comment #16 on issue 489 by chipx86: [ ] DON'T SHIP IT button http://code.google.com/p/reviewboard/issues/detail?id=489 A lot of discussion and design needs to take place regarding this and the proposals for comment types (which likely will not be happening as part of the core Review Board product, but that's another discussion). I don't feel comfortable making this part of the 1.1 plans. Moving back to 1.5. -- 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 1346 in reviewboard: Unable to view/delete unnamed discarded review request
Updates: Status: Fixed Owner: chipx86 Labels: Component-Admin Milestone-Release1.0.x Comment #3 on issue 1346 by chipx86: Unable to view/delete unnamed discarded review request http://code.google.com/p/reviewboard/issues/detail?id=1346 Fixed on master and release-1.0.x (r2947b00) -- 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 1125 in reviewboard: server port number is missing in review request email
Updates: Status: NeedInfo Comment #6 on issue 1125 by chipx86: server port number is missing in review request email http://code.google.com/p/reviewboard/issues/detail?id=1125 Can you guys try this with 1.0.4? This should be fixed now. We loosened restrictions on the server field, and I'm able to specify a port fine. -- 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 1356 in reviewboard: error while updating diff on review board
Updates: Status: WontFix Labels: VMware Comment #1 on issue 1356 by chipx86: error while updating diff on review board http://code.google.com/p/reviewboard/issues/detail?id=1356 Please file a helpzilla ticket at work. The version at VMware is pretty old now, there's not enough information in this report for us to determine anything, and any information you would have that we'd need probably couldn't be posted here (such as the diff). -- 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 488 in reviewboard: Instant messaging integration
Comment #4 on issue 488 by trowbrds: Instant messaging integration http://code.google.com/p/reviewboard/issues/detail?id=488 Issue 1357 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 1357 in reviewboard: XMPP chat notification of request changes
Updates: Status: Duplicate Mergedinto: 488 Comment #1 on issue 1357 by trowbrds: XMPP chat notification of request changes http://code.google.com/p/reviewboard/issues/detail?id=1357 (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 1357 in reviewboard: XMPP chat notification of request changes
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 1357 by gic...@gmail.com: XMPP chat notification of request changes http://code.google.com/p/reviewboard/issues/detail?id=1357 Is there a general notification API one can implement to observe changes in review requests? It would be cool if an RB bot could be created to hang out on an XMPP chat and provide updates as relevant review requests change, spit out information on reviews when they are mentioned or when asked, etc. Idea could also be extended to IRC or to simple "unicast" notifications for any IM network. -- 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 1277 in reviewboard: Comment Types
Comment #4 on issue 1277 by vi...@djgraphite.com: Comment Types http://code.google.com/p/reviewboard/issues/detail?id=1277 I think that while this is a good idea in spirit, I'm not sure it would work in practice. I don't think severity is useful in the case of code reviews in the same way that severity isn't all that useful in the case of bugs (wait for the explanation). In the case of bugs, in most cases, there are a few people who have a the big picture of the product and set the relative severity of the bugs. In this case however, it sounds like the commenter would set the priority. That's probably not a good way to go. I do like the idea of having "comment types" - a defect checkbox, or a checkbox that additionally says that security might be a concern. This classifies them without priority. I think it's up to the developer to read _all_ of the comments and then prioritize and address them. -- 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 1277 in reviewboard: Comment Types
Comment #3 on issue 1277 by cybrocop: Comment Types http://code.google.com/p/reviewboard/issues/detail?id=1277 Here is a use case for Severity. It is Thursday, 5:00 pm. You must submit your code by Friday according to the project plan so that QA can test it at least 1 full day and hopefully there are no problems, so that a release can be done on Monday. You submit a 2000-line changeset. It gets reviewed by 5 people and overall you have 30 comments to deal with. It is clear that by Monday, you will not be able to deal with all 20. How do you know which one to focus on. Some issues may be minor, some may be major. All of them are "defects". But you need to triage and find the most pressing ones. The defects need to be categorized and labeled. If they had some visible label, then I would scroll down through the review and focus on the 10 most critical defects. The other defects, would get reported to the management and if the management accepts the risk, then we could add a "//FIXME:" tag wherever there is an issue in the code and commit it. The defect type and severity should be prominently visible in the review screen. -- 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 1355 in reviewboard: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows
Updates: Status: Confirmed Labels: Milestone-Release1.1 Component-Reviews EasyFix Comment #1 on issue 1355 by chipx86: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows http://code.google.com/p/reviewboard/issues/detail?id=1355 (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 733 in reviewboard: Review Board as a VMWare Virtual Appliance
Comment #4 on issue 733 by gic...@gmail.com: Review Board as a VMWare Virtual Appliance http://code.google.com/p/reviewboard/issues/detail?id=733 Are you concerned about hosting the built VM or about a place to build the VM? -- 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 1355 in reviewboard: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1355 by hovigpda: Text Box Label and Size Issues for "New Review Request" and "Update Diff" windows http://code.google.com/p/reviewboard/issues/detail?id=1355 *NOTE: Do not post confidential information in this bug report.* What version are you running? ReviewBoard-1.1alpha1.dev_r2026 What's the URL of the page containing the problem? N/A What steps will reproduce the problem? N/A What is the expected output? What do you see instead? There are several cosmetic issues. 1. When creating a new review request: a) "Base Diff Path" textbox is only 20 chars wide, whereas the rest of the line is empty. b) "Diff" textbox is only 20 chars wide, whereas the rest of the line is empty. 2. When updating a diff on an existing review Same issues as above. 3. In one screen the label says "Base directory" while in another it says "Base Diff Path". The naming should be the same and all text box labels should follow the capitalization style. What operating system are you using? What browser? Firefox 3.5.3 on Windows XP Please provide any additional information below. Screen resolution: 1280 x 1024 Attachments: UploadDiff.jpg 28.1 KB NewReviewRequest.JPG 13.6 KB -- 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 1354 in reviewboard: Google chrome text editing fields expand
Comment #3 on issue 1354 by chipx86: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 It should say the version next to the "Review Board" text on the top of the page. If it says "Beta" next to it, then it's pre-1.0, in which case it's too old to have the fix for this. -- 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 1177 in reviewboard: Search Indexing Issue
Updates: Status: Fixed Owner: chipx86 Comment #1 on issue 1177 by chipx86: Search Indexing Issue http://code.google.com/p/reviewboard/issues/detail?id=1177 There were bugs with the crontab generation before. It's now fixed. I've also added new docs (which will be available in tomorrow's update) for how to enable search. It's mostly information found in other parts of the docs, but consolidated. -- 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 1353 in reviewboard: Change selection frame sticks to screen position when expanding unchanged lines
Updates: Status: Fixed Owner: chipx86 Labels: Component-DiffViewer Milestone-Release1.0.x Comment #1 on issue 1353 by chipx86: Change selection frame sticks to screen position when expanding unchanged lines http://code.google.com/p/reviewboard/issues/detail?id=1353 Fixed on master and release-1.0.x (rddfc9bd) -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Updates: Status: Fixed Comment #9 on issue 1348 by chipx86: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 Fixed on release-1.0.x and master (r331a1bf). -- 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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Updates: Status: Confirmed Owner: chipx86 Comment #8 on issue 1348 by chipx86: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 (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 1348 in reviewboard: Expanding hidden lines broken by the 1.0.3 update
Updates: Labels: -Priority-Medium Priority-High Milestone-Release1.0.x Component-DiffViewer Comment #7 on issue 1348 by chipx86: Expanding hidden lines broken by the 1.0.3 update http://code.google.com/p/reviewboard/issues/detail?id=1348 (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 1196 in reviewboard: Statistics are not available for this backend
Updates: Status: Fixed Comment #7 on issue 1196 by chipx86: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 We can continue discussion on here, though at this point it's an issue with your setup or components. The core bug (failing to match the memcached string) is fixed. -- 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 1173 in reviewboard: Binary files aren't listed in the diff?
Updates: Status: Fixed Owner: chipx86 Comment #5 on issue 1173 by chipx86: Binary files aren't listed in the diff? http://code.google.com/p/reviewboard/issues/detail?id=1173 Fixed in master (reb80a58) and release-1.0.x (reb80a58) -- 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 1354 in reviewboard: Google chrome text editing fields expand
Comment #4 on issue 1354 by christopher.james.lahey: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 Indeed, it says Beta. -- 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 1306 in reviewboard: Chrome: description and testing done boxes expand with every letter typed
Comment #3 on issue 1306 by chipx86: Chrome: description and testing done boxes expand with every letter typed http://code.google.com/p/reviewboard/issues/detail?id=1306 Issue 1354 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 1354 in reviewboard: Google chrome text editing fields expand
Updates: Status: Duplicate Mergedinto: 1306 Comment #5 on issue 1354 by chipx86: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 Okay. So, this was fixed in 1.0.4. You'll need to get the administrator to upgrade. -- 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 1354 in reviewboard: Google chrome text editing fields expand
Comment #2 on issue 1354 by christopher.james.lahey: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 How do I find out what version I'm using? -- 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 1354 in reviewboard: Google chrome text editing fields expand
Updates: Status: NeedInfo Comment #1 on issue 1354 by chipx86: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 What version of Review Board are you using? -- 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 1354 in reviewboard: Google chrome text editing fields expand
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1354 by christopher.james.lahey: Google chrome text editing fields expand http://code.google.com/p/reviewboard/issues/detail?id=1354 *NOTE: Do not post confidential information in this bug report.* What's the URL of the page containing the problem? Any page with a multi line edit box. What steps will reproduce the problem? 1. View a review request under google chrome on linux. 2. Click review 3. Edit your review What is the expected output? What do you see instead? Expected: Box expands to fit typed data. Experienced: Box expands one line with every character typed. What operating system are you using? What browser? Ubuntu 9.04 Chrome 4.0.221.8 -- 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 294 in reviewboard: Set Submitted Should Be Automatic
Comment #13 on issue 294 by christopher.james.lahey: Set Submitted Should Be Automatic http://code.google.com/p/reviewboard/issues/detail?id=294 Just 2 cents, but I don't think we should automatically mark a review as submitted. Occasionally, my process will require that I've submitted a review to perforce, but it's still important that the review be on rb. -- 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 -~--~~~~--~~--~--~---
Comment on IdeasFor2_0 in reviewboard
Comment by katsnelson: At some point the number of reviews becomes quite large. It is difficult to sort them out or find the one of interest. Will it be possible to ad folders to the dashboard? In this case users can sort the reviews int different topics or subprojects. For more information: http://code.google.com/p/reviewboard/wiki/IdeasFor2_0 --~--~-~--~~~---~--~~ 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 733 in reviewboard: Review Board as a VMWare Virtual Appliance
Updates: Labels: -Milestone-Release1.0 Milestone-Release1.1 Comment #3 on issue 733 by chipx86: Review Board as a VMWare Virtual Appliance http://code.google.com/p/reviewboard/issues/detail?id=733 We do want to get to this. Hosting is a concern, as it's going to be costly for us and we have no budget for this. -- 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 1196 in reviewboard: Statistics are not available for this backend
Comment #6 on issue 1196 by chipx86: Statistics are not available for this backend http://code.google.com/p/reviewboard/issues/detail?id=1196 Are you using python-memcached or cmemcached? What version of memcached are you running? What version of Python? What happens if you telnet to localhost:11211 and, at the prompt, type: stats -- 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 1285 in reviewboard: Expanding hidden lines between revisions
Updates: Status: Fixed Owner: chipx86 Comment #2 on issue 1285 by chipx86: Expanding hidden lines between revisions http://code.google.com/p/reviewboard/issues/detail?id=1285 Yeah, this should have been fixed a while ago in 1.0.2. Please upgrade to 1.0.4 and see if it's fixed. -- 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 -~--~~~~--~~--~--~---
