]> git.ipfire.org Git - thirdparty/git.git/commitdiff
Merge branch 'ag/rebase-i-in-c'
authorJunio C Hamano <gitster@pobox.com>
Fri, 2 Nov 2018 02:04:53 +0000 (11:04 +0900)
committerJunio C Hamano <gitster@pobox.com>
Fri, 2 Nov 2018 02:04:53 +0000 (11:04 +0900)
Rewrite of the remaining "rebase -i" machinery in C.

* ag/rebase-i-in-c:
  rebase -i: move rebase--helper modes to rebase--interactive
  rebase -i: remove git-rebase--interactive.sh
  rebase--interactive2: rewrite the submodes of interactive rebase in C
  rebase -i: implement the main part of interactive rebase as a builtin
  rebase -i: rewrite init_basic_state() in C
  rebase -i: rewrite write_basic_state() in C
  rebase -i: rewrite the rest of init_revisions_and_shortrevisions() in C
  rebase -i: implement the logic to initialize $revisions in C
  rebase -i: remove unused modes and functions
  rebase -i: rewrite complete_action() in C
  t3404: todo list with commented-out commands only aborts
  sequencer: change the way skip_unnecessary_picks() returns its result
  sequencer: refactor append_todo_help() to write its message to a buffer
  rebase -i: rewrite checkout_onto() in C
  rebase -i: rewrite setup_reflog_action() in C
  sequencer: add a new function to silence a command, except if it fails
  rebase -i: rewrite the edit-todo functionality in C
  editor: add a function to launch the sequence editor
  rebase -i: rewrite append_todo_help() in C
  sequencer: make three functions and an enum from sequencer.c public

1  2 
.gitignore
Makefile
builtin.h
cache.h
git-legacy-rebase.sh
git-rebase--preserve-merges.sh
git.c
sequencer.c
sequencer.h
strbuf.h
t/t3404-rebase-interactive.sh

diff --cc .gitignore
index 4d5de166e85fb8cf249bff8a15d4d1c7209697bc,406f26d0507961c9e39da636bb87ff62fae6a90b..0d77ea5894274c43c4b348c8b52b8e665a1a339e
  /git-read-tree
  /git-rebase
  /git-rebase--am
- /git-rebase--helper
 +/git-rebase--common
  /git-rebase--interactive
  /git-rebase--merge
  /git-rebase--preserve-merges
diff --cc Makefile
index 95b93c709d65f1a3b8a96e5911c33c1322220e8c,ca3a0888ddfdb48e85ed5d785d34d9194ff3c8a1..bbfbb4292d49e99704d75a093a97d432c537cd8d
+++ b/Makefile
@@@ -626,8 -619,6 +626,7 @@@ SCRIPT_SH += git-web--browse.s
  SCRIPT_LIB += git-mergetool--lib
  SCRIPT_LIB += git-parse-remote
  SCRIPT_LIB += git-rebase--am
- SCRIPT_LIB += git-rebase--interactive
 +SCRIPT_LIB += git-rebase--common
  SCRIPT_LIB += git-rebase--preserve-merges
  SCRIPT_LIB += git-rebase--merge
  SCRIPT_LIB += git-sh-setup
@@@ -952,9 -919,9 +951,10 @@@ LIB_OBJS += progress.
  LIB_OBJS += prompt.o
  LIB_OBJS += protocol.o
  LIB_OBJS += quote.o
 +LIB_OBJS += range-diff.o
  LIB_OBJS += reachable.o
  LIB_OBJS += read-cache.o
+ LIB_OBJS += rebase-interactive.o
  LIB_OBJS += reflog-walk.o
  LIB_OBJS += refs.o
  LIB_OBJS += refs/files-backend.o
@@@ -1092,10 -1058,8 +1092,10 @@@ BUILTIN_OBJS += builtin/prune-packed.
  BUILTIN_OBJS += builtin/prune.o
  BUILTIN_OBJS += builtin/pull.o
  BUILTIN_OBJS += builtin/push.o
 +BUILTIN_OBJS += builtin/range-diff.o
  BUILTIN_OBJS += builtin/read-tree.o
- BUILTIN_OBJS += builtin/rebase--helper.o
 +BUILTIN_OBJS += builtin/rebase.o
+ BUILTIN_OBJS += builtin/rebase--interactive.o
  BUILTIN_OBJS += builtin/receive-pack.o
  BUILTIN_OBJS += builtin/reflog.o
  BUILTIN_OBJS += builtin/remote.o
