Merge branch 'jk/rerere-forget-check-enabled'

"git rerere forget" in a repository without rerere enabled gave a
cryptic error message; it should be a silent no-op instead.

* jk/rerere-forget-check-enabled:
rerere: exit silently on "forget" when rerere is disabled

diff --combined rerere.c
index 31644dec04fe4a77d43624720ff516de2d746dbc,fd65a93873625d96c9824183cd81be1b408a8952..94aea9a36f16d9d72c1769aa9e258c5370c7dc84
--- 1/rerere.c
--- 2/rerere.c
+++ b/rerere.c
@@@ -1,5 -1,4 +1,5 @@@
  #include "cache.h"
 +#include "lockfile.h"
  #include "string-list.h"
  #include "rerere.h"
  #include "xdiff-interface.h"
@@@ -208,11 -207,11 +208,11 @@@ static int handle_path(unsigned char *s
                        strbuf_reset(&one);
                        strbuf_reset(&two);
                } else if (hunk == RR_SIDE_1)
 -                      strbuf_addstr(&one, buf.buf);
 +                      strbuf_addbuf(&one, &buf);
                else if (hunk == RR_ORIGINAL)
                        ; /* discard */
                else if (hunk == RR_SIDE_2)
 -                      strbuf_addstr(&two, buf.buf);
 +                      strbuf_addbuf(&two, &buf);
                else
                        rerere_io_putstr(buf.buf, io);
                continue;
@@@ -477,23 -476,28 +477,23 @@@ out
  
  static struct lock_file index_lock;
  
 -static int update_paths(struct string_list *update)
 +static void update_paths(struct string_list *update)
  {
        int i;
 -      int fd = hold_locked_index(&index_lock, 0);
 -      int status = 0;
  
 -      if (fd < 0)
 -              return -1;
 +      hold_locked_index(&index_lock, 1);
  
        for (i = 0; i < update->nr; i++) {
                struct string_list_item *item = &update->items[i];
 -              if (add_file_to_cache(item->string, ADD_CACHE_IGNORE_ERRORS))
 -                      status = -1;
 +              if (add_file_to_cache(item->string, 0))
 +                      exit(128);
        }
  
 -      if (!status && active_cache_changed) {
 -              if (write_cache(fd, active_cache, active_nr) ||
 -                  commit_locked_index(&index_lock))
 +      if (active_cache_changed) {
 +              if (write_locked_index(&the_index, &index_lock, COMMIT_LOCK))
                        die("Unable to write new index file");
 -      } else if (fd >= 0)
 +      } else
                rollback_lock_file(&index_lock);
 -      return status;
  }
  
  static int do_plain_rerere(struct string_list *rr, int fd)
@@@ -570,11 -574,15 +570,11 @@@
        return write_rr(rr, fd);
  }
  
 -static int git_rerere_config(const char *var, const char *value, void *cb)
 +static void git_rerere_config(void)
  {
 -      if (!strcmp(var, "rerere.enabled"))
 -              rerere_enabled = git_config_bool(var, value);
 -      else if (!strcmp(var, "rerere.autoupdate"))
 -              rerere_autoupdate = git_config_bool(var, value);
 -      else
 -              return git_default_config(var, value, cb);
 -      return 0;
 +      git_config_get_bool("rerere.enabled", &rerere_enabled);
 +      git_config_get_bool("rerere.autoupdate", &rerere_autoupdate);
 +      git_config(git_default_config, NULL);
  }
  
  static int is_rerere_enabled(void)
@@@ -599,7 -607,7 +599,7 @@@ int setup_rerere(struct string_list *me
  {
        int fd;
  
 -      git_config(git_rerere_config, NULL);
 +      git_rerere_config();
        if (!is_rerere_enabled())
                return -1;
  
@@@ -659,6 -667,8 +659,8 @@@ int rerere_forget(struct pathspec *path
                return error("Could not read index");
  
        fd = setup_rerere(&merge_rr, RERERE_NOAUTOUPDATE);
+       if (fd < 0)
+               return 0;
  
        unmerge_cache(pathspec);
        find_conflict(&conflict);
@@@ -692,6 -702,24 +694,6 @@@ static void unlink_rr_item(const char *
        rmdir(git_path("rr-cache/%s", name));
  }
  
 -struct rerere_gc_config_cb {
 -      int cutoff_noresolve;
 -      int cutoff_resolve;
 -};
 -
 -static int git_rerere_gc_config(const char *var, const char *value, void *cb)
 -{
 -      struct rerere_gc_config_cb *cf = cb;
 -
 -      if (!strcmp(var, "gc.rerereresolved"))
 -              cf->cutoff_resolve = git_config_int(var, value);
 -      else if (!strcmp(var, "gc.rerereunresolved"))
 -              cf->cutoff_noresolve = git_config_int(var, value);
 -      else
 -              return git_default_config(var, value, cb);
 -      return 0;
 -}
 -
  void rerere_gc(struct string_list *rr)
  {
        struct string_list to_remove = STRING_LIST_INIT_DUP;
@@@ -699,12 -727,9 +701,12 @@@
        struct dirent *e;
        int i, cutoff;
        time_t now = time(NULL), then;
 -      struct rerere_gc_config_cb cf = { 15, 60 };
 +      int cutoff_noresolve = 15;
 +      int cutoff_resolve = 60;
  
 -      git_config(git_rerere_gc_config, &cf);
 +      git_config_get_int("gc.rerereresolved", &cutoff_resolve);
 +      git_config_get_int("gc.rerereunresolved", &cutoff_noresolve);
 +      git_config(git_default_config, NULL);
        dir = opendir(git_path("rr-cache"));
        if (!dir)
                die_errno("unable to open rr-cache directory");
@@@ -714,12 -739,12 +716,12 @@@
  
                then = rerere_last_used_at(e->d_name);
                if (then) {
 -                      cutoff = cf.cutoff_resolve;
 +                      cutoff = cutoff_resolve;
                } else {
                        then = rerere_created_at(e->d_name);
                        if (!then)
                                continue;
 -                      cutoff = cf.cutoff_noresolve;
 +                      cutoff = cutoff_noresolve;
                }
                if (then < now - cutoff * 86400)
                        string_list_append(&to_remove, e->d_name);