[PATCH 02 of 16 V3] py3: make largefiles/__init__.py use absolute_import
Augie Fackler
raf at durin42.com
Mon Jun 6 12:33:13 EDT 2016
On Mon, Jun 06, 2016 at 08:36:08AM -0700, Martijn Pieters wrote:
> On 4 June 2016 at 08:23, liscju <piotr.listkiewicz at gmail.com> wrote:
> > # HG changeset patch
> > # User liscju <piotr.listkiewicz at gmail.com>
> > # Date 1462537712 -7200
> > # Fri May 06 14:28:32 2016 +0200
> > # Node ID 8df452a3387dfd47f901665b34b19d1a11cabe7d
> > # Parent b37877a19b851acd4a598f8b8214593651bd9089
> > py3: make largefiles/__init__.py use absolute_import
>
> To reduce noise on the list and to reduce the pressure on reviewers,
> can you please combine these into one patch? All 'absolute_import'
> changes for the largefiles extension really only need one patch, not
> 11.
Yeah, that probably would have been for the best. As it is, they were
quick to work through.
>
>
> > diff --git a/hgext/largefiles/__init__.py b/hgext/largefiles/__init__.py
> > --- a/hgext/largefiles/__init__.py
> > +++ b/hgext/largefiles/__init__.py
> > @@ -104,14 +104,20 @@ largefile. To add the first largefile to
> > explicitly do so with the --large flag passed to the :hg:`add`
> > command.
> > '''
> > -
> > -from mercurial import hg, localrepo
> > +from __future__ import absolute_import
> >
> > -import lfcommands
> > -import proto
> > -import reposetup
> > -import uisetup as uisetupmod
> > -import overrides
> > +from mercurial import (
> > + hg,
> > + localrepo,
> > +)
> > +
> > +from . import (
> > + lfcommands,
> > + overrides,
> > + proto,
> > + reposetup,
> > + uisetup as uisetupmod,
> > +)
> >
> > # Note for extension authors: ONLY specify testedwith = 'internal' for
> > # extensions which SHIP WITH MERCURIAL. Non-mainline extensions should
> > diff --git a/tests/test-check-py3-compat.t b/tests/test-check-py3-compat.t
> > --- a/tests/test-check-py3-compat.t
> > +++ b/tests/test-check-py3-compat.t
> > @@ -9,7 +9,6 @@
> > hgext/fsmonitor/pywatchman/pybser.py not using absolute_import
> > hgext/highlight/__init__.py not using absolute_import
> > hgext/highlight/highlight.py not using absolute_import
> > - hgext/largefiles/__init__.py not using absolute_import
> > hgext/largefiles/basestore.py not using absolute_import
> > hgext/largefiles/lfcommands.py not using absolute_import
> > hgext/largefiles/lfutil.py not using absolute_import
> > _______________________________________________
> > Mercurial-devel mailing list
> > Mercurial-devel at mercurial-scm.org
> > https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
>
>
>
> --
> Martijn Pieters
> _______________________________________________
> 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