pep8

Author: zworkb

pythonforandroid/bdistapk.py

@@ -21,7 +21,7 @@ class Bdist(Command):
 
     @property
     def package_type(self):
-        raise NotImplemented("Subclass must define package_type")
+        raise NotImplementedError("Subclass must define package_type")
 
     def initialize_options(self):
         for option in self.user_options:

pythonforandroid/toolchain.py

@@ -483,7 +483,7 @@ def add_parser(subparsers, *args, **kwargs):
             help='Symlink the dist instead of copying')
 
         parser_packaging = argparse.ArgumentParser(
-            parents = [generic_parser],
+            parents=[generic_parser],
             add_help=False,
             description='common options for packaging (apk, aar)')
 
@@ -528,12 +528,12 @@ def add_parser(subparsers, *args, **kwargs):
             '--signkeypw', dest='signkeypw', action='store', default=None,
             help='Password for key alias')
 
-        parser_aar = add_parser(
+        add_parser(
             subparsers,
             'aar', help='Build an AAR',
             parents=[parser_packaging])
 
-        parser_apk = add_parser(
+        add_parser(
             subparsers,
             'apk', help='Build an APK',
             parents=[parser_packaging])
@@ -993,7 +993,7 @@ def _build_package(self, args, package_type):
             self.hook("before_apk_build")
             os.environ["ANDROID_API"] = str(self.ctx.android_api)
             build = imp.load_source('build', join(dist.dist_dir, 'build.py'))
-            build_args = build.parse_args(args.unknown_args) # this call triggers build.make_package()
+            build_args = build.parse_args(args.unknown_args)  # this call triggers build.make_package()
 
             self.hook("after_apk_build")
             self.hook("before_apk_assemble")
@@ -1044,7 +1044,7 @@ def _finish_package(self, args, output, build_args, package_type, output_dir):
         :param package_type: one of 'apk', 'aar'
         :param output_dir: where to put the package file
         """
-        dist = self._dist
+
         with current_directory(self._dist.dist_dir):
             package_glob = "*-{}.%s" % package_type
             package_add_version = True