Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Christian Hammond
Hey everyone, We've been accepted again this year in Google's Summer of Code, a yearly event where students from around the world participate in open source projects, developing work experience and getting paid for it. We're expecting about 4 or 5 students this year, but we can grow that number

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Robert Munteanu
Hi, As far as I know a past GSOC session was the one that gave the initial version of the ReviewBoard connector for Eclipse ( ereviewboard ). I have picked up that integration and improved it in terms of - stability - support for the 2.0 REST API - feature additions

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Alexander Solovets
http://groups.google.com/group/reviewboard-gsoc/browse_thread/thread/fc38a36b2f93fa21 -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~--~~~~--~~--~--~---

Re: Linux/Cross-Platform Installer Framework

2011-03-21 Thread Sam Batista
Surendran K: I think that's a brilliant idea, I've spent the past few days trying to get review board to work on either Ubuntu or Windows. The process seems quite involved either way, and for someone not used to set up Apache servers and modules and configuring all the settings, it is a daunting

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Sam Batista
I think setting up some kind of all encompassing setup for this software would great to drive up its user base, which I believe is essential for its continuous improvement, now that it's reached a pretty stable and useful state. On Mar 21, 4:24 pm, Alexander Solovets asolov...@gmail.com wrote:

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Christian Hammond
Agreed. We have a prototype Windows installer, which gets you part of the way there on a standard Windows setup. What's needed at that point is, I think, a graphical front-end to rb-site, like we have on Linux. For Linux (and this carries to Windows as well), I'd like to eventually see the

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Alexander Solovets
Does it mean that you reject ReviewBoard CLI project? If you need more details I can provide it here. I spoke with other RB users and at least one of them said it would be a good idea. Mostly, because console interface makes it possible to add non-web GUI on the top of it. And a GUI in turn having

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Christian Hammond
The installation work is an entirely separate body of work from the post-review proposal. It's not one or the other. However, we tend to get 15+ applications each year, and can only accept generally 3 or 4, so I can't promise anyone at this point that they'll be accepted. We won't be able to make

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Christian Hammond
A bit early, yeah. I have in my head what I'd like to do for it, and I want to make sure whatever student we work with is on the same page. Too many efforts around installation will just complicate things. Christian -- Christian Hammond - chip...@chipx86.com Review Board -

Re: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

2011-03-21 Thread Sam Batista
You sound experienced. Alright, but it's about time it gets done heh? On Mar 21, 7:31 pm, Christian Hammond chip...@chipx86.com wrote: A bit early, yeah. I have in my head what I'd like to do for it, and I want to make sure whatever student we work with is on the same page. Too many efforts

Re: Linux/Cross-Platform Installer Framework

2011-03-21 Thread Mario Đanić
Hi all, you might, or might not know that like a year ago I've started working on solving the problem of a rather complex RB installation procedure by creating a framework, and then the actual installation functionality for Linux on top of it. Why Linux? To this date, RB is easiest to install on

Issue 2038 in reviewboard: Automatic hyperlink regexes are broken

2011-03-21 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2038 by sbress...@gmail.com: Automatic hyperlink regexes are broken http://code.google.com/p/reviewboard/issues/detail?id=2038 * NOTE: Do not post confidential information in this bug report. * * If you need

Re: Issue 1998 in reviewboard: Error when visiting user profile page: [Django] Error (EXTERNAL IP): /users/

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 1998 by trowb...@gmail.com: Error when visiting user profile page: [Django] Error (EXTERNAL IP): /users/ http://code.google.com/p/reviewboard/issues/detail?id=1998 Oops, this page was doing something monumentally silly. Fixed in master

Re: Issue 2038 in reviewboard: Automatic hyperlink regexes are broken

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 2002 Comment #1 on issue 2038 by trowb...@gmail.com: Automatic hyperlink regexes are broken http://code.google.com/p/reviewboard/issues/detail?id=2038 (No comment was entered for this change.) -- You received this message because you

Re: Issue 2002 in reviewboard: linkified URL has succeeding parenthesis included in link's target URL

2011-03-21 Thread reviewboard
Comment #1 on issue 2002 by trowb...@gmail.com: linkified URL has succeeding parenthesis included in link's target URL http://code.google.com/p/reviewboard/issues/detail?id=2002 Issue 2038 has been merged into this issue. -- You received this message because you are subscribed to the Google

