From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 27 Jun 2012 22:30:55 GMT
Subject: Re: [PATCH v2 2/3] fast-import: allow "merge $null_sha1" command
Message-ID: <7v395g75gg.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1340818825-13754-3-git-send-email-divanorama@gmail.com>

Dmitry Ivankov <divanorama@gmail.com> writes:

> "from $null_sha1" and "merge $empty_branch" are already allowed so
> allow "merge $null_sha1" command too.

Would accepting such a "merge oops-do-not-do-anything" allow
exporters' job to be simpler?

Without a convincing "it makes sense to treat this nonsense request
as a no-op" argument, I fail to see why this is a change in the
right direction.  If there are two other nonsense request that
silently become no-op, shouldn't they be diagnosed as bugs in the
input stream, or do these two have valid uses?

Very confused.