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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/01/2010 07:28 PM, Dmitrijs Ledkovs wrote:
> On 1 May 2010 18:19, Junio C Hamano<gitster@xxxxxxxxx>  wrote:
>> Dmitrijs Ledkovs<dmitrij.ledkov@xxxxxxxxxx>  writes:
>>
>>> On 1 May 2010 17:36, Andreas Schwab<schwab@xxxxxxxxxxxxxx>  wrote:
>>>> Giedrius Slavinskas<giedrius.slavinskas@xxxxxxxxx>  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)?

Add

Co-authored-by: Some One <some.one@xxxxxxxxxxx>

and parse that when sending back to bzr. That way you get attribution
properly in both tools, in a way.

-- 
Andreas Ericsson                   andreas.ericsson@xxxxxx
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]