I'm currently trying to to code-style checking on the PRs of a (github) repository, and I want to deliver patches to the submitters with which they can easily fix the codestyle. To this end, I'm pulling down their PR, run our uncrustify script over it to fix any style errors, and want to create a .patch file they can easily apply. However, it consistently breaks on some files.
I do (git version 1.7.10.4 with core.autocrlf=input
, core.filemode=false
):
$ git checkout pr-branch
$ git log -1 (shows: commit dbb8d3f)
$ git status (nothing to commit, working directory clean)
$ <run the code styler script, which modifies some files>
$ git diff > ../style.patch (so the patch file lands outside the repo)
$ git reset --hard HEAD (to simulate the situation at the submitter's end)
$ git log -1 (shows: commit dbb8d3f)
$ git status (nothing to commit, working directory clean, so we are where we started)
$ git apply ../style.patch
error: patch failed: somefile.cpp:195
error: somefile.cpp: patch does not apply (same output using the --check option)
This only applies to some files, not all of them. I don't know how to troubleshoot this, i.e. how to get git to tell me exactly where it goes wrong - it only tells me a hunk# when I dig, but that's still pretty huge.
What I've tried so far (without success):
apply --reverse
, apply --whitespace=nowarn
diff HEAD
instead of diff
aloneformat-patch
, delete the dummy commit, apply patch with git-am
with or without -3
, or apply with git-apply
patch
commandI don't know what could be wrong with the diff. Whitespace things should only warn, right? In any case, I won't want to ignore them, since it's a style fix which obviously involves whitespace.
How can I fix/diagnose this or even find out where it bails exactly? Would it help if I posted the diff of one of the culprit files? What baffles me also is that the committ works without problem, but the patch created from the commit does not??
After wrestling with this for several hours I'm at the end of my knowledge...
Update:
You can use git apply -v
to see more detailed info about what's going on, git apply --check
to just verify the operation, or git apply --index
to rebuild the local index file.
Based on your comment, it seems your local index was corrupted, and so index
solved it.
I will leave my original answer and the comments mostly to give people context on what was going on, as I suspect other people would jump to the same initial conclusions I had based on the problem description.
------
Most likely nothing's wrong with the diff. Instead look at the target git repository. While you are doing git reset --hard HEAD
, nothing guarantees you that the HEAD
on that other repository is the same as the HEAD
on your.
Do git log
on the target repo and look at the commit at the top. Is it the same as the one you produced the diff from? Most likely it is not. Look down the history and check if the commit you need is there. If it is, then the target repo is ahead of yours, and you have to go back, do git pull
(or git rebase
) and produce a new diff. If it isn't, then the target repo is behind yours, and you need to do git pull
(or git rebase
) on the target repo to bring it up to speed.
Keep in mind that if you have other people committing to your "master" repo (the one where bot yours and the target repositories are pulling from), you might have to git pull
both repositories, to get them to a reasonably recent common commit.