From: Joey Hess <joeyh@joeyh.name>
To: git@vger.kernel.org
Cc: Joey Hess <joeyh@joeyh.name>
Subject: [PATCH v5 7/8] use smudgeToFile filter in git am
Date: Mon, 11 Jul 2016 18:45:11 -0400 [thread overview]
Message-ID: <1468277112-9909-8-git-send-email-joeyh@joeyh.name> (raw)
In-Reply-To: <1468277112-9909-1-git-send-email-joeyh@joeyh.name>
git am updates the work tree and so should use the smudgeToFile filter.
This includes some refactoring into convert_to_working_tree_filter_to_file
to make it check the file after running the smudgeToFile command, and clean
up from a failing command.
Signed-off-by: Joey Hess <joeyh@joeyh.name>
---
apply.c | 16 ++++++++++++++++
convert.c | 25 +++++++++++++++++++++++--
entry.c | 21 ++++-----------------
t/t0021-conversion.sh | 13 +++++++++++++
4 files changed, 56 insertions(+), 19 deletions(-)
diff --git a/apply.c b/apply.c
index 4a6b2db..7db8344 100644
--- a/apply.c
+++ b/apply.c
@@ -4322,6 +4322,22 @@ static int try_create_file(const char *path, unsigned int mode, const char *buf,
if (fd < 0)
return 1;
+ if (can_smudge_to_file(path)) {
+ close(fd);
+ fd = convert_to_working_tree_filter_to_file(path, path, buf, size);
+ if (fd < 0) {
+ /* smudgeToFile filter failed; continue
+ * with regular file creation instead. */
+ fd = open(path, O_CREAT | O_EXCL | O_WRONLY, (mode & 0100) ? 0777 : 0666);
+ if (fd < 0)
+ return -1;
+ }
+ else {
+ close(fd);
+ return 0;
+ }
+ }
+
if (convert_to_working_tree(path, buf, size, &nbuf)) {
size = nbuf.len;
buf = nbuf.buf;
diff --git a/convert.c b/convert.c
index e1b0b44..3746ad5 100644
--- a/convert.c
+++ b/convert.c
@@ -1030,13 +1030,34 @@ int convert_to_working_tree(const char *path, const char *src, size_t len, struc
return convert_to_working_tree_internal(path, NULL, src, len, dst, 0);
}
+/*
+ * Returns fd open to read the worktree file on success.
+ * On failure, the worktree file will not exist.
+ */
int convert_to_working_tree_filter_to_file(const char *path, const char *destpath, const char *src, size_t len)
{
struct strbuf output = STRBUF_INIT;
- int ret = convert_to_working_tree_internal(path, destpath, src, len, &output, 0);
+ int ok = convert_to_working_tree_internal(path, destpath, src, len, &output, 0);
/* The smudgeToFile filter stdout is not used. */
strbuf_release(&output);
- return ret;
+ if (ok) {
+ /*
+ * Open the file to make sure that it's present
+ * (and readable) after the command populated it.
+ */
+ int fd = open(path, O_RDONLY);
+ if (fd < 0)
+ unlink(path);
+ return fd;
+ }
+ else {
+ /*
+ * The command could have created the file before failing,
+ * so delete it.
+ */
+ unlink(path);
+ return -1;
+ }
}
int renormalize_buffer(const char *path, const char *src, size_t len, struct strbuf *dst)
diff --git a/entry.c b/entry.c
index 7811e31..40662eb 100644
--- a/entry.c
+++ b/entry.c
@@ -191,34 +191,21 @@ static int write_entry(struct cache_entry *ce,
if (smudge_to_file) {
close(fd);
- if (convert_to_working_tree_filter_to_file(ce->name, path, new, size)) {
+ fd = convert_to_working_tree_filter_to_file(ce->name, path, new, size);
+ if (fd >= 0) {
free(new);
- /*
- * The smudgeToFile filter may have replaced
- * or deleted the file; reopen it to make
- * sure that the file exists.
- */
- fd = open(path, O_RDONLY);
- if (fd < 0)
- return error_errno("unable to create file %s", path);
if (!to_tempfile)
fstat_done = fstat_output(fd, state, &st);
close(fd);
}
else {
- /*
- * The failing smudgeToFile filter may have
- * deleted or replaced the file; delete
- * the file and re-open for recovery write.
- */
- unlink(path);
+ /* Fall through to normal write below. */
+ smudge_to_file = 0;
fd = open_output_fd(path, ce, to_tempfile);
if (fd < 0) {
free(new);
return error_errno("unable to create file %s", path);
}
- /* Fall through to normal write below. */
- smudge_to_file = 0;
}
}
diff --git a/t/t0021-conversion.sh b/t/t0021-conversion.sh
index 0efad9b..42b28aa 100755
--- a/t/t0021-conversion.sh
+++ b/t/t0021-conversion.sh
@@ -334,6 +334,19 @@ test_expect_success 'recovery from failure of smudgeToFile filter that deletes t
test_cmp test fstest.t
'
+test_expect_success 'smudgeToFile filter is used by git am' '
+ test_config filter.rot13.smudgeToFile ./rot13-to-file.sh &&
+
+ git commit fstest.t -m "added fstest.t" &&
+ git format-patch HEAD^ --stdout >fstest.patch &&
+ git reset --hard HEAD^ &&
+ git am fstest.patch &&
+
+ test -e rot13-to-file.ran &&
+ rm -f rot13-to-file.ran &&
+ test_cmp test fstest.t
+'
+
test_expect_success 'cleanFromFile filter is not used when clean filter is not configured' '
test_config filter.noclean.smudge ./rot13.sh &&
test_config filter.noclean.cleanFromFile ./rot13-from-file.sh &&
--
2.8.1
next prev parent reply other threads:[~2016-07-11 22:47 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-11 22:45 [PATCH v5 0/8] extend smudge/clean filters with direct file access (for pu) Joey Hess
2016-07-11 22:45 ` [PATCH v5 1/8] clarify %f documentation Joey Hess
2016-07-11 22:45 ` [PATCH v5 2/8] add smudgeToFile and cleanFromFile filter configs Joey Hess
2016-07-13 16:02 ` Lars Schneider
2016-07-11 22:45 ` [PATCH v5 3/8] use cleanFromFile in git add Joey Hess
2016-07-11 22:45 ` [PATCH v5 4/8] use smudgeToFile in git checkout etc Joey Hess
2016-07-11 22:45 ` [PATCH v5 5/8] warn on unusable smudgeToFile/cleanFromFile config Joey Hess
2016-07-11 22:45 ` [PATCH v5 6/8] better recovery from failure of smudgeToFile filter Joey Hess
2016-07-11 22:45 ` Joey Hess [this message]
2016-07-11 22:45 ` [PATCH v5 8/8] use smudgeToFile filter in recursive merge Joey Hess
2016-07-12 19:52 ` [PATCH v5 0/8] extend smudge/clean filters with direct file access (for pu) Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1468277112-9909-8-git-send-email-joeyh@joeyh.name \
--to=joeyh@joeyh.name \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).