summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTatyana Krasnukha <tatyana@synopsys.com>2018-02-15 01:29:34 +0100
committerJunio C Hamano <gitster@pobox.com>2018-02-15 11:09:02 -0800
commite454ad4becff73588b1b3cc24dba3a0e740c7f7c (patch)
treee15d28dc555858b6227d1b2ca96276ded66cd697
parentf16ef7bd4c37fba8c030b87610f646a1fce5d52d (diff)
downloadgit-tk/apply-dev-null-verify-name-fix.tar.gz
apply: handle Subversion diffs with /dev/null gracefullytk/apply-dev-null-verify-name-fix
Subversion generates diffs that can contain lines like this one: --- /dev/null (nonexistent) Let's teach Git's apply machinery to handle such a line gracefully. This fixes https://github.com/git-for-windows/git/isues/1489 Signed-off-by: Tatyana Krasnukha <tatyana@synopsys.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--apply.c2
-rwxr-xr-xt/t4135-apply-weird-filenames.sh2
2 files changed, 2 insertions, 2 deletions
diff --git a/apply.c b/apply.c
index 0c7b25989c..3aadbae305 100644
--- a/apply.c
+++ b/apply.c
@@ -975,7 +975,7 @@ static int gitdiff_verify_name(struct apply_state *state,
}
free(another);
} else {
- if (!starts_with(line, "/dev/null\n"))
+ if (!is_dev_null(line))
return error(_("git apply: bad git-diff - expected /dev/null on line %d"), state->linenr);
}
diff --git a/t/t4135-apply-weird-filenames.sh b/t/t4135-apply-weird-filenames.sh
index b14b808578..c7c688fcc4 100755
--- a/t/t4135-apply-weird-filenames.sh
+++ b/t/t4135-apply-weird-filenames.sh
@@ -100,7 +100,7 @@ deleted file mode 100644
-
EOF
-test_expect_failure 'apply handles a diff generated by Subversion' '
+test_expect_success 'apply handles a diff generated by Subversion' '
>Makefile &&
git apply -p2 diff-from-svn &&
test_path_is_missing Makefile