From patchwork Wed Mar  2 15:51:57 2016
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [7,of,8] convert: p4 use absolute_import
From: timeless@mozdev.org
X-Patchwork-Id: 13545
Message-Id: <6ff6ca04d1ecdb683550.1456933917@waste.org>
To: mercurial-devel@mercurial-scm.org
Date: Wed, 02 Mar 2016 09:51:57 -0600

# HG changeset patch
# User timeless <timeless@mozdev.org>
# Date 1456932675 0
#      Wed Mar 02 15:31:15 2016 +0000
# Node ID 6ff6ca04d1ecdb6835501f553f3559aef6c9bdd7
# Parent  e6d6124d7bdb342016128a649b161e1c15535718
convert: p4 use absolute_import

diff --git a/hgext/convert/p4.py b/hgext/convert/p4.py
--- a/hgext/convert/p4.py
+++ b/hgext/convert/p4.py
@@ -4,13 +4,18 @@
 #
 # This software may be used and distributed according to the terms of the
 # GNU General Public License version 2 or any later version.
+from __future__ import absolute_import
 
-from mercurial import util, error
+import marshal
+import re
+
+from mercurial import (
+    error,
+    util,
+)
 from mercurial.i18n import _
 
-from common import commit, converter_source, checktool, NoRepo
-import marshal
-import re
+from . import common
 
 def loaditer(f):
     "Yield the dictionary objects generated by p4"
@@ -37,7 +42,7 @@
         filename = filename.replace(k, v)
     return filename
 
-class p4_source(converter_source):
+class p4_source(common.converter_source):
     def __init__(self, ui, path, revs=None):
         # avoid import cycle
         import convcmd
@@ -45,9 +50,10 @@
         super(p4_source, self).__init__(ui, path, revs=revs)
 
         if "/" in path and not path.startswith('//'):
-            raise NoRepo(_('%s does not look like a P4 repository') % path)
+            raise common.NoRepo(_('%s does not look like a P4 repository') %
+                                path)
 
-        checktool('p4', abort=False)
+        common.checktool('p4', abort=False)
 
         self.p4changes = {}
         self.heads = {}
@@ -142,10 +148,10 @@
                 parents = []
 
             date = (int(d["time"]), 0)     # timezone not set
-            c = commit(author=self.recode(d["user"]),
-                       date=util.datestr(date, '%Y-%m-%d %H:%M:%S %1%2'),
-                       parents=parents, desc=desc, branch=None,
-                       extra={"p4": change})
+            c = common.commit(author=self.recode(d["user"]),
+                              date=util.datestr(date, '%Y-%m-%d %H:%M:%S %1%2'),
+                              parents=parents, desc=desc, branch=None,
+                              extra={"p4": change})
 
             files = []
             copies = {}
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
@@ -39,7 +39,6 @@
   hgext/convert/convcmd.py not using absolute_import
   hgext/convert/cvs.py not using absolute_import
   hgext/convert/monotone.py not using absolute_import
-  hgext/convert/p4.py not using absolute_import
   hgext/convert/subversion.py not using absolute_import
   hgext/convert/transport.py not using absolute_import
   hgext/eol.py not using absolute_import