commit 16ce790443728cc8edaef91b882adefb48e4b273
author Achuith Bhandarkar <achuith@chromium.org> Thu Sep 26 11:17:01 2019 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Sep 30 21:07:26 2019 +0000
tree 46568eb6b0613009bf600c1510909bdb6967d9bf
parent 1a8a7f91acc5164d93743e4435dcdd807beafaee

build_artifact: Fix lint errors.

BUG=chromium:403086
TEST=cros flash --debug --clear-cache ssh://100.107.71.157 xBuddy://remote/chell-release/R79-12539.0.0

Change-Id: Iccc786c0a3418633efa65720e8d944c5aeaeb9ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/1827158
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Sean Abraham <seanabraham@chromium.org>

build_artifact.py

diff --git a/build_artifact.py b/build_artifact.py
index d9ccb16..f5c0148 100755
--- a/build_artifact.py
+++ b/build_artifact.py
@@ -16,6 +16,8 @@
 import subprocess
 import traceback
 
+import six
+
 import artifact_info
 import common_util
 import devserver_constants
@@ -99,6 +101,7 @@
     return str(cls)
 
 
+@six.add_metaclass(ArtifactMeta)
 class Artifact(log_util.Loggable):
   """Wrapper around an artifact to download using a fetcher.
 
@@ -139,8 +142,6 @@
                            marker file.
   """
 
-  __metaclass__ = ArtifactMeta
-
   def __init__(self, name, install_dir, build, install_subdir='',
                is_regex_name=False, optional_name=None):
     """Constructor.
@@ -353,7 +354,7 @@
           self._SaveException(e)
 
           # Convert an unknown exception into an ArtifactDownloadError.
-          if type(e) is ArtifactDownloadError:
+          if isinstance(e, ArtifactDownloadError):
             raise
           else:
             raise ArtifactDownloadError('An error occurred: %s' % e)