Re: git fast-import crashes importing commits with more than one author

From: Dmitrijs Ledkovs <dmitrij.ledkov@ubuntu.com>
Date: May 1, 2010, 17:28 UTC
Message-ID: <y2m86ecb3c71005011028t7f581ce7t98256aa8f8bbb0bf@mail.gmail.com>
In-Reply-To: <7vbpcz5ylz.fsf@alter.siamese.dyndns.org>

On 1 May 2010 18:19, Junio C Hamano <gitster@pobox.com> wrote:

Show 19 quoted lines

> Dmitrijs Ledkovs <dmitrij.ledkov@ubuntu.com> writes:
>
>> On 1 May 2010 17:36, Andreas Schwab <schwab@linux-m68k.org> wrote:
>>> Giedrius Slavinskas <giedrius.slavinskas@gmail.com> writes:
>>>
>>>> git fast-import does not support commits with more than one author. I
>>>> try to import bzr repository and it crashes.
>>>
>>> You can find a patch here:
>>> <http://bazaar.launchpad.net/~schwab-linux-m68k/bzr-fastimport/fastimport.dev/revision/262>
>>>
>>> Andreas.
>>
>> Although this is fine shouldn't we put the additional authors to the
>> signed-off: lines in the git commit message?
>
> If an author didn't sign-off in the log message, a tool shouldn't add it
> automatically.  Doing so makes S-o-b meaningless.
>

Fair enough. I'm justing thinking on how to support round-tripping to bzr with import/export cause there are a few other properties bzr needs to recreate identical repository. Is it more appropriate for the marks format revision (bzr side of it)?