diff --cc builtin.h
index 6fb66f5ba40bca73ba62c6d340ab8f4f58ce1e32,7feb689d87c765fdb59f1b8c0e368bcd5819712f..6538932e99a72f1be97c41806ca6e394f18f9a0c
+++ b/builtin.h
@@@ -202,10 -201,8 +202,10 @@@ extern int cmd_prune(int argc, const ch
  extern int cmd_prune_packed(int argc, const char **argv, const char *prefix);
  extern int cmd_pull(int argc, const char **argv, const char *prefix);
  extern int cmd_push(int argc, const char **argv, const char *prefix);
 +extern int cmd_range_diff(int argc, const char **argv, const char *prefix);
  extern int cmd_read_tree(int argc, const char **argv, const char *prefix);
- extern int cmd_rebase__helper(int argc, const char **argv, const char *prefix);
 +extern int cmd_rebase(int argc, const char **argv, const char *prefix);
+ extern int cmd_rebase__interactive(int argc, const char **argv, const char *prefix);
  extern int cmd_receive_pack(int argc, const char **argv, const char *prefix);
  extern int cmd_reflog(int argc, const char **argv, const char *prefix);
  extern int cmd_remote(int argc, const char **argv, const char *prefix);
diff --cc cache.h
Simple merge
Simple merge
Simple merge
diff --cc git.c
index 8e522768315581a7a401e8cc6a09900e9ab9ad68,81aabd1423a99dad6ad17ffaedb6447de04fd51c..2f604a41eaf620f4d185c612cd7c6aee975faa67
--- 1/git.c
--- 2/git.c
+++ b/git.c
@@@ -524,15 -517,8 +524,15 @@@ static struct cmd_struct commands[] = 
        { "prune-packed", cmd_prune_packed, RUN_SETUP },
        { "pull", cmd_pull, RUN_SETUP | NEED_WORK_TREE },
        { "push", cmd_push, RUN_SETUP },
 +      { "range-diff", cmd_range_diff, RUN_SETUP | USE_PAGER },
        { "read-tree", cmd_read_tree, RUN_SETUP | SUPPORT_SUPER_PREFIX},
-       { "rebase--helper", cmd_rebase__helper, RUN_SETUP | NEED_WORK_TREE },
 +      /*
 +       * NEEDSWORK: Until the rebase is independent and needs no redirection
 +       * to rebase shell script this is kept as is, then should be changed to
 +       * RUN_SETUP | NEED_WORK_TREE
 +       */
 +      { "rebase", cmd_rebase },
+       { "rebase--interactive", cmd_rebase__interactive, RUN_SETUP | NEED_WORK_TREE },
        { "receive-pack", cmd_receive_pack },
        { "reflog", cmd_reflog, RUN_SETUP },
        { "remote", cmd_remote, RUN_SETUP },
diff --cc sequencer.c
index 0c164d5f98f152156ef21a79159f6348fecb0273,8dd6db5a017030da8468a48fe868d2982394699d..3c86c7694be5c364eaa7b4ea9724c66854b7477a
@@@ -30,7 -30,7 +30,8 @@@
  #include "oidset.h"
  #include "commit-slab.h"
  #include "alias.h"
 +#include "commit-reach.h"
+ #include "rebase-interactive.h"
  
  #define GIT_REFLOG_ACTION "GIT_REFLOG_ACTION"
  
@@@ -903,24 -876,13 +918,13 @@@ static int run_git_commit(const char *d
        if ((flags & ALLOW_EMPTY))
                argv_array_push(&cmd.args, "--allow-empty");
  
 -      if (opts->allow_empty_message)
 +      if (!(flags & EDIT_MSG))
                argv_array_push(&cmd.args, "--allow-empty-message");
  
-       if (cmd.err == -1) {
-               /* hide stderr on success */
-               struct strbuf buf = STRBUF_INIT;
-               int rc = pipe_command(&cmd,
-                                     NULL, 0,
-                                     /* stdout is already redirected */
-                                     NULL, 0,
-                                     &buf, 0);
-               if (rc)
-                       fputs(buf.buf, stderr);
-               strbuf_release(&buf);
-               return rc;
-       }
-       return run_command(&cmd);
+       if (is_rebase_i(opts) && !(flags & EDIT_MSG))
+               return run_command_silent_on_success(&cmd);
+       else
+               return run_command(&cmd);
  }
  
  static int rest_is_empty(const struct strbuf *sb, int start)
@@@ -4587,9 -4515,9 +4690,9 @@@ static int skip_unnecessary_picks(struc
                if (item->commit->parents->next)
                        break; /* merge commit */
                parent_oid = &item->commit->parents->item->object.oid;
-               if (!oideq(parent_oid, oid))
 -              if (hashcmp(parent_oid->hash, output_oid->hash))
++              if (!oideq(parent_oid, output_oid))
                        break;
-               oid = &item->commit->object.oid;
+               oidcpy(output_oid, &item->commit->object.oid);
        }
        if (i > 0) {
                int offset = get_item_line_offset(&todo_list, i);
diff --cc sequencer.h
index c986bc825161f1f4702a0cd435c6d9705e3be2df,aab280f276a55bdc0af81027dc497aa34354ee52..660cff5050b39e38e721182861ada83e95e8378b
@@@ -1,13 -1,9 +1,14 @@@
  #ifndef SEQUENCER_H
  #define SEQUENCER_H
  
 +#include "cache.h"
 +#include "strbuf.h"
 +
 +struct commit;
 +
  const char *git_path_commit_editmsg(void);
  const char *git_path_seq_dir(void);
+ const char *rebase_path_todo(void);
  
  #define APPEND_SIGNOFF_DEDUP (1u << 0)
  
diff --cc strbuf.h
index bf18fddb5bb7883c089b30cfabc54cc5a9d055b9,66da9822fd860cab6321505c368afa78496a4520..9981f782b2573c1a4360879e03884ec753999aec
+++ b/strbuf.h
@@@ -582,14 -574,11 +582,16 @@@ void strbuf_add_unique_abbrev(struct st
   * run in. If the buffer is NULL the editor is launched as usual but the
   * file's contents are not read into the buffer upon completion.
   */
- int launch_editor(const char *path,
-                 struct strbuf *buffer,
 -extern int launch_editor(const char *path, struct strbuf *buffer, const char *const *env);
 -extern int launch_sequence_editor(const char *path, struct strbuf *buffer,
 -                                const char *const *env);
++int launch_editor(const char *path, struct strbuf *buffer,
 +                const char *const *env);
 +
++int launch_sequence_editor(const char *path, struct strbuf *buffer,
++                         const char *const *env);
 -extern void strbuf_add_lines(struct strbuf *sb, const char *prefix, const char *buf, size_t size);
 +void strbuf_add_lines(struct strbuf *sb,
 +                    const char *prefix,
 +                    const char *buf,
 +                    size_t size);
  
  /**
   * Append s to sb, with the characters '<', '>', '&' and '"' converted
Simple merge