Re: [PATCH v2 3/4] fast-export: don't handle uninteresting refs

On Tue, Oct 30, 2012 at 8:01 PM, Jonathan Nieder [off-list ref] wrote:

Felipe Contreras wrote:

quoted

On Tue, Oct 30, 2012 at 7:47 PM, Jonathan Nieder [off-list ref] wrote:

quoted

quoted

and in the examples listed in the patch
description the changed behavior does not look like an improvement.

I disagree.

% git log master ^master

What do you expect? Nothing.

Yep.

quoted

% git fast-export master ^master

What do you expect? Nothing.

So you think what we have now is the correct behavior:

% git fast-export master ^master
reset refs/heads/master
from :0

That of course would crash fast-import. But hey, it's your opinion.

Would be interesting to see if other people think the above is correct.

Cheers.

-- 
Felipe Contreras