Skip to content

Commit

Permalink
Make "git resolve" take the merge message in $3
Browse files Browse the repository at this point in the history
It used to do "Merge $3" as the message, but that ends up being
inconvenient, and much more easily done inside git-pull-script instead.

This makes the third argument to "git resolve" much easier to explain.
  • Loading branch information
Linus Torvalds committed Jul 9, 2005
1 parent 69a97f1 commit 3ba513c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion git-pull-script
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ git-fetch-script "$merge_repo" "$merge_head" || exit 1
git-resolve-script \
"$(cat "$GIT_DIR"/HEAD)" \
"$(cat "$GIT_DIR"/FETCH_HEAD)" \
"$merge_name"
"Merge $merge_name"
7 changes: 3 additions & 4 deletions git-resolve-script
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

head=$(git-rev-parse --revs-only "$1")
merge=$(git-rev-parse --revs-only "$2")
merge_repo="$3"
merge_msg="$3"

dropheads() {
rm -f -- "$GIT_DIR/MERGE_HEAD" \
Expand All @@ -19,8 +19,8 @@ dropheads() {
# The remote name is just used for the message,
# but we do want it.
#
if [ -z "$head" -o -z "$merge" -o -z "$merge_repo" ]; then
die "git-resolve-script <head> <remote> <merge-repo-name>"
if [ -z "$head" -o -z "$merge" -o -z "$merge_msg" ]; then
die "git-resolve-script <head> <remote> <merge-message>"
fi

dropheads
Expand All @@ -47,7 +47,6 @@ if [ "$common" == "$head" ]; then
fi
echo "Trying to merge $merge into $head"
git-read-tree -u -m $common $head $merge || exit 1
merge_msg="Merge $merge_repo"
result_tree=$(git-write-tree 2> /dev/null)
if [ $? -ne 0 ]; then
echo "Simple merge failed, trying Automatic merge"
Expand Down

0 comments on commit 3ba513c

Please sign in to comment.