diff options
author | Brian Dolbec <dolsen@gentoo.org> | 2016-04-26 20:12:53 -0700 |
---|---|---|
committer | Brian Dolbec <dolsen@gentoo.org> | 2016-04-28 07:48:47 -0700 |
commit | e5d298ccb05b630f17035f31a79c2fe55b90bf07 (patch) | |
tree | 37beeb23d0352181150b197adf78b5575b212a57 | |
parent | Scanner: remove self.pkgs attribute (diff) | |
download | portage-e5d298cc.tar.gz portage-e5d298cc.tar.bz2 portage-e5d298cc.zip |
repoman/scanner.py: Make some variables local instead of classwide
Some were not needed outside of __init__.
Some are included self.kwargs.
Several classwide variables repolevel, reposplit, categories are accessed
outside of the scanner class. So therefore can not be made local.
-rw-r--r-- | pym/repoman/scanner.py | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/pym/repoman/scanner.py b/pym/repoman/scanner.py index d9665139d..a5c54b15d 100644 --- a/pym/repoman/scanner.py +++ b/pym/repoman/scanner.py @@ -107,13 +107,13 @@ class Scanner(object): self.repo_settings.repoman_settings['PORTAGE_ARCHLIST'] = ' '.join(sorted(kwlist)) self.repo_settings.repoman_settings.backup_changes('PORTAGE_ARCHLIST') - self.profiles = setup_profile(profile_list) + profiles = setup_profile(profile_list) - check_profiles(self.profiles, self.repo_settings.repoman_settings.archlist()) + check_profiles(profiles, self.repo_settings.repoman_settings.archlist()) scanlist = scan(self.repolevel, self.reposplit, startdir, self.categories, self.repo_settings) - self.dev_keywords = dev_profile_keywords(self.profiles) + self.dev_keywords = dev_profile_keywords(profiles) self.qatracker = self.vcs_settings.qatracker @@ -123,7 +123,7 @@ class Scanner(object): if self.vcs_settings.vcs is None: self.options.echangelog = 'n' - self.checks = {} + checks = {} # The --echangelog option causes automatic ChangeLog generation, # which invalidates changelog.ebuildadded and changelog.missing # checks. @@ -135,7 +135,7 @@ class Scanner(object): # TODO: shouldn't this just be switched on the repo, iso the VCS? is_echangelog_enabled = self.options.echangelog in ('y', 'force') self.vcs_settings.vcs_is_cvs_or_svn = self.vcs_settings.vcs in ('cvs', 'svn') - self.checks['changelog'] = not is_echangelog_enabled and self.vcs_settings.vcs_is_cvs_or_svn + checks['changelog'] = not is_echangelog_enabled and self.vcs_settings.vcs_is_cvs_or_svn if self.options.mode == "manifest" or self.options.quiet: pass @@ -170,7 +170,7 @@ class Scanner(object): # Disable the "self.modules['Ebuild'].notadded" check when not in commit mode and # running `svn status` in every package dir will be too expensive. - self.checks['ebuild_notadded'] = not \ + checks['ebuild_notadded'] = not \ (self.vcs_settings.vcs == "svn" and self.repolevel < 3 and self.options.mode != "commit") self.effective_scanlist = scanlist @@ -188,9 +188,9 @@ class Scanner(object): "options": self.options, "metadata_xsd": metadata_xsd, "uselist": uselist, - "checks": self.checks, + "checks": checks, "repo_metadata": self.repo_metadata, - "profiles": self.profiles, + "profiles": profiles, "include_arches": self.include_arches, "caches": self.caches, "repoman_incrementals": self.repoman_incrementals, @@ -291,7 +291,7 @@ class Scanner(object): checkdirlist = os.listdir(checkdir) # Run the status check - if self.checks['ebuild_notadded']: + if self.kwargs['checks']['ebuild_notadded']: self.vcs_settings.status.check(checkdir, checkdir_relative, xpkg) dynamic_data = { @@ -350,7 +350,7 @@ class Scanner(object): ebuildlist = sorted(pkgs.values()) ebuildlist = [pkg.pf for pkg in ebuildlist] - if self.checks['changelog'] and "ChangeLog" not in checkdirlist: + if self.kwargs['checks']['changelog'] and "ChangeLog" not in checkdirlist: self.qatracker.add_error("changelog.missing", xpkg + "/ChangeLog") changelog_path = os.path.join(checkdir_relative, "ChangeLog") |