Merge branch 'cc/reset-more'
authorJunio C Hamano <gitster@pobox.com>
Wed, 13 Jan 2010 19:58:56 +0000 (11:58 -0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 Jan 2010 19:58:56 +0000 (11:58 -0800)
* cc/reset-more:
  t7111: check that reset options work as described in the tables
  Documentation: reset: add some missing tables
  Fix bit assignment for CE_CONFLICTED
  "reset --merge": fix unmerged case
  reset: use "unpack_trees()" directly instead of "git read-tree"
  reset: add a few tests for "git reset --merge"
  Documentation: reset: add some tables to describe the different options
  reset: improve mixed reset error message when in a bare repo

1  2 
Documentation/git-reset.txt
builtin-reset.c
cache.h
read-cache.c
unpack-trees.c

Simple merge
diff --cc builtin-reset.c
@@@ -285,9 -305,15 +304,13 @@@ int cmd_reset(int argc, const char **ar
        if (reset_type == NONE)
                reset_type = MIXED; /* by default */
  
 -      if ((reset_type == HARD || reset_type == MERGE)
 -          && !is_inside_work_tree())
 -              die("%s reset requires a work tree",
 -                  reset_type_names[reset_type]);
 +      if (reset_type == HARD || reset_type == MERGE)
 +              setup_work_tree();
  
+       if (reset_type == MIXED && is_bare_repository())
+               die("%s reset is not allowed in a bare repository",
+                   reset_type_names[reset_type]);
        /* Soft reset does not touch the index file nor the working tree
         * at all, but requires them in a good order.  Other resets reset
         * the index file to the tree object we are switching to. */
diff --cc cache.h
+++ b/cache.h
@@@ -177,10 -177,8 +177,11 @@@ struct cache_entry 
  
  #define CE_HASHED    (0x100000)
  #define CE_UNHASHED  (0x200000)
+ #define CE_CONFLICTED (0x800000)
  
 +/* Only remove in work directory, not index */
 +#define CE_WT_REMOVE (0x400000)
 +
  /*
   * Extended on-disk flags
   */
diff --cc read-cache.c
Simple merge
diff --cc unpack-trees.c
@@@ -823,15 -686,14 +829,16 @@@ static int merged_entry(struct cache_en
                } else {
                        if (verify_uptodate(old, o))
                                return -1;
 +                      if (ce_skip_worktree(old))
 +                              update |= CE_SKIP_WORKTREE;
                        invalidate_ce_path(old, o);
                }
-       }
-       else {
-               if (verify_absent(merge, "overwritten", o))
-                       return -1;
-               invalidate_ce_path(merge, o);
+       } else {
+               /*
+                * Previously unmerged entry left as an existence
+                * marker by read_index_unmerged();
+                */
+               invalidate_ce_path(old, o);
        }
  
        add_entry(o, merge, update, CE_STAGEMASK);