[PATCH 10 of 10] manifest: delete manifest.manifest class

Durham Goode durham at fb.com
Fri Nov 11 04:19:30 EST 2016



On 11/10/16 10:54 PM, Martin von Zweigbergk wrote:
> On Wed, Nov 9, 2016 at 11:37 AM, Durham Goode <durham at fb.com> wrote:
>> Also, this series depends on the 7 patch series before. It seems like 1-6 of
>> that series was queued, but #7 has not been yet (though the email says it's
>> to be queued).
>>
>>
>>
>> On 11/9/16 7:35 PM, Durham Goode wrote:
>>> Sorry for the long series, but this is the final series in the manifest
>>> refactor. The last patch deletes the manifest class.  All the patches are
>>> pretty easy too.
> Even after this series, there is still localrepo._constructmanifest()
> that's documented like this:
>
>          # This is a temporary function while we migrate from manifest to
>          # manifestlog. It allows bundlerepo and unionrepo to intercept the
>          # manifest creation.
Yea, there's a couple patches of cleanup left (getting rid of the 
repo->manifestlog->repo cycle, and cleaning up _construct), but this is 
the last series around getting rid of the manifest class.


More information about the Mercurial-devel mailing list