Re: Issue 2359 in reviewboard: Review creation fails with patches created with Subversion 1.7.x if they include a property change.
Comment #37 on issue 2359 by lrozenbl...@gmail.com: Review creation fails with patches created with Subversion 1.7.x if they include a property change. http://code.google.com/p/reviewboard/issues/detail?id=2359 Hello guys, was this bug really fixed? We've installed ReviewBoard 1.7.11 (and we're using SVN client 1.8, while SVN server 1.6), and the problem still exists. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2359 in reviewboard: Review creation fails with patches created with Subversion 1.7.x if they include a property change.
Comment #38 on issue 2359 by denis.ba...@gmail.com: Review creation fails with patches created with Subversion 1.7.x if they include a property change. http://code.google.com/p/reviewboard/issues/detail?id=2359 We patch diff using third-party module for diff parsing: http://bazaar.launchpad.net/~bzr-pqm/bzr/bzr.dev/view/head:/bzrlib/patches.py We have moved to RB 1.6.11 but still upload patched diff with property changes stripped off. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2359 in reviewboard: Review creation fails with patches created with Subversion 1.7.x if they include a property change.
Comment #39 on issue 2359 by lrozenbl...@gmail.com: Review creation fails with patches created with Subversion 1.7.x if they include a property change. http://code.google.com/p/reviewboard/issues/detail?id=2359 I see - It means it's not fixed (as for me). Probably I'll start some discussion in mailing list - since I'm not sure what's the policy about commenting close issues here. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Issue 3041 in reviewboard: Diffs no longer required...?
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3041 by joerschn...@gmail.com: Diffs no longer required...? http://code.google.com/p/reviewboard/issues/detail?id=3041 *** For customer support, please post to reviewbo...@googlegroups.com *** If you have a patch, please submit it to http://reviews.reviewboard.org/ *** *** Do not post confidential information in this bug report! What version are you running? 1.7.11 What's the URL of the page containing the problem? https://[baseurl]/r/new/ What steps will reproduce the problem? 1. Create a new review 2. do not attach a diff or a file 3. click on create review request What is the expected output? What do you see instead? Expected behavior: It should fail saying you need to attach a diff or a file. Actually does: let's you create the Review Request, even though there is nothing to review... What operating system are you using? What browser? Centos (server side), Windows 7 client side, Chrome Broswer Please provide any additional information below. It used to behave as expected (and desired) until upgrading to 1.7.11, and no longer does. I can't find anywhere in the admin portal that could require diffs/files, nor on the server conf files. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 3041 in reviewboard: Diffs no longer required...?
Updates: Status: NeedInfo Comment #1 on issue 3041 by chip...@gmail.com: Diffs no longer required...? http://code.google.com/p/reviewboard/issues/detail?id=3041 It's valid to have a review request without diffs, for, say, file review. This is generally done by selecting the None - File Attachments Only option in the repository list, but either way, it's allowed. Is this causing problems for you? You can still upload a diff after the fact. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 3041 in reviewboard: Diffs no longer required...?
Comment #2 on issue 3041 by joerschn...@gmail.com: Diffs no longer required...? http://code.google.com/p/reviewboard/issues/detail?id=3041 Yeah I can totally see not needing diffs IF you have a file. The problem is, you can still submit a Review Request if you don't have either... I remember not long ago (although I don't remember which version), you needed to attach something or it wouldn't let you finish making the Review Request. Was this sanity check feature removed or am I just not seeing it? I can go through the entire workflow of publishing a Review Request without ever having a diff. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2359 in reviewboard: Review creation fails with patches created with Subversion 1.7.x if they include a property change.
Comment #40 on issue 2359 by elguerom...@gmail.com: Review creation fails with patches created with Subversion 1.7.x if they include a property change. http://code.google.com/p/reviewboard/issues/detail?id=2359 We are using latest RB 1.7 with SVN 1.7 without any issues. This issue was fixed. I seem to recall reading in the release notes of SVN 1.8, there may have been changes made again to the output of svn diff. If I were you, I would open a new issue against SVN 1.8 with the appropriate information, perhaps a sample header, to help with any necessary code changes in order to work with patches produced by SVN 1.8. -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at http://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/groups/opt_out.
