Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools: fix Python 3 deprecation warning in test.py #30208

Closed
wants to merge 2 commits into from
Closed

tools: fix Python 3 deprecation warning in test.py #30208

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
06247bf
tools: fix Python 3 deprecation warning in test.py
Hellzed Nov 1, 2019
fa6ac9b
tools: indent test.py with 2 spaces
Hellzed Nov 1, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 31 additions & 13 deletions tools/test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@


from __future__ import print_function
import imp
import logging
import optparse
import os
Expand All @@ -45,6 +44,30 @@
import errno
import copy


if sys.version_info >= (3, 5):
from importlib import machinery, util
def get_module(name, path):
loader_details = (
machinery.SourceFileLoader,
machinery.SOURCE_SUFFIXES
)
spec = machinery.FileFinder(path, loader_details).find_spec(name)
module = util.module_from_spec(spec)
spec.loader.exec_module(module)
return module
else:
import imp
def get_module(name, path):
file = None
try:
(file, pathname, description) = imp.find_module(name, [path])
return imp.load_module(name, file, pathname, description)
finally:
if file:
file.close()


from io import open
from os.path import join, dirname, abspath, basename, isdir, exists
from datetime import datetime
Expand Down Expand Up @@ -791,18 +814,13 @@ def GetConfiguration(self, context):
if self.is_loaded:
return self.config
self.is_loaded = True
file = None
try:
(file, pathname, description) = imp.find_module('testcfg', [ self.path ])
module = imp.load_module('testcfg', file, pathname, description)
self.config = module.GetConfiguration(context, self.path)
if hasattr(self.config, 'additional_flags'):
self.config.additional_flags += context.node_args
else:
self.config.additional_flags = context.node_args
finally:
if file:
file.close()

module = get_module('testcfg', self.path)
self.config = module.GetConfiguration(context, self.path)
if hasattr(self.config, 'additional_flags'):
self.config.additional_flags += context.node_args
else:
self.config.additional_flags = context.node_args
return self.config

def GetBuildRequirements(self, path, context):
Expand Down