diff options
author | Junio C Hamano <gitster@pobox.com> | 2008-04-06 19:21:45 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-04-06 19:21:45 -0700 |
commit | ee5a317e0130ab2db59db97c644576335530512d (patch) | |
tree | 840d35f9f8b587a0d9dc12df15205fcd1265bafc /t | |
parent | f8dd64fdbf47ac65c1b8d2ed25b789ae2655f8c8 (diff) | |
download | git-ee5a317e0130ab2db59db97c644576335530512d.tar.gz git-ee5a317e0130ab2db59db97c644576335530512d.tar.xz |
Fix "git apply" to correctly enforce "match at the beginning"
An earlier commit 4be6096 (apply --unidiff-zero: loosen sanity checks for
--unidiff=0 patches, 2006-09-17) made match_beginning and match_end
computed incorrectly. If a hunk inserts at the beginning, old position
recorded at the hunk is line 0, and if a hunk changes at the beginning, it
is line 1. The new test added to t4104 exposes that the old code did not
insist on matching at the beginning for a patch to add a line to an empty
file.
An even older 65aadb9 (apply: force matching at the beginning.,
2006-05-24) was equally wrong in that it tried to take hints from the
number of leading context lines, to decide if the hunk must match at the
beginning, but we can just look at the line number in the hunk to decide.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
-rwxr-xr-x | t/t4104-apply-boundary.sh | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/t/t4104-apply-boundary.sh b/t/t4104-apply-boundary.sh index 64f34e329..43943ab8c 100755 --- a/t/t4104-apply-boundary.sh +++ b/t/t4104-apply-boundary.sh @@ -112,4 +112,17 @@ do ' done +test_expect_success 'two lines' ' + + >file && + git add file && + echo aaa >file && + git diff >patch && + git add file && + echo bbb >file && + git add file && + test_must_fail git apply --check patch + +' + test_done |