Merge branch 'db/checkout'
authorJunio C Hamano <gitster@pobox.com>
Wed, 27 Feb 2008 20:53:26 +0000 (12:53 -0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 27 Feb 2008 20:53:26 +0000 (12:53 -0800)
* db/checkout: (21 commits)
  checkout: error out when index is unmerged even with -m
  checkout: show progress when checkout takes long time while switching branches
  Add merge-subtree back
  checkout: updates to tracking report
  builtin-checkout.c: Remove unused prefix arguments in switch_branches path
  checkout: work from a subdirectory
  checkout: tone down the "forked status" diagnostic messages
  Clean up reporting differences on branch switch
  builtin-checkout.c: fix possible usage segfault
  checkout: notice when the switched branch is behind or forked
  Build in checkout
  Move code to clean up after a branch change to branch.c
  Library function to check for unmerged index entries
  Use diff -u instead of diff in t7201
  Move create_branch into a library file
  Build-in merge-recursive
  Add "skip_unmerged" option to unpack_trees.
  Discard "deleted" cache entries after using them to update the working tree
  Send unpack-trees debugging output to stderr
  Add flag to make unpack_trees() not print errors.
  ...

Conflicts:

Makefile

12 files changed:
1  2 
Makefile
builtin-branch.c
builtin-commit.c
builtin-merge-recursive.c
builtin-read-tree.c
builtin.h
cache.h
contrib/examples/git-checkout.sh
git.c
read-cache.c
t/t7201-co.sh
unpack-trees.c

diff --combined Makefile
index 8d9f11e75e8fb1095761fd7fd25e66b1d73e6fa2,40fa41b5f3a914cfb82366158e8e27bac3bb8938..a5b6eebf1a0b65b481f6dd8f1e38006b46e87c83
+++ b/Makefile
@@@ -3,9 -3,6 +3,9 @@@ all:
  
  # Define V=1 to have a more verbose compile.
  #
 +# Define FREAD_READS_DIRECTORIES if your are on a system which succeeds
 +# when attempting to read from an fopen'ed directory.
 +#
  # Define NO_OPENSSL environment variable if you do not have OpenSSL.
  # This also implies MOZILLA_SHA1.
  #
@@@ -45,8 -42,6 +45,8 @@@
  #
  # Define NO_MKDTEMP if you don't have mkdtemp in the C library.
  #
 +# Define NO_SYS_SELECT_H if you don't have sys/select.h.
 +#
  # Define NO_SYMLINK_HEAD if you never want .git/HEAD to be a symbolic link.
  # Enable it on Windows.  By default, symrefs are still used.
  #
  # Define THREADED_DELTA_SEARCH if you have pthreads and wish to exploit
  # parallel delta searching when packing objects.
  #
 +# Define INTERNAL_QSORT to use Git's implementation of qsort(), which
 +# is a simplified version of the merge sort used in glibc. This is
 +# recommended if Git triggers O(n^2) behavior in your platform's qsort().
 +#
  
  GIT-VERSION-FILE: .FORCE-GIT-VERSION-FILE
        @$(SHELL_PATH) ./GIT-VERSION-GEN
@@@ -226,7 -217,7 +226,7 @@@ BASIC_CFLAGS 
  BASIC_LDFLAGS =
  
  SCRIPT_SH = \
-       git-bisect.sh git-checkout.sh \
+       git-bisect.sh \
        git-clone.sh \
        git-merge-one-file.sh git-mergetool.sh git-parse-remote.sh \
        git-pull.sh git-rebase.sh git-rebase--interactive.sh \
        git-lost-found.sh git-quiltimport.sh git-submodule.sh \
        git-filter-branch.sh \
        git-stash.sh \
 -      git-help--browse.sh
 +      git-web--browse.sh
  
  SCRIPT_PERL = \
        git-add--interactive.perl \
@@@ -265,23 -256,23 +265,23 @@@ PROGRAMS = 
        git-upload-pack$X \
        git-pack-redundant$X git-var$X \
        git-merge-tree$X git-imap-send$X \
-       git-merge-recursive$X \
        $(EXTRA_PROGRAMS)
  
  # Empty...
  EXTRA_PROGRAMS =
  
+ # List built-in command $C whose implementation cmd_$C() is not in
+ # builtin-$C.o but is linked in as part of some other command.
  BUILT_INS = \
        git-format-patch$X git-show$X git-whatchanged$X git-cherry$X \
        git-get-tar-commit-id$X git-init$X git-repo-config$X \
        git-fsck-objects$X git-cherry-pick$X git-peek-remote$X git-status$X \
+       git-merge-subtree$X \
        $(patsubst builtin-%.o,git-%$X,$(BUILTIN_OBJS))
  
  # what 'all' will build and 'install' will install, in gitexecdir
  ALL_PROGRAMS = $(PROGRAMS) $(SCRIPTS)
  
- ALL_PROGRAMS += git-merge-subtree$X
  # what 'all' will build but not install in gitexecdir
  OTHER_PROGRAMS = git$X gitweb/gitweb.cgi
  
@@@ -327,8 -318,7 +327,8 @@@ LIB_OBJS = 
        alloc.o merge-file.o path-list.o help.o unpack-trees.o $(DIFF_OBJS) \
        color.o wt-status.o archive-zip.o archive-tar.o shallow.o utf8.o \
        convert.o attr.o decorate.o progress.o mailmap.o symlinks.o remote.o \
-       transport.o bundle.o walker.o parse-options.o ws.o archive.o \
 -      transport.o bundle.o walker.o parse-options.o ws.o archive.o branch.o
++      transport.o bundle.o walker.o parse-options.o ws.o archive.o branch.o \
 +      alias.o
  
  BUILTIN_OBJS = \
        builtin-add.o \
        builtin-bundle.o \
        builtin-cat-file.o \
        builtin-check-attr.o \
+       builtin-checkout.o \
        builtin-checkout-index.o \
        builtin-check-ref-format.o \
        builtin-clean.o \
        builtin-merge-base.o \
        builtin-merge-file.o \
        builtin-merge-ours.o \
+       builtin-merge-recursive.o \
        builtin-mv.o \
        builtin-name-rev.o \
        builtin-pack-objects.o \
@@@ -513,17 -505,6 +515,17 @@@ ifeq ($(uname_S),IRIX64
        # for now, build 32-bit version
        BASIC_LDFLAGS += -L/usr/lib32
  endif
 +ifeq ($(uname_S),HP-UX)
 +      NO_IPV6=YesPlease
 +      NO_SETENV=YesPlease
 +      NO_STRCASESTR=YesPlease
 +      NO_MEMMEM = YesPlease
 +      NO_STRLCPY = YesPlease
 +      NO_MKDTEMP = YesPlease
 +      NO_UNSETENV = YesPlease
 +      NO_HSTRERROR = YesPlease
 +      NO_SYS_SELECT_H = YesPlease
 +endif
  ifneq (,$(findstring arm,$(uname_M)))
        ARM_SHA1 = YesPlease
  endif
@@@ -626,10 -607,6 +628,10 @@@ endi
  ifdef NO_C99_FORMAT
        BASIC_CFLAGS += -DNO_C99_FORMAT
  endif
 +ifdef FREAD_READS_DIRECTORIES
 +      COMPAT_CFLAGS += -DFREAD_READS_DIRECTORIES
 +      COMPAT_OBJS += compat/fopen.o
 +endif
  ifdef NO_SYMLINK_HEAD
        BASIC_CFLAGS += -DNO_SYMLINK_HEAD
  endif
@@@ -660,9 -637,6 +662,9 @@@ ifdef NO_UNSETEN
        COMPAT_CFLAGS += -DNO_UNSETENV
        COMPAT_OBJS += compat/unsetenv.o
  endif
 +ifdef NO_SYS_SELECT_H
 +      BASIC_CFLAGS += -DNO_SYS_SELECT_H
 +endif
  ifdef NO_MMAP
        COMPAT_CFLAGS += -DNO_MMAP
        COMPAT_OBJS += compat/mmap.o
@@@ -734,15 -708,10 +736,15 @@@ ifdef NO_MEMME
        COMPAT_CFLAGS += -DNO_MEMMEM
        COMPAT_OBJS += compat/memmem.o
  endif
 +ifdef INTERNAL_QSORT
 +      COMPAT_CFLAGS += -DINTERNAL_QSORT
 +      COMPAT_OBJS += compat/qsort.o
 +endif
  
  ifdef THREADED_DELTA_SEARCH
        BASIC_CFLAGS += -DTHREADED_DELTA_SEARCH
        EXTLIBS += -lpthread
 +      LIB_OBJS += thread-utils.o
  endif
  
  ifeq ($(TCLTK_PATH),)
@@@ -836,13 -805,9 +838,10 @@@ git$X: git.o $(BUILTIN_OBJS) $(GITLIBS
  
  help.o: help.c common-cmds.h GIT-CFLAGS
        $(QUIET_CC)$(CC) -o $*.o -c $(ALL_CFLAGS) \
 +              '-DGIT_HTML_PATH="$(htmldir_SQ)"' \
                '-DGIT_MAN_PATH="$(mandir_SQ)"' \
                '-DGIT_INFO_PATH="$(infodir_SQ)"' $<
  
- git-merge-subtree$X: git-merge-recursive$X
-       $(QUIET_BUILT_IN)$(RM) $@ && ln git-merge-recursive$X $@
  $(BUILT_INS): git$X
        $(QUIET_BUILT_IN)$(RM) $@ && ln git$X $@
  
@@@ -857,6 -822,7 +856,6 @@@ $(patsubst %.sh,%,$(SCRIPT_SH)) : % : %
            -e 's|@@PERL@@|$(PERL_PATH_SQ)|g' \
            -e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
            -e 's/@@NO_CURL@@/$(NO_CURL)/g' \
 -          -e 's|@@HTMLDIR@@|$(htmldir_SQ)|g' \
            $@.sh >$@+ && \
        chmod +x $@+ && \
        mv $@+ $@
@@@ -961,7 -927,7 +960,7 @@@ git-%$X: %.o $(GITLIBS
  
  git-imap-send$X: imap-send.o $(LIB_FILE)
  
 -http.o http-walker.o http-push.o: http.h
 +http.o http-walker.o http-push.o transport.o: http.h
  
  git-http-push$X: revision.o http.o http-push.o $(GITLIBS)
        $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) \
@@@ -1104,7 -1070,7 +1103,7 @@@ git.spec: git.spec.i
        mv $@+ $@
  
  GIT_TARNAME=git-$(GIT_VERSION)
 -dist: git.spec git-archive configure
 +dist: git.spec git-archive$(X) configure
        ./git-archive --format=tar \
                --prefix=$(GIT_TARNAME)/ HEAD^{tree} > $(GIT_TARNAME).tar
        @mkdir -p $(GIT_TARNAME)
diff --combined builtin-branch.c
index 9edf2eb8162eb7214e1b4d76dca8949c63e72847,1e0c9dea3f8a22f7714d51ad4ad00c30d436eac3..7e991030ca4def9e2be2f595809529e13a898c24
@@@ -12,6 -12,7 +12,7 @@@
  #include "builtin.h"
  #include "remote.h"
  #include "parse-options.h"
+ #include "branch.h"
  
  static const char * const builtin_branch_usage[] = {
        "git-branch [options] [-r | -a]",
@@@ -31,7 -32,7 +32,7 @@@ static unsigned char head_sha1[20]
  
  static int branch_track = 1;
  
 -static int branch_use_color;
 +static int branch_use_color = -1;
  static char branch_colors[][COLOR_MAXLEN] = {
        "\033[m",       /* reset */
        "",             /* PLAIN (normal) */
@@@ -70,21 -71,18 +71,21 @@@ static int git_branch_config(const cha
        }
        if (!prefixcmp(var, "color.branch.")) {
                int slot = parse_branch_color_slot(var, 13);
 +              if (!value)
 +                      return config_error_nonbool(var);
                color_parse(value, var, branch_colors[slot]);
                return 0;
        }
 -      if (!strcmp(var, "branch.autosetupmerge"))
 -                      branch_track = git_config_bool(var, value);
 -
 -      return git_default_config(var, value);
 +      if (!strcmp(var, "branch.autosetupmerge")) {
 +              branch_track = git_config_bool(var, value);
 +              return 0;
 +      }
 +      return git_color_default_config(var, value);
  }
  
  static const char *branch_get_color(enum color_branch ix)
  {
 -      if (branch_use_color)
 +      if (branch_use_color > 0)
                return branch_colors[ix];
        return "";
  }
@@@ -359,141 -357,6 +360,6 @@@ static void print_ref_list(int kinds, i
        free_ref_list(&ref_list);
  }
  
- struct tracking {
-       struct refspec spec;
-       char *src;
-       const char *remote;
-       int matches;
- };
- static int find_tracked_branch(struct remote *remote, void *priv)
- {
-       struct tracking *tracking = priv;
-       if (!remote_find_tracking(remote, &tracking->spec)) {
-               if (++tracking->matches == 1) {
-                       tracking->src = tracking->spec.src;
-                       tracking->remote = remote->name;
-               } else {
-                       free(tracking->spec.src);
-                       if (tracking->src) {
-                               free(tracking->src);
-                               tracking->src = NULL;
-                       }
-               }
-               tracking->spec.src = NULL;
-       }
-       return 0;
- }
- /*
-  * This is called when new_ref is branched off of orig_ref, and tries
-  * to infer the settings for branch.<new_ref>.{remote,merge} from the
-  * config.
-  */
- static int setup_tracking(const char *new_ref, const char *orig_ref)
- {
-       char key[1024];
-       struct tracking tracking;
-       if (strlen(new_ref) > 1024 - 7 - 7 - 1)
-               return error("Tracking not set up: name too long: %s",
-                               new_ref);
-       memset(&tracking, 0, sizeof(tracking));
-       tracking.spec.dst = (char *)orig_ref;
-       if (for_each_remote(find_tracked_branch, &tracking) ||
-                       !tracking.matches)
-               return 1;
-       if (tracking.matches > 1)
-               return error("Not tracking: ambiguous information for ref %s",
-                               orig_ref);
-       if (tracking.matches == 1) {
-               sprintf(key, "branch.%s.remote", new_ref);
-               git_config_set(key, tracking.remote ?  tracking.remote : ".");
-               sprintf(key, "branch.%s.merge", new_ref);
-               git_config_set(key, tracking.src);
-               free(tracking.src);
-               printf("Branch %s set up to track remote branch %s.\n",
-                              new_ref, orig_ref);
-       }
-       return 0;
- }
- static void create_branch(const char *name, const char *start_name,
-                         int force, int reflog, int track)
- {
-       struct ref_lock *lock;
-       struct commit *commit;
-       unsigned char sha1[20];
-       char *real_ref, ref[PATH_MAX], msg[PATH_MAX + 20];
-       int forcing = 0;
-       snprintf(ref, sizeof ref, "refs/heads/%s", name);
-       if (check_ref_format(ref))
-               die("'%s' is not a valid branch name.", name);
-       if (resolve_ref(ref, sha1, 1, NULL)) {
-               if (!force)
-                       die("A branch named '%s' already exists.", name);
-               else if (!is_bare_repository() && !strcmp(head, name))
-                       die("Cannot force update the current branch.");
-               forcing = 1;
-       }
-       real_ref = NULL;
-       if (get_sha1(start_name, sha1))
-               die("Not a valid object name: '%s'.", start_name);
-       switch (dwim_ref(start_name, strlen(start_name), sha1, &real_ref)) {
-       case 0:
-               /* Not branching from any existing branch */
-               real_ref = NULL;
-               break;
-       case 1:
-               /* Unique completion -- good */
-               break;
-       default:
-               die("Ambiguous object name: '%s'.", start_name);
-               break;
-       }
-       if ((commit = lookup_commit_reference(sha1)) == NULL)
-               die("Not a valid branch point: '%s'.", start_name);
-       hashcpy(sha1, commit->object.sha1);
-       lock = lock_any_ref_for_update(ref, NULL, 0);
-       if (!lock)
-               die("Failed to lock ref for update: %s.", strerror(errno));
-       if (reflog)
-               log_all_ref_updates = 1;
-       if (forcing)
-               snprintf(msg, sizeof msg, "branch: Reset from %s",
-                        start_name);
-       else
-               snprintf(msg, sizeof msg, "branch: Created from %s",
-                        start_name);
-       /* When branching off a remote branch, set up so that git-pull
-          automatically merges from there.  So far, this is only done for
-          remotes registered via .git/config.  */
-       if (real_ref && track)
-               setup_tracking(name, real_ref);
-       if (write_ref_sha1(lock, sha1, msg) < 0)
-               die("Failed to write ref: %s.", strerror(errno));
-       if (real_ref)
-               free(real_ref);
- }
  static void rename_branch(const char *oldname, const char *newname, int force)
  {
        char oldref[PATH_MAX], newref[PATH_MAX], logmsg[PATH_MAX*2 + 100];
@@@ -588,10 -451,6 +454,10 @@@ int cmd_branch(int argc, const char **a
        };
  
        git_config(git_branch_config);
 +
 +      if (branch_use_color == -1)
 +              branch_use_color = git_use_color_default;
 +
        track = branch_track;
        argc = parse_options(argc, argv, options, builtin_branch_usage, 0);
        if (!!delete + !!rename + !!force_create > 1)
        else if (rename && (argc == 2))
                rename_branch(argv[0], argv[1], rename > 1);
        else if (argc <= 2)
-               create_branch(argv[0], (argc == 2) ? argv[1] : head,
+               create_branch(head, argv[0], (argc == 2) ? argv[1] : head,
                              force_create, reflog, track);
        else
                usage_with_options(builtin_branch_usage, options);
diff --combined builtin-commit.c
index 065e1f7b7fbf426210d08062800ed81448a30d4a,5b5b7c0f4df81f4200707e0cbe688b3873443125..f49c22e64255225e492614bb628c1d1776521424
@@@ -7,7 -7,6 +7,7 @@@
  
  #include "cache.h"
  #include "cache-tree.h"
 +#include "color.h"
  #include "dir.h"
  #include "builtin.h"
  #include "diff.h"
@@@ -123,23 -122,19 +123,23 @@@ static void rollback_index_files(void
        }
  }
  
 -static void commit_index_files(void)
 +static int commit_index_files(void)
  {
 +      int err = 0;
 +
        switch (commit_style) {
        case COMMIT_AS_IS:
                break; /* nothing to do */
        case COMMIT_NORMAL:
 -              commit_lock_file(&index_lock);
 +              err = commit_lock_file(&index_lock);
                break;
        case COMMIT_PARTIAL:
 -              commit_lock_file(&index_lock);
 +              err = commit_lock_file(&index_lock);
                rollback_lock_file(&false_lock);
                break;
        }
 +
 +      return err;
  }
  
  /*
@@@ -205,7 -200,8 +205,8 @@@ static void create_base_index(void
                die("failed to unpack HEAD tree object");
        parse_tree(tree);
        init_tree_desc(&t, tree->buffer, tree->size);
-       unpack_trees(1, &t, &opts);
+       if (unpack_trees(1, &t, &opts))
+               exit(128); /* We've already reported the error, finish dying */
  }
  
  static char *prepare_index(int argc, const char **argv, const char *prefix)
        if (write_cache(fd, active_cache, active_nr) ||
            close_lock_file(&false_lock))
                die("unable to write temporary index file");
 +
 +      discard_cache();
 +      read_cache_from(false_lock.filename);
 +
        return false_lock.filename;
  }
  
@@@ -348,107 -340,45 +349,107 @@@ static int run_status(FILE *fp, const c
        return s.commitable;
  }
  
 +static int run_hook(const char *index_file, const char *name, ...)
 +{
 +      struct child_process hook;
 +      const char *argv[10], *env[2];
 +      char index[PATH_MAX];
 +      va_list args;
 +      int i;
 +
 +      va_start(args, name);
 +      argv[0] = git_path("hooks/%s", name);
 +      i = 0;
 +      do {
 +              if (++i >= ARRAY_SIZE(argv))
 +                      die ("run_hook(): too many arguments");
 +              argv[i] = va_arg(args, const char *);
 +      } while (argv[i]);
 +      va_end(args);
 +
 +      snprintf(index, sizeof(index), "GIT_INDEX_FILE=%s", index_file);
 +      env[0] = index;
 +      env[1] = NULL;
 +
 +      if (access(argv[0], X_OK) < 0)
 +              return 0;
 +
 +      memset(&hook, 0, sizeof(hook));
 +      hook.argv = argv;
 +      hook.no_stdin = 1;
 +      hook.stdout_to_stderr = 1;
 +      hook.env = env;
 +
 +      return run_command(&hook);
 +}
 +
 +static int is_a_merge(const unsigned char *sha1)
 +{
 +      struct commit *commit = lookup_commit(sha1);
 +      if (!commit || parse_commit(commit))
 +              die("could not parse HEAD commit");
 +      return !!(commit->parents && commit->parents->next);
 +}
 +
  static const char sign_off_header[] = "Signed-off-by: ";
  
 -static int prepare_log_message(const char *index_file, const char *prefix)
 +static int prepare_to_commit(const char *index_file, const char *prefix)
  {
        struct stat statbuf;
        int commitable, saved_color_setting;
        struct strbuf sb;
        char *buffer;
        FILE *fp;
 +      const char *hook_arg1 = NULL;
 +      const char *hook_arg2 = NULL;
 +
 +      if (!no_verify && run_hook(index_file, "pre-commit", NULL))
 +              return 0;
  
        strbuf_init(&sb, 0);
        if (message.len) {
                strbuf_addbuf(&sb, &message);
 +              hook_arg1 = "message";
        } else if (logfile && !strcmp(logfile, "-")) {
                if (isatty(0))
                        fprintf(stderr, "(reading log message from standard input)\n");
                if (strbuf_read(&sb, 0, 0) < 0)
                        die("could not read log from standard input");
 +              hook_arg1 = "message";
        } else if (logfile) {
                if (strbuf_read_file(&sb, logfile, 0) < 0)
                        die("could not read log file '%s': %s",
                            logfile, strerror(errno));
 +              hook_arg1 = "message";
        } else if (use_message) {
                buffer = strstr(use_message_buffer, "\n\n");
                if (!buffer || buffer[2] == '\0')
                        die("commit has empty message");
                strbuf_add(&sb, buffer + 2, strlen(buffer + 2));
 +              hook_arg1 = "commit";
 +              hook_arg2 = use_message;
        } else if (!stat(git_path("MERGE_MSG"), &statbuf)) {
                if (strbuf_read_file(&sb, git_path("MERGE_MSG"), 0) < 0)
                        die("could not read MERGE_MSG: %s", strerror(errno));
 +              hook_arg1 = "merge";
        } else if (!stat(git_path("SQUASH_MSG"), &statbuf)) {
                if (strbuf_read_file(&sb, git_path("SQUASH_MSG"), 0) < 0)
                        die("could not read SQUASH_MSG: %s", strerror(errno));
 +              hook_arg1 = "squash";
        } else if (template_file && !stat(template_file, &statbuf)) {
                if (strbuf_read_file(&sb, template_file, 0) < 0)
                        die("could not read %s: %s",
                            template_file, strerror(errno));
 +              hook_arg1 = "template";
        }
  
 +      /*
 +       * This final case does not modify the template message,
 +       * it just sets the argument to the prepare-commit-msg hook.
 +       */
 +      else if (in_merge)
 +              hook_arg1 = "merge";
 +
        fp = fopen(git_path(commit_editmsg), "w");
        if (fp == NULL)
                die("could not open %s", git_path(commit_editmsg));
  
        strbuf_release(&sb);
  
 -      if (!use_editor) {
 +      if (use_editor) {
 +              if (in_merge)
 +                      fprintf(fp,
 +                              "#\n"
 +                              "# It looks like you may be committing a MERGE.\n"
 +                              "# If this is not correct, please remove the file\n"
 +                              "#      %s\n"
 +                              "# and try again.\n"
 +                              "#\n",
 +                              git_path("MERGE_HEAD"));
 +
 +              fprintf(fp,
 +                      "\n"
 +                      "# Please enter the commit message for your changes.\n"
 +                      "# (Comment lines starting with '#' will ");
 +              if (cleanup_mode == CLEANUP_ALL)
 +                      fprintf(fp, "not be included)\n");
 +              else /* CLEANUP_SPACE, that is. */
 +                      fprintf(fp, "be kept.\n"
 +                              "# You can remove them yourself if you want to)\n");
 +              if (only_include_assumed)
 +                      fprintf(fp, "# %s\n", only_include_assumed);
 +
 +              saved_color_setting = wt_status_use_color;
 +              wt_status_use_color = 0;
 +              commitable = run_status(fp, index_file, prefix, 1);
 +              wt_status_use_color = saved_color_setting;
 +      } else {
                struct rev_info rev;
                unsigned char sha1[20];
                const char *parent = "HEAD";
  
 -              fclose(fp);
 -
                if (!active_nr && read_cache() < 0)
                        die("Cannot read index");
  
                        parent = "HEAD^1";
  
                if (get_sha1(parent, sha1))
 -                      return !!active_nr;
 +                      commitable = !!active_nr;
 +              else {
 +                      init_revisions(&rev, "");
 +                      rev.abbrev = 0;
 +                      setup_revisions(0, NULL, &rev, parent);
 +                      DIFF_OPT_SET(&rev.diffopt, QUIET);
 +                      DIFF_OPT_SET(&rev.diffopt, EXIT_WITH_STATUS);
 +                      run_diff_index(&rev, 1 /* cached */);
 +
 +                      commitable = !!DIFF_OPT_TST(&rev.diffopt, HAS_CHANGES);
 +              }
 +      }
  
 -              init_revisions(&rev, "");
 -              rev.abbrev = 0;
 -              setup_revisions(0, NULL, &rev, parent);
 -              DIFF_OPT_SET(&rev.diffopt, QUIET);
 -              DIFF_OPT_SET(&rev.diffopt, EXIT_WITH_STATUS);
 -              run_diff_index(&rev, 1 /* cached */);
 +      fclose(fp);
  
 -              return !!DIFF_OPT_TST(&rev.diffopt, HAS_CHANGES);
 +      if (!commitable && !in_merge && !allow_empty &&
 +          !(amend && is_a_merge(head_sha1))) {
 +              run_status(stdout, index_file, prefix, 0);
 +              unlink(commit_editmsg);
 +              return 0;
        }
  
 -      if (in_merge)
 -              fprintf(fp,
 -                      "#\n"
 -                      "# It looks like you may be committing a MERGE.\n"
 -                      "# If this is not correct, please remove the file\n"
 -                      "#      %s\n"
 -                      "# and try again.\n"
 -                      "#\n",
 -                      git_path("MERGE_HEAD"));
 -
 -      fprintf(fp,
 -              "\n"
 -              "# Please enter the commit message for your changes.\n"
 -              "# (Comment lines starting with '#' will ");
 -      if (cleanup_mode == CLEANUP_ALL)
 -              fprintf(fp, "not be included)\n");
 -      else /* CLEANUP_SPACE, that is. */
 -              fprintf(fp, "be kept.\n"
 -                      "# You can remove them yourself if you want to)\n");
 -      if (only_include_assumed)
 -              fprintf(fp, "# %s\n", only_include_assumed);
 -
 -      saved_color_setting = wt_status_use_color;
 -      wt_status_use_color = 0;
 -      commitable = run_status(fp, index_file, prefix, 1);
 -      wt_status_use_color = saved_color_setting;
 +      /*
 +       * Re-read the index as pre-commit hook could have updated it,
 +       * and write it out as a tree.  We must do this before we invoke
 +       * the editor and after we invoke run_status above.
 +       */
 +      discard_cache();
 +      read_cache_from(index_file);
 +      if (!active_cache_tree)
 +              active_cache_tree = cache_tree();
 +      if (cache_tree_update(active_cache_tree,
 +                            active_cache, active_nr, 0, 0) < 0) {
 +              error("Error building trees");
 +              return 0;
 +      }
  
 -      fclose(fp);
 +      if (run_hook(index_file, "prepare-commit-msg",
 +                   git_path(commit_editmsg), hook_arg1, hook_arg2, NULL))
 +              return 0;
  
 -      return commitable;
 +      if (use_editor) {
 +              char index[PATH_MAX];
 +              const char *env[2] = { index, NULL };
 +              snprintf(index, sizeof(index), "GIT_INDEX_FILE=%s", index_file);
 +              launch_editor(git_path(commit_editmsg), NULL, env);
 +      }
 +
 +      if (!no_verify &&
 +          run_hook(index_file, "commit-msg", git_path(commit_editmsg), NULL)) {
 +              return 0;
 +      }
 +
 +      return 1;
  }
  
  /*
@@@ -669,8 -562,6 +670,8 @@@ static int parse_and_validate_options(i
                use_editor = 0;
        if (edit_flag)
                use_editor = 1;
 +      if (!use_editor)
 +              setenv("GIT_EDITOR", ":", 1);
  
        if (get_sha1("HEAD", head_sha1))
                initial_commit = 1;
  
                if (get_sha1(use_message, sha1))
                        die("could not lookup commit %s", use_message);
 -              commit = lookup_commit(sha1);
 +              commit = lookup_commit_reference(sha1);
                if (!commit || parse_commit(commit))
                        die("could not parse commit %s", use_message);
  
@@@ -772,9 -663,6 +773,9 @@@ int cmd_status(int argc, const char **a
  
        git_config(git_status_config);
  
 +      if (wt_status_use_color == -1)
 +              wt_status_use_color = git_use_color_default;
 +
        argc = parse_and_validate_options(argc, argv, builtin_status_usage);
  
        index_file = prepare_index(argc, argv, prefix);
        return commitable ? 0 : 1;
  }
  
 -static int run_hook(const char *index_file, const char *name, const char *arg)
 -{
 -      struct child_process hook;
 -      const char *argv[3], *env[2];
 -      char index[PATH_MAX];
 -
 -      argv[0] = git_path("hooks/%s", name);
 -      argv[1] = arg;
 -      argv[2] = NULL;
 -      snprintf(index, sizeof(index), "GIT_INDEX_FILE=%s", index_file);
 -      env[0] = index;
 -      env[1] = NULL;
 -
 -      if (access(argv[0], X_OK) < 0)
 -              return 0;
 -
 -      memset(&hook, 0, sizeof(hook));
 -      hook.argv = argv;
 -      hook.no_stdin = 1;
 -      hook.stdout_to_stderr = 1;
 -      hook.env = env;
 -
 -      return run_command(&hook);
 -}
 -
  static void print_summary(const char *prefix, const unsigned char *sha1)
  {
        struct rev_info rev;
  int git_commit_config(const char *k, const char *v)
  {
        if (!strcmp(k, "commit.template")) {
 +              if (!v)
 +                      return config_error_nonbool(v);
                template_file = xstrdup(v);
                return 0;
        }
        return git_status_config(k, v);
  }
  
 -static int is_a_merge(const unsigned char *sha1)
 -{
 -      struct commit *commit = lookup_commit(sha1);
 -      if (!commit || parse_commit(commit))
 -              die("could not parse HEAD commit");
 -      return !!(commit->parents && commit->parents->next);
 -}
 -
  static const char commit_utf8_warn[] =
  "Warning: commit message does not conform to UTF-8.\n"
  "You may want to amend it after fixing the message, or set the config\n"
@@@ -867,13 -786,33 +868,13 @@@ int cmd_commit(int argc, const char **a
  
        index_file = prepare_index(argc, argv, prefix);
  
 -      if (!no_verify && run_hook(index_file, "pre-commit", NULL)) {
 +      /* Set up everything for writing the commit object.  This includes
 +         running hooks, writing the trees, and interacting with the user.  */
 +      if (!prepare_to_commit(index_file, prefix)) {
                rollback_index_files();
                return 1;
        }
  
 -      if (!prepare_log_message(index_file, prefix) && !in_merge &&
 -          !allow_empty && !(amend && is_a_merge(head_sha1))) {
 -              run_status(stdout, index_file, prefix, 0);
 -              rollback_index_files();
 -              unlink(commit_editmsg);
 -              return 1;
 -      }
 -
 -      /*
 -       * Re-read the index as pre-commit hook could have updated it,
 -       * and write it out as a tree.
 -       */
 -      discard_cache();
 -      read_cache_from(index_file);
 -      if (!active_cache_tree)
 -              active_cache_tree = cache_tree();
 -      if (cache_tree_update(active_cache_tree,
 -                            active_cache, active_nr, 0, 0) < 0) {
 -              rollback_index_files();
 -              die("Error building trees");
 -      }
 -
        /*
         * The commit object
         */
                strbuf_addf(&sb, "encoding %s\n", git_commit_encoding);
        strbuf_addch(&sb, '\n');
  
 -      /* Get the commit message and validate it */
 +      /* Finally, get the commit message */
        header_len = sb.len;
 -      if (use_editor) {
 -              char index[PATH_MAX];
 -              const char *env[2] = { index, NULL };
 -              snprintf(index, sizeof(index), "GIT_INDEX_FILE=%s", index_file);
 -              launch_editor(git_path(commit_editmsg), NULL, env);
 -      }
 -      if (!no_verify &&
 -          run_hook(index_file, "commit-msg", git_path(commit_editmsg))) {
 -              rollback_index_files();
 -              exit(1);
 -      }
        if (strbuf_read_file(&sb, git_path(commit_editmsg), 0) < 0) {
                rollback_index_files();
                die("could not read commit message");
  
        unlink(git_path("MERGE_HEAD"));
        unlink(git_path("MERGE_MSG"));
 +      unlink(git_path("SQUASH_MSG"));
  
 -      commit_index_files();
 +      if (commit_index_files())
 +              die ("Repository has been updated, but unable to write\n"
 +                   "new_index file. Check that disk is not full or quota is\n"
 +                   "not exceeded, and then \"git reset HEAD\" to recover.");
  
        rerere();
        run_hook(get_index_file(), "post-commit", NULL);
index 55ef76f5a5f855475f796dfda39fa3417179be8b,558a58e4d3aaff6e66f25220b30ea9749b50be08..6fe4102c0cfb29031f7fdce68ca4d1937e2fefd7
@@@ -7,6 -7,7 +7,7 @@@
  #include "cache-tree.h"
  #include "commit.h"
  #include "blob.h"
+ #include "builtin.h"
  #include "tree-walk.h"
  #include "diff.h"
  #include "diffcore.h"
@@@ -17,6 -18,7 +18,7 @@@
  #include "xdiff-interface.h"
  #include "interpolate.h"
  #include "attr.h"
+ #include "merge-recursive.h"
  
  static int subtree_merge;
  
@@@ -221,22 -223,11 +223,11 @@@ static int git_merge_trees(int index_on
        return rc;
  }
  
- static int unmerged_index(void)
- {
-       int i;
-       for (i = 0; i < active_nr; i++) {
-               struct cache_entry *ce = active_cache[i];
-               if (ce_stage(ce))
-                       return 1;
-       }
-       return 0;
- }
- static struct tree *git_write_tree(void)
+ struct tree *write_tree_from_memory(void)
  {
        struct tree *result = NULL;
  
-       if (unmerged_index()) {
+       if (unmerged_cache()) {
                int i;
                output(0, "There are unmerged index entries:");
                for (i = 0; i < active_nr; i++) {
@@@ -844,9 -835,8 +835,9 @@@ static int read_merge_config(const cha
        int namelen;
  
        if (!strcmp(var, "merge.default")) {
 -              if (value)
 -                      default_ll_merge = strdup(value);
 +              if (!value)
 +                      return config_error_nonbool(var);
 +              default_ll_merge = strdup(value);
                return 0;
        }
  
  
        if (!strcmp("name", ep)) {
                if (!value)
 -                      return error("%s: lacks value", var);
 +                      return config_error_nonbool(var);
                fn->description = strdup(value);
                return 0;
        }
  
        if (!strcmp("driver", ep)) {
                if (!value)
 -                      return error("%s: lacks value", var);
 +                      return config_error_nonbool(var);
                /*
                 * merge.<name>.driver specifies the command line:
                 *
  
        if (!strcmp("recursive", ep)) {
                if (!value)
 -                      return error("%s: lacks value", var);
 +                      return config_error_nonbool(var);
                fn->recursive = strdup(value);
                return 0;
        }
@@@ -1496,12 -1486,12 +1487,12 @@@ static int process_entry(const char *pa
        return clean_merge;
  }
  
static int merge_trees(struct tree *head,
-                      struct tree *merge,
-                      struct tree *common,
-                      const char *branch1,
-                      const char *branch2,
-                      struct tree **result)
+ int merge_trees(struct tree *head,
+               struct tree *merge,
+               struct tree *common,
+               const char *branch1,
+               const char *branch2,
+               struct tree **result)
  {
        int code, clean;
  
                    sha1_to_hex(head->object.sha1),
                    sha1_to_hex(merge->object.sha1));
  
-       if (unmerged_index()) {
+       if (unmerged_cache()) {
                struct path_list *entries, *re_head, *re_merge;
                int i;
                path_list_clear(&current_file_set, 1);
                clean = 1;
  
        if (index_only)
-               *result = git_write_tree();
+               *result = write_tree_from_memory();
  
        return clean;
  }
@@@ -1573,12 -1563,12 +1564,12 @@@ static struct commit_list *reverse_comm
   * Merge the commits h1 and h2, return the resulting virtual
   * commit object and a flag indicating the cleanness of the merge.
   */
static int merge(struct commit *h1,
-                struct commit *h2,
-                const char *branch1,
-                const char *branch2,
-                struct commit_list *ca,
-                struct commit **result)
int merge_recursive(struct commit *h1,
+                   struct commit *h2,
+                   const char *branch1,
+                   const char *branch2,
+                   struct commit_list *ca,
+                   struct commit **result)
  {
        struct commit_list *iter;
        struct commit *merged_common_ancestors;
                 * "conflicts" were already resolved.
                 */
                discard_cache();
-               merge(merged_common_ancestors, iter->item,
-                     "Temporary merge branch 1",
-                     "Temporary merge branch 2",
-                     NULL,
-                     &merged_common_ancestors);
+               merge_recursive(merged_common_ancestors, iter->item,
+                               "Temporary merge branch 1",
+                               "Temporary merge branch 2",
+                               NULL,
+                               &merged_common_ancestors);
                call_depth--;
  
                if (!merged_common_ancestors)
@@@ -1673,8 -1663,6 +1664,8 @@@ static struct commit *get_ref(const cha
        if (get_sha1(ref, sha1))
                die("Could not resolve ref '%s'", ref);
        object = deref_tag(parse_object(sha1), ref, strlen(ref));
 +      if (!object)
 +              return NULL;
        if (object->type == OBJ_TREE)
                return make_virtual_commit((struct tree*)object,
                        better_branch_name(ref));
@@@ -1698,7 -1686,7 +1689,7 @@@ static int merge_config(const char *var
        return git_default_config(var, value);
  }
  
- int main(int argc, char *argv[])
+ int cmd_merge_recursive(int argc, const char **argv, const char *prefix)
  {
        static const char *bases[20];
        static unsigned bases_count = 0;
                struct commit *ancestor = get_ref(bases[i]);
                ca = commit_list_insert(ancestor, &ca);
        }
-       clean = merge(h1, h2, branch1, branch2, ca, &result);
+       clean = merge_recursive(h1, h2, branch1, branch2, ca, &result);
  
        if (active_cache_changed &&
            (write_cache(index_fd, active_cache, active_nr) ||
diff --combined builtin-read-tree.c
index 7bdc312e3861103fa83b20b939962a8bc8e6cc0a,1d9d125b91f976ebbc5ef0b1ca0fe7c0caeabb8a..0138f5a9172034b2ce34222ff077a975f8998005
@@@ -41,12 -41,11 +41,12 @@@ static int read_cache_unmerged(void
        for (i = 0; i < active_nr; i++) {
                struct cache_entry *ce = active_cache[i];
                if (ce_stage(ce)) {
 +                      remove_index_entry(ce);
                        if (last && !strcmp(ce->name, last->name))
                                continue;
                        cache_tree_invalidate_path(active_cache_tree, ce->name);
                        last = ce;
 -                      ce->ce_flags |= CE_REMOVE;
 +                      continue;
                }
                *dst++ = ce;
        }
@@@ -269,7 -268,8 +269,8 @@@ int cmd_read_tree(int argc, const char 
                parse_tree(tree);
                init_tree_desc(t+i, tree->buffer, tree->size);
        }
-       unpack_trees(nr_trees, t, &opts);
+       if (unpack_trees(nr_trees, t, &opts))
+               return 128;
  
        /*
         * When reading only one tree (either the most basic form,
diff --combined builtin.h
index 3d1628c597b21cc22b750809c27d4499e725d259,25d91bbfb21ea3c1ea067b10f7ea033d3563936a..674c8a141faf808883c9de283d10da01b3f9c2d5
+++ b/builtin.h
@@@ -8,6 -8,7 +8,6 @@@ extern const char git_usage_string[]
  
  extern void list_common_cmds_help(void);
  extern void help_unknown_cmd(const char *cmd);
 -extern int write_tree(unsigned char *sha1, int missing_ok, const char *prefix);
  extern void prune_packed_objects(int);
  
  extern int cmd_add(int argc, const char **argv, const char *prefix);
@@@ -18,6 -19,7 +18,7 @@@ extern int cmd_blame(int argc, const ch
  extern int cmd_branch(int argc, const char **argv, const char *prefix);
  extern int cmd_bundle(int argc, const char **argv, const char *prefix);
  extern int cmd_cat_file(int argc, const char **argv, const char *prefix);
+ extern int cmd_checkout(int argc, const char **argv, const char *prefix);
  extern int cmd_checkout_index(int argc, const char **argv, const char *prefix);
  extern int cmd_check_attr(int argc, const char **argv, const char *prefix);
  extern int cmd_check_ref_format(int argc, const char **argv, const char *prefix);
@@@ -56,6 -58,7 +57,7 @@@ extern int cmd_mailsplit(int argc, cons
  extern int cmd_merge_base(int argc, const char **argv, const char *prefix);
  extern int cmd_merge_ours(int argc, const char **argv, const char *prefix);
  extern int cmd_merge_file(int argc, const char **argv, const char *prefix);
+ extern int cmd_merge_recursive(int argc, const char **argv, const char *prefix);
  extern int cmd_mv(int argc, const char **argv, const char *prefix);
  extern int cmd_name_rev(int argc, const char **argv, const char *prefix);
  extern int cmd_pack_objects(int argc, const char **argv, const char *prefix);
diff --combined cache.h
index 7769f055e964e4c1e29a1129050dd439940e29f0,888895a9695f9362df8c0ac03f3e762fd61d2a55..9ebe7913b01aed7cbfa089ce811e57bbc64a6e15
+++ b/cache.h
@@@ -110,6 -110,7 +110,6 @@@ struct ondisk_cache_entry 
  };
  
  struct cache_entry {
 -      struct cache_entry *next;
        unsigned int ce_ctime;
        unsigned int ce_mtime;
        unsigned int ce_dev;
        unsigned int ce_size;
        unsigned int ce_flags;
        unsigned char sha1[20];
 +      struct cache_entry *next;
        char name[FLEX_ARRAY]; /* more */
  };
  
  #define CE_UPDATE    (0x10000)
  #define CE_REMOVE    (0x20000)
  #define CE_UPTODATE  (0x40000)
 -#define CE_UNHASHED  (0x80000)
 +
 +#define CE_HASHED    (0x100000)
 +#define CE_UNHASHED  (0x200000)
 +
 +/*
 + * Copy the sha1 and stat state of a cache entry from one to
 + * another. But we never change the name, or the hash state!
 + */
 +#define CE_STATE_MASK (CE_HASHED | CE_UNHASHED)
 +static inline void copy_cache_entry(struct cache_entry *dst, struct cache_entry *src)
 +{
 +      unsigned int state = dst->ce_flags & CE_STATE_MASK;
 +
 +      /* Don't copy hash chain and name */
 +      memcpy(dst, src, offsetof(struct cache_entry, next));
 +
 +      /* Restore the hash state */
 +      dst->ce_flags = (dst->ce_flags & ~CE_STATE_MASK) | state;
 +}
 +
 +/*
 + * We don't actually *remove* it, we can just mark it invalid so that
 + * we won't find it in lookups.
 + *
 + * Not only would we have to search the lists (simple enough), but
 + * we'd also have to rehash other hash buckets in case this makes the
 + * hash bucket empty (common). So it's much better to just mark
 + * it.
 + */
 +static inline void remove_index_entry(struct cache_entry *ce)
 +{
 +      ce->ce_flags |= CE_UNHASHED;
 +}
  
  static inline unsigned create_ce_flags(size_t len, unsigned stage)
  {
@@@ -210,18 -178,6 +210,18 @@@ static inline unsigned int ce_mode_from
        }
        return create_ce_mode(mode);
  }
 +static inline int ce_to_dtype(const struct cache_entry *ce)
 +{
 +      unsigned ce_mode = ntohl(ce->ce_mode);
 +      if (S_ISREG(ce_mode))
 +              return DT_REG;
 +      else if (S_ISDIR(ce_mode) || S_ISGITLINK(ce_mode))
 +              return DT_DIR;
 +      else if (S_ISLNK(ce_mode))
 +              return DT_LNK;
 +      else
 +              return DT_UNKNOWN;
 +}
  #define canon_mode(mode) \
        (S_ISREG(mode) ? (S_IFREG | ce_permissions(mode)) : \
        S_ISLNK(mode) ? S_IFLNK : S_ISDIR(mode) ? S_IFDIR : S_IFGITLINK)
@@@ -252,6 -208,7 +252,7 @@@ extern struct index_state the_index
  #define read_cache_from(path) read_index_from(&the_index, (path))
  #define write_cache(newfd, cache, entries) write_index(&the_index, (newfd))
  #define discard_cache() discard_index(&the_index)
+ #define unmerged_cache() unmerged_index(&the_index)
  #define cache_name_pos(name, namelen) index_name_pos(&the_index,(name),(namelen))
  #define add_cache_entry(ce, option) add_index_entry(&the_index, (ce), (option))
  #define remove_cache_entry_at(pos) remove_index_entry_at(&the_index, (pos))
@@@ -346,6 -303,7 +347,7 @@@ extern int read_index(struct index_stat
  extern int read_index_from(struct index_state *, const char *path);
  extern int write_index(struct index_state *, int newfd);
  extern int discard_index(struct index_state *);
+ extern int unmerged_index(struct index_state *);
  extern int verify_path(const char *path);
  extern int index_name_exists(struct index_state *istate, const char *name, int namelen);
  extern int index_name_pos(struct index_state *, const char *name, int namelen);
@@@ -415,14 -373,6 +417,14 @@@ extern size_t packed_git_limit
  extern size_t delta_base_cache_limit;
  extern int auto_crlf;
  
 +enum safe_crlf {
 +      SAFE_CRLF_FALSE = 0,
 +      SAFE_CRLF_FAIL = 1,
 +      SAFE_CRLF_WARN = 2,
 +};
 +
 +extern enum safe_crlf safe_crlf;
 +
  #define GIT_REPO_VERSION 0
  extern int repository_format_version;
  extern int check_repository_format(void);
@@@ -677,16 -627,11 +679,16 @@@ extern int git_parse_ulong(const char *
  extern int git_config_int(const char *, const char *);
  extern unsigned long git_config_ulong(const char *, const char *);
  extern int git_config_bool(const char *, const char *);
 +extern int git_config_string(const char **, const char *, const char *);
  extern int git_config_set(const char *, const char *);
  extern int git_config_set_multivar(const char *, const char *, const char *, int);
  extern int git_config_rename_section(const char *, const char *);
  extern const char *git_etc_gitconfig(void);
  extern int check_repository_format_version(const char *var, const char *value);
 +extern int git_env_bool(const char *, int);
 +extern int git_config_system(void);
 +extern int git_config_global(void);
 +extern int config_error_nonbool(const char *);
  
  #define MAX_GITNAME (1000)
  extern char git_default_email[MAX_GITNAME];
@@@ -698,7 -643,6 +700,7 @@@ extern const char *git_log_output_encod
  /* IO helper functions */
  extern void maybe_flush_or_die(FILE *, const char *);
  extern int copy_fd(int ifd, int ofd);
 +extern int copy_file(const char *dst, const char *src, int mode);
  extern int read_in_full(int fd, void *buf, size_t count);
  extern int write_in_full(int fd, const void *buf, size_t count);
  extern void write_or_die(int fd, const void *buf, size_t count);
@@@ -707,12 -651,12 +709,12 @@@ extern int write_or_whine_pipe(int fd, 
  
  /* pager.c */
  extern void setup_pager(void);
 -extern char *pager_program;
 +extern const char *pager_program;
  extern int pager_in_use(void);
  extern int pager_use_color;
  
 -extern char *editor_program;
 -extern char *excludes_file;
 +extern const char *editor_program;
 +extern const char *excludes_file;
  
  /* base85 */
  int decode_85(char *dst, const char *line, int linelen);
@@@ -732,8 -676,7 +734,8 @@@ extern void trace_argv_printf(const cha
  
  /* convert.c */
  /* returns 1 if *dst was used */
 -extern int convert_to_git(const char *path, const char *src, size_t len, struct strbuf *dst);
 +extern int convert_to_git(const char *path, const char *src, size_t len,
 +                          struct strbuf *dst, enum safe_crlf checksafe);
  extern int convert_to_working_tree(const char *path, const char *src, size_t len, struct strbuf *dst);
  
  /* add */
@@@ -752,7 -695,6 +754,7 @@@ void shift_tree(const unsigned char *, 
  #define WS_TRAILING_SPACE     01
  #define WS_SPACE_BEFORE_TAB   02
  #define WS_INDENT_WITH_NON_TAB        04
 +#define WS_CR_AT_EOL           010
  #define WS_DEFAULT_RULE (WS_TRAILING_SPACE|WS_SPACE_BEFORE_TAB)
  extern unsigned whitespace_rule_cfg;
  extern unsigned whitespace_rule(const char *);
@@@ -761,13 -703,10 +763,13 @@@ extern unsigned check_and_emit_line(con
      FILE *stream, const char *set,
      const char *reset, const char *ws);
  extern char *whitespace_error_string(unsigned ws);
 +extern int ws_fix_copy(char *, const char *, int, unsigned, int *);
  
  /* ls-files */
  int pathspec_match(const char **spec, char *matched, const char *filename, int skiplen);
  int report_path_error(const char *ps_matched, const char **pathspec, int prefix_offset);
  void overlay_tree_on_cache(const char *tree_name, const char *prefix);
  
 +char *alias_lookup(const char *alias);
 +
  #endif /* CACHE_H */
index 1a7689a48f07a6ed2bb156f745bfea19a10e3eb9,5621c69d86062c7c75c0b8c2749d34efc78cafb4..1a7689a48f07a6ed2bb156f745bfea19a10e3eb9
@@@ -71,8 -71,7 +71,8 @@@ while test $# != 0; d
  done
  
  arg="$1"
 -if rev=$(git rev-parse --verify "$arg^0" 2>/dev/null)
 +rev=$(git rev-parse --verify "$arg" 2>/dev/null)
 +if rev=$(git rev-parse --verify "$rev^0" 2>/dev/null)
  then
        [ -z "$rev" ] && die "unknown flag $arg"
        new_name="$arg"
        fi
        new="$rev"
        shift
 -elif rev=$(git rev-parse --verify "$arg^{tree}" 2>/dev/null)
 +elif rev=$(git rev-parse --verify "$rev^{tree}" 2>/dev/null)
  then
        # checking out selected paths from a tree-ish.
        new="$rev"
 -      new_name="$arg^{tree}"
 +      new_name="$rev^{tree}"
        shift
  fi
  [ "$1" = "--" ] && shift
@@@ -210,14 -209,11 +210,14 @@@ the
      git read-tree $v --reset -u $new
  else
      git update-index --refresh >/dev/null
 -    merge_error=$(git read-tree -m -u --exclude-per-directory=.gitignore $old $new 2>&1) || (
 -      case "$merge" in
 -      '')
 -              echo >&2 "$merge_error"
 +    git read-tree $v -m -u --exclude-per-directory=.gitignore $old $new || (
 +      case "$merge,$v" in
 +      ,*)
                exit 1 ;;
 +      1,)
 +              ;; # quiet
 +      *)
 +              echo >&2 "Falling back to 3-way merge..." ;;
        esac
  
        # Match the index to the working tree, and do a three-way.
diff --combined git.c
index 8f08b12295e1641577ffd61ee71c7b7247645f81,bd424ea9bd9e54a214f587acfc2ca266105d0168..9cca81a60e6d4f93cf3132b76fd8f147a6a5d98f
--- 1/git.c
--- 2/git.c
+++ b/git.c
@@@ -87,6 -87,17 +87,6 @@@ static int handle_options(const char**
        return handled;
  }
  
 -static const char *alias_command;
 -static char *alias_string;
 -
 -static int git_alias_config(const char *var, const char *value)
 -{
 -      if (!prefixcmp(var, "alias.") && !strcmp(var + 6, alias_command)) {
 -              alias_string = xstrdup(value);
 -      }
 -      return 0;
 -}
 -
  static int split_cmdline(char *cmdline, const char ***argv)
  {
        int src, dst, count = 0, size = 16;
@@@ -146,13 -157,11 +146,13 @@@ static int handle_alias(int *argcp, con
        const char *subdir;
        int count, option_count;
        const char** new_argv;
 +      const char *alias_command;
 +      char *alias_string;
  
        subdir = setup_git_directory_gently(&nongit);
  
        alias_command = (*argv)[0];
 -      git_config(git_alias_config);
 +      alias_string = alias_lookup(alias_command);
        if (alias_string) {
                if (alias_string[0] == '!') {
                        if (*argcp > 1) {
@@@ -278,6 -287,7 +278,7 @@@ static void handle_internal_command(in
                { "branch", cmd_branch, RUN_SETUP },
                { "bundle", cmd_bundle },
                { "cat-file", cmd_cat_file, RUN_SETUP },
+               { "checkout", cmd_checkout, RUN_SETUP | NEED_WORK_TREE },
                { "checkout-index", cmd_checkout_index,
                        RUN_SETUP | NEED_WORK_TREE},
                { "check-ref-format", cmd_check_ref_format },
                { "merge-base", cmd_merge_base, RUN_SETUP },
                { "merge-file", cmd_merge_file },
                { "merge-ours", cmd_merge_ours, RUN_SETUP },
+               { "merge-recursive", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE },
+               { "merge-subtree", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE },
                { "mv", cmd_mv, RUN_SETUP | NEED_WORK_TREE },
                { "name-rev", cmd_name_rev, RUN_SETUP },
                { "pack-objects", cmd_pack_objects, RUN_SETUP },
diff --combined read-cache.c
index fee0c80734db2d5106c4c6fd6bc396af1831c392,22d7b462454463fb149d02ac62415fc63aca7672..657f0c5894c65831b80ceee54d161d0beac1d733
@@@ -37,13 -37,8 +37,13 @@@ static unsigned int hash_name(const cha
  static void hash_index_entry(struct index_state *istate, struct cache_entry *ce)
  {
        void **pos;
 -      unsigned int hash = hash_name(ce->name, ce_namelen(ce));
 +      unsigned int hash;
  
 +      if (ce->ce_flags & CE_HASHED)
 +              return;
 +      ce->ce_flags |= CE_HASHED;
 +      ce->next = NULL;
 +      hash = hash_name(ce->name, ce_namelen(ce));
        pos = insert_hash(hash, ce, &istate->name_hash);
        if (pos) {
                ce->next = *pos;
@@@ -64,18 -59,33 +64,18 @@@ static void lazy_init_name_hash(struct 
  
  static void set_index_entry(struct index_state *istate, int nr, struct cache_entry *ce)
  {
 +      ce->ce_flags &= ~CE_UNHASHED;
        istate->cache[nr] = ce;
        if (istate->name_hash_initialized)
                hash_index_entry(istate, ce);
  }
  
 -/*
 - * We don't actually *remove* it, we can just mark it invalid so that
 - * we won't find it in lookups.
 - *
 - * Not only would we have to search the lists (simple enough), but
 - * we'd also have to rehash other hash buckets in case this makes the
 - * hash bucket empty (common). So it's much better to just mark
 - * it.
 - */
 -static void remove_hash_entry(struct index_state *istate, struct cache_entry *ce)
 -{
 -      ce->ce_flags |= CE_UNHASHED;
 -}
 -
  static void replace_index_entry(struct index_state *istate, int nr, struct cache_entry *ce)
  {
        struct cache_entry *old = istate->cache[nr];
  
 -      if (ce != old) {
 -              remove_hash_entry(istate, old);
 -              set_index_entry(istate, nr, ce);
 -      }
 +      remove_index_entry(old);
 +      set_index_entry(istate, nr, ce);
        istate->cache_changed = 1;
  }
  
@@@ -403,7 -413,7 +403,7 @@@ int remove_index_entry_at(struct index_
  {
        struct cache_entry *ce = istate->cache[pos];
  
 -      remove_hash_entry(istate, ce);
 +      remove_index_entry(ce);
        istate->cache_changed = 1;
        istate->cache_nr--;
        if (pos >= istate->cache_nr)
@@@ -1166,6 -1176,16 +1166,16 @@@ int discard_index(struct index_state *i
        return 0;
  }
  
+ int unmerged_index(struct index_state *istate)
+ {
+       int i;
+       for (i = 0; i < istate->cache_nr; i++) {
+               if (ce_stage(istate->cache[i]))
+                       return 1;
+       }
+       return 0;
+ }
  #define WRITE_BUFFER_SIZE 8192
  static unsigned char write_buffer[WRITE_BUFFER_SIZE];
  static unsigned long write_buffer_len;
diff --combined t/t7201-co.sh
index dbf1ace29ef8ad178a0ad8539e6bde30482ee60f,0fa94678ad223e7a6f9db21a26c32648483701bc..724adef0bfdc1f9028f112451c76d9053b60223f
@@@ -83,13 -83,13 +83,13 @@@ test_expect_success "checkout with unre
        fill 0 1 2 3 4 5 6 7 8 >same &&
        cp same kept
        git checkout side >messages &&
-       git diff same kept
+       diff -u same kept
        (cat > messages.expect <<EOF
  M     same
  EOF
  ) &&
        touch messages.expect &&
-       git diff messages.expect messages
+       diff -u messages.expect messages
  '
  
  test_expect_success "checkout -m with dirty tree" '
        test "$(git symbolic-ref HEAD)" = "refs/heads/side" &&
  
        (cat >expect.messages <<EOF
- Merging side with local
- Merging:
- ab76817 Side M one, D two, A three
- virtual local
- found 1 common ancestor(s):
- 7329388 Initial A one, A two
- Auto-merged one
  M     one
  EOF
  ) &&
-       git diff expect.messages messages &&
+       diff -u expect.messages messages &&
  
        fill "M one" "A three" "D       two" >expect.master &&
        git diff --name-status master >current.master &&
-       diff expect.master current.master &&
+       diff -u expect.master current.master &&
  
        fill "M one" >expect.side &&
        git diff --name-status side >current.side &&
-       diff expect.side current.side &&
+       diff -u expect.side current.side &&
  
        : >expect.index &&
        git diff --cached >current.index &&
-       diff expect.index current.index
+       diff -u expect.index current.index
  '
  
  test_expect_success "checkout -m with dirty tree, renamed" '
  
        git checkout -m renamer &&
        fill 1 3 4 5 7 8 >expect &&
-       diff expect uno &&
+       diff -u expect uno &&
        ! test -f one &&
        git diff --cached >current &&
        ! test -s current
@@@ -168,7 -161,7 +161,7 @@@ test_expect_success 'checkout -m with m
        git diff master:one :3:uno |
        sed -e "1,/^@@/d" -e "/^ /d" -e "s/^-/d/" -e "s/^+/a/" >current &&
        fill d2 aT d7 aS >expect &&
-       diff current expect &&
+       diff -u current expect &&
        git diff --cached two >current &&
        ! test -s current
  '
@@@ -185,7 -178,7 +178,7 @@@ If you want to create a new branch fro
  HEAD is now at 7329388... Initial A one, A two
  EOF
  ) &&
-       git diff messages.expect messages &&
+       diff -u messages.expect messages &&
        H=$(git rev-parse --verify HEAD) &&
        M=$(git show-ref -s --verify refs/heads/master) &&
        test "z$H" = "z$M" &&
@@@ -214,22 -207,6 +207,22 @@@ test_expect_success 'checkout to detac
        fi
  '
  
 +test_expect_success 'checkout to detach HEAD with :/message' '
 +
 +      git checkout -f master && git clean -f &&
 +      git checkout ":/Initial" &&
 +      H=$(git rev-parse --verify HEAD) &&
 +      M=$(git show-ref -s --verify refs/heads/master) &&
 +      test "z$H" = "z$M" &&
 +      if git symbolic-ref HEAD >/dev/null 2>&1
 +      then
 +              echo "OOPS, HEAD is still symbolic???"
 +              false
 +      else
 +              : happy
 +      fi
 +'
 +
  test_expect_success 'checkout to detach HEAD with HEAD^0' '
  
        git checkout -f master && git clean -f &&
@@@ -286,4 -263,38 +279,38 @@@ test_expect_success 'checkout with ambi
  
  '
  
+ test_expect_success 'switch branches while in subdirectory' '
+       git reset --hard &&
+       git checkout master &&
+       mkdir subs &&
+       (
+               cd subs &&
+               git checkout side
+       ) &&
+       ! test -f subs/one &&
+       rm -fr subs
+ '
+ test_expect_success 'checkout specific path while in subdirectory' '
+       git reset --hard &&
+       git checkout side &&
+       mkdir subs &&
+       >subs/bero &&
+       git add subs/bero &&
+       git commit -m "add subs/bero" &&
+       git checkout master &&
+       mkdir -p subs &&
+       (
+               cd subs &&
+               git checkout side -- bero
+       ) &&
+       test -f subs/bero
+ '
  test_done
diff --combined unpack-trees.c
index 56c1ffbc199c534a53da9615aa16b4357656e320,470fa02e0886462f44fd0e188da6e8c65ab6fac0..3e448d8974eb6d738fec2c35cc5a9ffbc8764411
@@@ -85,6 -85,7 +85,7 @@@ static int unpack_trees_rec(struct tree
                int any_dirs = 0;
                char *cache_name;
                int ce_stage;
+               int skip_entry = 0;
  
                /* Find the first name in the input. */
  
  
  #if DBRT_DEBUG > 1
                if (first)
-                       printf("index %s\n", first);
+                       fprintf(stderr, "index %s\n", first);
  #endif
                for (i = 0; i < len; i++) {
                        if (!posns[i] || posns[i] == df_conflict_list)
                                continue;
  #if DBRT_DEBUG > 1
-                       printf("%d %s\n", i + 1, posns[i]->name);
+                       fprintf(stderr, "%d %s\n", i + 1, posns[i]->name);
  #endif
                        if (!first || entcmp(first, firstdir,
                                             posns[i]->name,
                        any_files = 1;
                        src[0] = active_cache[o->pos];
                        remove = o->pos;
+                       if (o->skip_unmerged && ce_stage(src[0]))
+                               skip_entry = 1;
                }
  
                for (i = 0; i < len; i++) {
                                continue;
                        }
  
+                       if (skip_entry) {
+                               subposns[i] = df_conflict_list;
+                               posns[i] = posns[i]->next;
+                               continue;
+                       }
                        if (!o->merge)
                                ce_stage = 0;
                        else if (i + 1 < o->head_idx)
                        posns[i] = posns[i]->next;
                }
                if (any_files) {
-                       if (o->merge) {
+                       if (skip_entry) {
+                               o->pos++;
+                               while (o->pos < active_nr &&
+                                      !strcmp(active_cache[o->pos]->name,
+                                              src[0]->name))
+                                       o->pos++;
+                       } else if (o->merge) {
                                int ret;
  
  #if DBRT_DEBUG > 1
-                               printf("%s:\n", first);
+                               fprintf(stderr, "%s:\n", first);
                                for (i = 0; i < src_size; i++) {
-                                       printf(" %d ", i);
+                                       fprintf(stderr, " %d ", i);
                                        if (src[i])
-                                               printf("%s\n", sha1_to_hex(src[i]->sha1));
+                                               fprintf(stderr, "%06x %s\n", src[i]->ce_mode, sha1_to_hex(src[i]->sha1));
                                        else
-                                               printf("\n");
+                                               fprintf(stderr, "\n");
                                }
  #endif
                                ret = o->fn(src, o, remove);
+                               if (ret < 0)
+                                       return ret;
  
  #if DBRT_DEBUG > 1
-                               printf("Added %d entries\n", ret);
+                               fprintf(stderr, "Added %d entries\n", ret);
  #endif
                                o->pos += ret;
                        } else {
@@@ -286,34 -303,36 +303,36 @@@ static void unlink_entry(char *name, ch
  }
  
  static struct checkout state;
- static void check_updates(struct cache_entry **src, int nr,
-                       struct unpack_trees_options *o)
+ static void check_updates(struct unpack_trees_options *o)
  {
        unsigned cnt = 0, total = 0;
        struct progress *progress = NULL;
        char last_symlink[PATH_MAX];
+       int i;
  
        if (o->update && o->verbose_update) {
-               for (total = cnt = 0; cnt < nr; cnt++) {
-                       struct cache_entry *ce = src[cnt];
+               for (total = cnt = 0; cnt < active_nr; cnt++) {
+                       struct cache_entry *ce = active_cache[cnt];
                        if (ce->ce_flags & (CE_UPDATE | CE_REMOVE))
                                total++;
                }
  
                progress = start_progress_delay("Checking out files",
 -                                              total, 50, 2);
 +                                              total, 50, 1);
                cnt = 0;
        }
  
        *last_symlink = '\0';
-       while (nr--) {
-               struct cache_entry *ce = *src++;
+       for (i = 0; i < active_nr; i++) {
+               struct cache_entry *ce = active_cache[i];
  
                if (ce->ce_flags & (CE_UPDATE | CE_REMOVE))
                        display_progress(progress, ++cnt);
                if (ce->ce_flags & CE_REMOVE) {
                        if (o->update)
                                unlink_entry(ce->name, last_symlink);
+                       remove_cache_entry_at(i);
+                       i--;
                        continue;
                }
                if (ce->ce_flags & CE_UPDATE) {
@@@ -354,23 -373,34 +373,34 @@@ int unpack_trees(unsigned len, struct t
                        posns[i] = create_tree_entry_list(t+i);
  
                if (unpack_trees_rec(posns, len, o->prefix ? o->prefix : "",
-                                    o, &df_conflict_list))
+                                    o, &df_conflict_list)) {
+                       if (o->gently) {
+                               discard_cache();
+                               read_cache();
+                       }
                        return -1;
+               }
        }
  
-       if (o->trivial_merges_only && o->nontrivial_merge)
-               die("Merge requires file-level merging");
+       if (o->trivial_merges_only && o->nontrivial_merge) {
+               if (o->gently) {
+                       discard_cache();
+                       read_cache();
+               }
+               return o->gently ? -1 :
+                       error("Merge requires file-level merging");
+       }
  
-       check_updates(active_cache, active_nr, o);
+       check_updates(o);
        return 0;
  }
  
  /* Here come the merge functions */
  
- static void reject_merge(struct cache_entry *ce)
+ static int reject_merge(struct cache_entry *ce)
  {
-       die("Entry '%s' would be overwritten by merge. Cannot merge.",
-           ce->name);
+       return error("Entry '%s' would be overwritten by merge. Cannot merge.",
+                    ce->name);
  }
  
  static int same(struct cache_entry *a, struct cache_entry *b)
   * When a CE gets turned into an unmerged entry, we
   * want it to be up-to-date
   */
- static void verify_uptodate(struct cache_entry *ce,
+ static int verify_uptodate(struct cache_entry *ce,
                struct unpack_trees_options *o)
  {
        struct stat st;
  
        if (o->index_only || o->reset)
-               return;
+               return 0;
  
        if (!lstat(ce->name, &st)) {
                unsigned changed = ce_match_stat(ce, &st, CE_MATCH_IGNORE_VALID);
                if (!changed)
-                       return;
+                       return 0;
                /*
                 * NEEDSWORK: the current default policy is to allow
                 * submodule to be out of sync wrt the supermodule
                 * checked out.
                 */
                if (S_ISGITLINK(ce->ce_mode))
-                       return;
+                       return 0;
                errno = 0;
        }
        if (errno == ENOENT)
-               return;
-       die("Entry '%s' not uptodate. Cannot merge.", ce->name);
+               return 0;
+       return o->gently ? -1 :
+               error("Entry '%s' not uptodate. Cannot merge.", ce->name);
  }
  
  static void invalidate_ce_path(struct cache_entry *ce)
@@@ -479,7 -510,8 +510,8 @@@ static int verify_clean_subdirectory(st
                 * ce->name is an entry in the subdirectory.
                 */
                if (!ce_stage(ce)) {
-                       verify_uptodate(ce, o);
+                       if (verify_uptodate(ce, o))
+                               return -1;
                        ce->ce_flags |= CE_REMOVE;
                }
                cnt++;
                d.exclude_per_dir = o->dir->exclude_per_dir;
        i = read_directory(&d, ce->name, pathbuf, namelen+1, NULL);
        if (i)
-               die("Updating '%s' would lose untracked files in it",
-                   ce->name);
+               return o->gently ? -1 :
+                       error("Updating '%s' would lose untracked files in it",
+                             ce->name);
        free(pathbuf);
        return cnt;
  }
   * We do not want to remove or overwrite a working tree file that
   * is not tracked, unless it is ignored.
   */
- static void verify_absent(struct cache_entry *ce, const char *action,
-               struct unpack_trees_options *o)
+ static int verify_absent(struct cache_entry *ce, const char *action,
+                        struct unpack_trees_options *o)
  {
        struct stat st;
  
        if (o->index_only || o->reset || !o->update)
-               return;
+               return 0;
  
        if (has_symlink_leading_path(ce->name, NULL))
-               return;
+               return 0;
  
        if (!lstat(ce->name, &st)) {
                int cnt;
 +              int dtype = ce_to_dtype(ce);
  
 -              if (o->dir && excluded(o->dir, ce->name))
 +              if (o->dir && excluded(o->dir, ce->name, &dtype))
                        /*
                         * ce->name is explicitly excluded, so it is Ok to
                         * overwrite it.
                         */
-                       return;
+                       return 0;
                if (S_ISDIR(st.st_mode)) {
                        /*
                         * We are checking out path "foo" and
                         * deleted entries here.
                         */
                        o->pos += cnt;
-                       return;
+                       return 0;
                }
  
                /*
                if (0 <= cnt) {
                        struct cache_entry *ce = active_cache[cnt];
                        if (ce->ce_flags & CE_REMOVE)
-                               return;
+                               return 0;
                }
  
-               die("Untracked working tree file '%s' "
-                   "would be %s by merge.", ce->name, action);
+               return o->gently ? -1 :
+                       error("Untracked working tree file '%s' "
+                             "would be %s by merge.", ce->name, action);
        }
+       return 0;
  }
  
  static int merged_entry(struct cache_entry *merge, struct cache_entry *old,
                 * a match.
                 */
                if (same(old, merge)) {
 -                      memcpy(merge, old, offsetof(struct cache_entry, name));
 +                      copy_cache_entry(merge, old);
                } else {
-                       verify_uptodate(old, o);
+                       if (verify_uptodate(old, o))
+                               return -1;
                        invalidate_ce_path(old);
                }
        }
        else {
-               verify_absent(merge, "overwritten", o);
+               if (verify_absent(merge, "overwritten", o))
+                       return -1;
                invalidate_ce_path(merge);
        }
  
  static int deleted_entry(struct cache_entry *ce, struct cache_entry *old,
                struct unpack_trees_options *o)
  {
-       if (old)
-               verify_uptodate(old, o);
-       else
-               verify_absent(ce, "removed", o);
+       if (old) {
+               if (verify_uptodate(old, o))
+                       return -1;
+       } else
+               if (verify_absent(ce, "removed", o))
+                       return -1;
        ce->ce_flags |= CE_REMOVE;
        add_cache_entry(ce, ADD_CACHE_OK_TO_ADD|ADD_CACHE_OK_TO_REPLACE);
        invalidate_ce_path(ce);
@@@ -700,16 -738,15 +739,15 @@@ int threeway_merge(struct cache_entry *
        /* #14, #14ALT, #2ALT */
        if (remote && !df_conflict_head && head_match && !remote_match) {
                if (index && !same(index, remote) && !same(index, head))
-                       reject_merge(index);
+                       return o->gently ? -1 : reject_merge(index);
                return merged_entry(remote, index, o);
        }
        /*
         * If we have an entry in the index cache, then we want to
         * make sure that it matches head.
         */
-       if (index && !same(index, head)) {
-               reject_merge(index);
-       }
+       if (index && !same(index, head))
+               return o->gently ? -1 : reject_merge(index);
  
        if (head) {
                /* #5ALT, #15 */
                        remove_entry(remove);
                        if (index)
                                return deleted_entry(index, index, o);
-                       else if (ce && !head_deleted)
-                               verify_absent(ce, "removed", o);
+                       else if (ce && !head_deleted) {
+                               if (verify_absent(ce, "removed", o))
+                                       return -1;
+                       }
                        return 0;
                }
                /*
         * conflict resolution files.
         */
        if (index) {
-               verify_uptodate(index, o);
+               if (verify_uptodate(index, o))
+                       return -1;
        }
  
        remove_entry(remove);
@@@ -856,11 -896,11 +897,11 @@@ int twoway_merge(struct cache_entry **s
                        /* all other failures */
                        remove_entry(remove);
                        if (oldtree)
-                               reject_merge(oldtree);
+                               return o->gently ? -1 : reject_merge(oldtree);
                        if (current)
-                               reject_merge(current);
+                               return o->gently ? -1 : reject_merge(current);
                        if (newtree)
-                               reject_merge(newtree);
+                               return o->gently ? -1 : reject_merge(newtree);
                        return -1;
                }
        }
@@@ -887,7 -927,8 +928,8 @@@ int bind_merge(struct cache_entry **src
                return error("Cannot do a bind merge of %d trees\n",
                             o->merge_size);
        if (a && old)
-               die("Entry '%s' overlaps.  Cannot bind.", a->name);
+               return o->gently ? -1 :
+                       error("Entry '%s' overlaps.  Cannot bind.", a->name);
        if (!a)
                return keep_entry(old, o);
        else