Skip to content

Commit

Permalink
git-commit: do not muck with commit message when no_edit is set.
Browse files Browse the repository at this point in the history
Spotted by Linus and Darrin Thompson.  When we took a commit
message from -F <file> with an incomplete line, we appended "git
status" output, which ended up attaching a lone "#" at the end.

We still need the "do we have anything to commit?" check by
running "status" (which has to know what to do in different
cases with -i/-o/-a), but there is no point appending its output
to the proposed commit message given by the user.

Signed-off-by: Junio C Hamano <junkio@cox.net>
  • Loading branch information
Junio C Hamano committed Apr 12, 2006
1 parent 684958a commit 475443c
Showing 1 changed file with 12 additions and 5 deletions.
17 changes: 12 additions & 5 deletions git-commit.sh
Original file line number Diff line number Diff line change
Expand Up @@ -537,7 +537,7 @@ t)
;;
esac

if [ -f "$GIT_DIR/MERGE_HEAD" ]; then
if test -f "$GIT_DIR/MERGE_HEAD" && test -z "$no_edit"; then
echo "#"
echo "# It looks like you may be committing a MERGE."
echo "# If this is not correct, please remove the file"
Expand Down Expand Up @@ -605,16 +605,23 @@ else
current=
fi

{
test -z "$only_include_assumed" || echo "$only_include_assumed"
run_status
} >>"$GIT_DIR"/COMMIT_EDITMSG
if test -z "$no_edit"
then
{
test -z "$only_include_assumed" || echo "$only_include_assumed"
run_status
} >>"$GIT_DIR"/COMMIT_EDITMSG
else
# we need to check if there is anything to commit
run_status >/dev/null
fi
if [ "$?" != "0" -a ! -f "$GIT_DIR/MERGE_HEAD" -a -z "$amend" ]
then
rm -f "$GIT_DIR/COMMIT_EDITMSG"
run_status
exit 1
fi

case "$no_edit" in
'')
case "${VISUAL:-$EDITOR},$TERM" in
Expand Down

0 comments on commit 475443c

Please sign in to comment.