[PATCH 6 of 6 v3] convert: Create commits from revmap list if needed

Kostia Balytskyi kobalyts at outlook.com
Thu Dec 8 06:00:12 EST 2016



On 12/8/16 12:18 AM, David Soria Parra wrote:
> On Wed, Dec 07, 2016 at 10:23:08PM +0000, Kostia Balytskyi wrote:
>> On 12/7/16 9:51 PM, David Soria Parra wrote:
>>> # HG changeset patch
>>> # User David Soria Parra <davidsp at fb.com>
>>> # Date 1481143876 28800
>>> #      Wed Dec 07 12:51:16 2016 -0800
>>> # Node ID 109de539306c5bc49d38d6f1c802c4a8d092b485
>>> # Parent  be68e4436851b7e20f3b8cb34666418f5840dd66
>>> convert: Create commits from revmap list if needed
>> This commit demonstrates the need to add some sort of comment to
>> self.revmap = {} in constructor. Otherwise, it's very confusing IMO.
> I honestly don't undersetand what is so confusing about it. It's a
> revmap and it seems straight forward to me that it's a dictionary
> that maps commits, at least thats what all revmaps in convert ever do.
The confusing (at least to me) part is what revmap maps commits to. 
There's no example
of setrevmap usage, so it's unclear how it is used. Feel free to ignore 
this if you think this is unreasonable though.
>
>>> diff --git a/hgext/convert/p4.py b/hgext/convert/p4.py
>>> --- a/hgext/convert/p4.py
>>> +++ b/hgext/convert/p4.py
>>> @@ -314,6 +314,12 @@
>>>            return marshal.load(stdout)
>>>    
>>>        def getcommit(self, rev):
>>> +        if rev not in self.changeset and rev not in self.revmap:
>>> +            raise error.Abort(
>>> +                _("cannot find %s in the revmap or parsed changesets") % rev)
>>> +        if rev not in self.changeset:
>>> +            d = self._fetch_revision(rev)
>>> +            return self._construct_commit(d, parents=None)
>>>            return self.changeset[rev]
>>>    
>>>        def gettags(self):
>>> _______________________________________________
>>> Mercurial-devel mailing list
>>> Mercurial-devel at mercurial-scm.org
>>> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
>> _______________________________________________
>> Mercurial-devel mailing list
>> Mercurial-devel at mercurial-scm.org
>> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel



More information about the Mercurial-devel mailing list