Re: [PATCH 4/4] fast-import: only store commit objects

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: May 7, 2013, 04:39 UTC
Message-ID: <alpine.DEB.1.00.1305062332470.8213@s15462909.onlinehome-server.info>
In-Reply-To: <518874A5.5050002@alum.mit.edu>

Hi,

On Tue, 7 May 2013, Michael Haggerty wrote:

> On 05/06/2013 11:04 PM, Felipe Contreras wrote:
> > On Mon, May 6, 2013 at 5:45 AM, Michael Haggerty <mhagger@alum.mit.edu> wrote:
> >> On 05/06/2013 12:32 PM, Thomas Rast wrote:

Show 17 quoted lines

> >> So the proposed change would break a documented use of cvs2git.
> > 
> > It's documented as an alternative. How many people actually use this
> > form over the other? Is there really any advantage? It's possibly that
> > basically nobody is using this form, and there's no benefits.
> 
> You conjectured earlier that nobody uses blob marks, and I provided a
> counterexample.  Then you proposed a workaround that would require
> changes to the cvs2git documentation, and I even explained how your
> proposed workaround is not as flexible as the status quo.  Do you want
> to go through the same argument with every possible user of
> git-fast-import?
> 
> It would be insanity to change the default behavior when a workaround on
> the Git side (namely adding an option that tells git-fast-import *not*
> to emit blob marks) would be quite straightforward to implement and have
> little maintenance cost.

I really wonder how many more counterexamples are required to settle this discussion.

There is no good reason to artificially limit Git's capabilities here, especially when it has been demonstrated that supporting that capability is not only possible, but also outright easy.

Ciao, Dscho