From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 4 Sep 2010 05:23:49 +0000 (-0700)
Subject: Merge branch 'dg/local-mod-error-messages'
X-Git-Tag: v1.7.3-rc0~5
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=c208e05bd9880028c980fafbc5dda3d17a603ac1;p=git.git

Merge branch 'dg/local-mod-error-messages'

* dg/local-mod-error-messages:
  t7609-merge-co-error-msgs: test non-fast forward case too.
  Move "show_all_errors = 1" to setup_unpack_trees_porcelain()
  setup_unpack_trees_porcelain: take the whole options struct as parameter
  Move set_porcelain_error_msgs to unpack-trees.c and rename it

Conflicts:
	merge-recursive.c
---

c208e05bd9880028c980fafbc5dda3d17a603ac1
diff --cc merge-recursive.c
index aadd48c4f,084f54b88..20e177942
--- a/merge-recursive.c
+++ b/merge-recursive.c
@@@ -1230,108 -1178,6 +1230,64 @@@ static int process_entry(struct merge_o
  	return clean_merge;
  }
  
 +/*
 + * Per entry merge function for D/F conflicts, to be called only after
 + * all files below dir have been processed.  We do this because in the
 + * cases we can cleanly resolve D/F conflicts, process_entry() can clean
 + * out all the files below the directory for us.
 + */
 +static int process_df_entry(struct merge_options *o,
 +			 const char *path, struct stage_data *entry)
 +{
 +	int clean_merge = 1;
 +	unsigned o_mode = entry->stages[1].mode;
 +	unsigned a_mode = entry->stages[2].mode;
 +	unsigned b_mode = entry->stages[3].mode;
 +	unsigned char *o_sha = stage_sha(entry->stages[1].sha, o_mode);
 +	unsigned char *a_sha = stage_sha(entry->stages[2].sha, a_mode);
 +	unsigned char *b_sha = stage_sha(entry->stages[3].sha, b_mode);
 +	const char *add_branch;
 +	const char *other_branch;
 +	unsigned mode;
 +	const unsigned char *sha;
 +	const char *conf;
 +	struct stat st;
 +
 +	/* We currently only handle D->F cases */
 +	assert((!o_sha && a_sha && !b_sha) ||
 +	       (!o_sha && !a_sha && b_sha));
 +
 +	entry->processed = 1;
 +
 +	if (a_sha) {
 +		add_branch = o->branch1;
 +		other_branch = o->branch2;
 +		mode = a_mode;
 +		sha = a_sha;
 +		conf = "file/directory";
 +	} else {
 +		add_branch = o->branch2;
 +		other_branch = o->branch1;
 +		mode = b_mode;
 +		sha = b_sha;
 +		conf = "directory/file";
 +	}
 +	if (lstat(path, &st) == 0 && S_ISDIR(st.st_mode)) {
 +		const char *new_path = unique_path(o, path, add_branch);
 +		clean_merge = 0;
 +		output(o, 1, "CONFLICT (%s): There is a directory with name %s in %s. "
 +		       "Adding %s as %s",
 +		       conf, path, other_branch, path, new_path);
 +		remove_file(o, 0, path, 0);
 +		update_file(o, 0, sha, mode, new_path);
 +	} else {
 +		output(o, 2, "Adding %s", path);
 +		update_file(o, 1, sha, mode, path);
 +	}
 +
 +	return clean_merge;
 +}
 +
- void set_porcelain_error_msgs(const char **msgs, const char *cmd)
- {
- 	const char *msg;
- 	char *tmp;
- 	const char *cmd2 = strcmp(cmd, "checkout") ? cmd : "switch branches";
- 	if (advice_commit_before_merge)
- 		msg = "Your local changes to the following files would be overwritten by %s:\n%%s"
- 			"Please, commit your changes or stash them before you can %s.";
- 	else
- 		msg = "Your local changes to the following files would be overwritten by %s:\n%%s";
- 	tmp = xmalloc(strlen(msg) + strlen(cmd) + strlen(cmd2) - 2);
- 	sprintf(tmp, msg, cmd, cmd2);
- 	msgs[ERROR_WOULD_OVERWRITE] = tmp;
- 	msgs[ERROR_NOT_UPTODATE_FILE] = tmp;
- 
- 	msgs[ERROR_NOT_UPTODATE_DIR] =
- 		"Updating the following directories would lose untracked files in it:\n%s";
- 
- 	if (advice_commit_before_merge)
- 		msg = "The following untracked working tree files would be %s by %s:\n%%s"
- 			"Please move or remove them before you can %s.";
- 	else
- 		msg = "The following untracked working tree files would be %s by %s:\n%%s";
- 	tmp = xmalloc(strlen(msg) + strlen(cmd) + strlen("removed") + strlen(cmd2) - 4);
- 	sprintf(tmp, msg, "removed", cmd, cmd2);
- 	msgs[ERROR_WOULD_LOSE_UNTRACKED_REMOVED] = tmp;
- 	tmp = xmalloc(strlen(msg) + strlen(cmd) + strlen("overwritten") + strlen(cmd2) - 4);
- 	sprintf(tmp, msg, "overwritten", cmd, cmd2);
- 	msgs[ERROR_WOULD_LOSE_UNTRACKED_OVERWRITTEN] = tmp;
- 
- 	/*
- 	 * Special case: ERROR_BIND_OVERLAP refers to a pair of paths, we
- 	 * cannot easily display it as a list.
- 	 */
- 	msgs[ERROR_BIND_OVERLAP] = "Entry '%s' overlaps with '%s'.  Cannot bind.";
- 
- 	msgs[ERROR_SPARSE_NOT_UPTODATE_FILE] =
- 		"Cannot update sparse checkout: the following entries are not up-to-date:\n%s";
- 	msgs[ERROR_WOULD_LOSE_ORPHANED_OVERWRITTEN] =
- 		"The following Working tree files would be overwritten by sparse checkout update:\n%s";
- 	msgs[ERROR_WOULD_LOSE_ORPHANED_REMOVED] =
- 		"The following Working tree files would be removed by sparse checkout update:\n%s";
- }
- 
  int merge_trees(struct merge_options *o,
  		struct tree *head,
  		struct tree *merge,