2008-05-24 07:28:56 +02:00
|
|
|
/*
|
|
|
|
* "diff --no-index" support
|
|
|
|
* Copyright (c) 2007 by Johannes Schindelin
|
|
|
|
* Copyright (c) 2008 by Junio C Hamano
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include "cache.h"
|
|
|
|
#include "color.h"
|
|
|
|
#include "commit.h"
|
|
|
|
#include "blob.h"
|
|
|
|
#include "tag.h"
|
|
|
|
#include "diff.h"
|
|
|
|
#include "diffcore.h"
|
|
|
|
#include "revision.h"
|
|
|
|
#include "log-tree.h"
|
|
|
|
#include "builtin.h"
|
2008-07-21 20:03:49 +02:00
|
|
|
#include "string-list.h"
|
2008-05-24 07:28:56 +02:00
|
|
|
|
2008-07-21 20:03:49 +02:00
|
|
|
static int read_directory(const char *path, struct string_list *list)
|
2008-05-24 07:28:56 +02:00
|
|
|
{
|
|
|
|
DIR *dir;
|
|
|
|
struct dirent *e;
|
|
|
|
|
|
|
|
if (!(dir = opendir(path)))
|
|
|
|
return error("Could not open directory %s", path);
|
|
|
|
|
|
|
|
while ((e = readdir(dir)))
|
|
|
|
if (strcmp(".", e->d_name) && strcmp("..", e->d_name))
|
2010-06-26 01:41:35 +02:00
|
|
|
string_list_insert(list, e->d_name);
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
closedir(dir);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2012-06-28 05:14:47 +02:00
|
|
|
/*
|
|
|
|
* This should be "(standard input)" or something, but it will
|
|
|
|
* probably expose many more breakages in the way no-index code
|
|
|
|
* is bolted onto the diff callchain.
|
|
|
|
*/
|
|
|
|
static const char file_from_standard_input[] = "-";
|
|
|
|
|
2008-05-24 07:28:56 +02:00
|
|
|
static int get_mode(const char *path, int *mode)
|
|
|
|
{
|
|
|
|
struct stat st;
|
|
|
|
|
|
|
|
if (!path || !strcmp(path, "/dev/null"))
|
|
|
|
*mode = 0;
|
2013-05-02 21:26:08 +02:00
|
|
|
#ifdef GIT_WINDOWS_NATIVE
|
2009-03-07 16:51:33 +01:00
|
|
|
else if (!strcasecmp(path, "nul"))
|
|
|
|
*mode = 0;
|
|
|
|
#endif
|
2012-06-28 05:14:47 +02:00
|
|
|
else if (path == file_from_standard_input)
|
2008-05-24 07:28:56 +02:00
|
|
|
*mode = create_ce_mode(0666);
|
2009-01-29 17:30:51 +01:00
|
|
|
else if (lstat(path, &st))
|
2008-05-24 07:28:56 +02:00
|
|
|
return error("Could not access '%s'", path);
|
|
|
|
else
|
|
|
|
*mode = st.st_mode;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2012-06-28 05:14:47 +02:00
|
|
|
static int populate_from_stdin(struct diff_filespec *s)
|
|
|
|
{
|
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
size_t size = 0;
|
|
|
|
|
|
|
|
if (strbuf_read(&buf, 0, 0) < 0)
|
|
|
|
return error("error while reading from stdin %s",
|
|
|
|
strerror(errno));
|
|
|
|
|
|
|
|
s->should_munmap = 0;
|
|
|
|
s->data = strbuf_detach(&buf, &size);
|
|
|
|
s->size = size;
|
|
|
|
s->should_free = 1;
|
|
|
|
s->is_stdin = 1;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct diff_filespec *noindex_filespec(const char *name, int mode)
|
|
|
|
{
|
|
|
|
struct diff_filespec *s;
|
|
|
|
|
|
|
|
if (!name)
|
|
|
|
name = "/dev/null";
|
|
|
|
s = alloc_filespec(name);
|
2012-08-27 20:54:28 +02:00
|
|
|
fill_filespec(s, null_sha1, 0, mode);
|
2012-06-28 05:14:47 +02:00
|
|
|
if (name == file_from_standard_input)
|
|
|
|
populate_from_stdin(s);
|
|
|
|
return s;
|
|
|
|
}
|
|
|
|
|
2008-05-24 07:28:56 +02:00
|
|
|
static int queue_diff(struct diff_options *o,
|
2012-04-25 21:37:38 +02:00
|
|
|
const char *name1, const char *name2)
|
2008-05-24 07:28:56 +02:00
|
|
|
{
|
|
|
|
int mode1 = 0, mode2 = 0;
|
|
|
|
|
|
|
|
if (get_mode(name1, &mode1) || get_mode(name2, &mode2))
|
|
|
|
return -1;
|
|
|
|
|
|
|
|
if (mode1 && mode2 && S_ISDIR(mode1) != S_ISDIR(mode2))
|
|
|
|
return error("file/directory conflict: %s, %s", name1, name2);
|
|
|
|
|
|
|
|
if (S_ISDIR(mode1) || S_ISDIR(mode2)) {
|
2012-04-25 21:37:38 +02:00
|
|
|
struct strbuf buffer1 = STRBUF_INIT;
|
|
|
|
struct strbuf buffer2 = STRBUF_INIT;
|
2010-07-04 21:46:19 +02:00
|
|
|
struct string_list p1 = STRING_LIST_INIT_DUP;
|
|
|
|
struct string_list p2 = STRING_LIST_INIT_DUP;
|
2012-04-25 21:37:38 +02:00
|
|
|
int i1, i2, ret = 0;
|
2012-05-16 16:28:31 +02:00
|
|
|
size_t len1 = 0, len2 = 0;
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
if (name1 && read_directory(name1, &p1))
|
|
|
|
return -1;
|
|
|
|
if (name2 && read_directory(name2, &p2)) {
|
2008-07-21 20:03:49 +02:00
|
|
|
string_list_clear(&p1, 0);
|
2008-05-24 07:28:56 +02:00
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (name1) {
|
2012-04-25 21:37:38 +02:00
|
|
|
strbuf_addstr(&buffer1, name1);
|
|
|
|
if (buffer1.len && buffer1.buf[buffer1.len - 1] != '/')
|
|
|
|
strbuf_addch(&buffer1, '/');
|
2012-05-16 16:28:31 +02:00
|
|
|
len1 = buffer1.len;
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
if (name2) {
|
2012-04-25 21:37:38 +02:00
|
|
|
strbuf_addstr(&buffer2, name2);
|
|
|
|
if (buffer2.len && buffer2.buf[buffer2.len - 1] != '/')
|
|
|
|
strbuf_addch(&buffer2, '/');
|
2012-05-16 16:28:31 +02:00
|
|
|
len2 = buffer2.len;
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
for (i1 = i2 = 0; !ret && (i1 < p1.nr || i2 < p2.nr); ) {
|
|
|
|
const char *n1, *n2;
|
|
|
|
int comp;
|
|
|
|
|
2012-05-16 16:28:31 +02:00
|
|
|
strbuf_setlen(&buffer1, len1);
|
|
|
|
strbuf_setlen(&buffer2, len2);
|
|
|
|
|
2008-05-24 07:28:56 +02:00
|
|
|
if (i1 == p1.nr)
|
|
|
|
comp = 1;
|
|
|
|
else if (i2 == p2.nr)
|
|
|
|
comp = -1;
|
|
|
|
else
|
2012-04-25 21:37:38 +02:00
|
|
|
comp = strcmp(p1.items[i1].string, p2.items[i2].string);
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
if (comp > 0)
|
|
|
|
n1 = NULL;
|
|
|
|
else {
|
2012-04-25 21:37:38 +02:00
|
|
|
strbuf_addstr(&buffer1, p1.items[i1++].string);
|
|
|
|
n1 = buffer1.buf;
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
if (comp < 0)
|
|
|
|
n2 = NULL;
|
|
|
|
else {
|
2012-04-25 21:37:38 +02:00
|
|
|
strbuf_addstr(&buffer2, p2.items[i2++].string);
|
|
|
|
n2 = buffer2.buf;
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
ret = queue_diff(o, n1, n2);
|
|
|
|
}
|
2008-07-21 20:03:49 +02:00
|
|
|
string_list_clear(&p1, 0);
|
|
|
|
string_list_clear(&p2, 0);
|
2012-05-16 16:50:31 +02:00
|
|
|
strbuf_release(&buffer1);
|
|
|
|
strbuf_release(&buffer2);
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
return ret;
|
|
|
|
} else {
|
|
|
|
struct diff_filespec *d1, *d2;
|
|
|
|
|
|
|
|
if (DIFF_OPT_TST(o, REVERSE_DIFF)) {
|
|
|
|
unsigned tmp;
|
|
|
|
const char *tmp_c;
|
|
|
|
tmp = mode1; mode1 = mode2; mode2 = tmp;
|
|
|
|
tmp_c = name1; name1 = name2; name2 = tmp_c;
|
|
|
|
}
|
|
|
|
|
2012-06-28 05:14:47 +02:00
|
|
|
d1 = noindex_filespec(name1, mode1);
|
|
|
|
d2 = noindex_filespec(name2, mode2);
|
2008-05-24 07:28:56 +02:00
|
|
|
diff_queue(&diff_queued_diff, d1, d2);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
void diff_no_index(struct rev_info *revs,
|
|
|
|
int argc, const char **argv,
|
2013-12-11 10:58:42 +01:00
|
|
|
const char *prefix)
|
2008-05-24 07:28:56 +02:00
|
|
|
{
|
2012-06-27 21:05:52 +02:00
|
|
|
int i, prefixlen;
|
2013-07-18 02:18:32 +02:00
|
|
|
unsigned deprecated_show_diff_q_option_used = 0;
|
2012-06-27 20:51:15 +02:00
|
|
|
const char *paths[2];
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
diff_setup(&revs->diffopt);
|
|
|
|
for (i = 1; i < argc - 2; ) {
|
|
|
|
int j;
|
|
|
|
if (!strcmp(argv[i], "--no-index"))
|
|
|
|
i++;
|
2009-01-07 12:15:30 +01:00
|
|
|
else if (!strcmp(argv[i], "-q")) {
|
2013-07-18 02:18:32 +02:00
|
|
|
deprecated_show_diff_q_option_used = 1;
|
2009-01-07 12:15:30 +01:00
|
|
|
i++;
|
|
|
|
}
|
2009-01-06 19:53:32 +01:00
|
|
|
else if (!strcmp(argv[i], "--"))
|
|
|
|
i++;
|
2008-05-24 07:28:56 +02:00
|
|
|
else {
|
|
|
|
j = diff_opt_parse(&revs->diffopt, argv + i, argc - i);
|
diff-no-index: correctly diagnose error return from diff_opt_parse()
diff_opt_parse() returns the number of options parsed, or often
returns error() which is defined to return -1. Yes, return value of
0 is "I did not process that option at all", which should cause the
caller to say that, but negative return should not be forgotten.
This bug caused "diff --no-index" to infinitely show the same error
message because the returned value was used to decrement the loop
control variable, e.g.
$ git diff --no-index --color=words a b
error: option `color' expects "always", "auto", or "never"
error: option `color' expects "always", "auto", or "never"
...
Instead, make it act like so:
$ git diff --no-index --color=words a b
error: option `color' expects "always", "auto", or "never"
fatal: invalid diff option/value: --color=words
Reported-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-03-31 20:47:17 +02:00
|
|
|
if (j <= 0)
|
2008-05-24 07:28:56 +02:00
|
|
|
die("invalid diff option/value: %s", argv[i]);
|
|
|
|
i += j;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-06-27 21:05:52 +02:00
|
|
|
prefixlen = prefix ? strlen(prefix) : 0;
|
|
|
|
for (i = 0; i < 2; i++) {
|
|
|
|
const char *p = argv[argc - 2 + i];
|
|
|
|
if (!strcmp(p, "-"))
|
2008-05-24 07:28:56 +02:00
|
|
|
/*
|
2012-06-27 21:05:52 +02:00
|
|
|
* stdin should be spelled as "-"; if you have
|
|
|
|
* path that is "-", spell it as "./-".
|
2008-05-24 07:28:56 +02:00
|
|
|
*/
|
2012-06-28 05:14:47 +02:00
|
|
|
p = file_from_standard_input;
|
2012-06-27 21:05:52 +02:00
|
|
|
else if (prefixlen)
|
|
|
|
p = xstrdup(prefix_filename(prefix, prefixlen, p));
|
|
|
|
paths[i] = p;
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|
2009-02-18 07:48:06 +01:00
|
|
|
revs->diffopt.skip_stat_unmatch = 1;
|
2009-03-25 18:19:46 +01:00
|
|
|
if (!revs->diffopt.output_format)
|
|
|
|
revs->diffopt.output_format = DIFF_FORMAT_PATCH;
|
2008-05-24 07:28:56 +02:00
|
|
|
|
|
|
|
DIFF_OPT_SET(&revs->diffopt, NO_INDEX);
|
|
|
|
|
|
|
|
revs->max_count = -2;
|
2012-08-03 14:16:24 +02:00
|
|
|
diff_setup_done(&revs->diffopt);
|
2008-05-24 07:28:56 +02:00
|
|
|
|
2013-07-18 02:18:32 +02:00
|
|
|
if (deprecated_show_diff_q_option_used)
|
|
|
|
handle_deprecated_show_diff_q(&revs->diffopt);
|
|
|
|
|
2012-06-15 22:32:55 +02:00
|
|
|
setup_diff_pager(&revs->diffopt);
|
|
|
|
DIFF_OPT_SET(&revs->diffopt, EXIT_WITH_STATUS);
|
|
|
|
|
2012-06-27 20:51:15 +02:00
|
|
|
if (queue_diff(&revs->diffopt, paths[0], paths[1]))
|
2008-05-24 07:28:56 +02:00
|
|
|
exit(1);
|
2008-08-19 05:08:09 +02:00
|
|
|
diff_set_mnemonic_prefix(&revs->diffopt, "1/", "2/");
|
2008-05-24 07:28:56 +02:00
|
|
|
diffcore_std(&revs->diffopt);
|
|
|
|
diff_flush(&revs->diffopt);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* The return code for --no-index imitates diff(1):
|
|
|
|
* 0 = no changes, 1 = changes, else error
|
|
|
|
*/
|
2012-06-21 20:09:51 +02:00
|
|
|
exit(diff_result_code(&revs->diffopt, 0));
|
2008-05-24 07:28:56 +02:00
|
|
|
}
|