[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