2007-03-01 05:26:30 +01:00
|
|
|
#include "cache.h"
|
|
|
|
#include "builtin.h"
|
2007-10-07 23:02:29 +02:00
|
|
|
#include "parse-options.h"
|
2008-03-03 07:30:56 +01:00
|
|
|
#include "diff.h"
|
|
|
|
#include "revision.h"
|
2008-08-10 13:48:55 +02:00
|
|
|
#include "rerere.h"
|
2011-08-04 12:39:08 +02:00
|
|
|
#include "dir.h"
|
2011-08-04 12:39:11 +02:00
|
|
|
#include "sequencer.h"
|
2007-03-01 05:26:30 +01:00
|
|
|
|
|
|
|
/*
|
|
|
|
* This implements the builtins revert and cherry-pick.
|
|
|
|
*
|
|
|
|
* Copyright (c) 2007 Johannes E. Schindelin
|
|
|
|
*
|
|
|
|
* Based on git-revert.sh, which is
|
|
|
|
*
|
|
|
|
* Copyright (c) 2005 Linus Torvalds
|
|
|
|
* Copyright (c) 2005 Junio C Hamano
|
|
|
|
*/
|
|
|
|
|
2007-10-07 23:02:29 +02:00
|
|
|
static const char * const revert_usage[] = {
|
2013-04-22 17:57:37 +02:00
|
|
|
N_("git revert [options] <commit-ish>..."),
|
2012-08-20 14:32:41 +02:00
|
|
|
N_("git revert <subcommand>"),
|
2007-10-07 23:02:29 +02:00
|
|
|
NULL
|
|
|
|
};
|
2007-03-01 05:26:30 +01:00
|
|
|
|
2007-10-07 23:02:29 +02:00
|
|
|
static const char * const cherry_pick_usage[] = {
|
2013-04-22 17:57:37 +02:00
|
|
|
N_("git cherry-pick [options] <commit-ish>..."),
|
2012-08-20 14:32:41 +02:00
|
|
|
N_("git cherry-pick <subcommand>"),
|
2007-10-07 23:02:29 +02:00
|
|
|
NULL
|
|
|
|
};
|
2007-03-01 05:26:30 +01:00
|
|
|
|
2011-08-04 12:39:05 +02:00
|
|
|
static const char *action_name(const struct replay_opts *opts)
|
|
|
|
{
|
2012-01-11 19:15:56 +01:00
|
|
|
return opts->action == REPLAY_REVERT ? "revert" : "cherry-pick";
|
2011-08-04 12:39:05 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static const char * const *revert_or_cherry_pick_usage(struct replay_opts *opts)
|
2010-06-14 07:32:09 +02:00
|
|
|
{
|
2012-01-11 19:15:56 +01:00
|
|
|
return opts->action == REPLAY_REVERT ? revert_usage : cherry_pick_usage;
|
2010-06-14 07:32:09 +02:00
|
|
|
}
|
|
|
|
|
2010-12-11 01:51:44 +01:00
|
|
|
static int option_parse_x(const struct option *opt,
|
|
|
|
const char *arg, int unset)
|
|
|
|
{
|
2011-08-04 12:39:05 +02:00
|
|
|
struct replay_opts **opts_ptr = opt->value;
|
|
|
|
struct replay_opts *opts = *opts_ptr;
|
|
|
|
|
2010-12-11 01:51:44 +01:00
|
|
|
if (unset)
|
|
|
|
return 0;
|
|
|
|
|
2011-08-04 12:39:05 +02:00
|
|
|
ALLOC_GROW(opts->xopts, opts->xopts_nr + 1, opts->xopts_alloc);
|
|
|
|
opts->xopts[opts->xopts_nr++] = xstrdup(arg);
|
2010-12-11 01:51:44 +01:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-07-18 22:02:12 +02:00
|
|
|
LAST_ARG_MUST_BE_NULL
|
2011-08-04 12:39:07 +02:00
|
|
|
static void verify_opt_compatible(const char *me, const char *base_opt, ...)
|
|
|
|
{
|
|
|
|
const char *this_opt;
|
|
|
|
va_list ap;
|
|
|
|
|
|
|
|
va_start(ap, base_opt);
|
|
|
|
while ((this_opt = va_arg(ap, const char *))) {
|
|
|
|
if (va_arg(ap, int))
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
va_end(ap);
|
|
|
|
|
|
|
|
if (this_opt)
|
|
|
|
die(_("%s: %s cannot be used with %s"), me, this_opt, base_opt);
|
|
|
|
}
|
|
|
|
|
2011-08-04 12:39:05 +02:00
|
|
|
static void parse_args(int argc, const char **argv, struct replay_opts *opts)
|
2007-03-01 05:26:30 +01:00
|
|
|
{
|
2011-08-04 12:39:05 +02:00
|
|
|
const char * const * usage_str = revert_or_cherry_pick_usage(opts);
|
2011-08-04 12:39:07 +02:00
|
|
|
const char *me = action_name(opts);
|
2013-08-03 13:51:26 +02:00
|
|
|
int cmd = 0;
|
2007-10-07 23:02:29 +02:00
|
|
|
struct option options[] = {
|
2013-08-03 13:51:26 +02:00
|
|
|
OPT_CMDMODE(0, "quit", &cmd, N_("end revert or cherry-pick sequence"), 'q'),
|
|
|
|
OPT_CMDMODE(0, "continue", &cmd, N_("resume revert or cherry-pick sequence"), 'c'),
|
|
|
|
OPT_CMDMODE(0, "abort", &cmd, N_("cancel revert or cherry-pick sequence"), 'a'),
|
|
|
|
OPT_BOOL('n', "no-commit", &opts->no_commit, N_("don't automatically commit")),
|
|
|
|
OPT_BOOL('e', "edit", &opts->edit, N_("edit the commit message")),
|
2011-09-28 19:47:50 +02:00
|
|
|
OPT_NOOP_NOARG('r', NULL),
|
2013-08-03 13:51:26 +02:00
|
|
|
OPT_BOOL('s', "signoff", &opts->signoff, N_("add Signed-off-by:")),
|
2012-08-20 14:32:41 +02:00
|
|
|
OPT_INTEGER('m', "mainline", &opts->mainline, N_("parent number")),
|
2011-08-04 12:39:05 +02:00
|
|
|
OPT_RERERE_AUTOUPDATE(&opts->allow_rerere_auto),
|
2012-08-20 14:32:41 +02:00
|
|
|
OPT_STRING(0, "strategy", &opts->strategy, N_("strategy"), N_("merge strategy")),
|
|
|
|
OPT_CALLBACK('X', "strategy-option", &opts, N_("option"),
|
|
|
|
N_("option for merge strategy"), option_parse_x),
|
2007-10-07 23:02:29 +02:00
|
|
|
OPT_END(),
|
2010-03-06 21:34:42 +01:00
|
|
|
OPT_END(),
|
|
|
|
OPT_END(),
|
2012-04-11 22:21:53 +02:00
|
|
|
OPT_END(),
|
git-cherry-pick: Add keep-redundant-commits option
The git-cherry-pick --allow-empty command by default only preserves empty
commits that were originally empty, i.e only those commits for which
<commit>^{tree} and <commit>^^{tree} are equal. By default commits which are
non-empty, but were made empty by the inclusion of a prior commit on the current
history are filtered out. This option allows us to override that behavior and
include redundant commits as empty commits in the change history.
Note that this patch changes the default behavior of git cherry-pick slightly.
Prior to this patch all commits in a cherry-pick sequence were applied and git
commit was run. The implication here was that, if a commit was redundant, and
the commit did not trigger the fast forward logic, the git commit operation, and
therefore the git cherry-pick operation would fail, displaying the cherry pick
advice (i.e. run git commit --allow-empty). With this patch however, such
redundant commits are automatically skipped without stopping, unless
--keep-redundant-commits is specified, in which case, they are automatically
applied as empty commits.
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-04-20 16:36:15 +02:00
|
|
|
OPT_END(),
|
2012-08-02 12:38:51 +02:00
|
|
|
OPT_END(),
|
2007-10-07 23:02:29 +02:00
|
|
|
};
|
|
|
|
|
2012-01-11 19:15:56 +01:00
|
|
|
if (opts->action == REPLAY_PICK) {
|
2010-03-06 21:34:42 +01:00
|
|
|
struct option cp_extra[] = {
|
2013-08-03 13:51:26 +02:00
|
|
|
OPT_BOOL('x', NULL, &opts->record_origin, N_("append commit name")),
|
|
|
|
OPT_BOOL(0, "ff", &opts->allow_ff, N_("allow fast-forward")),
|
|
|
|
OPT_BOOL(0, "allow-empty", &opts->allow_empty, N_("preserve initially empty commits")),
|
|
|
|
OPT_BOOL(0, "allow-empty-message", &opts->allow_empty_message, N_("allow commits with empty messages")),
|
|
|
|
OPT_BOOL(0, "keep-redundant-commits", &opts->keep_redundant_commits, N_("keep redundant, empty commits")),
|
2010-03-06 21:34:42 +01:00
|
|
|
OPT_END(),
|
|
|
|
};
|
|
|
|
if (parse_options_concat(options, ARRAY_SIZE(options), cp_extra))
|
2011-02-23 00:42:28 +01:00
|
|
|
die(_("program error"));
|
2010-03-06 21:34:42 +01:00
|
|
|
}
|
|
|
|
|
2011-08-13 19:06:23 +02:00
|
|
|
argc = parse_options(argc, argv, NULL, options, usage_str,
|
|
|
|
PARSE_OPT_KEEP_ARGV0 |
|
|
|
|
PARSE_OPT_KEEP_UNKNOWN);
|
2011-08-04 12:39:11 +02:00
|
|
|
|
git-cherry-pick: Add keep-redundant-commits option
The git-cherry-pick --allow-empty command by default only preserves empty
commits that were originally empty, i.e only those commits for which
<commit>^{tree} and <commit>^^{tree} are equal. By default commits which are
non-empty, but were made empty by the inclusion of a prior commit on the current
history are filtered out. This option allows us to override that behavior and
include redundant commits as empty commits in the change history.
Note that this patch changes the default behavior of git cherry-pick slightly.
Prior to this patch all commits in a cherry-pick sequence were applied and git
commit was run. The implication here was that, if a commit was redundant, and
the commit did not trigger the fast forward logic, the git commit operation, and
therefore the git cherry-pick operation would fail, displaying the cherry pick
advice (i.e. run git commit --allow-empty). With this patch however, such
redundant commits are automatically skipped without stopping, unless
--keep-redundant-commits is specified, in which case, they are automatically
applied as empty commits.
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-04-20 16:36:15 +02:00
|
|
|
/* implies allow_empty */
|
|
|
|
if (opts->keep_redundant_commits)
|
|
|
|
opts->allow_empty = 1;
|
|
|
|
|
2011-08-04 12:39:11 +02:00
|
|
|
/* Set the subcommand */
|
2013-08-03 13:51:26 +02:00
|
|
|
if (cmd == 'q')
|
2011-11-22 12:14:29 +01:00
|
|
|
opts->subcommand = REPLAY_REMOVE_STATE;
|
2013-08-03 13:51:26 +02:00
|
|
|
else if (cmd == 'c')
|
2011-08-04 12:39:15 +02:00
|
|
|
opts->subcommand = REPLAY_CONTINUE;
|
2013-08-03 13:51:26 +02:00
|
|
|
else if (cmd == 'a')
|
2011-11-23 02:27:21 +01:00
|
|
|
opts->subcommand = REPLAY_ROLLBACK;
|
2011-08-04 12:39:11 +02:00
|
|
|
else
|
|
|
|
opts->subcommand = REPLAY_NONE;
|
|
|
|
|
|
|
|
/* Check for incompatible command line arguments */
|
2011-08-04 12:39:15 +02:00
|
|
|
if (opts->subcommand != REPLAY_NONE) {
|
|
|
|
char *this_operation;
|
2011-11-22 12:14:29 +01:00
|
|
|
if (opts->subcommand == REPLAY_REMOVE_STATE)
|
|
|
|
this_operation = "--quit";
|
2011-11-23 02:27:21 +01:00
|
|
|
else if (opts->subcommand == REPLAY_CONTINUE)
|
2011-08-04 12:39:15 +02:00
|
|
|
this_operation = "--continue";
|
2011-11-23 02:27:21 +01:00
|
|
|
else {
|
|
|
|
assert(opts->subcommand == REPLAY_ROLLBACK);
|
|
|
|
this_operation = "--abort";
|
|
|
|
}
|
2011-08-04 12:39:15 +02:00
|
|
|
|
|
|
|
verify_opt_compatible(me, this_operation,
|
2011-08-04 12:39:11 +02:00
|
|
|
"--no-commit", opts->no_commit,
|
|
|
|
"--signoff", opts->signoff,
|
|
|
|
"--mainline", opts->mainline,
|
|
|
|
"--strategy", opts->strategy ? 1 : 0,
|
|
|
|
"--strategy-option", opts->xopts ? 1 : 0,
|
|
|
|
"-x", opts->record_origin,
|
|
|
|
"--ff", opts->allow_ff,
|
|
|
|
NULL);
|
|
|
|
}
|
|
|
|
|
2011-08-04 12:39:07 +02:00
|
|
|
if (opts->allow_ff)
|
|
|
|
verify_opt_compatible(me, "--ff",
|
|
|
|
"--signoff", opts->signoff,
|
|
|
|
"--no-commit", opts->no_commit,
|
|
|
|
"-x", opts->record_origin,
|
|
|
|
"--edit", opts->edit,
|
|
|
|
NULL);
|
2011-08-13 19:06:23 +02:00
|
|
|
|
|
|
|
if (opts->subcommand != REPLAY_NONE) {
|
|
|
|
opts->revs = NULL;
|
|
|
|
} else {
|
2012-04-14 21:04:48 +02:00
|
|
|
struct setup_revision_opt s_r_opt;
|
2011-08-13 19:06:23 +02:00
|
|
|
opts->revs = xmalloc(sizeof(*opts->revs));
|
|
|
|
init_revisions(opts->revs, NULL);
|
2012-08-29 08:15:56 +02:00
|
|
|
opts->revs->no_walk = REVISION_WALK_NO_WALK_UNSORTED;
|
2011-08-13 19:06:23 +02:00
|
|
|
if (argc < 2)
|
|
|
|
usage_with_options(usage_str, options);
|
2013-10-10 18:41:17 +02:00
|
|
|
if (!strcmp(argv[1], "-"))
|
|
|
|
argv[1] = "@{-1}";
|
2012-04-14 21:04:48 +02:00
|
|
|
memset(&s_r_opt, 0, sizeof(s_r_opt));
|
|
|
|
s_r_opt.assume_dashdash = 1;
|
|
|
|
argc = setup_revisions(argc, argv, opts->revs, &s_r_opt);
|
2011-08-13 19:06:23 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
if (argc > 1)
|
|
|
|
usage_with_options(usage_str, options);
|
2007-03-01 05:26:30 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
int cmd_revert(int argc, const char **argv, const char *prefix)
|
|
|
|
{
|
2011-08-04 12:39:05 +02:00
|
|
|
struct replay_opts opts;
|
2011-08-04 12:39:16 +02:00
|
|
|
int res;
|
2011-08-04 12:39:05 +02:00
|
|
|
|
|
|
|
memset(&opts, 0, sizeof(opts));
|
2007-03-01 05:26:30 +01:00
|
|
|
if (isatty(0))
|
2011-08-04 12:39:05 +02:00
|
|
|
opts.edit = 1;
|
2012-01-11 19:15:56 +01:00
|
|
|
opts.action = REPLAY_REVERT;
|
revert: Separate cmdline parsing from functional code
Currently, revert_or_cherry_pick sets up a default git config, parses
command-line arguments, before preparing to pick commits. This makes
for a bad API as the central worry of callers is to assert whether or
not a conflict occured while cherry picking. The current API is like:
if (revert_or_cherry_pick(argc, argv, opts) < 0)
print "Something failed, we're not sure what"
Simplify and rename revert_or_cherry_pick to pick_commits so that it
only has the responsibility of setting up the revision walker and
picking commits in a loop. Transfer the remaining work to its
callers. Now, the API is simplified as:
if (parse_args(argc, argv, opts) < 0)
print "Can't parse arguments"
if (pick_commits(opts) < 0)
print "Error encountered in picking machinery"
Later in the series, pick_commits will also serve as the starting
point for continuing a cherry-pick or revert.
Inspired-by: Christian Couder <chriscool@tuxfamily.org>
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-08-04 12:39:06 +02:00
|
|
|
git_config(git_default_config, NULL);
|
|
|
|
parse_args(argc, argv, &opts);
|
2012-01-11 19:15:57 +01:00
|
|
|
res = sequencer_pick_revisions(&opts);
|
2011-08-04 12:39:16 +02:00
|
|
|
if (res < 0)
|
|
|
|
die(_("revert failed"));
|
|
|
|
return res;
|
2007-03-01 05:26:30 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
int cmd_cherry_pick(int argc, const char **argv, const char *prefix)
|
|
|
|
{
|
2011-08-04 12:39:05 +02:00
|
|
|
struct replay_opts opts;
|
2011-08-04 12:39:16 +02:00
|
|
|
int res;
|
2011-08-04 12:39:05 +02:00
|
|
|
|
|
|
|
memset(&opts, 0, sizeof(opts));
|
2012-01-11 19:15:56 +01:00
|
|
|
opts.action = REPLAY_PICK;
|
revert: Separate cmdline parsing from functional code
Currently, revert_or_cherry_pick sets up a default git config, parses
command-line arguments, before preparing to pick commits. This makes
for a bad API as the central worry of callers is to assert whether or
not a conflict occured while cherry picking. The current API is like:
if (revert_or_cherry_pick(argc, argv, opts) < 0)
print "Something failed, we're not sure what"
Simplify and rename revert_or_cherry_pick to pick_commits so that it
only has the responsibility of setting up the revision walker and
picking commits in a loop. Transfer the remaining work to its
callers. Now, the API is simplified as:
if (parse_args(argc, argv, opts) < 0)
print "Can't parse arguments"
if (pick_commits(opts) < 0)
print "Error encountered in picking machinery"
Later in the series, pick_commits will also serve as the starting
point for continuing a cherry-pick or revert.
Inspired-by: Christian Couder <chriscool@tuxfamily.org>
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-08-04 12:39:06 +02:00
|
|
|
git_config(git_default_config, NULL);
|
|
|
|
parse_args(argc, argv, &opts);
|
2012-01-11 19:15:57 +01:00
|
|
|
res = sequencer_pick_revisions(&opts);
|
2011-08-04 12:39:16 +02:00
|
|
|
if (res < 0)
|
|
|
|
die(_("cherry-pick failed"));
|
|
|
|
return res;
|
2007-03-01 05:26:30 +01:00
|
|
|
}
|