2008-07-07 19:24:20 +02:00
|
|
|
/*
|
|
|
|
* Builtin "git merge"
|
|
|
|
*
|
|
|
|
* Copyright (c) 2008 Miklos Vajna <vmiklos@frugalware.org>
|
|
|
|
*
|
|
|
|
* Based on git-merge.sh by Junio C Hamano.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include "cache.h"
|
|
|
|
#include "parse-options.h"
|
|
|
|
#include "builtin.h"
|
|
|
|
#include "run-command.h"
|
|
|
|
#include "diff.h"
|
|
|
|
#include "refs.h"
|
|
|
|
#include "commit.h"
|
|
|
|
#include "diffcore.h"
|
|
|
|
#include "revision.h"
|
|
|
|
#include "unpack-trees.h"
|
|
|
|
#include "cache-tree.h"
|
|
|
|
#include "dir.h"
|
|
|
|
#include "utf8.h"
|
|
|
|
#include "log-tree.h"
|
|
|
|
#include "color.h"
|
2008-07-16 04:09:46 +02:00
|
|
|
#include "rerere.h"
|
2008-07-30 01:16:59 +02:00
|
|
|
#include "help.h"
|
2008-08-28 15:43:00 +02:00
|
|
|
#include "merge-recursive.h"
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
#define DEFAULT_TWOHEAD (1<<0)
|
|
|
|
#define DEFAULT_OCTOPUS (1<<1)
|
|
|
|
#define NO_FAST_FORWARD (1<<2)
|
|
|
|
#define NO_TRIVIAL (1<<3)
|
|
|
|
|
|
|
|
struct strategy {
|
|
|
|
const char *name;
|
|
|
|
unsigned attr;
|
|
|
|
};
|
|
|
|
|
|
|
|
static const char * const builtin_merge_usage[] = {
|
2009-01-04 19:39:27 +01:00
|
|
|
"git merge [options] <remote>...",
|
|
|
|
"git merge [options] <msg> HEAD <remote>",
|
2008-07-07 19:24:20 +02:00
|
|
|
NULL
|
|
|
|
};
|
|
|
|
|
|
|
|
static int show_diffstat = 1, option_log, squash;
|
|
|
|
static int option_commit = 1, allow_fast_forward = 1;
|
|
|
|
static int allow_trivial = 1, have_message;
|
|
|
|
static struct strbuf merge_msg;
|
|
|
|
static struct commit_list *remoteheads;
|
|
|
|
static unsigned char head[20], stash[20];
|
|
|
|
static struct strategy **use_strategies;
|
|
|
|
static size_t use_strategies_nr, use_strategies_alloc;
|
|
|
|
static const char *branch;
|
2008-11-15 01:14:24 +01:00
|
|
|
static int verbosity;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
static struct strategy all_strategy[] = {
|
|
|
|
{ "recursive", DEFAULT_TWOHEAD | NO_TRIVIAL },
|
|
|
|
{ "octopus", DEFAULT_OCTOPUS },
|
|
|
|
{ "resolve", 0 },
|
|
|
|
{ "ours", NO_FAST_FORWARD | NO_TRIVIAL },
|
|
|
|
{ "subtree", NO_FAST_FORWARD | NO_TRIVIAL },
|
|
|
|
};
|
|
|
|
|
|
|
|
static const char *pull_twohead, *pull_octopus;
|
|
|
|
|
|
|
|
static int option_parse_message(const struct option *opt,
|
|
|
|
const char *arg, int unset)
|
|
|
|
{
|
|
|
|
struct strbuf *buf = opt->value;
|
|
|
|
|
|
|
|
if (unset)
|
|
|
|
strbuf_setlen(buf, 0);
|
2008-07-20 14:34:47 +02:00
|
|
|
else if (arg) {
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_addf(buf, "%s\n\n", arg);
|
|
|
|
have_message = 1;
|
2008-07-20 14:34:47 +02:00
|
|
|
} else
|
|
|
|
return error("switch `m' requires a value");
|
2008-07-07 19:24:20 +02:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct strategy *get_strategy(const char *name)
|
|
|
|
{
|
|
|
|
int i;
|
2008-07-30 01:16:59 +02:00
|
|
|
struct strategy *ret;
|
|
|
|
static struct cmdnames main_cmds, other_cmds;
|
2008-08-28 19:15:33 +02:00
|
|
|
static int loaded;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
if (!name)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(all_strategy); i++)
|
|
|
|
if (!strcmp(name, all_strategy[i].name))
|
|
|
|
return &all_strategy[i];
|
2008-07-21 18:10:47 +02:00
|
|
|
|
2008-08-28 19:15:33 +02:00
|
|
|
if (!loaded) {
|
2008-07-30 01:16:59 +02:00
|
|
|
struct cmdnames not_strategies;
|
2008-08-28 19:15:33 +02:00
|
|
|
loaded = 1;
|
2008-07-30 01:16:59 +02:00
|
|
|
|
|
|
|
memset(¬_strategies, 0, sizeof(struct cmdnames));
|
2008-08-28 19:15:33 +02:00
|
|
|
load_command_list("git-merge-", &main_cmds, &other_cmds);
|
2008-07-30 01:16:59 +02:00
|
|
|
for (i = 0; i < main_cmds.cnt; i++) {
|
|
|
|
int j, found = 0;
|
|
|
|
struct cmdname *ent = main_cmds.names[i];
|
|
|
|
for (j = 0; j < ARRAY_SIZE(all_strategy); j++)
|
|
|
|
if (!strncmp(ent->name, all_strategy[j].name, ent->len)
|
|
|
|
&& !all_strategy[j].name[ent->len])
|
|
|
|
found = 1;
|
|
|
|
if (!found)
|
|
|
|
add_cmdname(¬_strategies, ent->name, ent->len);
|
|
|
|
exclude_cmds(&main_cmds, ¬_strategies);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (!is_in_cmdlist(&main_cmds, name) && !is_in_cmdlist(&other_cmds, name)) {
|
2008-08-21 07:07:55 +02:00
|
|
|
fprintf(stderr, "Could not find merge strategy '%s'.\n", name);
|
|
|
|
fprintf(stderr, "Available strategies are:");
|
|
|
|
for (i = 0; i < main_cmds.cnt; i++)
|
|
|
|
fprintf(stderr, " %s", main_cmds.names[i]->name);
|
|
|
|
fprintf(stderr, ".\n");
|
|
|
|
if (other_cmds.cnt) {
|
|
|
|
fprintf(stderr, "Available custom strategies are:");
|
|
|
|
for (i = 0; i < other_cmds.cnt; i++)
|
|
|
|
fprintf(stderr, " %s", other_cmds.names[i]->name);
|
|
|
|
fprintf(stderr, ".\n");
|
|
|
|
}
|
2008-07-30 01:16:59 +02:00
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
2008-10-07 01:39:10 +02:00
|
|
|
ret = xcalloc(1, sizeof(struct strategy));
|
2008-07-30 01:16:59 +02:00
|
|
|
ret->name = xstrdup(name);
|
|
|
|
return ret;
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void append_strategy(struct strategy *s)
|
|
|
|
{
|
|
|
|
ALLOC_GROW(use_strategies, use_strategies_nr + 1, use_strategies_alloc);
|
|
|
|
use_strategies[use_strategies_nr++] = s;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int option_parse_strategy(const struct option *opt,
|
|
|
|
const char *name, int unset)
|
|
|
|
{
|
|
|
|
if (unset)
|
|
|
|
return 0;
|
|
|
|
|
2008-07-21 18:10:47 +02:00
|
|
|
append_strategy(get_strategy(name));
|
2008-07-07 19:24:20 +02:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int option_parse_n(const struct option *opt,
|
|
|
|
const char *arg, int unset)
|
|
|
|
{
|
|
|
|
show_diffstat = unset;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct option builtin_merge_options[] = {
|
|
|
|
{ OPTION_CALLBACK, 'n', NULL, NULL, NULL,
|
|
|
|
"do not show a diffstat at the end of the merge",
|
|
|
|
PARSE_OPT_NOARG, option_parse_n },
|
|
|
|
OPT_BOOLEAN(0, "stat", &show_diffstat,
|
|
|
|
"show a diffstat at the end of the merge"),
|
|
|
|
OPT_BOOLEAN(0, "summary", &show_diffstat, "(synonym to --stat)"),
|
|
|
|
OPT_BOOLEAN(0, "log", &option_log,
|
|
|
|
"add list of one-line log to merge commit message"),
|
|
|
|
OPT_BOOLEAN(0, "squash", &squash,
|
|
|
|
"create a single commit instead of doing a merge"),
|
|
|
|
OPT_BOOLEAN(0, "commit", &option_commit,
|
|
|
|
"perform a commit if the merge succeeds (default)"),
|
|
|
|
OPT_BOOLEAN(0, "ff", &allow_fast_forward,
|
|
|
|
"allow fast forward (default)"),
|
|
|
|
OPT_CALLBACK('s', "strategy", &use_strategies, "strategy",
|
|
|
|
"merge strategy to use", option_parse_strategy),
|
|
|
|
OPT_CALLBACK('m', "message", &merge_msg, "message",
|
|
|
|
"message to be used for the merge commit (if any)",
|
|
|
|
option_parse_message),
|
2008-11-15 01:14:24 +01:00
|
|
|
OPT__VERBOSITY(&verbosity),
|
2008-07-07 19:24:20 +02:00
|
|
|
OPT_END()
|
|
|
|
};
|
|
|
|
|
|
|
|
/* Cleans up metadata that is uninteresting after a succeeded merge. */
|
|
|
|
static void drop_save(void)
|
|
|
|
{
|
|
|
|
unlink(git_path("MERGE_HEAD"));
|
|
|
|
unlink(git_path("MERGE_MSG"));
|
builtin-commit: use reduce_heads() only when appropriate
Since commit 6bb6b034 (builtin-commit: use commit_tree(), 2008-09-10),
builtin-commit performs a reduce_heads() unconditionally. However,
it's not always needed, and in some cases even harmful.
reduce_heads() is not needed for the initial commit or for an
"ordinary" commit, because they don't have any or have only one
parent, respectively.
reduce_heads() must be avoided when 'git commit' is run after a 'git
merge --no-ff --no-commit', otherwise it will turn the
non-fast-forward merge into fast-forward. For the same reason,
reduce_heads() must be avoided when amending such a merge commit.
To resolve this issue, 'git merge' will write info about whether
fast-forward is allowed or not to $GIT_DIR/MERGE_MODE. Based on this
info, 'git commit' will only perform reduce_heads() when it's
committing a merge and fast-forward is enabled.
Also add test cases to ensure that non-fast-forward merges are
committed and amended properly.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
2008-10-03 14:04:47 +02:00
|
|
|
unlink(git_path("MERGE_MODE"));
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void save_state(void)
|
|
|
|
{
|
|
|
|
int len;
|
|
|
|
struct child_process cp;
|
|
|
|
struct strbuf buffer = STRBUF_INIT;
|
|
|
|
const char *argv[] = {"stash", "create", NULL};
|
|
|
|
|
|
|
|
memset(&cp, 0, sizeof(cp));
|
|
|
|
cp.argv = argv;
|
|
|
|
cp.out = -1;
|
|
|
|
cp.git_cmd = 1;
|
|
|
|
|
|
|
|
if (start_command(&cp))
|
|
|
|
die("could not run stash.");
|
|
|
|
len = strbuf_read(&buffer, cp.out, 1024);
|
|
|
|
close(cp.out);
|
|
|
|
|
|
|
|
if (finish_command(&cp) || len < 0)
|
|
|
|
die("stash failed");
|
|
|
|
else if (!len)
|
|
|
|
return;
|
|
|
|
strbuf_setlen(&buffer, buffer.len-1);
|
|
|
|
if (get_sha1(buffer.buf, stash))
|
|
|
|
die("not a valid object: %s", buffer.buf);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void reset_hard(unsigned const char *sha1, int verbose)
|
|
|
|
{
|
|
|
|
int i = 0;
|
|
|
|
const char *args[6];
|
|
|
|
|
|
|
|
args[i++] = "read-tree";
|
|
|
|
if (verbose)
|
|
|
|
args[i++] = "-v";
|
|
|
|
args[i++] = "--reset";
|
|
|
|
args[i++] = "-u";
|
|
|
|
args[i++] = sha1_to_hex(sha1);
|
|
|
|
args[i] = NULL;
|
|
|
|
|
|
|
|
if (run_command_v_opt(args, RUN_GIT_CMD))
|
|
|
|
die("read-tree failed");
|
|
|
|
}
|
|
|
|
|
|
|
|
static void restore_state(void)
|
|
|
|
{
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf sb = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
const char *args[] = { "stash", "apply", NULL, NULL };
|
|
|
|
|
|
|
|
if (is_null_sha1(stash))
|
|
|
|
return;
|
|
|
|
|
|
|
|
reset_hard(head, 1);
|
|
|
|
|
|
|
|
args[2] = sha1_to_hex(stash);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* It is OK to ignore error here, for example when there was
|
|
|
|
* nothing to restore.
|
|
|
|
*/
|
|
|
|
run_command_v_opt(args, RUN_GIT_CMD);
|
|
|
|
|
|
|
|
strbuf_release(&sb);
|
|
|
|
refresh_cache(REFRESH_QUIET);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* This is called when no merge was necessary. */
|
|
|
|
static void finish_up_to_date(const char *msg)
|
|
|
|
{
|
2008-11-15 01:14:24 +01:00
|
|
|
if (verbosity >= 0)
|
|
|
|
printf("%s%s\n", squash ? " (nothing to squash)" : "", msg);
|
2008-07-07 19:24:20 +02:00
|
|
|
drop_save();
|
|
|
|
}
|
|
|
|
|
|
|
|
static void squash_message(void)
|
|
|
|
{
|
|
|
|
struct rev_info rev;
|
|
|
|
struct commit *commit;
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf out = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
struct commit_list *j;
|
|
|
|
int fd;
|
|
|
|
|
|
|
|
printf("Squash commit -- not updating HEAD\n");
|
|
|
|
fd = open(git_path("SQUASH_MSG"), O_WRONLY | O_CREAT, 0666);
|
|
|
|
if (fd < 0)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not write to '%s'", git_path("SQUASH_MSG"));
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
init_revisions(&rev, NULL);
|
|
|
|
rev.ignore_merges = 1;
|
|
|
|
rev.commit_format = CMIT_FMT_MEDIUM;
|
|
|
|
|
|
|
|
commit = lookup_commit(head);
|
|
|
|
commit->object.flags |= UNINTERESTING;
|
|
|
|
add_pending_object(&rev, &commit->object, NULL);
|
|
|
|
|
|
|
|
for (j = remoteheads; j; j = j->next)
|
|
|
|
add_pending_object(&rev, &j->item->object, NULL);
|
|
|
|
|
|
|
|
setup_revisions(0, NULL, &rev, NULL);
|
|
|
|
if (prepare_revision_walk(&rev))
|
|
|
|
die("revision walk setup failed");
|
|
|
|
|
|
|
|
strbuf_addstr(&out, "Squashed commit of the following:\n");
|
|
|
|
while ((commit = get_revision(&rev)) != NULL) {
|
|
|
|
strbuf_addch(&out, '\n');
|
|
|
|
strbuf_addf(&out, "commit %s\n",
|
|
|
|
sha1_to_hex(commit->object.sha1));
|
|
|
|
pretty_print_commit(rev.commit_format, commit, &out, rev.abbrev,
|
|
|
|
NULL, NULL, rev.date_mode, 0);
|
|
|
|
}
|
2008-12-05 01:35:48 +01:00
|
|
|
if (write(fd, out.buf, out.len) < 0)
|
2009-06-27 17:58:46 +02:00
|
|
|
die_errno("Writing SQUASH_MSG");
|
2008-12-05 01:35:48 +01:00
|
|
|
if (close(fd))
|
2009-06-27 17:58:46 +02:00
|
|
|
die_errno("Finishing SQUASH_MSG");
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_release(&out);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void finish(const unsigned char *new_head, const char *msg)
|
|
|
|
{
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf reflog_message = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
if (!msg)
|
|
|
|
strbuf_addstr(&reflog_message, getenv("GIT_REFLOG_ACTION"));
|
|
|
|
else {
|
2008-11-15 01:14:24 +01:00
|
|
|
if (verbosity >= 0)
|
|
|
|
printf("%s\n", msg);
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_addf(&reflog_message, "%s: %s",
|
|
|
|
getenv("GIT_REFLOG_ACTION"), msg);
|
|
|
|
}
|
|
|
|
if (squash) {
|
|
|
|
squash_message();
|
|
|
|
} else {
|
2008-11-15 01:14:24 +01:00
|
|
|
if (verbosity >= 0 && !merge_msg.len)
|
2008-07-07 19:24:20 +02:00
|
|
|
printf("No merge message -- not updating HEAD\n");
|
|
|
|
else {
|
|
|
|
const char *argv_gc_auto[] = { "gc", "--auto", NULL };
|
|
|
|
update_ref(reflog_message.buf, "HEAD",
|
|
|
|
new_head, head, 0,
|
|
|
|
DIE_ON_ERR);
|
|
|
|
/*
|
|
|
|
* We ignore errors in 'gc --auto', since the
|
|
|
|
* user should see them.
|
|
|
|
*/
|
|
|
|
run_command_v_opt(argv_gc_auto, RUN_GIT_CMD);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (new_head && show_diffstat) {
|
|
|
|
struct diff_options opts;
|
|
|
|
diff_setup(&opts);
|
|
|
|
opts.output_format |=
|
|
|
|
DIFF_FORMAT_SUMMARY | DIFF_FORMAT_DIFFSTAT;
|
|
|
|
opts.detect_rename = DIFF_DETECT_RENAME;
|
|
|
|
if (diff_use_color_default > 0)
|
|
|
|
DIFF_OPT_SET(&opts, COLOR_DIFF);
|
|
|
|
if (diff_setup_done(&opts) < 0)
|
|
|
|
die("diff_setup_done failed");
|
|
|
|
diff_tree_sha1(head, new_head, "", &opts);
|
|
|
|
diffcore_std(&opts);
|
|
|
|
diff_flush(&opts);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Run a post-merge hook */
|
2009-01-16 20:09:59 +01:00
|
|
|
run_hook(NULL, "post-merge", squash ? "1" : "0", NULL);
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
strbuf_release(&reflog_message);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Get the name for the merge commit's message. */
|
|
|
|
static void merge_name(const char *remote, struct strbuf *msg)
|
|
|
|
{
|
|
|
|
struct object *remote_head;
|
|
|
|
unsigned char branch_head[20], buf_sha[20];
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf buf = STRBUF_INIT;
|
2009-02-14 08:26:12 +01:00
|
|
|
struct strbuf bname = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
const char *ptr;
|
merge: fix incorrect merge message for ambiguous tag/branch
If we have both a tag and a branch named "foo", then calling
"git merge foo" will warn about the ambiguous ref, but merge
the tag.
When generating the commit message, though, we simply
checked whether "refs/heads/foo" existed, and if it did,
assumed it was a branch. This led to the statement "Merge
branch 'foo'" in the commit message, which is quite wrong.
Instead, we should use dwim_ref to find the actual ref used,
and describe it appropriately.
In addition to the test in t7608, we must also tweak the
expected output of t4202, which was accidentally triggering
this bug.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-08-09 12:02:24 +02:00
|
|
|
char *found_ref;
|
2008-07-07 19:24:20 +02:00
|
|
|
int len, early;
|
|
|
|
|
2009-03-21 21:17:30 +01:00
|
|
|
strbuf_branchname(&bname, remote);
|
|
|
|
remote = bname.buf;
|
2009-02-14 08:26:12 +01:00
|
|
|
|
2008-07-07 19:24:20 +02:00
|
|
|
memset(branch_head, 0, sizeof(branch_head));
|
|
|
|
remote_head = peel_to_type(remote, 0, NULL, OBJ_COMMIT);
|
|
|
|
if (!remote_head)
|
|
|
|
die("'%s' does not point to a commit", remote);
|
|
|
|
|
merge: fix incorrect merge message for ambiguous tag/branch
If we have both a tag and a branch named "foo", then calling
"git merge foo" will warn about the ambiguous ref, but merge
the tag.
When generating the commit message, though, we simply
checked whether "refs/heads/foo" existed, and if it did,
assumed it was a branch. This led to the statement "Merge
branch 'foo'" in the commit message, which is quite wrong.
Instead, we should use dwim_ref to find the actual ref used,
and describe it appropriately.
In addition to the test in t7608, we must also tweak the
expected output of t4202, which was accidentally triggering
this bug.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-08-09 12:02:24 +02:00
|
|
|
if (dwim_ref(remote, strlen(remote), branch_head, &found_ref) > 0) {
|
|
|
|
if (!prefixcmp(found_ref, "refs/heads/")) {
|
|
|
|
strbuf_addf(msg, "%s\t\tbranch '%s' of .\n",
|
|
|
|
sha1_to_hex(branch_head), remote);
|
|
|
|
goto cleanup;
|
|
|
|
}
|
2009-08-09 12:02:51 +02:00
|
|
|
if (!prefixcmp(found_ref, "refs/remotes/")) {
|
|
|
|
strbuf_addf(msg, "%s\t\tremote branch '%s' of .\n",
|
|
|
|
sha1_to_hex(branch_head), remote);
|
|
|
|
goto cleanup;
|
|
|
|
}
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
/* See if remote matches <name>^^^.. or <name>~<number> */
|
|
|
|
for (len = 0, ptr = remote + strlen(remote);
|
|
|
|
remote < ptr && ptr[-1] == '^';
|
|
|
|
ptr--)
|
|
|
|
len++;
|
|
|
|
if (len)
|
|
|
|
early = 1;
|
|
|
|
else {
|
|
|
|
early = 0;
|
|
|
|
ptr = strrchr(remote, '~');
|
|
|
|
if (ptr) {
|
|
|
|
int seen_nonzero = 0;
|
|
|
|
|
|
|
|
len++; /* count ~ */
|
|
|
|
while (*++ptr && isdigit(*ptr)) {
|
|
|
|
seen_nonzero |= (*ptr != '0');
|
|
|
|
len++;
|
|
|
|
}
|
|
|
|
if (*ptr)
|
|
|
|
len = 0; /* not ...~<number> */
|
|
|
|
else if (seen_nonzero)
|
|
|
|
early = 1;
|
|
|
|
else if (len == 1)
|
|
|
|
early = 1; /* "name~" is "name~1"! */
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (len) {
|
|
|
|
struct strbuf truname = STRBUF_INIT;
|
|
|
|
strbuf_addstr(&truname, "refs/heads/");
|
|
|
|
strbuf_addstr(&truname, remote);
|
Fix merge name generation in "merge in C"
When merging an early part of a branch, e.g. "git merge xyzzy~20", we were
supposed to say "branch 'xyzzy' (early part)", but it incorrectly said
"branch 'refs/heads/xy' (early part)" instead.
The logic was supposed to first strip away "~20" part to make sure that
what follows "~" is a non-zero posint, prefix it with "refs/heads/" and
ask resolve_ref() if it is a ref. If it is, then we know xyzzy was a
branch, and we can give the correct message.
However, there were a few bugs. First of all, the logic to build this
"true branch refname" did not count the characters correctly. At this
point of the code, "len" is the number of trailing, non-name part of the
given extended SHA-1 expression given by the user, i.e. number of bytes in
"~20" in the above example.
In addition, the message forgot to skip "refs/heads/" it prefixed from the
output.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-30 10:12:19 +02:00
|
|
|
strbuf_setlen(&truname, truname.len - len);
|
2009-06-18 19:28:43 +02:00
|
|
|
if (resolve_ref(truname.buf, buf_sha, 0, NULL)) {
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_addf(msg,
|
|
|
|
"%s\t\tbranch '%s'%s of .\n",
|
|
|
|
sha1_to_hex(remote_head->sha1),
|
Fix merge name generation in "merge in C"
When merging an early part of a branch, e.g. "git merge xyzzy~20", we were
supposed to say "branch 'xyzzy' (early part)", but it incorrectly said
"branch 'refs/heads/xy' (early part)" instead.
The logic was supposed to first strip away "~20" part to make sure that
what follows "~" is a non-zero posint, prefix it with "refs/heads/" and
ask resolve_ref() if it is a ref. If it is, then we know xyzzy was a
branch, and we can give the correct message.
However, there were a few bugs. First of all, the logic to build this
"true branch refname" did not count the characters correctly. At this
point of the code, "len" is the number of trailing, non-name part of the
given extended SHA-1 expression given by the user, i.e. number of bytes in
"~20" in the above example.
In addition, the message forgot to skip "refs/heads/" it prefixed from the
output.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-30 10:12:19 +02:00
|
|
|
truname.buf + 11,
|
2008-07-07 19:24:20 +02:00
|
|
|
(early ? " (early part)" : ""));
|
2009-02-14 08:26:12 +01:00
|
|
|
strbuf_release(&truname);
|
|
|
|
goto cleanup;
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!strcmp(remote, "FETCH_HEAD") &&
|
|
|
|
!access(git_path("FETCH_HEAD"), R_OK)) {
|
|
|
|
FILE *fp;
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf line = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
char *ptr;
|
|
|
|
|
|
|
|
fp = fopen(git_path("FETCH_HEAD"), "r");
|
|
|
|
if (!fp)
|
2009-06-27 17:58:46 +02:00
|
|
|
die_errno("could not open '%s' for reading",
|
|
|
|
git_path("FETCH_HEAD"));
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_getline(&line, fp, '\n');
|
|
|
|
fclose(fp);
|
|
|
|
ptr = strstr(line.buf, "\tnot-for-merge\t");
|
|
|
|
if (ptr)
|
|
|
|
strbuf_remove(&line, ptr-line.buf+1, 13);
|
|
|
|
strbuf_addbuf(msg, &line);
|
|
|
|
strbuf_release(&line);
|
2009-02-14 08:26:12 +01:00
|
|
|
goto cleanup;
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
strbuf_addf(msg, "%s\t\tcommit '%s'\n",
|
|
|
|
sha1_to_hex(remote_head->sha1), remote);
|
2009-02-14 08:26:12 +01:00
|
|
|
cleanup:
|
|
|
|
strbuf_release(&buf);
|
|
|
|
strbuf_release(&bname);
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
2008-07-24 01:09:35 +02:00
|
|
|
static int git_merge_config(const char *k, const char *v, void *cb)
|
2008-07-07 19:24:20 +02:00
|
|
|
{
|
|
|
|
if (branch && !prefixcmp(k, "branch.") &&
|
|
|
|
!prefixcmp(k + 7, branch) &&
|
|
|
|
!strcmp(k + 7 + strlen(branch), ".mergeoptions")) {
|
|
|
|
const char **argv;
|
|
|
|
int argc;
|
|
|
|
char *buf;
|
|
|
|
|
|
|
|
buf = xstrdup(v);
|
|
|
|
argc = split_cmdline(buf, &argv);
|
2008-09-22 17:06:41 +02:00
|
|
|
if (argc < 0)
|
|
|
|
die("Bad branch.%s.mergeoptions string", branch);
|
2008-07-07 19:24:20 +02:00
|
|
|
argv = xrealloc(argv, sizeof(*argv) * (argc + 2));
|
|
|
|
memmove(argv + 1, argv, sizeof(*argv) * (argc + 1));
|
|
|
|
argc++;
|
2009-05-23 20:53:12 +02:00
|
|
|
parse_options(argc, argv, NULL, builtin_merge_options,
|
2008-07-07 19:24:20 +02:00
|
|
|
builtin_merge_usage, 0);
|
|
|
|
free(buf);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!strcmp(k, "merge.diffstat") || !strcmp(k, "merge.stat"))
|
|
|
|
show_diffstat = git_config_bool(k, v);
|
|
|
|
else if (!strcmp(k, "pull.twohead"))
|
|
|
|
return git_config_string(&pull_twohead, k, v);
|
|
|
|
else if (!strcmp(k, "pull.octopus"))
|
|
|
|
return git_config_string(&pull_octopus, k, v);
|
2008-07-10 09:50:59 +02:00
|
|
|
else if (!strcmp(k, "merge.log") || !strcmp(k, "merge.summary"))
|
|
|
|
option_log = git_config_bool(k, v);
|
2008-07-07 19:24:20 +02:00
|
|
|
return git_diff_ui_config(k, v, cb);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int read_tree_trivial(unsigned char *common, unsigned char *head,
|
|
|
|
unsigned char *one)
|
|
|
|
{
|
|
|
|
int i, nr_trees = 0;
|
|
|
|
struct tree *trees[MAX_UNPACK_TREES];
|
|
|
|
struct tree_desc t[MAX_UNPACK_TREES];
|
|
|
|
struct unpack_trees_options opts;
|
|
|
|
|
|
|
|
memset(&opts, 0, sizeof(opts));
|
|
|
|
opts.head_idx = 2;
|
|
|
|
opts.src_index = &the_index;
|
|
|
|
opts.dst_index = &the_index;
|
|
|
|
opts.update = 1;
|
|
|
|
opts.verbose_update = 1;
|
|
|
|
opts.trivial_merges_only = 1;
|
|
|
|
opts.merge = 1;
|
|
|
|
trees[nr_trees] = parse_tree_indirect(common);
|
|
|
|
if (!trees[nr_trees++])
|
|
|
|
return -1;
|
|
|
|
trees[nr_trees] = parse_tree_indirect(head);
|
|
|
|
if (!trees[nr_trees++])
|
|
|
|
return -1;
|
|
|
|
trees[nr_trees] = parse_tree_indirect(one);
|
|
|
|
if (!trees[nr_trees++])
|
|
|
|
return -1;
|
|
|
|
opts.fn = threeway_merge;
|
|
|
|
cache_tree_free(&active_cache_tree);
|
|
|
|
for (i = 0; i < nr_trees; i++) {
|
|
|
|
parse_tree(trees[i]);
|
|
|
|
init_tree_desc(t+i, trees[i]->buffer, trees[i]->size);
|
|
|
|
}
|
|
|
|
if (unpack_trees(nr_trees, t, &opts))
|
|
|
|
return -1;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void write_tree_trivial(unsigned char *sha1)
|
|
|
|
{
|
|
|
|
if (write_cache_as_tree(sha1, 0, NULL))
|
|
|
|
die("git write-tree failed to write a tree");
|
|
|
|
}
|
|
|
|
|
|
|
|
static int try_merge_strategy(const char *strategy, struct commit_list *common,
|
|
|
|
const char *head_arg)
|
|
|
|
{
|
|
|
|
const char **args;
|
|
|
|
int i = 0, ret;
|
|
|
|
struct commit_list *j;
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf buf = STRBUF_INIT;
|
2008-10-03 15:02:31 +02:00
|
|
|
int index_fd;
|
|
|
|
struct lock_file *lock = xcalloc(1, sizeof(struct lock_file));
|
|
|
|
|
|
|
|
index_fd = hold_locked_index(lock, 1);
|
|
|
|
refresh_cache(REFRESH_QUIET);
|
|
|
|
if (active_cache_changed &&
|
|
|
|
(write_cache(index_fd, active_cache, active_nr) ||
|
|
|
|
commit_locked_index(lock)))
|
|
|
|
return error("Unable to write index.");
|
|
|
|
rollback_lock_file(lock);
|
2008-07-07 19:24:20 +02:00
|
|
|
|
2008-08-28 15:43:00 +02:00
|
|
|
if (!strcmp(strategy, "recursive") || !strcmp(strategy, "subtree")) {
|
|
|
|
int clean;
|
|
|
|
struct commit *result;
|
|
|
|
struct lock_file *lock = xcalloc(1, sizeof(struct lock_file));
|
|
|
|
int index_fd;
|
|
|
|
struct commit_list *reversed = NULL;
|
|
|
|
struct merge_options o;
|
|
|
|
|
|
|
|
if (remoteheads->next) {
|
|
|
|
error("Not handling anything other than two heads merge.");
|
|
|
|
return 2;
|
|
|
|
}
|
|
|
|
|
|
|
|
init_merge_options(&o);
|
|
|
|
if (!strcmp(strategy, "subtree"))
|
|
|
|
o.subtree_merge = 1;
|
|
|
|
|
|
|
|
o.branch1 = head_arg;
|
|
|
|
o.branch2 = remoteheads->item->util;
|
|
|
|
|
|
|
|
for (j = common; j; j = j->next)
|
|
|
|
commit_list_insert(j->item, &reversed);
|
|
|
|
|
|
|
|
index_fd = hold_locked_index(lock, 1);
|
|
|
|
clean = merge_recursive(&o, lookup_commit(head),
|
|
|
|
remoteheads->item, reversed, &result);
|
|
|
|
if (active_cache_changed &&
|
|
|
|
(write_cache(index_fd, active_cache, active_nr) ||
|
|
|
|
commit_locked_index(lock)))
|
|
|
|
die ("unable to write %s", get_index_file());
|
2008-09-06 18:29:49 +02:00
|
|
|
rollback_lock_file(lock);
|
2008-08-28 15:43:00 +02:00
|
|
|
return clean ? 0 : 1;
|
|
|
|
} else {
|
|
|
|
args = xmalloc((4 + commit_list_count(common) +
|
|
|
|
commit_list_count(remoteheads)) * sizeof(char *));
|
|
|
|
strbuf_addf(&buf, "merge-%s", strategy);
|
|
|
|
args[i++] = buf.buf;
|
|
|
|
for (j = common; j; j = j->next)
|
|
|
|
args[i++] = xstrdup(sha1_to_hex(j->item->object.sha1));
|
|
|
|
args[i++] = "--";
|
|
|
|
args[i++] = head_arg;
|
|
|
|
for (j = remoteheads; j; j = j->next)
|
|
|
|
args[i++] = xstrdup(sha1_to_hex(j->item->object.sha1));
|
|
|
|
args[i] = NULL;
|
|
|
|
ret = run_command_v_opt(args, RUN_GIT_CMD);
|
|
|
|
strbuf_release(&buf);
|
|
|
|
i = 1;
|
|
|
|
for (j = common; j; j = j->next)
|
|
|
|
free((void *)args[i++]);
|
|
|
|
i += 2;
|
|
|
|
for (j = remoteheads; j; j = j->next)
|
|
|
|
free((void *)args[i++]);
|
|
|
|
free(args);
|
|
|
|
discard_cache();
|
|
|
|
if (read_cache() < 0)
|
|
|
|
die("failed to read the cache");
|
2009-07-04 21:26:39 +02:00
|
|
|
return ret;
|
2008-08-28 15:43:00 +02:00
|
|
|
}
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void count_diff_files(struct diff_queue_struct *q,
|
|
|
|
struct diff_options *opt, void *data)
|
|
|
|
{
|
|
|
|
int *count = data;
|
|
|
|
|
|
|
|
(*count) += q->nr;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int count_unmerged_entries(void)
|
|
|
|
{
|
|
|
|
const struct index_state *state = &the_index;
|
|
|
|
int i, ret = 0;
|
|
|
|
|
|
|
|
for (i = 0; i < state->cache_nr; i++)
|
|
|
|
if (ce_stage(state->cache[i]))
|
|
|
|
ret++;
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int checkout_fast_forward(unsigned char *head, unsigned char *remote)
|
|
|
|
{
|
|
|
|
struct tree *trees[MAX_UNPACK_TREES];
|
|
|
|
struct unpack_trees_options opts;
|
|
|
|
struct tree_desc t[MAX_UNPACK_TREES];
|
|
|
|
int i, fd, nr_trees = 0;
|
|
|
|
struct dir_struct dir;
|
|
|
|
struct lock_file *lock_file = xcalloc(1, sizeof(struct lock_file));
|
|
|
|
|
2008-08-21 00:09:28 +02:00
|
|
|
refresh_cache(REFRESH_QUIET);
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
fd = hold_locked_index(lock_file, 1);
|
|
|
|
|
|
|
|
memset(&trees, 0, sizeof(trees));
|
|
|
|
memset(&opts, 0, sizeof(opts));
|
|
|
|
memset(&t, 0, sizeof(t));
|
2008-07-22 07:32:53 +02:00
|
|
|
memset(&dir, 0, sizeof(dir));
|
2009-02-16 13:20:25 +01:00
|
|
|
dir.flags |= DIR_SHOW_IGNORED;
|
2008-07-07 19:24:20 +02:00
|
|
|
dir.exclude_per_dir = ".gitignore";
|
|
|
|
opts.dir = &dir;
|
|
|
|
|
|
|
|
opts.head_idx = 1;
|
|
|
|
opts.src_index = &the_index;
|
|
|
|
opts.dst_index = &the_index;
|
|
|
|
opts.update = 1;
|
|
|
|
opts.verbose_update = 1;
|
|
|
|
opts.merge = 1;
|
|
|
|
opts.fn = twoway_merge;
|
|
|
|
|
|
|
|
trees[nr_trees] = parse_tree_indirect(head);
|
|
|
|
if (!trees[nr_trees++])
|
|
|
|
return -1;
|
|
|
|
trees[nr_trees] = parse_tree_indirect(remote);
|
|
|
|
if (!trees[nr_trees++])
|
|
|
|
return -1;
|
|
|
|
for (i = 0; i < nr_trees; i++) {
|
|
|
|
parse_tree(trees[i]);
|
|
|
|
init_tree_desc(t+i, trees[i]->buffer, trees[i]->size);
|
|
|
|
}
|
|
|
|
if (unpack_trees(nr_trees, t, &opts))
|
|
|
|
return -1;
|
|
|
|
if (write_cache(fd, active_cache, active_nr) ||
|
|
|
|
commit_locked_index(lock_file))
|
|
|
|
die("unable to write new index file");
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void split_merge_strategies(const char *string, struct strategy **list,
|
|
|
|
int *nr, int *alloc)
|
|
|
|
{
|
|
|
|
char *p, *q, *buf;
|
|
|
|
|
|
|
|
if (!string)
|
|
|
|
return;
|
|
|
|
|
|
|
|
buf = xstrdup(string);
|
|
|
|
q = buf;
|
|
|
|
for (;;) {
|
|
|
|
p = strchr(q, ' ');
|
|
|
|
if (!p) {
|
|
|
|
ALLOC_GROW(*list, *nr + 1, *alloc);
|
|
|
|
(*list)[(*nr)++].name = xstrdup(q);
|
|
|
|
free(buf);
|
|
|
|
return;
|
|
|
|
} else {
|
|
|
|
*p = '\0';
|
|
|
|
ALLOC_GROW(*list, *nr + 1, *alloc);
|
|
|
|
(*list)[(*nr)++].name = xstrdup(q);
|
|
|
|
q = ++p;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static void add_strategies(const char *string, unsigned attr)
|
|
|
|
{
|
|
|
|
struct strategy *list = NULL;
|
|
|
|
int list_alloc = 0, list_nr = 0, i;
|
|
|
|
|
|
|
|
memset(&list, 0, sizeof(list));
|
|
|
|
split_merge_strategies(string, &list, &list_nr, &list_alloc);
|
2008-07-21 18:10:47 +02:00
|
|
|
if (list) {
|
|
|
|
for (i = 0; i < list_nr; i++)
|
|
|
|
append_strategy(get_strategy(list[i].name));
|
2008-07-07 19:24:20 +02:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
for (i = 0; i < ARRAY_SIZE(all_strategy); i++)
|
|
|
|
if (all_strategy[i].attr & attr)
|
|
|
|
append_strategy(&all_strategy[i]);
|
|
|
|
|
|
|
|
}
|
|
|
|
|
|
|
|
static int merge_trivial(void)
|
|
|
|
{
|
|
|
|
unsigned char result_tree[20], result_commit[20];
|
2008-10-09 02:07:54 +02:00
|
|
|
struct commit_list *parent = xmalloc(sizeof(*parent));
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
write_tree_trivial(result_tree);
|
|
|
|
printf("Wonderful.\n");
|
merge: fix numerus bugs around "trivial merge" area
The "trivial merge" codepath wants to optimize itself by making an
internal call to the read-tree machinery, but it does not read the index
before doing so, and the codepath is never exercised. Incidentally, this
failure to read the index upfront means that the safety to refuse doing
anything when the index is unmerged does not kick in, either.
These two problem are fixed by using read_cache_unmerged() that does read
the index before checking if it is unmerged at the beginning of
cmd_merge().
The primary logic of the merge, however, assumes that the process never
reads the index in-core, and the call to write_cache_as_tree() it makes
from write_tree_trivial() will always read from the on-disk index that is
prepared the strategy back-ends. This assumption is now broken by the
above fix. To fix this issue, we now call discard_cache() before calling
write_tree_trivial() when it wants to write the on-disk index as a tree.
When multiple strategies are tried, their results are evaluated by reading
the resulting index and inspecting it. The codepath needs to make a call
to read_cache() for each successful strategy, and for that to work, they
need to discard_cache() the one read by the previous round.
Also the "trivial merge" forgot that the current commit is one of the
parents of the resulting commit.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-08-23 21:56:57 +02:00
|
|
|
parent->item = lookup_commit(head);
|
2008-10-09 02:07:54 +02:00
|
|
|
parent->next = xmalloc(sizeof(*parent->next));
|
merge: fix numerus bugs around "trivial merge" area
The "trivial merge" codepath wants to optimize itself by making an
internal call to the read-tree machinery, but it does not read the index
before doing so, and the codepath is never exercised. Incidentally, this
failure to read the index upfront means that the safety to refuse doing
anything when the index is unmerged does not kick in, either.
These two problem are fixed by using read_cache_unmerged() that does read
the index before checking if it is unmerged at the beginning of
cmd_merge().
The primary logic of the merge, however, assumes that the process never
reads the index in-core, and the call to write_cache_as_tree() it makes
from write_tree_trivial() will always read from the on-disk index that is
prepared the strategy back-ends. This assumption is now broken by the
above fix. To fix this issue, we now call discard_cache() before calling
write_tree_trivial() when it wants to write the on-disk index as a tree.
When multiple strategies are tried, their results are evaluated by reading
the resulting index and inspecting it. The codepath needs to make a call
to read_cache() for each successful strategy, and for that to work, they
need to discard_cache() the one read by the previous round.
Also the "trivial merge" forgot that the current commit is one of the
parents of the resulting commit.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-08-23 21:56:57 +02:00
|
|
|
parent->next->item = remoteheads->item;
|
|
|
|
parent->next->next = NULL;
|
2008-09-10 22:10:31 +02:00
|
|
|
commit_tree(merge_msg.buf, result_tree, parent, result_commit, NULL);
|
2008-07-07 19:24:20 +02:00
|
|
|
finish(result_commit, "In-index merge");
|
|
|
|
drop_save();
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int finish_automerge(struct commit_list *common,
|
|
|
|
unsigned char *result_tree,
|
|
|
|
const char *wt_strategy)
|
|
|
|
{
|
|
|
|
struct commit_list *parents = NULL, *j;
|
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
unsigned char result_commit[20];
|
|
|
|
|
|
|
|
free_commit_list(common);
|
|
|
|
if (allow_fast_forward) {
|
|
|
|
parents = remoteheads;
|
|
|
|
commit_list_insert(lookup_commit(head), &parents);
|
|
|
|
parents = reduce_heads(parents);
|
|
|
|
} else {
|
|
|
|
struct commit_list **pptr = &parents;
|
|
|
|
|
|
|
|
pptr = &commit_list_insert(lookup_commit(head),
|
|
|
|
pptr)->next;
|
|
|
|
for (j = remoteheads; j; j = j->next)
|
|
|
|
pptr = &commit_list_insert(j->item, pptr)->next;
|
|
|
|
}
|
|
|
|
free_commit_list(remoteheads);
|
|
|
|
strbuf_addch(&merge_msg, '\n');
|
2008-09-10 22:10:31 +02:00
|
|
|
commit_tree(merge_msg.buf, result_tree, parents, result_commit, NULL);
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_addf(&buf, "Merge made by %s.", wt_strategy);
|
|
|
|
finish(result_commit, buf.buf);
|
|
|
|
strbuf_release(&buf);
|
|
|
|
drop_save();
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int suggest_conflicts(void)
|
|
|
|
{
|
|
|
|
FILE *fp;
|
|
|
|
int pos;
|
|
|
|
|
|
|
|
fp = fopen(git_path("MERGE_MSG"), "a");
|
|
|
|
if (!fp)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not open '%s' for writing",
|
|
|
|
git_path("MERGE_MSG"));
|
2008-07-07 19:24:20 +02:00
|
|
|
fprintf(fp, "\nConflicts:\n");
|
|
|
|
for (pos = 0; pos < active_nr; pos++) {
|
|
|
|
struct cache_entry *ce = active_cache[pos];
|
|
|
|
|
|
|
|
if (ce_stage(ce)) {
|
|
|
|
fprintf(fp, "\t%s\n", ce->name);
|
|
|
|
while (pos + 1 < active_nr &&
|
|
|
|
!strcmp(ce->name,
|
|
|
|
active_cache[pos + 1]->name))
|
|
|
|
pos++;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
fclose(fp);
|
|
|
|
rerere();
|
|
|
|
printf("Automatic merge failed; "
|
|
|
|
"fix conflicts and then commit the result.\n");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct commit *is_old_style_invocation(int argc, const char **argv)
|
|
|
|
{
|
|
|
|
struct commit *second_token = NULL;
|
2009-12-02 18:59:35 +01:00
|
|
|
if (argc > 2) {
|
2008-07-07 19:24:20 +02:00
|
|
|
unsigned char second_sha1[20];
|
|
|
|
|
|
|
|
if (get_sha1(argv[1], second_sha1))
|
|
|
|
return NULL;
|
|
|
|
second_token = lookup_commit_reference_gently(second_sha1, 0);
|
|
|
|
if (!second_token)
|
|
|
|
die("'%s' is not a commit", argv[1]);
|
|
|
|
if (hashcmp(second_token->object.sha1, head))
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
return second_token;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int evaluate_result(void)
|
|
|
|
{
|
|
|
|
int cnt = 0;
|
|
|
|
struct rev_info rev;
|
|
|
|
|
|
|
|
/* Check how many files differ. */
|
|
|
|
init_revisions(&rev, "");
|
|
|
|
setup_revisions(0, NULL, &rev, NULL);
|
|
|
|
rev.diffopt.output_format |=
|
|
|
|
DIFF_FORMAT_CALLBACK;
|
|
|
|
rev.diffopt.format_callback = count_diff_files;
|
|
|
|
rev.diffopt.format_callback_data = &cnt;
|
|
|
|
run_diff_files(&rev, 0);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Check how many unmerged entries are
|
|
|
|
* there.
|
|
|
|
*/
|
|
|
|
cnt += count_unmerged_entries();
|
|
|
|
|
|
|
|
return cnt;
|
|
|
|
}
|
|
|
|
|
|
|
|
int cmd_merge(int argc, const char **argv, const char *prefix)
|
|
|
|
{
|
|
|
|
unsigned char result_tree[20];
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf buf = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
const char *head_arg;
|
|
|
|
int flag, head_invalid = 0, i;
|
|
|
|
int best_cnt = -1, merge_was_ok = 0, automerge_was_ok = 0;
|
|
|
|
struct commit_list *common = NULL;
|
|
|
|
const char *best_strategy = NULL, *wt_strategy = NULL;
|
|
|
|
struct commit_list **remotes = &remoteheads;
|
|
|
|
|
2009-06-01 11:20:56 +02:00
|
|
|
if (file_exists(git_path("MERGE_HEAD")))
|
|
|
|
die("You have not concluded your merge. (MERGE_HEAD exists)");
|
merge: fix numerus bugs around "trivial merge" area
The "trivial merge" codepath wants to optimize itself by making an
internal call to the read-tree machinery, but it does not read the index
before doing so, and the codepath is never exercised. Incidentally, this
failure to read the index upfront means that the safety to refuse doing
anything when the index is unmerged does not kick in, either.
These two problem are fixed by using read_cache_unmerged() that does read
the index before checking if it is unmerged at the beginning of
cmd_merge().
The primary logic of the merge, however, assumes that the process never
reads the index in-core, and the call to write_cache_as_tree() it makes
from write_tree_trivial() will always read from the on-disk index that is
prepared the strategy back-ends. This assumption is now broken by the
above fix. To fix this issue, we now call discard_cache() before calling
write_tree_trivial() when it wants to write the on-disk index as a tree.
When multiple strategies are tried, their results are evaluated by reading
the resulting index and inspecting it. The codepath needs to make a call
to read_cache() for each successful strategy, and for that to work, they
need to discard_cache() the one read by the previous round.
Also the "trivial merge" forgot that the current commit is one of the
parents of the resulting commit.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-08-23 21:56:57 +02:00
|
|
|
if (read_cache_unmerged())
|
2009-06-01 11:20:56 +02:00
|
|
|
die("You are in the middle of a conflicted merge."
|
|
|
|
" (index unmerged)");
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Check if we are _not_ on a detached HEAD, i.e. if there is a
|
|
|
|
* current branch.
|
|
|
|
*/
|
|
|
|
branch = resolve_ref("HEAD", head, 0, &flag);
|
|
|
|
if (branch && !prefixcmp(branch, "refs/heads/"))
|
|
|
|
branch += 11;
|
|
|
|
if (is_null_sha1(head))
|
|
|
|
head_invalid = 1;
|
|
|
|
|
|
|
|
git_config(git_merge_config, NULL);
|
|
|
|
|
|
|
|
/* for color.ui */
|
|
|
|
if (diff_use_color_default == -1)
|
|
|
|
diff_use_color_default = git_use_color_default;
|
|
|
|
|
2009-05-23 20:53:12 +02:00
|
|
|
argc = parse_options(argc, argv, prefix, builtin_merge_options,
|
2008-07-07 19:24:20 +02:00
|
|
|
builtin_merge_usage, 0);
|
2008-11-15 01:14:24 +01:00
|
|
|
if (verbosity < 0)
|
|
|
|
show_diffstat = 0;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
if (squash) {
|
|
|
|
if (!allow_fast_forward)
|
|
|
|
die("You cannot combine --squash with --no-ff.");
|
|
|
|
option_commit = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!argc)
|
|
|
|
usage_with_options(builtin_merge_usage,
|
|
|
|
builtin_merge_options);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* This could be traditional "merge <msg> HEAD <commit>..." and
|
|
|
|
* the way we can tell it is to see if the second token is HEAD,
|
|
|
|
* but some people might have misused the interface and used a
|
|
|
|
* committish that is the same as HEAD there instead.
|
|
|
|
* Traditional format never would have "-m" so it is an
|
|
|
|
* additional safety measure to check for it.
|
|
|
|
*/
|
|
|
|
|
|
|
|
if (!have_message && is_old_style_invocation(argc, argv)) {
|
|
|
|
strbuf_addstr(&merge_msg, argv[0]);
|
|
|
|
head_arg = argv[1];
|
|
|
|
argv += 2;
|
|
|
|
argc -= 2;
|
|
|
|
} else if (head_invalid) {
|
|
|
|
struct object *remote_head;
|
|
|
|
/*
|
|
|
|
* If the merged head is a valid one there is no reason
|
|
|
|
* to forbid "git merge" into a branch yet to be born.
|
|
|
|
* We do the same for "git pull".
|
|
|
|
*/
|
|
|
|
if (argc != 1)
|
|
|
|
die("Can merge only exactly one commit into "
|
|
|
|
"empty head");
|
2008-08-21 14:14:18 +02:00
|
|
|
if (squash)
|
|
|
|
die("Squash commit into empty head not supported yet");
|
|
|
|
if (!allow_fast_forward)
|
|
|
|
die("Non-fast-forward commit does not make sense into "
|
|
|
|
"an empty head");
|
2008-07-07 19:24:20 +02:00
|
|
|
remote_head = peel_to_type(argv[0], 0, NULL, OBJ_COMMIT);
|
|
|
|
if (!remote_head)
|
|
|
|
die("%s - not something we can merge", argv[0]);
|
|
|
|
update_ref("initial pull", "HEAD", remote_head->sha1, NULL, 0,
|
|
|
|
DIE_ON_ERR);
|
|
|
|
reset_hard(remote_head->sha1, 0);
|
|
|
|
return 0;
|
|
|
|
} else {
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf msg = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
/* We are invoked directly as the first-class UI. */
|
|
|
|
head_arg = "HEAD";
|
|
|
|
|
|
|
|
/*
|
|
|
|
* All the rest are the commits being merged;
|
|
|
|
* prepare the standard merge summary message to
|
|
|
|
* be appended to the given message. If remote
|
|
|
|
* is invalid we will die later in the common
|
|
|
|
* codepath so we discard the error in this
|
|
|
|
* loop.
|
|
|
|
*/
|
|
|
|
for (i = 0; i < argc; i++)
|
|
|
|
merge_name(argv[i], &msg);
|
|
|
|
fmt_merge_msg(option_log, &msg, &merge_msg);
|
|
|
|
if (merge_msg.len)
|
|
|
|
strbuf_setlen(&merge_msg, merge_msg.len-1);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (head_invalid || !argc)
|
|
|
|
usage_with_options(builtin_merge_usage,
|
|
|
|
builtin_merge_options);
|
|
|
|
|
|
|
|
strbuf_addstr(&buf, "merge");
|
|
|
|
for (i = 0; i < argc; i++)
|
|
|
|
strbuf_addf(&buf, " %s", argv[i]);
|
|
|
|
setenv("GIT_REFLOG_ACTION", buf.buf, 0);
|
|
|
|
strbuf_reset(&buf);
|
|
|
|
|
|
|
|
for (i = 0; i < argc; i++) {
|
|
|
|
struct object *o;
|
2008-08-28 15:43:00 +02:00
|
|
|
struct commit *commit;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
o = peel_to_type(argv[i], 0, NULL, OBJ_COMMIT);
|
|
|
|
if (!o)
|
|
|
|
die("%s - not something we can merge", argv[i]);
|
2008-08-28 15:43:00 +02:00
|
|
|
commit = lookup_commit(o->sha1);
|
|
|
|
commit->util = (void *)argv[i];
|
|
|
|
remotes = &commit_list_insert(commit, remotes)->next;
|
2008-07-07 19:24:20 +02:00
|
|
|
|
|
|
|
strbuf_addf(&buf, "GITHEAD_%s", sha1_to_hex(o->sha1));
|
|
|
|
setenv(buf.buf, argv[i], 1);
|
|
|
|
strbuf_reset(&buf);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!use_strategies) {
|
|
|
|
if (!remoteheads->next)
|
|
|
|
add_strategies(pull_twohead, DEFAULT_TWOHEAD);
|
|
|
|
else
|
|
|
|
add_strategies(pull_octopus, DEFAULT_OCTOPUS);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; i < use_strategies_nr; i++) {
|
|
|
|
if (use_strategies[i]->attr & NO_FAST_FORWARD)
|
|
|
|
allow_fast_forward = 0;
|
|
|
|
if (use_strategies[i]->attr & NO_TRIVIAL)
|
|
|
|
allow_trivial = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!remoteheads->next)
|
|
|
|
common = get_merge_bases(lookup_commit(head),
|
|
|
|
remoteheads->item, 1);
|
|
|
|
else {
|
|
|
|
struct commit_list *list = remoteheads;
|
|
|
|
commit_list_insert(lookup_commit(head), &list);
|
|
|
|
common = get_octopus_merge_bases(list);
|
|
|
|
free(list);
|
|
|
|
}
|
|
|
|
|
|
|
|
update_ref("updating ORIG_HEAD", "ORIG_HEAD", head, NULL, 0,
|
|
|
|
DIE_ON_ERR);
|
|
|
|
|
|
|
|
if (!common)
|
|
|
|
; /* No common ancestors found. We need a real merge. */
|
|
|
|
else if (!remoteheads->next && !common->next &&
|
|
|
|
common->item == remoteheads->item) {
|
|
|
|
/*
|
|
|
|
* If head can reach all the merge then we are up to date.
|
|
|
|
* but first the most common case of merging one remote.
|
|
|
|
*/
|
|
|
|
finish_up_to_date("Already up-to-date.");
|
|
|
|
return 0;
|
|
|
|
} else if (allow_fast_forward && !remoteheads->next &&
|
|
|
|
!common->next &&
|
|
|
|
!hashcmp(common->item->object.sha1, head)) {
|
|
|
|
/* Again the most common case of merging one remote. */
|
2008-10-09 21:12:12 +02:00
|
|
|
struct strbuf msg = STRBUF_INIT;
|
2008-07-07 19:24:20 +02:00
|
|
|
struct object *o;
|
|
|
|
char hex[41];
|
|
|
|
|
|
|
|
strcpy(hex, find_unique_abbrev(head, DEFAULT_ABBREV));
|
|
|
|
|
2008-11-15 01:14:24 +01:00
|
|
|
if (verbosity >= 0)
|
|
|
|
printf("Updating %s..%s\n",
|
|
|
|
hex,
|
|
|
|
find_unique_abbrev(remoteheads->item->object.sha1,
|
|
|
|
DEFAULT_ABBREV));
|
2008-07-07 19:24:20 +02:00
|
|
|
strbuf_addstr(&msg, "Fast forward");
|
|
|
|
if (have_message)
|
|
|
|
strbuf_addstr(&msg,
|
|
|
|
" (no commit created; -m option ignored)");
|
|
|
|
o = peel_to_type(sha1_to_hex(remoteheads->item->object.sha1),
|
|
|
|
0, NULL, OBJ_COMMIT);
|
|
|
|
if (!o)
|
|
|
|
return 1;
|
|
|
|
|
|
|
|
if (checkout_fast_forward(head, remoteheads->item->object.sha1))
|
|
|
|
return 1;
|
|
|
|
|
|
|
|
finish(o->sha1, msg.buf);
|
|
|
|
drop_save();
|
|
|
|
return 0;
|
|
|
|
} else if (!remoteheads->next && common->next)
|
|
|
|
;
|
|
|
|
/*
|
|
|
|
* We are not doing octopus and not fast forward. Need
|
|
|
|
* a real merge.
|
|
|
|
*/
|
|
|
|
else if (!remoteheads->next && !common->next && option_commit) {
|
|
|
|
/*
|
|
|
|
* We are not doing octopus, not fast forward, and have
|
|
|
|
* only one common.
|
|
|
|
*/
|
|
|
|
refresh_cache(REFRESH_QUIET);
|
|
|
|
if (allow_trivial) {
|
|
|
|
/* See if it is really trivial. */
|
|
|
|
git_committer_info(IDENT_ERROR_ON_NO_NAME);
|
|
|
|
printf("Trying really trivial in-index merge...\n");
|
|
|
|
if (!read_tree_trivial(common->item->object.sha1,
|
|
|
|
head, remoteheads->item->object.sha1))
|
|
|
|
return merge_trivial();
|
|
|
|
printf("Nope.\n");
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* An octopus. If we can reach all the remote we are up
|
|
|
|
* to date.
|
|
|
|
*/
|
|
|
|
int up_to_date = 1;
|
|
|
|
struct commit_list *j;
|
|
|
|
|
|
|
|
for (j = remoteheads; j; j = j->next) {
|
|
|
|
struct commit_list *common_one;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Here we *have* to calculate the individual
|
|
|
|
* merge_bases again, otherwise "git merge HEAD^
|
|
|
|
* HEAD^^" would be missed.
|
|
|
|
*/
|
|
|
|
common_one = get_merge_bases(lookup_commit(head),
|
|
|
|
j->item, 1);
|
|
|
|
if (hashcmp(common_one->item->object.sha1,
|
|
|
|
j->item->object.sha1)) {
|
|
|
|
up_to_date = 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (up_to_date) {
|
|
|
|
finish_up_to_date("Already up-to-date. Yeeah!");
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/* We are going to make a new commit. */
|
|
|
|
git_committer_info(IDENT_ERROR_ON_NO_NAME);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* At this point, we need a real merge. No matter what strategy
|
|
|
|
* we use, it would operate on the index, possibly affecting the
|
|
|
|
* working tree, and when resolved cleanly, have the desired
|
|
|
|
* tree in the index -- this means that the index must be in
|
|
|
|
* sync with the head commit. The strategies are responsible
|
|
|
|
* to ensure this.
|
|
|
|
*/
|
|
|
|
if (use_strategies_nr != 1) {
|
|
|
|
/*
|
|
|
|
* Stash away the local changes so that we can try more
|
|
|
|
* than one.
|
|
|
|
*/
|
|
|
|
save_state();
|
|
|
|
} else {
|
|
|
|
memcpy(stash, null_sha1, 20);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; i < use_strategies_nr; i++) {
|
|
|
|
int ret;
|
|
|
|
if (i) {
|
|
|
|
printf("Rewinding the tree to pristine...\n");
|
|
|
|
restore_state();
|
|
|
|
}
|
|
|
|
if (use_strategies_nr != 1)
|
|
|
|
printf("Trying merge strategy %s...\n",
|
|
|
|
use_strategies[i]->name);
|
|
|
|
/*
|
|
|
|
* Remember which strategy left the state in the working
|
|
|
|
* tree.
|
|
|
|
*/
|
|
|
|
wt_strategy = use_strategies[i]->name;
|
|
|
|
|
|
|
|
ret = try_merge_strategy(use_strategies[i]->name,
|
|
|
|
common, head_arg);
|
|
|
|
if (!option_commit && !ret) {
|
|
|
|
merge_was_ok = 1;
|
|
|
|
/*
|
|
|
|
* This is necessary here just to avoid writing
|
|
|
|
* the tree, but later we will *not* exit with
|
|
|
|
* status code 1 because merge_was_ok is set.
|
|
|
|
*/
|
|
|
|
ret = 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (ret) {
|
|
|
|
/*
|
|
|
|
* The backend exits with 1 when conflicts are
|
|
|
|
* left to be resolved, with 2 when it does not
|
|
|
|
* handle the given merge at all.
|
|
|
|
*/
|
|
|
|
if (ret == 1) {
|
|
|
|
int cnt = evaluate_result();
|
|
|
|
|
|
|
|
if (best_cnt <= 0 || cnt <= best_cnt) {
|
|
|
|
best_strategy = use_strategies[i]->name;
|
|
|
|
best_cnt = cnt;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (merge_was_ok)
|
|
|
|
break;
|
|
|
|
else
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Automerge succeeded. */
|
|
|
|
write_tree_trivial(result_tree);
|
|
|
|
automerge_was_ok = 1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If we have a resulting tree, that means the strategy module
|
|
|
|
* auto resolved the merge cleanly.
|
|
|
|
*/
|
|
|
|
if (automerge_was_ok)
|
|
|
|
return finish_automerge(common, result_tree, wt_strategy);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Pick the result from the best strategy and have the user fix
|
|
|
|
* it up.
|
|
|
|
*/
|
|
|
|
if (!best_strategy) {
|
|
|
|
restore_state();
|
|
|
|
if (use_strategies_nr > 1)
|
|
|
|
fprintf(stderr,
|
|
|
|
"No merge strategy handled the merge.\n");
|
|
|
|
else
|
|
|
|
fprintf(stderr, "Merge with strategy %s failed.\n",
|
|
|
|
use_strategies[0]->name);
|
|
|
|
return 2;
|
|
|
|
} else if (best_strategy == wt_strategy)
|
|
|
|
; /* We already have its result in the working tree. */
|
|
|
|
else {
|
|
|
|
printf("Rewinding the tree to pristine...\n");
|
|
|
|
restore_state();
|
|
|
|
printf("Using the %s to prepare resolving by hand.\n",
|
|
|
|
best_strategy);
|
|
|
|
try_merge_strategy(best_strategy, common, head_arg);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (squash)
|
|
|
|
finish(NULL, NULL);
|
|
|
|
else {
|
|
|
|
int fd;
|
|
|
|
struct commit_list *j;
|
|
|
|
|
|
|
|
for (j = remoteheads; j; j = j->next)
|
|
|
|
strbuf_addf(&buf, "%s\n",
|
|
|
|
sha1_to_hex(j->item->object.sha1));
|
|
|
|
fd = open(git_path("MERGE_HEAD"), O_WRONLY | O_CREAT, 0666);
|
|
|
|
if (fd < 0)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not open '%s' for writing",
|
|
|
|
git_path("MERGE_HEAD"));
|
2008-07-07 19:24:20 +02:00
|
|
|
if (write_in_full(fd, buf.buf, buf.len) != buf.len)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not write to '%s'", git_path("MERGE_HEAD"));
|
2008-07-07 19:24:20 +02:00
|
|
|
close(fd);
|
|
|
|
strbuf_addch(&merge_msg, '\n');
|
|
|
|
fd = open(git_path("MERGE_MSG"), O_WRONLY | O_CREAT, 0666);
|
|
|
|
if (fd < 0)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not open '%s' for writing",
|
|
|
|
git_path("MERGE_MSG"));
|
2008-07-07 19:24:20 +02:00
|
|
|
if (write_in_full(fd, merge_msg.buf, merge_msg.len) !=
|
|
|
|
merge_msg.len)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not write to '%s'", git_path("MERGE_MSG"));
|
2008-07-07 19:24:20 +02:00
|
|
|
close(fd);
|
builtin-commit: use reduce_heads() only when appropriate
Since commit 6bb6b034 (builtin-commit: use commit_tree(), 2008-09-10),
builtin-commit performs a reduce_heads() unconditionally. However,
it's not always needed, and in some cases even harmful.
reduce_heads() is not needed for the initial commit or for an
"ordinary" commit, because they don't have any or have only one
parent, respectively.
reduce_heads() must be avoided when 'git commit' is run after a 'git
merge --no-ff --no-commit', otherwise it will turn the
non-fast-forward merge into fast-forward. For the same reason,
reduce_heads() must be avoided when amending such a merge commit.
To resolve this issue, 'git merge' will write info about whether
fast-forward is allowed or not to $GIT_DIR/MERGE_MODE. Based on this
info, 'git commit' will only perform reduce_heads() when it's
committing a merge and fast-forward is enabled.
Also add test cases to ensure that non-fast-forward merges are
committed and amended properly.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
2008-10-03 14:04:47 +02:00
|
|
|
fd = open(git_path("MERGE_MODE"), O_WRONLY | O_CREAT | O_TRUNC, 0666);
|
|
|
|
if (fd < 0)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not open '%s' for writing",
|
|
|
|
git_path("MERGE_MODE"));
|
builtin-commit: use reduce_heads() only when appropriate
Since commit 6bb6b034 (builtin-commit: use commit_tree(), 2008-09-10),
builtin-commit performs a reduce_heads() unconditionally. However,
it's not always needed, and in some cases even harmful.
reduce_heads() is not needed for the initial commit or for an
"ordinary" commit, because they don't have any or have only one
parent, respectively.
reduce_heads() must be avoided when 'git commit' is run after a 'git
merge --no-ff --no-commit', otherwise it will turn the
non-fast-forward merge into fast-forward. For the same reason,
reduce_heads() must be avoided when amending such a merge commit.
To resolve this issue, 'git merge' will write info about whether
fast-forward is allowed or not to $GIT_DIR/MERGE_MODE. Based on this
info, 'git commit' will only perform reduce_heads() when it's
committing a merge and fast-forward is enabled.
Also add test cases to ensure that non-fast-forward merges are
committed and amended properly.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
2008-10-03 14:04:47 +02:00
|
|
|
strbuf_reset(&buf);
|
|
|
|
if (!allow_fast_forward)
|
|
|
|
strbuf_addf(&buf, "no-ff");
|
|
|
|
if (write_in_full(fd, buf.buf, buf.len) != buf.len)
|
2009-06-27 17:58:47 +02:00
|
|
|
die_errno("Could not write to '%s'", git_path("MERGE_MODE"));
|
builtin-commit: use reduce_heads() only when appropriate
Since commit 6bb6b034 (builtin-commit: use commit_tree(), 2008-09-10),
builtin-commit performs a reduce_heads() unconditionally. However,
it's not always needed, and in some cases even harmful.
reduce_heads() is not needed for the initial commit or for an
"ordinary" commit, because they don't have any or have only one
parent, respectively.
reduce_heads() must be avoided when 'git commit' is run after a 'git
merge --no-ff --no-commit', otherwise it will turn the
non-fast-forward merge into fast-forward. For the same reason,
reduce_heads() must be avoided when amending such a merge commit.
To resolve this issue, 'git merge' will write info about whether
fast-forward is allowed or not to $GIT_DIR/MERGE_MODE. Based on this
info, 'git commit' will only perform reduce_heads() when it's
committing a merge and fast-forward is enabled.
Also add test cases to ensure that non-fast-forward merges are
committed and amended properly.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
2008-10-03 14:04:47 +02:00
|
|
|
close(fd);
|
2008-07-07 19:24:20 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
if (merge_was_ok) {
|
|
|
|
fprintf(stderr, "Automatic merge went well; "
|
|
|
|
"stopped before committing as requested\n");
|
|
|
|
return 0;
|
|
|
|
} else
|
|
|
|
return suggest_conflicts();
|
|
|
|
}
|