Code

Merge branch 'jc/maint-format-patch'
authorJunio C Hamano <gitster@pobox.com>
Sun, 18 Jan 2009 07:05:50 +0000 (23:05 -0800)
committerJunio C Hamano <gitster@pobox.com>
Sun, 18 Jan 2009 07:05:50 +0000 (23:05 -0800)
* jc/maint-format-patch:
  format-patch: show patch text for the root commit

1  2 
builtin-log.c

diff --combined builtin-log.c
index c1e2c6a4c53a7b1b40f3868a0288a6be4c7cb247,5e302906fb026ca2ed9bd110780aefb058c0c36e..c7aa48e74821b9b8aea2f99041cf323c9057c8da
@@@ -14,9 -14,9 +14,9 @@@
  #include "tag.h"
  #include "reflog-walk.h"
  #include "patch-ids.h"
 -#include "refs.h"
  #include "run-command.h"
  #include "shortlog.h"
 +#include "remote.h"
  
  /* Set a default date-time format for git log ("log.date" config variable) */
  static const char *default_date_mode = NULL;
@@@ -25,10 -25,36 +25,10 @@@ static int default_show_root = 1
  static const char *fmt_patch_subject_prefix = "PATCH";
  static const char *fmt_pretty;
  
 -static void add_name_decoration(const char *prefix, const char *name, struct object *obj)
 -{
 -      int plen = strlen(prefix);
 -      int nlen = strlen(name);
 -      struct name_decoration *res = xmalloc(sizeof(struct name_decoration) + plen + nlen);
 -      memcpy(res->name, prefix, plen);
 -      memcpy(res->name + plen, name, nlen + 1);
 -      res->next = add_decoration(&name_decoration, obj, res);
 -}
 -
 -static int add_ref_decoration(const char *refname, const unsigned char *sha1, int flags, void *cb_data)
 -{
 -      struct object *obj = parse_object(sha1);
 -      if (!obj)
 -              return 0;
 -      add_name_decoration("", refname, obj);
 -      while (obj->type == OBJ_TAG) {
 -              obj = ((struct tag *)obj)->tagged;
 -              if (!obj)
 -                      break;
 -              add_name_decoration("tag: ", refname, obj);
 -      }
 -      return 0;
 -}
 -
  static void cmd_log_init(int argc, const char **argv, const char *prefix,
                      struct rev_info *rev)
  {
        int i;
 -      int decorate = 0;
  
        rev->abbrev = DEFAULT_ABBREV;
        rev->commit_format = CMIT_FMT_DEFAULT;
@@@ -38,7 -64,6 +38,7 @@@
        DIFF_OPT_SET(&rev->diffopt, RECURSIVE);
        rev->show_root_diff = default_show_root;
        rev->subject_prefix = fmt_patch_subject_prefix;
 +      DIFF_OPT_SET(&rev->diffopt, ALLOW_TEXTCONV);
  
        if (default_date_mode)
                rev->date_mode = parse_date_format(default_date_mode);
        for (i = 1; i < argc; i++) {
                const char *arg = argv[i];
                if (!strcmp(arg, "--decorate")) {
 -                      if (!decorate)
 -                              for_each_ref(add_ref_decoration, NULL);
 -                      decorate = 1;
 +                      load_ref_decorations();
 +                      rev->show_decorations = 1;
 +              } else if (!strcmp(arg, "--source")) {
 +                      rev->show_source = 1;
                } else
                        die("unrecognized argument: %s", arg);
        }
@@@ -193,11 -217,6 +193,11 @@@ static int cmd_log_walk(struct rev_inf
        if (rev->early_output)
                finish_early_output(rev);
  
 +      /*
 +       * For --check and --exit-code, the exit code is based on CHECK_FAILED
 +       * and HAS_CHANGES being accumulated in rev->diffopt, so be careful to
 +       * retain that state information if replacing rev->diffopt in this loop
 +       */
        while ((commit = get_revision(rev)) != NULL) {
                log_tree_commit(rev, commit);
                if (!rev->reflog_info) {
                free_commit_list(commit->parents);
                commit->parents = NULL;
        }
 -      return 0;
 +      if (rev->diffopt.output_format & DIFF_FORMAT_CHECKDIFF &&
 +          DIFF_OPT_TST(&rev->diffopt, CHECK_FAILED)) {
 +              return 02;
 +      }
 +      return diff_result_code(&rev->diffopt, 0);
  }
  
  static int git_log_config(const char *var, const char *value, void *cb)
@@@ -250,13 -265,22 +250,13 @@@ int cmd_whatchanged(int argc, const cha
  
  static void show_tagger(char *buf, int len, struct rev_info *rev)
  {
 -      char *email_end, *p;
 -      unsigned long date;
 -      int tz;
 +      struct strbuf out = STRBUF_INIT;
  
 -      email_end = memchr(buf, '>', len);
 -      if (!email_end)
 -              return;
 -      p = ++email_end;
 -      while (isspace(*p))
 -              p++;
 -      date = strtoul(p, &p, 10);
 -      while (isspace(*p))
 -              p++;
 -      tz = (int)strtol(p, NULL, 10);
 -      printf("Tagger: %.*s\nDate:   %s\n", (int)(email_end - buf), buf,
 -             show_date(date, tz, rev->date_mode));
 +      pp_user_info("Tagger", rev->commit_format, &out, buf, rev->date_mode,
 +              git_log_output_encoding ?
 +              git_log_output_encoding: git_commit_encoding);
 +      printf("%s\n", out.buf);
 +      strbuf_release(&out);
  }
  
  static int show_object(const unsigned char *sha1, int show_tag_object,
@@@ -426,7 -450,7 +426,7 @@@ static int istitlechar(char c
  
  static const char *fmt_patch_suffix = ".patch";
  static int numbered = 0;
 -static int auto_number = 0;
 +static int auto_number = 1;
  
  static char **extra_hdr;
  static int extra_hdr_nr;
@@@ -485,7 -509,6 +485,7 @@@ static int git_format_config(const cha
                        return 0;
                }
                numbered = git_config_bool(var, value);
 +              auto_number = auto_number && numbered;
                return 0;
        }
  
@@@ -629,9 -652,10 +629,9 @@@ static void gen_message_id(struct rev_i
        const char *committer = git_committer_info(IDENT_WARN_ON_NO_NAME);
        const char *email_start = strrchr(committer, '<');
        const char *email_end = strrchr(committer, '>');
 -      struct strbuf buf;
 +      struct strbuf buf = STRBUF_INIT;
        if (!email_start || !email_end || email_start > email_end - 1)
                die("Could not extract email from committer identity.");
 -      strbuf_init(&buf, 0);
        strbuf_addf(&buf, "%s.%lu.git.%.*s", base,
                    (unsigned long) time(NULL),
                    (int)(email_end - email_start - 1), email_start + 1);
@@@ -650,7 -674,7 +650,7 @@@ static void make_cover_letter(struct re
        const char *msg;
        const char *extra_headers = rev->extra_headers;
        struct shortlog log;
 -      struct strbuf sb;
 +      struct strbuf sb = STRBUF_INIT;
        int i;
        const char *encoding = "utf-8";
        struct diff_options opts;
        committer = git_committer_info(0);
  
        msg = body;
 -      strbuf_init(&sb, 0);
        pp_user_info(NULL, CMIT_FMT_EMAIL, &sb, committer, DATE_RFC2822,
                     encoding);
        pp_title_line(CMIT_FMT_EMAIL, &msg, &sb, subject_start, extra_headers,
@@@ -752,7 -777,7 +752,7 @@@ int cmd_format_patch(int argc, const ch
        const char *in_reply_to = NULL;
        struct patch_ids ids;
        char *add_signoff = NULL;
 -      struct strbuf buf;
 +      struct strbuf buf = STRBUF_INIT;
  
        git_config(git_format_config, NULL);
        init_revisions(&rev, prefix);
                        committer = git_committer_info(IDENT_ERROR_ON_NO_NAME);
                        endpos = strchr(committer, '>');
                        if (!endpos)
 -                              die("bogus committer info %s\n", committer);
 +                              die("bogus committer info %s", committer);
                        add_signoff = xmemdupz(committer, endpos - committer + 1);
                }
                else if (!strcmp(argv[i], "--attach")) {
        }
        argc = j;
  
 -      strbuf_init(&buf, 0);
 -
        for (i = 0; i < extra_hdr_nr; i++) {
                strbuf_addstr(&buf, extra_hdr[i]);
                strbuf_addch(&buf, '\n');
        if (argc > 1)
                die ("unrecognized argument: %s", argv[1]);
  
 -      if (!rev.diffopt.output_format)
 +      if (!rev.diffopt.output_format
 +              || rev.diffopt.output_format == DIFF_FORMAT_PATCH)
                rev.diffopt.output_format = DIFF_FORMAT_DIFFSTAT | DIFF_FORMAT_SUMMARY | DIFF_FORMAT_PATCH;
  
        if (!DIFF_OPT_TST(&rev.diffopt, TEXT) && !no_binary_diff)
                 * get_revision() to do the usual traversal.
                 */
        }
+       /*
+        * We cannot move this anywhere earlier because we do want to
+        * know if --root was given explicitly from the comand line.
+        */
+       rev.show_root_diff = 1;
        if (cover_letter) {
                /* remember the range */
                int i;
@@@ -1062,14 -1095,13 +1069,14 @@@ static int add_pending_commit(const cha
  }
  
  static const char cherry_usage[] =
 -"git cherry [-v] <upstream> [<head>] [<limit>]";
 +"git cherry [-v] [<upstream> [<head> [<limit>]]]";
  int cmd_cherry(int argc, const char **argv, const char *prefix)
  {
        struct rev_info revs;
        struct patch_ids ids;
        struct commit *commit;
        struct commit_list *list = NULL;
 +      struct branch *current_branch;
        const char *upstream;
        const char *head = "HEAD";
        const char *limit = NULL;
                upstream = argv[1];
                break;
        default:
 -              usage(cherry_usage);
 +              current_branch = branch_get(NULL);
 +              if (!current_branch || !current_branch->merge
 +                                      || !current_branch->merge[0]
 +                                      || !current_branch->merge[0]->dst) {
 +                      fprintf(stderr, "Could not find a tracked"
 +                                      " remote branch, please"
 +                                      " specify <upstream> manually.\n");
 +                      usage(cherry_usage);
 +              }
 +
 +              upstream = current_branch->merge[0]->dst;
        }
  
        init_revisions(&revs, prefix);
                        sign = '-';
  
                if (verbose) {
 -                      struct strbuf buf;
 -                      strbuf_init(&buf, 0);
 +                      struct strbuf buf = STRBUF_INIT;
                        pretty_print_commit(CMIT_FMT_ONELINE, commit,
                                            &buf, 0, NULL, NULL, 0, 0);
                        printf("%c %s %s\n", sign,