From: Junio C Hamano Date: Tue, 26 Apr 2005 01:26:45 +0000 (-0700) Subject: [PATCH] Introduce diff-tree-helper. X-Git-Tag: v0.99~741 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=d1df5743809614241883ecad51876607cf432034;p=git.git [PATCH] Introduce diff-tree-helper. This patch introduces a new program, diff-tree-helper. It reads output from diff-cache and diff-tree, and produces a patch file. The diff format customization can be done the same way the show-diff uses; the same external diff interface introduced by the previous patch to drive diff from show-diff is used so this is not surprising. It is used like the following examples: $ diff-cache --cached -z | diff-tree-helper -z -R paths... $ diff-tree -r -z | diff-tree-helper -z paths... - As usual, the use of the -z flag is recommended in the script to pass NUL-terminated filenames through the pipe between commands. - The -R flag is used to generate reverse diff. It does not matter for diff-tree case, but it is sometimes useful to get a patch in the desired direction out of diff-cache. - The paths parameters are used to restrict the paths that appears in the output. Again this is useful to use with diff-cache, which, unlike diff-tree, does not take such paths restriction parameters. Signed-off-by: Junio C Hamano Signed-off-by: Linus Torvalds --- diff --git a/Makefile b/Makefile index e108b98e6..2d2913b6b 100644 --- a/Makefile +++ b/Makefile @@ -17,7 +17,8 @@ SCRIPTS=git-merge-one-file-script git-prune-script git-pull-script git-tag-scrip PROG= update-cache show-diff init-db write-tree read-tree commit-tree \ cat-file fsck-cache checkout-cache diff-tree rev-tree show-files \ check-files ls-tree merge-base merge-cache unpack-file git-export \ - diff-cache convert-cache http-pull rpush rpull rev-list git-mktag + diff-cache convert-cache http-pull rpush rpull rev-list git-mktag \ + diff-tree-helper all: $(PROG) @@ -28,6 +29,9 @@ LIB_OBJS=read-cache.o sha1_file.o usage.o object.o commit.o tree.o blob.o LIB_FILE=libgit.a LIB_H=cache.h object.h +LIB_H += strbuf.h +LIB_OBJS += strbuf.o + LIB_H += diff.h LIB_OBJS += diff.o diff --git a/diff-tree-helper.c b/diff-tree-helper.c new file mode 100644 index 000000000..293e00b70 --- /dev/null +++ b/diff-tree-helper.c @@ -0,0 +1,302 @@ +#include "cache.h" +#include "strbuf.h" +#include "diff.h" + +static int matches_pathspec(const char *name, char **spec, int cnt) +{ + int i; + int namelen = strlen(name); + for (i = 0; i < cnt; i++) { + int speclen = strlen(spec[i]); + if (! strncmp(spec[i], name, speclen) && + speclen <= namelen && + (name[speclen] == 0 || + name[speclen] == '/')) + return 1; + } + return 0; +} + +static int parse_oneside_change(const char *cp, unsigned char *sha1, + char *path) { + int ch; + while ((ch = *cp) && '0' <= ch && ch <= '7') + cp++; /* skip mode bits */ + if (strncmp(cp, "\tblob\t", 6)) + return -1; + cp += 6; + if (get_sha1_hex(cp, sha1)) + return -1; + cp += 40; + if (*cp++ != '\t') + return -1; + strcpy(path, cp); + return 0; +} + +#define STATUS_CACHED 0 /* cached and sha1 valid */ +#define STATUS_ABSENT 1 /* diff-tree says old removed or new added */ +#define STATUS_UNCACHED 2 /* diff-cache output: read from working tree */ + +static int parse_diff_tree_output(const char *buf, + unsigned char *old_sha1, + int *old_status, + unsigned char *new_sha1, + int *new_status, + char *path) { + const char *cp = buf; + int ch; + static unsigned char null_sha[20] = { 0, }; + + switch (*cp++) { + case '+': + *old_status = STATUS_ABSENT; + *new_status = (memcmp(new_sha1, null_sha, sizeof(null_sha)) ? + STATUS_CACHED : STATUS_UNCACHED); + return parse_oneside_change(cp, new_sha1, path); + case '-': + *new_status = STATUS_ABSENT; + *old_status = (memcmp(old_sha1, null_sha, sizeof(null_sha)) ? + STATUS_CACHED : STATUS_UNCACHED); + return parse_oneside_change(cp, old_sha1, path); + case '*': + break; + default: + return -1; + } + + /* This is for '*' entries */ + while ((ch = *cp) && ('0' <= ch && ch <= '7')) + cp++; /* skip mode bits */ + if (strncmp(cp, "->", 2)) + return -1; + cp += 2; + while ((ch = *cp) && ('0' <= ch && ch <= '7')) + cp++; /* skip mode bits */ + if (strncmp(cp, "\tblob\t", 6)) + return -1; + cp += 6; + if (get_sha1_hex(cp, old_sha1)) + return -1; + cp += 40; + if (strncmp(cp, "->", 2)) + return -1; + cp += 2; + if (get_sha1_hex(cp, new_sha1)) + return -1; + cp += 40; + if (*cp++ != '\t') + return -1; + strcpy(path, cp); + *old_status = (memcmp(old_sha1, null_sha, sizeof(null_sha)) ? + STATUS_CACHED : STATUS_UNCACHED); + *new_status = (memcmp(new_sha1, null_sha, sizeof(null_sha)) ? + STATUS_CACHED : STATUS_UNCACHED); + return 0; +} + +static int sha1err(const char *path, const unsigned char *sha1) +{ + return error("diff-tree-helper: unable to read sha1 file of %s (%s)", + path, sha1_to_hex(sha1)); +} + +static int fserr(const char *path) +{ + return error("diff-tree-helper: unable to read file %s", path); +} + +static char *map_whole_file(const char *path, unsigned long *size) { + int fd; + struct stat st; + void *buf; + + if ((fd = open(path, O_RDONLY)) < 0) { + error("diff-tree-helper: unable to read file %s", path); + return 0; + } + if (fstat(fd, &st) < 0) { + close(fd); + error("diff-tree-helper: unable to stat file %s", path); + return 0; + } + *size = st.st_size; + buf = mmap(NULL, st.st_size, PROT_READ, MAP_PRIVATE, fd, 0); + close(fd); + return buf; +} + +static int show_diff(const unsigned char *old_sha1, int old_status, + const unsigned char *new_sha1, int new_status, + const char *path, int reverse_diff) +{ + char other[PATH_MAX]; + unsigned long size; + char type[20]; + int fd; + int reverse; + void *blob = 0; + const char *fs = 0; + int need_unmap = 0; + int need_unlink = 0; + + + switch (old_status) { + case STATUS_CACHED: + blob = read_sha1_file(old_sha1, type, &size); + if (! blob) + return sha1err(path, old_sha1); + + switch (new_status) { + case STATUS_CACHED: + strcpy(other, ".diff_tree_helper_XXXXXX"); + fd = mkstemp(other); + if (fd < 0) + die("unable to create temp-file"); + if (write(fd, blob, size) != size) + die("unable to write temp-file"); + close(fd); + free(blob); + + blob = read_sha1_file(new_sha1, type, &size); + if (! blob) + return sha1err(path, new_sha1); + + need_unlink = 1; + /* new = blob, old = fs */ + reverse = !reverse_diff; + fs = other; + break; + + case STATUS_ABSENT: + case STATUS_UNCACHED: + fs = ((new_status == STATUS_ABSENT) ? + "/dev/null" : path); + reverse = reverse_diff; + break; + + default: + reverse = reverse_diff; + } + break; + + case STATUS_ABSENT: + switch (new_status) { + case STATUS_CACHED: + blob = read_sha1_file(new_sha1, type, &size); + if (! blob) + return sha1err(path, new_sha1); + /* old = fs, new = blob */ + fs = "/dev/null"; + reverse = !reverse_diff; + break; + + case STATUS_ABSENT: + return error("diff-tree-helper: absent from both old and new?"); + case STATUS_UNCACHED: + fs = path; + blob = strdup(""); + size = 0; + /* old = blob, new = fs */ + reverse = reverse_diff; + break; + default: + reverse = reverse_diff; + } + break; + + case STATUS_UNCACHED: + fs = path; /* old = fs, new = blob */ + reverse = !reverse_diff; + + switch (new_status) { + case STATUS_CACHED: + blob = read_sha1_file(new_sha1, type, &size); + if (! blob) + return sha1err(path, new_sha1); + break; + + case STATUS_ABSENT: + blob = strdup(""); + size = 0; + break; + + case STATUS_UNCACHED: + /* old = fs */ + blob = map_whole_file(path, &size); + if (! blob) + return fserr(path); + need_unmap = 1; + break; + default: + reverse = reverse_diff; + } + break; + + default: + reverse = reverse_diff; + } + + if (fs) + show_differences(fs, + path, /* label */ + blob, + size, + reverse /* 0: diff blob fs + 1: diff fs blob */); + + if (need_unlink) + unlink(other); + if (need_unmap && blob) + munmap(blob, size); + else + free(blob); + return 0; +} + +static const char *diff_tree_helper_usage = +"diff-tree-helper [-R] [-z] paths..."; + +int main(int ac, char **av) { + struct strbuf sb; + int reverse_diff = 0; + int line_termination = '\n'; + + strbuf_init(&sb); + + while (1 < ac && av[1][0] == '-') { + if (av[1][1] == 'R') + reverse_diff = 1; + else if (av[1][1] == 'z') + line_termination = 0; + else + usage(diff_tree_helper_usage); + ac--; av++; + } + /* the remaining parameters are paths patterns */ + + prepare_diff_cmd(); + + while (1) { + int old_status, new_status; + unsigned char old_sha1[20], new_sha1[20]; + char path[PATH_MAX]; + read_line(&sb, stdin, line_termination); + if (sb.eof) + break; + if (parse_diff_tree_output(sb.buf, + old_sha1, &old_status, + new_sha1, &new_status, + path)) { + fprintf(stderr, "cannot parse %s\n", sb.buf); + continue; + } + if (1 < ac && ! matches_pathspec(path, av+1, ac-1)) + continue; + + show_diff(old_sha1, old_status, + new_sha1, new_status, + path, reverse_diff); + } + return 0; +} diff --git a/strbuf.c b/strbuf.c new file mode 100644 index 000000000..dac945c7c --- /dev/null +++ b/strbuf.c @@ -0,0 +1,43 @@ +#include +#include +#include "strbuf.h" + +void strbuf_init(struct strbuf *sb) { + sb->buf = 0; + sb->eof = sb->alloc = sb->len = 0; +} + +static void strbuf_begin(struct strbuf *sb) { + free(sb->buf); + strbuf_init(sb); +} + +static void inline strbuf_add(struct strbuf *sb, int ch) { + if (sb->alloc <= sb->len) { + sb->alloc = sb->alloc * 3 / 2 + 16; + sb->buf = realloc(sb->buf, sb->alloc); + } + sb->buf[sb->len++] = ch; +} + +static void strbuf_end(struct strbuf *sb) { + strbuf_add(sb, 0); +} + +void read_line(struct strbuf *sb, FILE *fp, int term) { + int ch; + strbuf_begin(sb); + if (feof(fp)) { + sb->eof = 1; + return; + } + while ((ch = fgetc(fp)) != EOF) { + if (ch == term) + break; + strbuf_add(sb, ch); + } + if (sb->len == 0) + sb->eof = 1; + strbuf_end(sb); +} + diff --git a/strbuf.h b/strbuf.h new file mode 100644 index 000000000..da678e506 --- /dev/null +++ b/strbuf.h @@ -0,0 +1,13 @@ +#ifndef STRBUF_H +#define STRBUF_H +struct strbuf { + int alloc; + int len; + int eof; + unsigned char *buf; +}; + +extern void strbuf_init(struct strbuf *); +extern void read_line(struct strbuf *, FILE *, int); + +#endif /* STRBUF_H */