logger: utils: fs: use lazy formatting

iterative · Jan 19, 2020 · 19acded · 19acded
1 parent 65c0c7e
commit 19acded
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/dvc/rwlock.py b/dvc/rwlock.py
@@ -9,7 +9,7 @@
 
 from .exceptions import DvcException
 from .lock import LockError
-from .utils.fs import relpath
+from .utils import relpath
 
 INFO_SCHEMA = {"pid": int, "cmd": str}
 

diff --git a/dvc/utils/fs.py b/dvc/utils/fs.py
@@ -13,7 +13,6 @@
 from dvc.utils import dict_md5
 from dvc.utils import fspath
 from dvc.utils import fspath_py35
-from dvc.utils import relpath
 
 
 logger = logging.getLogger(__name__)
@@ -30,7 +29,7 @@ def fs_copy(src, dst):
 
 def get_inode(path):
     inode = System.inode(path)
-    logger.debug("Path {} inode {}".format(path, inode))
+    logger.debug("Path {} inode {}", path, inode)
     return inode
 
 
@@ -131,10 +130,9 @@ def _chmod(func, p, excinfo):
 
 
 def remove(path):
-    path = fspath_py35(path)
-
-    logger.debug("Removing '{}'".format(relpath(path)))
+    logger.debug("Removing '{}'", path)
 
+    path = fspath_py35(path)
     try:
         if os.path.isdir(path):
             shutil.rmtree(path, onerror=_chmod)