@@ -17,12 +17,12 @@ import sys
import threading
import time
-import builderthread
+from buildman import builderthread
+from buildman import toolchain
import command
import gitutil
import terminal
from terminal import Print
-import toolchain
"""
Theory of Operation
@@ -5,18 +5,18 @@
import multiprocessing
import os
import shutil
+import subprocess
import sys
-import board
-import bsettings
-from builder import Builder
+from buildman import board
+from buildman import bsettings
+from buildman import toolchain
+from buildman.builder import Builder
+import command
import gitutil
import patchstream
import terminal
from terminal import Print
-import toolchain
-import command
-import subprocess
def GetPlural(count):
"""Returns a plural 's' if count is not 1"""
@@ -8,11 +8,12 @@ import sys
import tempfile
import unittest
-import board
-import bsettings
-import cmdline
+from buildman import board
+from buildman import bsettings
+from buildman import cmdline
+from buildman import control
+from buildman import toolchain
import command
-import control
import gitutil
import terminal
import toolchain
@@ -6,6 +6,7 @@
"""See README for more information"""
+import doctest
import multiprocessing
import os
import re
@@ -14,20 +15,19 @@ import unittest
# Bring in the patman libraries
our_path = os.path.dirname(os.path.realpath(__file__))
-sys.path.insert(1, os.path.join(our_path, '../patman'))
+sys.path.insert(1, os.path.join(our_path, '..'))
+sys.path.insert(2, os.path.join(our_path, '../patman'))
# Our modules
-import board
-import bsettings
-import builder
-import checkpatch
-import cmdline
-import control
-import doctest
-import gitutil
+from buildman import board
+from buildman import bsettings
+from buildman import builder
+from buildman import cmdline
+from buildman import control
+from buildman import toolchain
import patchstream
+import gitutil
import terminal
-import toolchain
def RunTests(skip_net_tests):
import func_test
@@ -13,15 +13,15 @@ import unittest
our_path = os.path.dirname(os.path.realpath(__file__))
sys.path.append(os.path.join(our_path, '../patman'))
-import board
-import bsettings
-import builder
-import control
-import command
+from buildman import board
+from buildman import bsettings
+from buildman import builder
+from buildman import control
+from buildman import toolchain
import commit
+import command
import terminal
import test_util
-import toolchain
import tools
use_network = True
@@ -10,7 +10,7 @@ import sys
import tempfile
import urllib.request, urllib.error, urllib.parse
-import bsettings
+from buildman import bsettings
import command
import terminal
import tools
@@ -314,11 +314,11 @@ import tempfile
import threading
import time
-sys.path.append(os.path.join(os.path.dirname(__file__), 'buildman'))
+sys.path.append(os.path.join(os.path.dirname(__file__), '..'))
sys.path.append(os.path.join(os.path.dirname(__file__), 'patman'))
-import bsettings
-import kconfiglib
-import toolchain
+from buildman import bsettings
+from buildman import kconfiglib
+from buildman import toolchain
SHOW_GNU_MAKE = 'scripts/show-gnu-make'
SLEEP_TIME=0.03
At present buildman sets the python path on startup so that it can access the libraries it needs. If we convert to use absolute imports this is not necessary. Move buildman to use absolute imports. Also adjust moveconfig.py too since it uses some buildman modules and cannot work without this. Signed-off-by: Simon Glass <sjg at chromium.org> --- Changes in v2: None tools/buildman/builder.py | 4 ++-- tools/buildman/control.py | 12 ++++++------ tools/buildman/func_test.py | 9 +++++---- tools/buildman/main.py | 20 ++++++++++---------- tools/buildman/test.py | 12 ++++++------ tools/buildman/toolchain.py | 2 +- tools/moveconfig.py | 8 ++++---- 7 files changed, 34 insertions(+), 33 deletions(-)