From 2c8a4d332e0e90a9c999e781a68af46ca1988747 Mon Sep 17 00:00:00 2001 From: Tomas Kopecek Date: Jun 01 2017 07:39:23 +0000 Subject: python-modernize -f libmodernize.fixes.fix_int_long_tuple --- diff --git a/builder/kojid b/builder/kojid index d0d1915..2da22fb 100755 --- a/builder/kojid +++ b/builder/kojid @@ -269,7 +269,7 @@ class BuildRoot(object): id_suffix = 'repo' name_prefix = 'Repository for Koji' for dep in self.deps: - if isinstance(dep, (int, long)): + if isinstance(dep, six.integer_types): # dep is a task ID, the url points to the task output directory repo_type = 'task' dep_url = pi.task(dep) @@ -1957,7 +1957,7 @@ class ChainMavenTask(MultiPlatformTask): pkg_to_wrap = params['buildrequires'][0] to_wrap = self.done[pkg_to_wrap] - if isinstance(to_wrap, (int, long)): + if isinstance(to_wrap, six.integer_types): task_to_wrap = self.session.getTaskInfo(to_wrap, request=True) build_to_wrap = None else: diff --git a/hub/kojihub.py b/hub/kojihub.py index 7253b14..0283ee5 100644 --- a/hub/kojihub.py +++ b/hub/kojihub.py @@ -2971,7 +2971,7 @@ def get_tag(tagInfo, strict=False, event=None): 'tag_config.maven_include_all': 'maven_include_all' } clauses = [eventCondition(event, table='tag_config')] - if isinstance(tagInfo, (int, long)): + if isinstance(tagInfo, six.integer_types): clauses.append("tag.id = %(tagInfo)i") elif isinstance(tagInfo, six.string_types): clauses.append("tag.name = %(tagInfo)s") @@ -3199,7 +3199,7 @@ def get_external_repos(info=None, url=None, event=None, queryOpts=None): if info is not None: if isinstance(info, str): clauses.append('name = %(info)s') - elif isinstance(info, (int, long)): + elif isinstance(info, six.integer_types): clauses.append('id = %(info)i') else: raise koji.GenericError('invalid type for lookup: %s' % type(info)) @@ -3638,7 +3638,7 @@ def get_rpm(rpminfo, strict=False, multi=False): ) # we can look up by id or NVRA data = None - if isinstance(rpminfo, (int, long)): + if isinstance(rpminfo, six.integer_types): data = {'id': rpminfo} elif isinstance(rpminfo, str): data = koji.parse_NVRA(rpminfo) @@ -5539,7 +5539,7 @@ def add_external_rpm(rpminfo, external_repo, strict=True): ('arch', six.string_types), ('payloadhash', str), ('size', int), - ('buildtime', (int, long))) + ('buildtime', six.integer_types)) for field, allowed in dtypes: if field not in rpminfo: raise koji.GenericError("%s field missing: %r" % (field, rpminfo)) @@ -9344,7 +9344,7 @@ class RootExports(object): before = calendar.timegm(before.utctimetuple()) elif isinstance(before, (str, six.text_type)): before = koji.util.parseTime(before) - elif isinstance(before, (int, long)): + elif isinstance(before, six.integer_types): pass else: raise koji.GenericError('invalid type for before: %s' % type(before)) @@ -9354,7 +9354,7 @@ class RootExports(object): after = calendar.timegm(after.utctimetuple()) elif isinstance(after, (str, six.text_type)): after = koji.util.parseTime(after) - elif isinstance(after, (int, long)): + elif isinstance(after, six.integer_types): pass else: raise koji.GenericError('invalid type for after: %s' % type(after)) @@ -9443,7 +9443,7 @@ class RootExports(object): def listTagged(self, tag, event=None, inherit=False, prefix=None, latest=False, package=None, owner=None, type=None): """List builds tagged with tag""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) results = readTaggedBuilds(tag, event, inherit=inherit, latest=latest, package=package, owner=owner, type=type) @@ -9454,14 +9454,14 @@ class RootExports(object): def listTaggedRPMS(self, tag, event=None, inherit=False, latest=False, package=None, arch=None, rpmsigs=False, owner=None, type=None): """List rpms and builds within tag""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) return readTaggedRPMS(tag, event=event, inherit=inherit, latest=latest, package=package, arch=arch, rpmsigs=rpmsigs, owner=owner, type=type) def listTaggedArchives(self, tag, event=None, inherit=False, latest=False, package=None, type=None): """List archives and builds within a tag""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): tag = get_tag_id(tag, strict=True) return readTaggedArchives(tag, event=event, inherit=inherit, latest=latest, package=package, type=type) @@ -9618,14 +9618,14 @@ class RootExports(object): def getLatestBuilds(self, tag, event=None, package=None, type=None): """List latest builds for tag (inheritance enabled)""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) return readTaggedBuilds(tag, event, inherit=True, latest=True, package=package, type=type) def getLatestRPMS(self, tag, package=None, arch=None, event=None, rpmsigs=False, type=None): """List latest RPMS for tag (inheritance enabled)""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) return readTaggedRPMS(tag, package=package, arch=arch, event=event, inherit=True, latest=True, rpmsigs=rpmsigs, type=type) @@ -9685,13 +9685,13 @@ class RootExports(object): def getInheritanceData(self, tag, event=None): """Return inheritance data for tag""" - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) return readInheritanceData(tag, event) def setInheritanceData(self, tag, data, clear=False): - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) context.session.assertPerm('admin') @@ -9702,7 +9702,7 @@ class RootExports(object): stops = {} if jumps is None: jumps = {} - if not isinstance(tag, (int, long)): + if not isinstance(tag, six.integer_types): #lookup tag id tag = get_tag_id(tag, strict=True) for mapping in [stops, jumps]: @@ -9729,7 +9729,7 @@ class RootExports(object): - buildroot_id If no build has the given ID, or the build generated no RPMs, an empty list is returned.""" - if not isinstance(build, (int, long)): + if not isinstance(build, six.integer_types): #lookup build id build = self.findBuildID(build, strict=True) return self.listRPMs(buildID=build) @@ -10120,7 +10120,7 @@ class RootExports(object): return taginfo def getRepo(self, tag, state=None, event=None, dist=False): - if isinstance(tag, (int, long)): + if isinstance(tag, six.integer_types): id = tag else: id = get_tag_id(tag, strict=True) @@ -12135,7 +12135,7 @@ class HostExports(object): extra_deps = [] task_deps = {} for dep in extra_deps: - if isinstance(dep, (int, long)): + if isinstance(dep, six.integer_types): task_output = list_task_output(dep, stat=True) for filepath, filestats in six.iteritems(task_output): if os.path.splitext(filepath)[1] in ['.log', '.md5', '.sha1']: diff --git a/koji/__init__.py b/koji/__init__.py index d20840b..0cbe502 100644 --- a/koji/__init__.py +++ b/koji/__init__.py @@ -464,7 +464,7 @@ def encode_int(n): def decode_int(n): """If n is not an integer, attempt to convert it""" - if isinstance(n, (int, long)): + if isinstance(n, six.integer_types): return n #else return int(n) @@ -914,7 +914,7 @@ def get_header_field(hdr, name, src_arch=False): # HACK: workaround for https://bugzilla.redhat.com/show_bug.cgi?id=991329 if result is None: result = [] - elif isinstance(result, (int, long)): + elif isinstance(result, six.integer_types): result = [result] return result