reviewboard - Project Hosting on Google Code

Today 4 hours ago issue 1446 (Remove "System" Dependencies) reported by bentilford - Assuming that dependencies will be available on a user's system (cmemcache for example) is a sure way to lose users. An application should contain it's dependencies and not rely that they be installed on the system. BTW cmemcache has been abandoned and can no longer be compiled with current versions of memcache, libmemcache, and gcc so it's impossible to install ReviewBoard using the dependencies currently available in repos. Assuming that dependencies will be available on a user's system (cmemcache for example) is a sure way to lose users. An application should contain it's dependencies and not rely that they be installed on the system. BTW cmemcache has been abandoned and can no longer be compiled with current versions of memcache, libmemcache, and gcc so it's impossible to install ReviewBoard using the dependencies currently available in repos. 12 hours ago issue 1445 (Provide suuport to bulk change reviews) reported by ksangani - NOTE: If you have a patch, please submit it to http://reviews.review- board.org/ What version are you running? 1.0.5.1 What's the URL of the page this enhancement relates to, if any? Describe the enhancement and the motivation for it. Provide support to bulk change reviews. For example bulk change all reviews which are submitted. What operating system are you using? What browser? Linux Firefox Please provide any additional information below. NOTE: If you have a patch, please submit it to http://reviews.review- board.org/ What version are you running? 1.0.5.1 What's the URL of the page this enhancement relates to, if any? Describe the enhancement and the motivation for it. Provide support to bulk change reviews. For example bulk change all reviews which are submitted. What operating system are you using? What browser? Linux Firefox Please provide any additional information below. Yesterday 37 hours ago issue 1444 (DLL load failed on trying to create SVN repository) reported by caffeinatedcoder - NOTE: Do not post confidential information in this bug report. What version are you running? 1.1 alpha 2 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? Windows 2003 server Please provide any additional information below. I got review board working fine on an XP workstation but when I put it on a Windows 2003 server and I try to create a SVN repository I get: "DLL load failed: The specified procedure could not be found." Now it would help if the message would tell me what DLL it is looking for. I am assuming it is something to do with SVN but am not sure. Can anyone tell me what it might be looking for or what the possible causes could be? Thanks. NOTE: Do not post confidential information in this bug report. What version are you running? 1.1 alpha 2 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? Windows 2003 server Please provide any additional information below. I got review board working fine on an XP workstation but when I put it on a Windows 2003 server and I try to create a SVN repository I get: "DLL load failed: The specified procedure could not be found." Now it would help if the message would tell me what DLL it is looking for. I am assuming it is something to do with SVN but am not sure. Can anyone tell me what it might be looking for or what the possible causes could be? Thanks. Last 7 days Dec 27, 2009 issue 1443 (process hangs when generating diff for large file with 1 lin...) reported by sokann - What version are you running? 1.0.5.1 What steps will reproduce the problem? 1. Commit http://jqueryjs.googlecode.com/files/jquery-1.3.1.min.js to code repository as jquery.min.js 2. Replace the file with http://jqueryjs.googlecode.com/files/ jquery-1.3.2.min.js 3. Post the diff to reviewboard 4. View the diff in reviewboard What is the expected output? What do you see instead? Expected: useless diff appears on the screen Actual: nothing on the screen; the apache process CPU consumption goes to 100% and stays there forever What operating system are you using? What browser? Linux, Opera/Firefox Please provide any additional information below. apache 2.2.8, mod_python 3.3.1, subversion 1.6.5 What version are you running? 1.0.5.1 What steps will reproduce the problem? 1. Commit http://jqueryjs.googlecode.com/files/jquery-1.3.1.min.js to code repository as jquery.min.js 2. Replace the file with http://jqueryjs.googlecode.com/files/ jquery-1.3.2.min.js 3. Post the diff to reviewboard 4. View the diff in reviewboard What is the expected output? What do you see instead? Expected: useless diff appears on the screen Actual: nothing on the screen; the apache process CPU consumption goes to 100% and stays there forever What operating system are you using? What browser? Linux, Opera/Firefox Please provide any additional information below. apache 2.2.8, mod_python 3.3.1, subversion 1.6.5 Dec 23, 2009 issue 1442 (post-review can't work in a BAT file under windows system wh...) reported by jackhe1...@gmail.com - NOTE: Do not post confidential information in this bug report. What version are you running? RBTools-0.2beta2-py2.6.egg Review Board 1.0.5.1 What's the URL of the page containing the problem? It's not on the webpage, but in the client side. What steps will reproduce the problem? 1. Execute the post-review.exe in the command console could successfully post a new review request in the server. Everything's OK. 2. Put the same command line for post-review.exe into a BAT file and keep in the same folder as the console startup. 3. Double click the BAT file through the File Explorer. What is the expected output? What do you see instead? It should be the same as running from the console window. I should see a new request created in server side. However, it's failed with an error message "The current directory does not contain a checkout from a supported source code repository.". The directory was exactly the root folder of my working copies. What operating system are you using? What browser? Windows XP SP3, there's no dependency of browser since it's not be opened yet. Please provide any additional information below. NOTE: Do not post confidential information in this bug report. What version are you running? RBTools-0.2beta2-py2.6.egg Review Board 1.0.5.1 What's the URL of the page containing the problem? It's not on the webpage, but in the client side. What steps will reproduce the problem? 1. Execute the post-review.exe in the command console could successfully post a new review request in the server. Everything's OK. 2. Put the same command line for post-review.exe into a BAT file and keep in the same folder as the console startup. 3. Double click the BAT file through the File Explorer. What is the expected output? What do you see instead? It should be the same as running from the console window. I should see a new request created in server side. However, it's failed with an error message "The current directory does not contain a checkout from a supported source code repository.". The directory was exactly the root folder of my working copies. What operating system are you using? What browser? Windows XP SP3, there's no dependency of browser since it's not be opened yet. Please provide any additional information below. Dec 23, 2009 issue 1440 (Comment flag not visible in View Diff) commented on by lonico - There is also an issue with this assignment: 877 } else if (row != null) { 878 /* 879 * We collapsed the rows (unless someone mucked with the DB), 880 * so the desired row is less than the row number retrieved. 881 / 882 high = parseInt(row.getAttribute('line')) 883 } high should not be set to a linenum, but to a row number. And the following statement 896 if (!value) { 897 i++; 898 continue; 899 } assumes that value will not be NULL eventually as i keeps increasing. This is not very robust. There is also an issue with this assignment: 877 } else if (row != null) { 878 / 879 * We collapsed the rows (unless someone mucked with the DB), 880 * so the desired row is less than the row number retrieved. 881 / 882 high = parseInt(row.getAttribute('line')) 883 } high should not be set to a linenum, but to a row number. And the following statement 896 if (!value) { 897 i++; 898 continue; 899 } assumes that value will not be NULL eventually as i keeps increasing. This is not very robust. Dec 23, 2009 issue 1430 (Cannot store local certificate for HTTPS SVN when SVN runnin...) commented on by caffeinatedcoder - Thanks for the tip, I will try it. Thanks for the tip, I will try it. Dec 23, 2009 issue 1431 (Unable to load diff for review) commented on by caffeinatedcoder - Well the index line causes a problem but I can work around it by editing. Unfortunately, I am trying to set this up so the department can use it for reviews and an extra steps can be an issue when trying to convince people to use it. TortoiseSVN support is a must because nobody has a copy of svn on their local workstations only the Tortoise client (or the Subclipse plugin which I have not tested). Thanks. Well the index line causes a problem but I can work around it by editing. Unfortunately, I am trying to set this up so the department can use it for reviews and an extra steps can be an issue when trying to convince people to use it. TortoiseSVN support is a must because nobody has a copy of svn on their local workstations only the Tortoise client (or the Subclipse plugin which I have not tested). Thanks. Dec 23, 2009 issue 1419 (Error uploading diff: The file was not found in the reposito...) commented on by lianne2005 - hi chiox86, you means, I need test such command like belowed: post-review.exe -- server=http://rb.corp.XXXXX.com/ --repository- url=http://svn.corp.XXXXX.com/svn/XXXX/trunk/ --revision-range=5548:5549 -- summary="test diff post by XXXXX" --output-diff I runned it and see every files diff info between the 2 version. hi chiox86, you means, I need test such command like belowed: post-review.exe -- server=http://rb.corp.XXXXX.com/ --repository- url=http://svn.corp.XXXXX.com/svn/XXXX/trunk/ --revision-range=5548:5549 -- summary="test diff post by XXXXX" --output-diff I runned it and see every files diff info between the 2 version. Dec 23, 2009 issue 1441 (Bug# in review listing) reported by KrosseKrabbe - What version are you running? 1.1 a2 What's the URL of the page this enhancement relates to, if any? /r/ Describe the enhancement and the motivation for it. In a review's details it's possible to input the related Bug#. It should be possible to add that Bug# as column to the review listing (like it's currently possible with "last updated", "diff updated", ...). That would be useful because when i'm talking about reviews with our developer's they often say something like "I've updated review of ticket #255". It's a little bit hard to find the corresponding review for the ticket currently.. would be much easier if it would be possible to add the bug# as column to the listing. What operating system are you using? What browser? n/a Please provide any additional information below. n/a What version are you running? 1.1 a2 What's the URL of the page this enhancement relates to, if any? /r/ Describe the enhancement and the motivation for it. In a review's details it's possible to input the related Bug#. It should be possible to add that Bug# as column to the review listing (like it's currently possible with "last updated", "diff updated", ...). That would be useful because when i'm talking about reviews with our developer's they often say something like "I've updated review of ticket #255". It's a little bit hard to find the corresponding review for the ticket currently.. would be much easier if it would be possible to add the bug# as column to the listing. What operating system are you using? What browser? n/a Please provide any additional information below. n/a Last 30 days Dec 22, 2009 issue 1440 (Comment flag not visible in View Diff) reported by lonico - 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? http://reviewboard.xxxxxxx.com/r/4174/diff/1/#index_header What steps will reproduce the problem? Unknown. This problem was only noticed for a few comments. What is the expected output? What do you see instead? A comment flag is missing in view diffs, though the comment is visible in the reviews page (and the comment is also visible in the html source for the view diffs page). What operating system are you using? What browser? Windows, Firefox Please provide any additional information below. Bug if row is undefined for the first iteration. i is not updated, so row stays undefined, and the loop continues until high reaches low. The function then exits and failed to find the correct row. In diffviewer.js function findLineNumRow(table, linenum, startRow, endRow) { 885 /* Binary search for this cell. / 886 for (var i = Math.round((low + high) / 2); low < high - 1;) { 887 row = table.rows[row_offset + i]; 888 889 if (!row) { 890 high--; 891 continue; 892 } 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? http://reviewboard.xxxxxxx.com/r/4174/diff/1/#index_header What steps will reproduce the problem? Unknown. This problem was only noticed for a few comments. What is the expected output? What do you see instead? A comment flag is missing in view diffs, though the comment is visible in the reviews page (and the comment is also visible in the html source for the view diffs page). What operating system are you using? What browser? Windows, Firefox Please provide any additional information below. Bug if row is undefined for the first iteration. i is not updated, so row stays undefined, and the loop continues until high reaches low. The function then exits and failed to find the correct row. In diffviewer.js function findLineNumRow(table, linenum, startRow, endRow) { 885 /* Binary search for this cell. / 886 for (var i = Math.round((low + high) / 2); low < high - 1;) { 887 row = table.rows[row_offset + i]; 888 889 if (!row) { 890 high--; 891 continue; 892 } Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by chipx86 - Nope, all we do is attempt to make an SSH connection in order to check the key so that we can verify it, and do an svn info on the repository. Both should work. I don't know off-hand what's causing the timeout. If you upgrade to last night's nightly, the log file should contain some more information on the verification attempt. Nope, all we do is attempt to make an SSH connection in order to check the key so that we can verify it, and do an svn info on the repository. Both should work. I don't know off-hand what's causing the timeout. If you upgrade to last night's nightly, the log file should contain some more information on the verification attempt. Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by ruddct - Ahah. Other operations involving my svn server are working (uploading diffs, viewing context, etc), so I don't think it's a configuration problem. Latency to my SVN server is quite high (due to physical distance), and the server has a large number of branches/revisions. When you add/edit a repository in RB, is there some kind of scan of all revision numbers or branches that is done? Might the combination of this in addition to high latency be the cause of my timeouts? Ahah. Other operations involving my svn server are working (uploading diffs, viewing context, etc), so I don't think it's a configuration problem. Latency to my SVN server is quite high (due to physical distance), and the server has a large number of branches/revisions. When you add/edit a repository in RB, is there some kind of scan of all revision numbers or branches that is done? Might the combination of this in addition to high latency be the cause of my timeouts? Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by chipx86 - No, there's no way to bypass this. We need to figure out why it's timing out. One of the leading causes of support issues we encounter is people setting up their repository wrong and not catching it until later. We added repository verification in order to get around this. However, this is an alpha release, and isn't meant for production servers yet, so it's quite possible to hit problems. In particular, the SSH verification is absolutely essential, as we need to detect if the host key has changed and to allow verification of the key. Otherwise, the SSH operations would not work. No, there's no way to bypass this. We need to figure out why it's timing out. One of the leading causes of support issues we encounter is people setting up their repository wrong and not catching it until later. We added repository verification in order to get around this. However, this is an alpha release, and isn't meant for production servers yet, so it's quite possible to hit problems. In particular, the SSH verification is absolutely essential, as we need to detect if the host key has changed and to allow verification of the key. Otherwise, the SSH operations would not work. Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by ruddct - Also, in what might be a related issue, any attempt to add or modify an svn+ssh server results in the error "(110, 'Connection timed out')". Operations with this server from other tools (command-line svn, etc) work as expected. Is there some way I can manually override the server path without making a network connection? Also, in what might be a related issue, any attempt to add or modify an svn+ssh server results in the error "(110, 'Connection timed out')". Operations with this server from other tools (command-line svn, etc) work as expected. Is there some way I can manually override the server path without making a network connection? Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by chipx86 - If that's the top level of the SVN repository, then that's fine. It's the top-level of the repository that matters, not the website. If that's the top level of the SVN repository, then that's fine. It's the top-level of the repository that matters, not the website. Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by ruddct - The root of my svn server is "svn+ssh://svn@SERVER/cpp/". Is the "/cpp/" going to cause problems? The root of my svn server is "svn+ssh://svn@SERVER/cpp/". Is the "/cpp/" going to cause problems? Dec 22, 2009 issue 1098 (Add support to post-review for review board installations us...) Labels changed by trowbrds - Labels: Type-Enhancement −Type-Defect Labels: Type-Enhancement −Type-Defect Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by chipx86 - The key part is that the file wasn't found in the repository. So, the problem you're hitting is that the repository URL is wrong. You don't want to point it to trunk. It needs to point to the very root of the SVN server. Otherwise, invalid path names will be generated. When sending a diff, a base path (the path between the root of the repository and current directory within the repository) is computed and sent along with the diff. This is appended to the repository URL, and the filename in the diff is then appended to this. If the repository URL is pointing to a subdirectory within the server, you will hit these errors. The key part is that the file wasn't found in the repository. So, the problem you're hitting is that the repository URL is wrong. You don't want to point it to trunk. It needs to point to the very root of the SVN server. Otherwise, invalid path names will be generated. When sending a diff, a base path (the path between the root of the repository and current directory within the repository) is computed and sent along with the diff. This is appended to the repository URL, and the filename in the diff is then appended to this. If the repository URL is pointing to a subdirectory within the server, you will hit these errors. Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by ruddct - Sorry it's taken me so long to get around to trying this using post-review. Results are the same when using it. Nothing shows up in the logs. desktop% post-review --server=https://localhost:8080 --repository-url=svn+ssh://SERVER/cpp/trunk/ --revision-range=1000:HEAD ==> Review Board Login Required Enter username and password for Review Board at https://localhost:8080/ Username: USERNAME Password: Error uploading diff: The file was not found in the repository (207) Your review request still exists, but the diff is not attached. Sorry it's taken me so long to get around to trying this using post-review. Results are the same when using it. Nothing shows up in the logs. desktop% post-review --server=https://localhost:8080 --repository-url=svn+ssh://SERVER/cpp/trunk/ --revision-range=1000:HEAD ==> Review Board Login Required Enter username and password for Review Board at https://localhost:8080/ Username: USERNAME Password: Error uploading diff: The file was not found in the repository (207) Your review request still exists, but the diff is not attached. Dec 22, 2009 issue 1439 (URL /r/xyz always redirects to /xyz) reported by youssef.adnan - What version are you running? 1.1 alpha 2 What's the URL of the page containing the problem? /r/(anything) This would be the same for /r/34 (to display a review) or /r/new or anything under /r What steps will reproduce the problem? 1. From the dashboard (or any page), click any review 2. From the dashboard (or any page), click "New Review Request" What is the expected output? What do you see instead? Expected output is the review or the new review request page. Instead, I get a 404 as here is what happens: http://reviewboard.example.com/r/34 would redirect to http://reviewboard.example.com/34 or http://reviewboard.example.com/r/new would redirect to http://reviewboard.example.com/new would redirect to What operating system are you using? What browser? OS: CentOS 5.4 Browser: Linux Chrome, Firefox 3.5, and Opera 10. Please provide any additional information below. It seems to me that the get_absolute_url call in /review/models.py is returning the url without the /r piece of it. Regards, Youssef What version are you running? 1.1 alpha 2 What's the URL of the page containing the problem? /r/(anything) This would be the same for /r/34 (to display a review) or /r/new or anything under /r What steps will reproduce the problem? 1. From the dashboard (or any page), click any review 2. From the dashboard (or any page), click "New Review Request" What is the expected output? What do you see instead? Expected output is the review or the new review request page. Instead, I get a 404 as here is what happens: http://reviewboard.example.com/r/34 would redirect to http://reviewboard.example.com/34 or http://reviewboard.example.com/r/new would redirect to http://reviewboard.example.com/new would redirect to What operating system are you using? What browser? OS: CentOS 5.4 Browser: Linux Chrome, Firefox 3.5, and Opera 10. Please provide any additional information below. It seems to me that the get_absolute_url call in /review/models.py is returning the url without the /r piece of it. Regards, Youssef Dec 22, 2009 issue 1420 (When updating a review without any diff change reviewboard r...) commented on by ryan.gallagher - Thanks for the heads-up on --change-only. Was this always the case? I'm probably mistaken, but I thought in the past that on the server-side it did an interdiff upon updating the diff before recording a new one. Thanks for the heads-up on --change-only. Was this always the case? I'm probably mistaken, but I thought in the past that on the server-side it did an interdiff upon updating the diff before recording a new one. Dec 22, 2009 issue 1438 (Review files being marked erroneously as deleted.) reported by trevboxmonster - 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? http://reviewboard.4sight.local:10081/r/222/diff/#index_header (this is a local address of a review prior to publishing) What steps will reproduce the problem? 1. From the perforce P4V GUI, create a new pending changelist containing files for review. 2. Run the post-review.exe tool against the changelist. 3. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing from the list of files for review under the diff section. What is the expected output? What do you see instead? I expect to see the edited and newly created files in my changelist to be reviewed. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing. What operating system are you using? What browser? Windows XP Professional Internet Explorer 8 Please provide any additional information below. 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? http://reviewboard.4sight.local:10081/r/222/diff/#index_header (this is a local address of a review prior to publishing) What steps will reproduce the problem? 1. From the perforce P4V GUI, create a new pending changelist containing files for review. 2. Run the post-review.exe tool against the changelist. 3. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing from the list of files for review under the diff section. What is the expected output? What do you see instead? I expect to see the edited and newly created files in my changelist to be reviewed. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing. What operating system are you using? What browser? Windows XP Professional Internet Explorer 8 Please provide any additional information below. Dec 22, 2009 issue 1038 (Add time zone to settings page.) commented on by birbilis - So far there is only one global setting in the admin page. Are there any plans on having per-user timezones? That would be very useful for cross-world developement. So far there is only one global setting in the admin page. Are there any plans on having per-user timezones? That would be very useful for cross-world developement. Dec 22, 2009 issue 1386 (Full file access via API) changed by chipx86 - Status: Confirmed Owner: chipx86 Labels: Milestone-Release1.5 Component-API Status: Confirmed Owner: chipx86 Labels: Milestone-Release1.5 Component-API Dec 22, 2009 issue 1417 (Can't make a review for a p4 changelist with a file move/add) Status changed by chipx86 - Tomorrow's nightly of RBTools should fix this. Please let me know. Status: Fixed Tomorrow's nightly of RBTools should fix this. Please let me know. Status: Fixed Dec 22, 2009 issue 1417 (Can't make a review for a p4 changelist with a file move/add) changed by chipx86 - Status: Confirmed Owner: chipx86 Labels: Priority-High Component-RBTools Milestone-RBTools-Release1.0 −Priority-Medium Status: Confirmed Owner: chipx86 Labels: Priority-High Component-RBTools Milestone-RBTools-Release1.0 −Priority-Medium Dec 22, 2009 issue 1181 (Wrong content of search-cron.conf after installation) commented on by chipx86 - Issue 1367 has been merged into this issue. Issue 1367 has been merged into this issue. Dec 22, 2009 issue 1367 (Missing double-quotes in reviewboard/cmdline/conf/search-cro...) changed by chipx86 - Status: Duplicate Status: Duplicate Dec 22, 2009 issue 1420 (When updating a review without any diff change reviewboard r...) changed by chipx86 - Hi Ryan, post-review intentionally does not do any complex diff comparisons or anything. If you only need to update the change description, you can run post-review with the --change-only flag. Status: NotABug Labels: Component-RBTools Hi Ryan, post-review intentionally does not do any complex diff comparisons or anything. If you only need to update the change description, you can run post-review with the --change-only flag. Status: NotABug Labels: Component-RBTools Dec 22, 2009 issue 1401 (Creating new review requests does not work) commented on by chipx86 - Sorry, for whatever reason, we're no longer seeing e-mails on updates to tickets. ramjisriram: Did you run `rb-site upgrade` on the new site? Keep in mind that 1.1 alpha is an alpha version, and as such it may be prone to problems like this. It's really best to make sure you've completely backed up your database beforehand and are not using it for production servers. If you use post-review to upload a diff, do you have the same problem? Is there anything in the log files? Sorry, for whatever reason, we're no longer seeing e-mails on updates to tickets. ramjisriram: Did you run `rb-site upgrade` on the new site? Keep in mind that 1.1 alpha is an alpha version, and as such it may be prone to problems like this. It's really best to make sure you've completely backed up your database beforehand and are not using it for production servers. If you use post-review to upload a diff, do you have the same problem? Is there anything in the log files? Dec 22, 2009 issue 1419 (Error uploading diff: The file was not found in the reposito...) Status changed by chipx86 - I need more information on the file and its circumstances. Is the file that it references a brand new file in one of those revisions? If you run with --output-diff, do you see a diff that makes sense for that file? Status: NeedInfo I need more information on the file and its circumstances. Is the file that it references a brand new file in one of those revisions? If you run with --output-diff, do you see a diff that makes sense for that file? Status: NeedInfo Dec 22, 2009 issue 1421 (post-review doesn't gracefully handle ClearCase deployments ...) commented on by chipx86 - Please post patches on http://reviews.reviewboard.org/ Please post patches on http://reviews.reviewboard.org/ Dec 22, 2009 issue 1423 (post-review expects an array internally and a string is retu...) Labels changed by chipx86 - Please post patches on http://reviews.reviewboard.org/ Labels: Component-RBTools Please post patches on http://reviews.reviewboard.org/ Labels: Component-RBTools Dec 22, 2009 issue 1425 (post-review + subversion + empty files Produces diffs That R...) changed by chipx86 - Status: Confirmed Labels: Component-DiffParser Milestone-Release1.0.x Status: Confirmed Labels: Component-DiffParser Milestone-Release1.0.x Dec 22, 2009 issue 1427 (Problems with post-review while uploading diffs) changed by chipx86 - Are you sure that this repository is set for Subversion and not Bazaar? Status: NeedInfo Labels: Component-SCMTools Are you sure that this repository is set for Subversion and not Bazaar? Status: NeedInfo Labels: Component-SCMTools Dec 22, 2009 issue 1429 (Update Diff doesn't work) Status changed by chipx86 - More information is needed than this. Please re-file the bug report. Status: Invalid More information is needed than this. Please re-file the bug report. Status: Invalid Dec 22, 2009 issue 1433 (DiffViewer "Files Changed" list is incomplete when png image...) Labels changed by chipx86 - Labels: Component-DiffViewer Labels: Component-DiffViewer Dec 22, 2009 issue 1428 (website - Allow yearly donations) Status changed by chipx86 - Thanks. The website now supports yearly donations. Status: Fixed Thanks. The website now supports yearly donations. Status: Fixed Dec 22, 2009 issue 1428 (website - Allow yearly donations) commented on by onkarshinde - Paypal certainly allows yearly donations. Freenode.net accepts yearly donations through paypal. Please check the form on home page - http://freenode.net/ Paypal certainly allows yearly donations. Freenode.net accepts yearly donations through paypal. Please check the form on home page - http://freenode.net/ Dec 22, 2009 issue 1437 (ReviewBoard Installation failed on Windows XP SP2 VMWare.) commented on by kranthi.vskp - Is there any other way to install ReviewBoard without using easy_install, by downloading ReviewBoard and all its dependent softwared and install it. Is there any other way to install ReviewBoard without using easy_install, by downloading ReviewBoard and all its dependent softwared and install it. Dec 22, 2009 issue 1437 (ReviewBoard Installation failed on Windows XP SP2 VMWare.) commented on by kranthi.vskp - pypi.python.org was not down. I could see the ReviewBoard hyperlink too on this page which is pointing to download links. For some strange reasons, I am unable to install ReviewBoard. I tired installing ReviewBoard on my dev-box instead of VMWare and faced the same issue. pypi.python.org was not down. I could see the ReviewBoard hyperlink too on this page which is pointing to download links. For some strange reasons, I am unable to install ReviewBoard. I tired installing ReviewBoard on my dev-box instead of VMWare and faced the same issue. Dec 22, 2009 issue 1368 (LDAP not properly translating %s for LDAP) Status changed by trowbrds - Status: Incomplete Status: Incomplete Dec 21, 2009 issue 1431 (Unable to load diff for review) commented on by chipx86 - The Index line should work fine. We used it all the time. However, you're right that TortoiseSVN-generated diffs don't work, as they generate incompatible diffs. We would like to support them better. The Index line should work fine. We used it all the time. However, you're right that TortoiseSVN-generated diffs don't work, as they generate incompatible diffs. We would like to support them better. Dec 21, 2009 issue 1415 (Not able to creat a new review request (Critical)) Status changed by chipx86 - Make sure you're running 1.0.5.1, not 1.0.5. This was a bug in 1.0.5. Status: SetupIssue Make sure you're running 1.0.5.1, not 1.0.5. This was a bug in 1.0.5. Status: SetupIssue Dec 21, 2009 issue 1430 (Cannot store local certificate for HTTPS SVN when SVN runnin...) commented on by chipx86 - The issue is that we don't have much control over these directories. We call into libsvn, which writes in ~/.subversion. So we're basically limited to what svn can do as a service. As far as what that is, I don't know. What you should be able to do, however, is modify your Apache configuration to set HOME to be some other directory. You should be able to do this with a SetEnv directive in the Apache configuration for Review Board. Just specify a directory where the .subversion directory can be stored and restart the web server. The issue is that we don't have much control over these directories. We call into libsvn, which writes in ~/.subversion. So we're basically limited to what svn can do as a service. As far as what that is, I don't know. What you should be able to do, however, is modify your Apache configuration to set HOME to be some other directory. You should be able to do this with a SetEnv directive in the Apache configuration for Review Board. Just specify a directory where the .subversion directory can be stored and restart the web server. Dec 21, 2009 issue 1416 (Error 500 when publishing) Status changed by trowbrds - It looks like the hostname hasn't been configured properly in the admin interface. Status: SetupIssue It looks like the hostname hasn't been configured properly in the admin interface. Status: SetupIssue Dec 21, 2009 issue 1435 (Feature request: show "Ship It" status on dashboard) Status changed by chipx86 - Actually, both are available. If you have the My Comments column turned on, there will be an icon showing you selected Ship It. It's the comment flag with a checkmark on it. Status: NotABug Actually, both are available. If you have the My Comments column turned on, there will be an icon showing you selected Ship It. It's the comment flag with a checkmark on it. Status: NotABug Dec 21, 2009 issue 1437 (ReviewBoard Installation failed on Windows XP SP2 VMWare.) commented on by chipx86 - This sounds more like a network configuration issue than anything. We have no control over pypi.python.org, but if it was down, it's certainly back up now. This sounds more like a network configuration issue than anything. We have no control over pypi.python.org, but if it was down, it's certainly back up now. Dec 21, 2009 issue 1412 (Can't see repository name in the repository drop list) Status changed by trowbrds - Sounds like you got it fixed. Status: SetupIssue Sounds like you got it fixed. Status: SetupIssue