Re: Issue 1648 in reviewboard: post-review throws an exception for Perforce changesets with huge descriptions.

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1481 Comment #2 on issue 1648 by trowb...@gmail.com: post-review throws an exception for Perforce changesets with huge descriptions. http://code.google.com/p/reviewboard/issues/detail?id=1648 (No comment was entered for this change.) --

Re: Issue 1481 in reviewboard: Review Board 500 error if testing done is over 65535 characters (or a bit smaller)

2011-03-21 Thread reviewboard
Comment #4 on issue 1481 by trowb...@gmail.com: Review Board 500 error if testing done is over 65535 characters (or a bit smaller) http://code.google.com/p/reviewboard/issues/detail?id=1481 Issue 1648 has been merged into this issue. -- You received this message because you are subscribed

Re: Issue 2024 in reviewboard: MultipleObjectsReturned: get() returned more than one ReviewRequest

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 2024 by trowb...@gmail.com: MultipleObjectsReturned: get() returned more than one ReviewRequest http://code.google.com/p/reviewboard/issues/detail?id=2024 Fixed in release-1.5.x. This will ship with 1.5.5. -- You received this message

Re: Issue 1791 in reviewboard: Hide unreasonably large diffs.

2011-03-21 Thread reviewboard
Updates: Status: WontFix Comment #2 on issue 1791 by trowb...@gmail.com: Hide unreasonably large diffs. http://code.google.com/p/reviewboard/issues/detail?id=1791 I think this is actually a social problem rather than a technical one. If your users are posting code reviews of

Re: Issue 1695 in reviewboard: filename containing spaces is not handled properly in UI

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1533 Comment #2 on issue 1695 by trowb...@gmail.com: filename containing spaces is not handled properly in UI http://code.google.com/p/reviewboard/issues/detail?id=1695 (No comment was entered for this change.) -- You received this

Re: Issue 1533 in reviewboard: File Names Cut off in Diff Viewer

2011-03-21 Thread reviewboard
Comment #5 on issue 1533 by trowb...@gmail.com: File Names Cut off in Diff Viewer http://code.google.com/p/reviewboard/issues/detail?id=1533 Issue 1695 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To

Re: Issue 1633 in reviewboard: Repository configuration should not expose passwords

2011-03-21 Thread reviewboard
Updates: Status: WontFix Comment #9 on issue 1633 by trowb...@gmail.com: Repository configuration should not expose passwords http://code.google.com/p/reviewboard/issues/detail?id=1633 After thinking about this for a while, I think the solution is use https -- You received this

Re: Issue 1724 in reviewboard: error when trying to view diff for create_gse_esl.mac

2011-03-21 Thread reviewboard
Updates: Status: UserError Comment #1 on issue 1724 by trowb...@gmail.com: error when trying to view diff for create_gse_esl.mac http://code.google.com/p/reviewboard/issues/detail?id=1724 This is something that you'll have to work with your server admin to track down. Typically

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 1983 by trowb...@gmail.com: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Fixed in release-1.5.x as 529227b. This will ship in 1.5.5. Thanks! -- You received this message because

Re: Issue 774 in reviewboard: Review tracker for user

2011-03-21 Thread reviewboard
Comment #7 on issue 774 by trowb...@gmail.com: Review tracker for user http://code.google.com/p/reviewboard/issues/detail?id=774 I'm marking bug 1798 as a dupe, because it's profoundly similar, but it's asking for the ability to hide files in a diff behind disclosure triangles (which could

Re: Issue 1798 in reviewboard: Allow diffs on view diff page to be folded.

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 774 Comment #2 on issue 1798 by trowb...@gmail.com: Allow diffs on view diff page to be folded. http://code.google.com/p/reviewboard/issues/detail?id=1798 This is similar enough to bug 774 that I'm going to combine them. -- You received

Re: Issue 1747 in reviewboard: post-review --revison-range=XXXXX crashes (implement post-commit posting)

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 921 Comment #3 on issue 1747 by trowb...@gmail.com: post-review --revison-range=X crashes (implement post-commit posting) http://code.google.com/p/reviewboard/issues/detail?id=1747 (No comment was entered for this change.) -- You

Re: Issue 921 in reviewboard: Add post-commit review submission to post-review for perforce

