Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #11 on issue 2184 by trowb...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 Sure, we can do that. -- 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/d/optout.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #10 on issue 2184 by bruce.c...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 I wonder if there should be a note in the documentation (install guide or release notes?) mentioning that people might want to have patch 1.7 or newer installed on the RB server since it's relatively recent (2012). -- 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/d/optout.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Updates: Status: ThirdParty Comment #9 on issue 2184 by trowb...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 That sounds like great news. :) -- 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/d/optout.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #8 on issue 2184 by bruce.c...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 I came across this problem today: the problem is that patch didn't used to understand features of git-style diffs, including renames. Installing patch 2.7 (http://savannah.gnu.org/projects/patch/) or newer should fix this. -- 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/d/optout.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #7 on issue 2184 by derpmhic...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 1.7.16 We have a similar issue (maybe exactly the same) using SVN. I suspect it has something to do with the patch file containing 2 diffs for the same file (one for the deletion one for the addition). Error information: The patch to 'https://devsubversion/svn/Development/Branches/SW22.5.0/Projects/SiteWatch/Common/deltime.pas' didn't apply cleanly. The temporary files have been left in '/tmp/reviewboard.ZD632t' for debugging purposes. `patch` returned: patching file /tmp/reviewboard.ZD632t/tmp3eUZCF 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.ZD632t/tmp3eUZCF-new.rej I've saved the files/logs in case there's anything useful. I have not attempted to duplicate. -- 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/d/optout.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #6 on issue 2184 by philip.w...@googlemail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 I think hg remove might cause a similar issue. -- 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 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #5 on issue 2184 by jon.bark...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 Still seeing this in 1.7.6 -- 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?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Updates: Labels: Component-SCMTools Component-DiffViewer Comment #4 on issue 2184 by trowb...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 (No comment was entered for this change.) -- 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 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #3 on issue 2184 by d...@wifislam.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 Still seeing this in 1.6.1 :(. -- 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 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #2 on issue 2184 by carl.van...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 This bug applies to more than just Mercurial changesets. I've tested in both mercurial and git that: (reviewboard 1.6RC2) 1. Renames without modifications 2. Deletes of empty files 3. Addition of empty files All three have similar properties in that the diff file contains meta data, but no actual diff headers, thus reviewboard internal patching fails. -- 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 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Comment #1 on issue 2184 by yuzi...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 In comparison, the following WORKS PERFECTLY 1. hg mv OLD_FILENAME NEW_FILENAME 2. vim NEW_FILENAME 3. (make some changes to the file) 4. hg commit -m "Rename a file without changing it" 5. hg export -g `hg parent --template '{node}'` > my_patch.diff 6. Log in to reviewboard 7. Create a review and upload this patch to reviewboard 8. Open your newly created review 9. Click [View Diff] This works as expected. The only difference is whether you modify the file after renaming it or not. It seems that patch just has trouble applying empty patches. -- 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 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly"
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 2184 by yuzi...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with "Diff did not apply cleanly" http://code.google.com/p/reviewboard/issues/detail?id=2184 * NOTE: Do not post confidential information in this bug report. * * If you need immediate support, please contact* * reviewbo...@googlegroups.com * What version are you running? Reviewboard 1.6 RC2 (also reproducible on RC1) What's the URL of the page containing the problem? What steps will reproduce the problem? 1. hg mv OLD_FILENAME NEW_FILENAME 2. hg commit -m "Rename a file without changing it" 3. hg export -g `hg parent --template '{node}'` > my_patch.diff 4. Log in to reviewboard 5. Create a review and upload this patch to reviewboard 6. Open your newly created review 7. Click [View Diff] What is the expected output? What do you see instead? You should see the usual diff view, with: Files changed: 1. NEW_FILENAME (was OLD_FILENAME) [no changes] Instead, you see: The patch to 'NEW_FILENAME' didn't apply cleanly. The temporary files have been left in '/tmp/reviewboard.IzIH1V' for debugging purposes. `patch` returned: patch: Only garbage was found in the patch input. Traceback (most recent call last): File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/views.py", line 151, in view_diff interdiffset, highlighting, True) File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/diffutils.py", line 1071, in get_diff_files large_data=True) File "/usr/local/lib/python2.7/dist-packages/Djblets-0.6.9-py2.7.egg/djblets/util/misc.py", line 166, in cache_memoize data = lookup_callable() File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/diffutils.py", line 1070, in enable_syntax_highlighting)), File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/diffutils.py", line 552, in get_chunks new = get_patched_file(old, filediff) File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/diffutils.py", line 374, in get_patched_file return patch(filediff.diff, buffer, filediff.dest_file) File "/usr/local/lib/python2.7/dist-packages/ReviewBoard-1.6rc2-py2.7.egg/reviewboard/diffviewer/diffutils.py", line 242, in patch (filename, tempdir, patch_output)) Exception: The patch to 'LICENSE' didn't apply cleanly. The temporary files have been left in '/tmp/reviewboard.IzIH1V' for debugging purposes. `patch` returned: patch: Only garbage was found in the patch input. Contents of /tmp/reviewboard.IzIH1V are attached to this bug report. What operating system are you using? What browser? Reviewboard is installed on Ubuntu (AWS/EC2) % lsb_release -a No LSB modules are available. Distributor ID: Ubuntu Description:Ubuntu 11.04 Release:11.04 Codename: natty % uname -a Linux hostname 2.6.38-8-virtual #42-Ubuntu SMP Mon Apr 11 04:06:34 UTC 2011 x86_64 x86_64 x86_64 GNU/Linux Please provide any additional information below. Attachments: NEW_FILENAME.diff 40 bytes tmpBNKTGj 14 bytes tmpBNKTGj-new 0 bytes -- 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.