Merge branch 'js/apply-root'
authorJunio C Hamano <gitster@pobox.com>
Wed, 9 Jul 2008 23:58:21 +0000 (16:58 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 9 Jul 2008 23:58:21 +0000 (16:58 -0700)
* js/apply-root:
git-apply --directory: make --root more similar to GNU diff
apply --root: thinkofix.
Teach "git apply" to prepend a prefix with "--root=<root>"

1  2 
Documentation/git-apply.txt
builtin-apply.c
index e9f724b2fa35cf18978dbcba9dcef7ed28a1b19d,3cd3179ff17f45ec0c0cbd3687fdf8020cd8e5f4..feb51f124ac8a806e65d41f6274c58de64d2991f
@@@ -9,16 -9,16 +9,16 @@@ git-apply - Apply a patch on a git inde
  SYNOPSIS
  --------
  [verse]
 -'git-apply' [--stat] [--numstat] [--summary] [--check] [--index]
 +'git apply' [--stat] [--numstat] [--summary] [--check] [--index]
          [--apply] [--no-add] [--build-fake-ancestor <file>] [-R | --reverse]
          [--allow-binary-replacement | --binary] [--reject] [-z]
 -        [-pNUM] [-CNUM] [--inaccurate-eof] [--cached]
 +        [-pNUM] [-CNUM] [--inaccurate-eof] [--recount] [--cached]
          [--whitespace=<nowarn|warn|fix|error|error-all>]
-         [--exclude=PATH] [--verbose] [<patch>...]
+         [--exclude=PATH] [--directory=<root>] [--verbose] [<patch>...]
  
  DESCRIPTION
  -----------
 -Reads supplied diff output and applies it on a git index file
 +Reads supplied 'diff' output and applies it on a git index file
  and a work tree.
  
  OPTIONS
@@@ -64,7 -64,7 +64,7 @@@
        without using the working tree. This implies '--index'.
  
  --build-fake-ancestor <file>::
 -      Newer git-diff output has embedded 'index information'
 +      Newer 'git-diff' output has embedded 'index information'
        for each blob to help identify the original version that
        the patch applies to.  When this flag is given, and if
        the original versions of the blobs is available locally,
@@@ -78,7 -78,7 +78,7 @@@ the information is read from the curren
        Apply the patch in reverse.
  
  --reject::
 -      For atomicity, linkgit:git-apply[1] by default fails the whole patch and
 +      For atomicity, 'git-apply' by default fails the whole patch and
        does not touch the working tree when some of the hunks
        do not apply.  This option makes it apply
        the parts of the patch that are applicable, and leave the
        ever ignored.
  
  --unidiff-zero::
 -      By default, linkgit:git-apply[1] expects that the patch being
 +      By default, 'git-apply' expects that the patch being
        applied is a unified diff with at least one line of context.
        This provides good safety measures, but breaks down when
        applying a diff generated with --unified=0. To bypass these
@@@ -113,7 -113,7 +113,7 @@@ discouraged
  
  --apply::
        If you use any of the options marked "Turns off
 -      'apply'" above, linkgit:git-apply[1] reads and outputs the
 +      'apply'" above, 'git-apply' reads and outputs the
        information you asked without actually applying the
        patch.  Give this flag after those flags to also apply
        the patch.
  --no-add::
        When applying a patch, ignore additions made by the
        patch.  This can be used to extract the common part between
 -      two files by first running `diff` on them and applying
 +      two files by first running 'diff' on them and applying
        the result with this option, which would apply the
        deletion part but not addition part.
  
        considered whitespace errors.
  +
  By default, the command outputs warning messages but applies the patch.
 -When linkgit:git-apply[1] is used for statistics and not applying a
 +When `git-apply is used for statistics and not applying a
  patch, it defaults to `nowarn`.
  +
  You can use different `<action>` to control this
@@@ -165,9 -165,9 +165,9 @@@ behavior
  * `error-all` is similar to `error` but shows all errors.
  
  --inaccurate-eof::
 -      Under certain circumstances, some versions of diff do not correctly
 +      Under certain circumstances, some versions of 'diff' do not correctly
        detect a missing new-line at the end of the file. As a result, patches
 -      created by such diff programs do not record incomplete lines
 +      created by such 'diff' programs do not record incomplete lines
        correctly. This option adds support for applying such patches by
        working around this bug.
  
        current patch being applied will be printed. This option will cause
        additional information to be reported.
  
 +--recount::
 +      Do not trust the line counts in the hunk headers, but infer them
 +      by inspecting the patch (e.g. after editing the patch without
 +      adjusting the hunk headers appropriately).
 +
+ --directory=<root>::
+       Prepend <root> to all filenames.  If a "-p" argument was passed, too,
+       it is applied before prepending the new root.
+ +
+ For example, a patch that talks about updating `a/git-gui.sh` to `b/git-gui.sh`
+ can be applied to the file in the working tree `modules/git-gui/git-gui.sh` by
+ running `git apply --directory=modules/git-gui`.
  Configuration
  -------------
  
@@@ -191,7 -194,7 +199,7 @@@ apply.whitespace:
  
  Submodules
  ----------
 -If the patch contains any changes to submodules then linkgit:git-apply[1]
 +If the patch contains any changes to submodules then 'git-apply'
  treats these changes as follows.
  
  If --index is specified (explicitly or implicitly), then the submodule
diff --combined builtin-apply.c
index c0f867daed351d4b32f871941e14773a44508eaf,c242bbd83bd0f154ccf5d4a8be7f85386216cebc..b3fc290ff33e6388b25b6cb046ad97856d81169d
@@@ -12,7 -12,6 +12,7 @@@
  #include "blob.h"
  #include "delta.h"
  #include "builtin.h"
 +#include "path-list.h"
  
  /*
   *  --check turns on checking that the working tree matches the
@@@ -58,6 -57,8 +58,8 @@@ static int whitespace_error
  static int squelch_whitespace_errors = 5;
  static int applied_after_fixing_ws;
  static const char *patch_input_file;
+ static const char *root;
+ static int root_len;
  
  static void parse_whitespace_option(const char *option)
  {
@@@ -154,7 -155,6 +156,7 @@@ struct patch 
        unsigned int is_binary:1;
        unsigned int is_copy:1;
        unsigned int is_rename:1;
 +      unsigned int recount:1;
        struct fragment *fragments;
        char *result;
        size_t resultsize;
@@@ -187,13 -187,6 +189,13 @@@ struct image 
        struct line *line;
  };
  
 +/*
 + * Records filenames that have been touched, in order to handle
 + * the case where more than one patches touch the same file.
 + */
 +
 +static struct path_list fn_table;
 +
  static uint32_t hash_line(const char *cp, size_t len)
  {
        size_t i;
@@@ -340,6 -333,8 +342,8 @@@ static char *find_name(const char *line
                                 */
                                strbuf_remove(&name, 0, cp - name.buf);
                                free(def);
+                               if (root)
+                                       strbuf_insert(&name, 0, root, root_len);
                                return strbuf_detach(&name, NULL);
                        }
                }
                free(def);
        }
  
+       if (root) {
+               char *ret = xmalloc(root_len + len + 1);
+               strcpy(ret, root);
+               memcpy(ret + root_len, start, len);
+               ret[root_len + len] = '\0';
+               return ret;
+       }
        return xmemdupz(start, len);
  }
  
@@@ -891,56 -894,6 +903,56 @@@ static int parse_range(const char *line
        return offset + ex;
  }
  
 +static void recount_diff(char *line, int size, struct fragment *fragment)
 +{
 +      int oldlines = 0, newlines = 0, ret = 0;
 +
 +      if (size < 1) {
 +              warning("recount: ignore empty hunk");
 +              return;
 +      }
 +
 +      for (;;) {
 +              int len = linelen(line, size);
 +              size -= len;
 +              line += len;
 +
 +              if (size < 1)
 +                      break;
 +
 +              switch (*line) {
 +              case ' ': case '\n':
 +                      newlines++;
 +                      /* fall through */
 +              case '-':
 +                      oldlines++;
 +                      continue;
 +              case '+':
 +                      newlines++;
 +                      continue;
 +              case '\\':
 +                      continue;
 +              case '@':
 +                      ret = size < 3 || prefixcmp(line, "@@ ");
 +                      break;
 +              case 'd':
 +                      ret = size < 5 || prefixcmp(line, "diff ");
 +                      break;
 +              default:
 +                      ret = -1;
 +                      break;
 +              }
 +              if (ret) {
 +                      warning("recount: unexpected line: %.*s",
 +                              (int)linelen(line, size), line);
 +                      return;
 +              }
 +              break;
 +      }
 +      fragment->oldlines = oldlines;
 +      fragment->newlines = newlines;
 +}
 +
  /*
   * Parse a unified diff fragment header of the
   * form "@@ -a,b +c,d @@"
@@@ -1038,7 -991,8 +1050,7 @@@ static int find_header(char *line, unsi
  static void check_whitespace(const char *line, int len, unsigned ws_rule)
  {
        char *err;
 -      unsigned result = check_and_emit_line(line + 1, len - 1, ws_rule,
 -          NULL, NULL, NULL, NULL);
 +      unsigned result = ws_check(line + 1, len - 1, ws_rule);
        if (!result)
                return;
  
        else {
                err = whitespace_error_string(result);
                fprintf(stderr, "%s:%d: %s.\n%.*s\n",
 -                   patch_input_file, linenr, err, len - 2, line + 1);
 +                      patch_input_file, linenr, err, len - 2, line + 1);
                free(err);
        }
  }
@@@ -1071,8 -1025,6 +1083,8 @@@ static int parse_fragment(char *line, u
        offset = parse_fragment_header(line, len, fragment);
        if (offset < 0)
                return -1;
 +      if (offset > 0 && patch->recount)
 +              recount_diff(line + offset, size - offset, fragment);
        oldlines = fragment->oldlines;
        newlines = fragment->newlines;
        leading = 0;
@@@ -2236,62 -2188,15 +2248,62 @@@ static int read_file_or_gitlink(struct 
        return 0;
  }
  
 +static struct patch *in_fn_table(const char *name)
 +{
 +      struct path_list_item *item;
 +
 +      if (name == NULL)
 +              return NULL;
 +
 +      item = path_list_lookup(name, &fn_table);
 +      if (item != NULL)
 +              return (struct patch *)item->util;
 +
 +      return NULL;
 +}
 +
 +static void add_to_fn_table(struct patch *patch)
 +{
 +      struct path_list_item *item;
 +
 +      /*
 +       * Always add new_name unless patch is a deletion
 +       * This should cover the cases for normal diffs,
 +       * file creations and copies
 +       */
 +      if (patch->new_name != NULL) {
 +              item = path_list_insert(patch->new_name, &fn_table);
 +              item->util = patch;
 +      }
 +
 +      /*
 +       * store a failure on rename/deletion cases because
 +       * later chunks shouldn't patch old names
 +       */
 +      if ((patch->new_name == NULL) || (patch->is_rename)) {
 +              item = path_list_insert(patch->old_name, &fn_table);
 +              item->util = (struct patch *) -1;
 +      }
 +}
 +
  static int apply_data(struct patch *patch, struct stat *st, struct cache_entry *ce)
  {
        struct strbuf buf;
        struct image image;
        size_t len;
        char *img;
 +      struct patch *tpatch;
  
        strbuf_init(&buf, 0);
 -      if (cached) {
 +
 +      if ((tpatch = in_fn_table(patch->old_name)) != NULL) {
 +              if (tpatch == (struct patch *) -1) {
 +                      return error("patch %s has been renamed/deleted",
 +                              patch->old_name);
 +              }
 +              /* We have a patched copy in memory use that */
 +              strbuf_add(&buf, tpatch->result, tpatch->resultsize);
 +      } else if (cached) {
                if (read_file_or_gitlink(ce, &buf))
                        return error("read of %s failed", patch->old_name);
        } else if (patch->old_name) {
                return -1; /* note with --reject this succeeds. */
        patch->result = image.buf;
        patch->resultsize = image.len;
 +      add_to_fn_table(patch);
        free(image.line_allocated);
  
        if (0 < patch->is_delete && patch->resultsize)
@@@ -2363,7 -2267,6 +2375,7 @@@ static int verify_index_match(struct ca
  static int check_preimage(struct patch *patch, struct cache_entry **ce, struct stat *st)
  {
        const char *old_name = patch->old_name;
 +      struct patch *tpatch;
        int stat_ret = 0;
        unsigned st_mode = 0;
  
                return 0;
  
        assert(patch->is_new <= 0);
 -      if (!cached) {
 +      if ((tpatch = in_fn_table(old_name)) != NULL) {
 +              if (tpatch == (struct patch *) -1) {
 +                      return error("%s: has been deleted/renamed", old_name);
 +              }
 +              st_mode = tpatch->new_mode;
 +      } else if (!cached) {
                stat_ret = lstat(old_name, st);
                if (stat_ret && errno != ENOENT)
                        return error("%s: %s", old_name, strerror(errno));
        }
 -      if (check_index) {
 +      if (check_index && !tpatch) {
                int pos = cache_name_pos(old_name, strlen(old_name));
                if (pos < 0) {
                        if (patch->is_new < 0)
        return 0;
  }
  
 -static int check_patch(struct patch *patch, struct patch *prev_patch)
 +static int check_patch(struct patch *patch)
  {
        struct stat st;
        const char *old_name = patch->old_name;
                return status;
        old_name = patch->old_name;
  
 -      if (new_name && prev_patch && 0 < prev_patch->is_delete &&
 -          !strcmp(prev_patch->old_name, new_name))
 +      if (in_fn_table(new_name) == (struct patch *) -1)
                /*
                 * A type-change diff is always split into a patch to
                 * delete old, immediately followed by a patch to
  
  static int check_patch_list(struct patch *patch)
  {
 -      struct patch *prev_patch = NULL;
        int err = 0;
  
 -      for (prev_patch = NULL; patch ; patch = patch->next) {
 +      while (patch) {
                if (apply_verbosely)
                        say_patch_name(stderr,
                                       "Checking patch ", patch, "...\n");
 -              err |= check_patch(patch, prev_patch);
 -              prev_patch = patch;
 +              err |= check_patch(patch);
 +              patch = patch->next;
        }
        return err;
  }
@@@ -3024,18 -2924,13 +3036,18 @@@ static void prefix_patches(struct patc
        }
  }
  
 -static int apply_patch(int fd, const char *filename, int inaccurate_eof)
 +#define INACCURATE_EOF        (1<<0)
 +#define RECOUNT               (1<<1)
 +
 +static int apply_patch(int fd, const char *filename, int options)
  {
        size_t offset;
        struct strbuf buf;
        struct patch *list = NULL, **listp = &list;
        int skipped_patch = 0;
  
 +      /* FIXME - memory leak when using multiple patch files as inputs */
 +      memset(&fn_table, 0, sizeof(struct path_list));
        strbuf_init(&buf, 0);
        patch_input_file = filename;
        read_patch_file(&buf, fd);
                int nr;
  
                patch = xcalloc(1, sizeof(*patch));
 -              patch->inaccurate_eof = inaccurate_eof;
 +              patch->inaccurate_eof = !!(options & INACCURATE_EOF);
 +              patch->recount =  !!(options & RECOUNT);
                nr = parse_chunk(buf.buf + offset, buf.len - offset, patch);
                if (nr < 0)
                        break;
@@@ -3115,7 -3009,7 +3127,7 @@@ int cmd_apply(int argc, const char **ar
  {
        int i;
        int read_stdin = 1;
 -      int inaccurate_eof = 0;
 +      int options = 0;
        int errs = 0;
        int is_not_gitdir;
  
                int fd;
  
                if (!strcmp(arg, "-")) {
 -                      errs |= apply_patch(0, "<stdin>", inaccurate_eof);
 +                      errs |= apply_patch(0, "<stdin>", options);
                        read_stdin = 0;
                        continue;
                }
                        continue;
                }
                if (!strcmp(arg, "--inaccurate-eof")) {
 -                      inaccurate_eof = 1;
 +                      options |= INACCURATE_EOF;
 +                      continue;
 +              }
 +              if (!strcmp(arg, "--recount")) {
 +                      options |= RECOUNT;
                        continue;
                }
+               if (!prefixcmp(arg, "--directory=")) {
+                       arg += strlen("--directory=");
+                       root_len = strlen(arg);
+                       if (root_len && arg[root_len - 1] != '/') {
+                               char *new_root;
+                               root = new_root = xmalloc(root_len + 2);
+                               strcpy(new_root, arg);
+                               strcpy(new_root + root_len++, "/");
+                       } else
+                               root = arg;
+                       continue;
+               }
                if (0 < prefix_length)
                        arg = prefix_filename(prefix, prefix_length, arg);
  
                        die("can't open patch '%s': %s", arg, strerror(errno));
                read_stdin = 0;
                set_default_whitespace_mode(whitespace_option);
 -              errs |= apply_patch(fd, arg, inaccurate_eof);
 +              errs |= apply_patch(fd, arg, options);
                close(fd);
        }
        set_default_whitespace_mode(whitespace_option);
        if (read_stdin)
 -              errs |= apply_patch(0, "<stdin>", inaccurate_eof);
 +              errs |= apply_patch(0, "<stdin>", options);
        if (whitespace_error) {
                if (squelch_whitespace_errors &&
                    squelch_whitespace_errors < whitespace_error) {