2011-03-21 Thread reviewboard
Comment #3 on issue 921 by trowb...@gmail.com: Add post-commit review submission to post-review for perforce http://code.google.com/p/reviewboard/issues/detail?id=921 Issue 1747 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1390 in reviewboard: Review emails should have Review ID # in title/subject of email

2011-03-21 Thread reviewboard
Updates: Status: WontFix Comment #3 on issue 1390 by trowb...@gmail.com: Review emails should have Review ID # in title/subject of email http://code.google.com/p/reviewboard/issues/detail?id=1390 I'm going to go ahead and claim that your desires are specific enough that you should

Re: Issue 1513 in reviewboard: Update diff : Error: Saving the form failed due to a server error: 0 n/a

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #5 on issue 1513 by trowb...@gmail.com: Update diff : Error: Saving the form failed due to a server error: 0 n/a http://code.google.com/p/reviewboard/issues/detail?id=1513 This has changed significantly since 1.0.x, and I haven't seen any reports of

Re: Issue 1433 in reviewboard: DiffViewer Files Changed list is incomplete when png images are on changelist

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #2 on issue 1433 by trowb...@gmail.com: DiffViewer Files Changed list is incomplete when png images are on changelist http://code.google.com/p/reviewboard/issues/detail?id=1433 I believe this was fixed in another change. -- You received this message

Re: Issue 1183 in reviewboard: Viewing diff in review request causes traceback involving memcache

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 387 Comment #11 on issue 1183 by trowb...@gmail.com: Viewing diff in review request causes traceback involving memcache http://code.google.com/p/reviewboard/issues/detail?id=1183 (No comment was entered for this change.) -- You received

Re: Issue 387 in reviewboard: Error Control Characters not allowed

2011-03-21 Thread reviewboard
Comment #7 on issue 387 by trowb...@gmail.com: Error Control Characters not allowed http://code.google.com/p/reviewboard/issues/detail?id=387 Issue 1183 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group.

Re: Issue 387 in reviewboard: Error Control Characters not allowed

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #8 on issue 387 by trowb...@gmail.com: Error Control Characters not allowed http://code.google.com/p/reviewboard/issues/detail?id=387 Fixed in release-1.5.x as 171d857. This will ship in 1.5.5. Thanks! -- You received this message because you are

Re: Issue 1770 in reviewboard: Provide feedback on post-review progress

2011-03-21 Thread reviewboard
Updates: Status: WontFix Comment #2 on issue 1770 by trowb...@gmail.com: Provide feedback on post-review progress http://code.google.com/p/reviewboard/issues/detail?id=1770 There are only really two places where it can block--talking to the SCM, and talking to review board. In

Re: Issue 1263 in reviewboard: Add file navigation popup list to view diffs page

2011-03-21 Thread reviewboard
Comment #2 on issue 1263 by trowb...@gmail.com: Add file navigation popup list to view diffs page http://code.google.com/p/reviewboard/issues/detail?id=1263 Issue 1591 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1591 in reviewboard: Improve the navigation by providing a vertical bar showing where you are, and where diffs are

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1263 Comment #4 on issue 1591 by trowb...@gmail.com: Improve the navigation by providing a vertical bar showing where you are, and where diffs are http://code.google.com/p/reviewboard/issues/detail?id=1591 (No comment was entered for

Re: Issue 1794 in reviewboard: alert (popup) the user when attempting to leave a page with unpublished changes

2011-03-21 Thread reviewboard
Comment #2 on issue 1794 by trowb...@gmail.com: alert (popup) the user when attempting to leave a page with unpublished changes http://code.google.com/p/reviewboard/issues/detail?id=1794 Issue 1706 has been merged into this issue. -- You received this message because you are subscribed to

Re: Issue 1706 in reviewboard: View Diff causes un-saved changes to Description to be dropped

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1794 Comment #2 on issue 1706 by trowb...@gmail.com: View Diff causes un-saved changes to Description to be dropped http://code.google.com/p/reviewboard/issues/detail?id=1706 1794 characterizes it better. -- You received this message

Re: Issue 2014 in reviewboard: get_interesting_headers raises IndexError with C++ end-of-file whitespace change

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #1 on issue 2014 by trowb...@gmail.com: get_interesting_headers raises IndexError with C++ end-of-file whitespace change http://code.google.com/p/reviewboard/issues/detail?id=2014 Fixed in release-1.5.x as beda363. This will ship in 1.5.5. Thanks! --

