diff options
| author | Sergio Ahumada <sahumada@blackberry.com> | 2015-02-14 11:26:46 +0100 |
|---|---|---|
| committer | Sergio Ahumada <sahumada@texla.cl> | 2015-03-10 12:28:45 +0000 |
| commit | 3cde3fb39489e8e9b4319d69031f27a2d113cc39 (patch) | |
| tree | b887d6c0741e6edc5f70e2516fcf00b49a0e9db5 /packaging-tools/archiveresolver.py | |
| parent | a2f7d382886120158329dba152c23b523159b938 (diff) | |
Clean up unused variables and imports
* remove unused imports
* remove unused variables
* replace unused variables with 'dummy', this is the suggested and
most common way to deal with variables that need to be defined
but are never used and it also makes pylint not to complain, eg:
for root, dir, files in os.walk(directory):
for basename in files:
if fnmatch.fnmatch(basename, '*.pro'):
filename = os.path.join(root, basename)
since 'dirs' is never used, we replace it with 'dummy'
for root, dummy, files in os.walk(directory):
for basename in files:
if fnmatch.fnmatch(basename, '*.pro'):
filename = os.path.join(root, basename)
Change-Id: I850c2c2a8ae60035d0cc5bf6c3429ff8640f9f11
Reviewed-by: Tim Jenssen <tim.jenssen@theqtcompany.com>
Diffstat (limited to 'packaging-tools/archiveresolver.py')
| -rw-r--r-- | packaging-tools/archiveresolver.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/packaging-tools/archiveresolver.py b/packaging-tools/archiveresolver.py index dee6e7260..f70d58825 100644 --- a/packaging-tools/archiveresolver.py +++ b/packaging-tools/archiveresolver.py @@ -89,7 +89,6 @@ class ArchiveLocationResolver: # get packages tempalates src dir first pkg_templates_dir = os.path.normpath(bldinstallercommon.config_section_map(target_config,'PackageTemplates')['template_dirs']) self.pkg_templates_dir_list = pkg_templates_dir.replace(' ', '').rstrip(',\n').split(',') - server_namespace = os.path.normpath(bldinstallercommon.config_section_map(target_config,'PackageTemplates')['template_dirs']) # next read server list if server_base_url_override: server_obj = ArchiveLocationResolver.ArchiveRemoteLocation('default_server_name', server_base_url_override, '') |