You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Allen Wittenauer (Jira)" <ji...@apache.org> on 2020/09/08 02:00:11 UTC

[jira] [Created] (YETUS-983) test-patch confused by multiple commits that include rename

Allen Wittenauer created YETUS-983:
--------------------------------------

             Summary: test-patch confused by multiple commits that include rename
                 Key: YETUS-983
                 URL: https://issues.apache.org/jira/browse/YETUS-983
             Project: Yetus
          Issue Type: Bug
            Reporter: Allen Wittenauer


A long-standing bug has been that precommit is confused by a patch with multiple commits. Based upon some observations in a few private repositories, there appears to be some correlation between:

* a commit that renames a file
* a commit that changes the content of the file post-rename

and git format-patch --dry-run.   Because it is in dry-run mode, the rename doesn't take place, so the next commit in the sequence to the renamed file fails.

Squashing the commits is the obvious workaround.  Fixing this will likely require much more in-depth analysis of the patchfile and determining if dry-run should be skipped.





--
This message was sent by Atlassian Jira
(v8.3.4#803005)