Code

Keep Porcelainish from failing by broken ident after making changes.
authorJunio C Hamano <junkio@cox.net>
Sun, 19 Feb 2006 04:51:26 +0000 (20:51 -0800)
committerJunio C Hamano <junkio@cox.net>
Wed, 22 Feb 2006 21:14:57 +0000 (13:14 -0800)
commitd64e6b04291e2313343866a6b206caf13313f1f9
tree511478e8e11aa4cdf1a9cedcd5000e4ba9960098
parent589e4f93c7fc31d73da3d0764c71d939c9332442
Keep Porcelainish from failing by broken ident after making changes.

"empty ident not allowed" error makes commit-tree fail, so we
are already safer in that we would not end up with commit
objects that have bogus names on the author or committer fields.
However, before commit-tree is called there are already changes
made to the index file and the working tree.  The operation can
be resumed after fixing the environment problem, but when this
triggers to a newcomer with unusable gecos, the first question
becomes "what did I lose and how would I recover".

This patch modifies some Porcelainish commands to verify
GIT_COMMITTER_IDENT as soon as we know we are going to make some
commits before doing much damage to prevent confusion.

Signed-off-by: Junio C Hamano <junkio@cox.net>
git-am.sh
git-applymbox.sh
git-merge.sh
git-resolve.sh