[PATCH 1 of 6 RFC] localrepo: remove all internal uses of localrepo.opener

Augie Fackler raf at durin42.com
Mon Jan 12 16:14:17 CST 2015


On Sat, Jan 10, 2015 at 10:12:52PM -0800, Pierre-Yves David wrote:
>
>
> On 01/10/2015 06:58 PM, Matt Harbison wrote:
> >On Sat, 10 Jan 2015 20:20:50 -0500, Angel Ezquerra
> ><angel.ezquerra at gmail.com> wrote:
> >
> >># HG changeset patch
> >># User Angel Ezquerra <angel.ezquerra at gmail.com>
> >># Date 1420927372 -3600
> >>#      Sat Jan 10 23:02:52 2015 +0100
> >># Node ID 459444107fc272ee7ca15cc9ac72602d60ed364f
> >># Parent  678f53865c6860a950392691814766957ee89316
> >>localrepo: remove all internal uses of localrepo.opener
> >
> >The series LGTM.  The test suite on my Linux VM is happy with all 6
> >applied, with the following skips:
>
> I'm not super eager to drop the opener version yet as this would break
> unsuspicious extension. I would go with a (new-feature) developper warning
> for a few version.

On the other hand, as the likely author of such extensions, I'd be a
huge fan of just ripping the band-aid off now.

>
> --
> Pierre-Yves David
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list