Re: Issue 2033 in reviewboard: System is broken after clicking save in File Storage

2011-03-21 Thread reviewboard
Updates: Status: Duplicate Mergedinto: 1888 Comment #1 on issue 2033 by trowb...@gmail.com: System is broken after clicking save in File Storage http://code.google.com/p/reviewboard/issues/detail?id=2033 (No comment was entered for this change.) -- You received this message

Re: Issue 1888 in reviewboard: KeyError at /review/admin/db/scmtools/repository/add/

2011-03-21 Thread reviewboard
Comment #4 on issue 1888 by trowb...@gmail.com: KeyError at /review/admin/db/scmtools/repository/add/ http://code.google.com/p/reviewboard/issues/detail?id=1888 Issue 2033 has been merged into this issue. -- You received this message because you are subscribed to the Google Groups

Re: Issue 1888 in reviewboard: KeyError at /review/admin/db/scmtools/repository/add/

2011-03-21 Thread reviewboard
Updates: Status: Fixed Comment #5 on issue 1888 by trowb...@gmail.com: KeyError at /review/admin/db/scmtools/repository/add/ http://code.google.com/p/reviewboard/issues/detail?id=1888 Fixed in release-1.5.x as 9bdc33f. Thanks! -- You received this message because you are subscribed

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

2011-03-21 Thread reviewboard
Comment #2 on issue 1983 by robert.munteanu: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Thanks! Are you sure the commit is correct? https://github.com/reviewboard/reviewboard/commit/529227b seems unrelated. -- You received

Re: Issue 2035 in reviewboard: document.title (tab name) should include bug number and submitter

2011-03-21 Thread reviewboard
Comment #2 on issue 2035 by keshav.a...@gmail.com: document.title (tab name) should include bug number and submitter http://code.google.com/p/reviewboard/issues/detail?id=2035 If you have 5 tabs open with 5 reviews, and you have to click through all of them just to find a single review,

Re: Issue 1949 in reviewboard: RBtools intermittently not submitting review requests.

2011-03-21 Thread reviewboard
Comment #2 on issue 1949 by mail.ax...@gmail.com: RBtools intermittently not submitting review requests. http://code.google.com/p/reviewboard/issues/detail?id=1949 I'm not sure whether the bad line is caused by the diff or something else as the commit usually goes through after a few

Issue 2039 in reviewboard: My Dashboard datagrid table is not properly aligned in Chrome browser (dev channel)

2011-03-21 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2039 by chu.e...@gmail.com: My Dashboard datagrid table is not properly aligned in Chrome browser (dev channel) http://code.google.com/p/reviewboard/issues/detail?id=2039 * NOTE: Do not post confidential information in

Re: Issue 1633 in reviewboard: Repository configuration should not expose passwords

2011-03-21 Thread reviewboard
Comment #10 on issue 1633 by rtimush: Repository configuration should not expose passwords http://code.google.com/p/reviewboard/issues/detail?id=1633 Use https is not a solution for the original issue (the problem was that passwords were visible when user clicks the view source button).

Issue 2041 in reviewboard: Option to add more context to the main page of a review

2011-03-21 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2041 by marc.san...@gmail.com: Option to add more context to the main page of a review http://code.google.com/p/reviewboard/issues/detail?id=2041 What version are you running? 1.5.4 What's the URL of the page this

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

2011-03-21 Thread reviewboard
Comment #3 on issue 1983 by trowb...@gmail.com: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 You're right. I meant https://github.com/reviewboard/reviewboard/commit/171d857102ee6890f90916aa8276b85ad7a80453 -- You received this

Re: Issue 1983 in reviewboard: Include the server's time zone in the Web API

2011-03-21 Thread reviewboard
Comment #4 on issue 1983 by robert.munteanu: Include the server's time zone in the Web API http://code.google.com/p/reviewboard/issues/detail?id=1983 Great, thank you. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this

Re: Issue 2041 in reviewboard: Option to add more context to the main page of a review

2011-03-21 Thread reviewboard
Updates: Status: Confirmed Labels: Component-Reviews Comment #1 on issue 2041 by chip...@gmail.com: Option to add more context to the main page of a review http://code.google.com/p/reviewboard/issues/detail?id=2041 I've wanted this before too. It'll depend on the incremental