[PATCH 4 of 7] convert: bzr use absolute_import

FUJIWARA Katsunori foozy at lares.dti.ne.jp
Fri Mar 11 03:52:55 EST 2016


At Fri, 11 Mar 2016 03:08:50 +0900,
FUJIWARA Katsunori wrote:
> 
> At Thu, 10 Mar 2016 15:35:28 +0000,
> Pierre-Yves David wrote:
> > 
> > On 03/09/2016 04:25 AM, timeless wrote:
> > > # HG changeset patch
> > > # User timeless <timeless at mozdev.org>
> > > # Date 1456936372 0
> > > #      Wed Mar 02 16:32:52 2016 +0000
> > > # Node ID 2a9a21afd35fbc2cad414078c6ec00ad0157834b
> > > # Parent  81f563d9b69385a37f1f45de5be67340dc87f075
> > > convert: bzr use absolute_import
> > 
> > this one make test-check-module-imports.t unhappy, I've dropped it from 
> > the clowncopter.
> > 
> > Timeless, use the following command to get a non-dead version:
> > 
> >    hg pull -r 5120b551e95f http://hg.netv6.net/marmoute-wip/mercurial/
> > 
> > --- /home/pyd/src/mercurial-dev/tests/test-check-module-imports.t
> > +++ /home/pyd/src/mercurial-dev/tests/test-check-module-imports.t.err
> > @@ -142,5 +142,8 @@
> >   these may expose other cycles.
> > 
> >     $ hg locate 'mercurial/**.py' 'hgext/**.py' | sed 's-\\-/-g' | 
> > python "$import_checker" -
> > +  hgext/convert/bzr.py:29: relative import of stdlib module
> > +  hgext/convert/bzr.py:34: relative import of stdlib module
> > +  hgext/convert/bzr.py:69: relative import of stdlib module
> >     Import cycle: hgext.largefiles.basestore -> 
> > hgext.largefiles.localstore -> hgext.largefiles.basestore
> >     [1]
> > 
> > ERROR: test-check-module-imports.t output changed
> 
> In my environment (Python 2.6.2 on Linux 2.6.32-5),
> test-check-module-import.t reports errors below after this series:
> 
>    $ hg locate 'mercurial/**.py' 'hgext/**.py' | sed 's-\\-/-g' | python "$import_checker" -
> +  hgext/convert/subversion.py:46: relative import of stdlib module
> +  hgext/convert/transport.py:25: stdlib import "svn.client" follows local import: mercurial
> +  hgext/convert/transport.py:26: stdlib import "svn.core" follows local import: mercurial
> +  hgext/convert/transport.py:27: stdlib import "svn.ra" follows local import: mercurial
>    Import cycle: hgext.largefiles.basestore -> hgext.largefiles.localstore -> hgext.largefiles.basestore
>    [1]
> 
> In addition to it, after "convert: subversion use absolute_import"
> change (e5aca467220e on clowncopter), tests for "hg convert svn" below
> fail by aborting "No module named transport!" .
> 
>     test-convert-svn-move.t
>     test-convert-svn-source.t
>     test-convert-svn-encoding.t
>     test-convert-hg-svn.t
>     test-convert-svn-branches.t
>     test-convert-svn-startrev.t
>     test-convert-svn-tags.t
> 
> Does this problem depend on environment ?

I confirmed that these problems are reproduced also with Python 2.7.x.

I'll post patches for them soon.


> > -- 
> > Pierre-Yves David
> > _______________________________________________
> > Mercurial-devel mailing list
> > Mercurial-devel at mercurial-scm.org
> > https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
> 
> ----------------------------------------------------------------------
> [FUJIWARA Katsunori]                             foozy at lares.dti.ne.jp
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
----------------------------------------------------------------------
[FUJIWARA Katsunori]                             foozy at lares.dti.ne.jp


More information about the Mercurial-devel mailing list