issue 1438 (Review files being marked erroneously as deleted.) reported by trevboxmonster - *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? http://reviewboard.4sight.local:10081/r/222/diff/#index_header (this is a local address of a review prior to publishing) What steps will reproduce the problem? 1. From the perforce P4V GUI, create a new pending changelist containing files for review. 2. Run the post-review.exe tool against the changelist. 3. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing from the list of files for review under the diff section. What is the expected output? What do you see instead? I expect to see the edited and newly created files in my changelist to be reviewed. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing. What operating system are you using? What browser? Windows XP Professional Internet Explorer 8 Please provide any additional information below.

*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? http://reviewboard.4sight.local:10081/r/222/diff/#index_header (this is a local address of a review prior to publishing) What steps will reproduce the problem? 1. From the perforce P4V GUI, create a new pending changelist containing files for review. 2. Run the post-review.exe tool against the changelist. 3. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing from the list of files for review under the diff section. What is the expected output? What do you see instead? I expect to see the edited and newly created files in my changelist to be reviewed. When the review appears in Reviewboard for publishing all the edited files in the changelist will be marked erroneously as deleted and all the newly created files will be missing. What operating system are you using? What browser? Windows XP Professional Internet Explorer 8 Please provide any additional information below.

Today

Yesterday

Last 7 days

Last 30 days