fixup! Merge 'cat-file-smudge'

This reverts commit 7576a30b53, reversing
changes made to 6de9d8ad27.

The patch series has been sent upstream and modifications were necessary
as a consequence. We will merge the new version of the patch series in
one moment.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
This commit is contained in:
Johannes Schindelin
2016-08-24 14:07:14 +02:00
parent a1db819446
commit 7720efbbee
3 changed files with 8 additions and 116 deletions

View File

@@ -9,15 +9,15 @@ git-cat-file - Provide content or type and size information for repository objec
SYNOPSIS
--------
[verse]
'git cat-file' (-t [--allow-unknown-type]| -s [--allow-unknown-type]| -e | -p | <type> | --textconv | --smudge ) [--use-path=<path>] <object>
'git cat-file' (-t [--allow-unknown-type]| -s [--allow-unknown-type]| -e | -p | <type> | --textconv ) <object>
'git cat-file' (--batch | --batch-check) [--follow-symlinks]
DESCRIPTION
-----------
In its first form, the command provides the content or the type of an object in
the repository. The type is required unless `-t` or `-p` is used to find the
object type, or `-s` is used to find the object size, or `--textconv` or
`--smudge` is used (which imply type "blob").
object type, or `-s` is used to find the object size, or `--textconv` is used
(which implies type "blob").
In the second form, a list of objects (separated by linefeeds) is provided on
stdin, and the SHA-1, type, and size of each object is printed on stdout.
@@ -54,19 +54,8 @@ OPTIONS
--textconv::
Show the content as transformed by a textconv filter. In this case,
<object> has to be of the form <tree-ish>:<path>, or :<path> in
order to apply the filter to the content recorded in the index at
<path>.
--smudge::
Show the content as transformed by the smudge filter configured in
the current working tree for the given <path>. In this case,
<object> has to be of the form <tree-ish>:<path>, or :<path>.
--use-path=<path>::
For use with --textconv or --smudge, to allow specifying an object
name and a path separately, e.g. when it is difficult to figure out
the revision from which the blob came.
<object> has be of the form <tree-ish>:<path>, or :<path> in order
to apply the filter to the content recorded in the index at <path>.
--batch::
--batch=<format>::

View File

@@ -20,34 +20,6 @@ struct batch_options {
const char *format;
};
static const char *force_path;
static int smudge_object(const char *path, unsigned mode, unsigned char *sha1,
char **buf, unsigned long *size)
{
enum object_type type;
*buf = read_sha1_file(sha1, &type, size);
if (!*buf)
return error(_("cannot read object %s '%s'"),
sha1_to_hex(sha1), path);
if (type != OBJ_BLOB) {
free(*buf);
return error(_("blob expected for %s '%s'"),
sha1_to_hex(sha1), path);
}
if (S_ISREG(mode)) {
struct strbuf strbuf = STRBUF_INIT;
if (convert_to_working_tree(path, *buf, *size, &strbuf)) {
free(*buf);
*size = strbuf.len;
*buf = strbuf_detach(&strbuf, NULL);
}
}
return 0;
}
static int cat_one_file(int opt, const char *exp_type, const char *obj_name,
int unknown_type)
{
@@ -89,25 +61,12 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name,
case 'e':
return !has_sha1_file(sha1);
case 'w':
if (!force_path && !obj_context.path[0])
die("git cat-file --smudge %s: <object> must be <sha1:path>",
obj_name);
if (smudge_object(force_path ? force_path : obj_context.path,
force_path ? 0100644 : obj_context.mode,
sha1, &buf, &size))
return -1;
break;
case 'c':
if (!force_path && !obj_context.path[0])
if (!obj_context.path[0])
die("git cat-file --textconv %s: <object> must be <sha1:path>",
obj_name);
if (textconv_object(force_path ? force_path : obj_context.path,
force_path ? 0100644 : obj_context.mode,
sha1, 1, &buf, &size))
if (textconv_object(obj_context.path, obj_context.mode, sha1, 1, &buf, &size))
break;
case 'p':
@@ -481,7 +440,7 @@ static int batch_objects(struct batch_options *opt)
}
static const char * const cat_file_usage[] = {
N_("git cat-file (-t [--allow-unknown-type]|-s [--allow-unknown-type]|-e|-p|<type>|--textconv|--smudge) [--use-path=<path>] <object>"),
N_("git cat-file (-t [--allow-unknown-type]|-s [--allow-unknown-type]|-e|-p|<type>|--textconv) <object>"),
N_("git cat-file (--batch | --batch-check) [--follow-symlinks]"),
NULL
};
@@ -527,10 +486,6 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix)
OPT_CMDMODE('p', NULL, &opt, N_("pretty-print object's content"), 'p'),
OPT_CMDMODE(0, "textconv", &opt,
N_("for blob objects, run textconv on object's content"), 'c'),
OPT_CMDMODE(0, "smudge", &opt,
N_("for blob objects, run smudge filters on object's content"), 'w'),
OPT_STRING(0, "use-path", &force_path, N_("blob"),
N_("use a specific path for --textconv/--smudge")),
OPT_BOOL(0, "allow-unknown-type", &unknown_type,
N_("allow -s and -t to work with broken/corrupt objects")),
OPT_BOOL(0, "buffer", &batch.buffer_output, N_("buffer --batch output")),
@@ -573,11 +528,6 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix)
usage_with_options(cat_file_usage, options);
}
if (force_path && opt != 'c' && opt != 'w') {
error("--use-path=<path> needs --textconv or --smudge");
usage_with_options(cat_file_usage, options);
}
if (batch.buffer_output < 0)
batch.buffer_output = batch.all_objects;

View File

@@ -1,47 +0,0 @@
#!/bin/sh
test_description='git cat-file filters support'
. ./test-lib.sh
test_expect_success 'setup ' '
echo "*.txt eol=crlf diff=txt" >.gitattributes &&
echo "hello" | append_cr >world.txt &&
git add .gitattributes world.txt &&
test_tick &&
git commit -m "Initial commit"
'
has_cr() {
tr '\015' Q <"$1" | grep Q >/dev/null
}
test_expect_success 'no filters with `git show`' '
git show HEAD:world.txt >actual &&
! has_cr actual
'
test_expect_success 'no filters with cat-file' '
git cat-file blob HEAD:world.txt >actual &&
! has_cr actual
'
test_expect_success 'cat-file --smudge converts to worktree version' '
git cat-file --smudge HEAD:world.txt >actual &&
has_cr actual
'
test_expect_success 'cat-file --smudge --use-path=<path> works' '
sha1=$(git rev-parse -q --verify HEAD:world.txt) &&
git cat-file --smudge --use-path=world.txt $sha1 >actual &&
has_cr actual
'
test_expect_success 'cat-file --textconv --use-path=<path> works' '
sha1=$(git rev-parse -q --verify HEAD:world.txt) &&
test_config diff.txt.textconv "tr A-Za-z N-ZA-Mn-za-m <" &&
git cat-file --textconv --use-path=hello.txt $sha1 >rot13 &&
test uryyb = "$(cat rot13)"
'
test_done