servo: Merge #6038 - Add --dev option to mach build (from mbrubeck:jinankjain-Bug#5933); r=mbrubeck
Require either `--dev` or `--release`, unless a default build.mode is set in `.servobuild`. This is @jinankjain's patch from #5965, rebased onto master. It is already reviewed, and only waiting for servo/saltfs#28 to be deployed. Source-Repo: https://github.com/servo/servo Source-Revision: c0d8488b719ea72e73e435618e015c2f705853cc
This commit is contained in:
@@ -137,6 +137,20 @@ class MachCommands(CommandBase):
|
||||
opts = params or []
|
||||
features = []
|
||||
|
||||
if not (release or dev):
|
||||
if self.config["build"]["mode"] == "dev":
|
||||
dev = True
|
||||
elif self.config["build"]["mode"] == "release":
|
||||
release = True
|
||||
else:
|
||||
print("Please specify either --dev (-d) for a development")
|
||||
print(" build, or --release (-r) for an optimized build.")
|
||||
sys.exit(1)
|
||||
|
||||
if release and dev:
|
||||
print("Please specify either --dev or --release.")
|
||||
sys.exit(1)
|
||||
|
||||
if release:
|
||||
opts += ["--release"]
|
||||
if target:
|
||||
|
||||
@@ -93,7 +93,7 @@ class CommandBase(object):
|
||||
|
||||
self.config.setdefault("build", {})
|
||||
self.config["build"].setdefault("android", False)
|
||||
|
||||
self.config["build"].setdefault("mode", "")
|
||||
self.config["build"].setdefault("debug-mozjs", False)
|
||||
|
||||
self.config.setdefault("android", {})
|
||||
|
||||
Reference in New Issue
Block a user