From: Junio C Hamano Date: Sat, 20 May 2006 01:06:08 +0000 (-0700) Subject: Merge branch 'master' into next X-Git-Tag: v1.4.1-rc1~81 X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=ae514b4c5b51e9b4ee1071898481c01b4d90f67b;p=git.git Merge branch 'master' into next * master: Fix build procedure for builtin-init-db Allow pickaxe and diff-filter options to be used by git log. Libify the index refresh logic Builtin git-init-db --- ae514b4c5b51e9b4ee1071898481c01b4d90f67b diff --cc Makefile index 06b31d8a2,4fd6520b7..abfc07358 --- a/Makefile +++ b/Makefile @@@ -170,8 -170,8 +170,9 @@@ PROGRAMS = BUILT_INS = git-log$X git-whatchanged$X git-show$X \ git-count-objects$X git-diff$X git-push$X \ - git-grep$X git-rev-list$X git-check-ref-format$X \ + git-grep$X git-add$X git-rm$X git-rev-list$X \ - git-check-ref-format$X ++ git-check-ref-format$X \ + git-init-db$X # what 'all' will build and 'install' will install, in gitexecdir ALL_PROGRAMS = $(PROGRAMS) $(SIMPLE_PROGRAMS) $(SCRIPTS) @@@ -219,8 -219,8 +220,8 @@@ LIB_OBJS = BUILTIN_OBJS = \ builtin-log.o builtin-help.o builtin-count.o builtin-diff.o builtin-push.o \ - builtin-grep.o builtin-rev-list.o builtin-check-ref-format.o \ - builtin-init-db.o + builtin-grep.o builtin-add.o builtin-rev-list.o builtin-check-ref-format.o \ - builtin-rm.o ++ builtin-rm.o builtin-init-db.o GITLIBS = $(LIB_FILE) $(XDIFF_LIB) LIBS = $(GITLIBS) -lz diff --cc builtin.h index d150c7ce5,60541262c..34d5e6a7a --- a/builtin.h +++ b/builtin.h @@@ -25,9 -24,8 +25,10 @@@ extern int cmd_count_objects(int argc, extern int cmd_push(int argc, const char **argv, char **envp); extern int cmd_grep(int argc, const char **argv, char **envp); +extern int cmd_rm(int argc, const char **argv, char **envp); +extern int cmd_add(int argc, const char **argv, char **envp); extern int cmd_rev_list(int argc, const char **argv, char **envp); extern int cmd_check_ref_format(int argc, const char **argv, char **envp); + extern int cmd_init_db(int argc, const char **argv, char **envp); #endif diff --cc git.c index 63aa31143,3216d311b..eab8c1f45 --- a/git.c +++ b/git.c @@@ -51,9 -50,8 +51,10 @@@ static void handle_internal_command(in { "count-objects", cmd_count_objects }, { "diff", cmd_diff }, { "grep", cmd_grep }, + { "rm", cmd_rm }, + { "add", cmd_add }, { "rev-list", cmd_rev_list }, + { "init-db", cmd_init_db }, { "check-ref-format", cmd_check_ref_format } }; int i; diff --cc update-index.c index 21448cc5c,7d6de821e..956b6b34f --- a/update-index.c +++ b/update-index.c @@@ -148,103 -120,70 +128,6 @@@ static int add_file_to_cache(const cha return 0; } --/* - * "refresh" does not calculate a new sha1 file or bring the - * cache up-to-date for mode/content changes. But what it - * _does_ do is to "re-match" the stat information of a file - * with the cache, so that you can refresh the cache for a - * file that hasn't been changed but where the stat entry is - * out of date. - * We fundamentally don't like some paths: we don't want - * dot or dot-dot anywhere, and for obvious reasons don't - * want to recurse into ".git" either. -- * - * For example, you'd want to do this after doing a "git-read-tree", - * to link up the stat cache details with the proper files. - * Also, we don't want double slashes or slashes at the - * end that can make pathnames ambiguous. -- */ - static struct cache_entry *refresh_entry(struct cache_entry *ce, int really) -static int verify_dotfile(const char *rest) --{ - struct stat st; - struct cache_entry *updated; - int changed, size; - - if (lstat(ce->name, &st) < 0) - return ERR_PTR(-errno); - - changed = ce_match_stat(ce, &st, really); - if (!changed) { - if (really && assume_unchanged && - !(ce->ce_flags & htons(CE_VALID))) - ; /* mark this one VALID again */ - else - return NULL; - } - - if (ce_modified(ce, &st, really)) - return ERR_PTR(-EINVAL); - - size = ce_size(ce); - updated = xmalloc(size); - memcpy(updated, ce, size); - fill_stat_cache_info(updated, &st); - - /* In this case, if really is not set, we should leave - * CE_VALID bit alone. Otherwise, paths marked with - * --no-assume-unchanged (i.e. things to be edited) will - * reacquire CE_VALID bit automatically, which is not - * really what we want. - /* - * The first character was '.', but that - * has already been discarded, we now test - * the rest. -- */ - if (!really && assume_unchanged && !(ce->ce_flags & htons(CE_VALID))) - updated->ce_flags &= ~htons(CE_VALID); - switch (*rest) { - /* "." is not allowed */ - case '\0': case '/': - return 0; -- - return updated; - /* - * ".git" followed by NUL or slash is bad. This - * shares the path end test with the ".." case. - */ - case 'g': - if (rest[1] != 'i') - break; - if (rest[2] != 't') - break; - rest += 2; - /* fallthrough */ - case '.': - if (rest[1] == '\0' || rest[1] == '/') - return 0; - } - return 1; --} -- - static int refresh_cache(int really) -static int verify_path(const char *path) --{ - int i; - int has_errors = 0; - - for (i = 0; i < active_nr; i++) { - struct cache_entry *ce, *new; - ce = active_cache[i]; - if (ce_stage(ce)) { - while ((i < active_nr) && - ! strcmp(active_cache[i]->name, ce->name)) - i++; - i--; - if (allow_unmerged) - continue; - printf("%s: needs merge\n", ce->name); - has_errors = 1; - continue; - } - char c; -- - new = refresh_entry(ce, really); - if (!new) - continue; - if (IS_ERR(new)) { - if (not_new && PTR_ERR(new) == -ENOENT) - goto inside; - for (;;) { - if (!c) - return 1; - if (c == '/') { -inside: - c = *path++; - switch (c) { - default: -- continue; - if (really && PTR_ERR(new) == -EINVAL) { - /* If we are doing --really-refresh that - * means the index is not valid anymore. - */ - ce->ce_flags &= ~htons(CE_VALID); - active_cache_changed = 1; - case '/': case '\0': - break; - case '.': - if (verify_dotfile(path)) - continue; -- } - if (quiet) - continue; - printf("%s: needs update\n", ce->name); - has_errors = 1; - continue; - return 0; -- } - active_cache_changed = 1; - /* You can NOT just free active_cache[i] here, since it - * might not be necessarily malloc()ed but can also come - * from mmap(). */ - active_cache[i] = new; - c = *path++; -- } - return has_errors; --} -- static int add_cacheinfo(unsigned int mode, const unsigned char *sha1, const char *path, int stage) {