Skip to content

Commit

Permalink
Bug 794509 - Part 2: Move mach command modules into a mach.commands s…
Browse files Browse the repository at this point in the history
…ub-module; r=jhammel

This patch on its own will break mach. Part 3 will refactor mach's
loader to discover and load modules using a modified module finding
method.
  • Loading branch information
indygreg committed Oct 5, 2012
1 parent 21c5e50 commit d946522
Show file tree
Hide file tree
Showing 10 changed files with 5 additions and 7 deletions.
1 change: 0 additions & 1 deletion mach
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ our_dir = os.path.dirname(os.path.abspath(__file__))
try:
import mach.main
except ImportError:
SEARCH_PATHS.reverse()
sys.path[0:0] = [os.path.join(our_dir, path) for path in SEARCH_PATHS]

import mach.main
Expand Down
Empty file.
File renamed without changes.
9 changes: 4 additions & 5 deletions python/mach/mach/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,10 @@
# Import sub-command modules
# TODO Bug 794509 do this via auto-discovery. Update README once this is
# done.
from mach.build import Build
from mach.settings import Settings
from mach.testing import Testing
from mach.warnings import Warnings

from mach.commands.build import Build
from mach.commands.settings import Settings
from mach.commands.testing import Testing
from mach.commands.warnings import Warnings

# Classes inheriting from ConfigProvider that provide settings.
# TODO this should come from auto-discovery somehow.
Expand Down
Empty file.
Empty file.
File renamed without changes.
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion python/mozbuild/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,6 @@
name='mozbuild',
description='Mozilla build system functionality.',
license='MPL 2.0',
packages=['mozbuild'],
packages=['mach', 'mozbuild'],
version=VERSION
)

0 comments on commit d946522

Please sign in to comment.