[PATCH 1 of 57] use absolute_import

Pierre-Yves David pierre-yves.david at ens-lyon.org
Mon Aug 10 02:04:31 CDT 2015



On 08/08/2015 08:53 PM, Gregory Szorc wrote:
> Pierre-Yves et al,
>
> I have added absolute_import to most mercurial.* modules it can be added
> to without other changes. There are still a number of stragglers due to:
>
> * circular dependencies (not captured by static analysis)
> * sub-package import triggering false positives in the import module checker
> * files importing modules with C extensions
> * me not touching extensions or support scripts
>
> But you have to start somewhere.
>
> hg pull -r 5ab6d3932f39 https://hg.mozilla.org/users/gszorc_mozilla.com/hg
>
> You are probably interesting in ee3d290b4b82::5ab6d3932f39.
>
> These patches may require the demandimport changes I sent to this list.
> Or at least we should queue the demandimport changes before these,
> otherwise performance may regress a bit since demandimport doesn't lazy
> load "from . import X" syntax without the patches. If you just "merge"
> 5ab6d3932f39 to clowncopter, etc, things should just work.

I've pulled 5ab6d3932f39, reviewed the whole stack (the demand import 
part seems fine to me, so let's move forward).

The result is pushed to the clowncopter (hooray?).

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list