Skip to content

Commit 9c05d88

Browse files
ConorPKeeganadbridge
authored andcommitted
Add app config switch to options.py and make.py
1 parent 9d36809 commit 9c05d88

File tree

3 files changed

+13
-9
lines changed

3 files changed

+13
-9
lines changed

tools/make.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252

5353
if __name__ == '__main__':
5454
# Parse Options
55-
parser = get_default_options_parser()
55+
parser = get_default_options_parser(add_app_config=True)
5656
group = parser.add_mutually_exclusive_group(required=False)
5757
group.add_argument("-p",
5858
type=argparse_many(test_known),
@@ -274,7 +274,8 @@
274274
silent=options.silent,
275275
macros=options.macros,
276276
jobs=options.jobs,
277-
name=options.artifact_name)
277+
name=options.artifact_name,
278+
app_config=options.app_config)
278279
print 'Image: %s'% bin_file
279280

280281
if options.disk:

tools/options.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,11 @@
1818
from tools.toolchains import TOOLCHAINS
1919
from tools.targets import TARGET_NAMES
2020
from tools.utils import argparse_force_uppercase_type, \
21-
argparse_lowercase_hyphen_type, argparse_many
21+
argparse_lowercase_hyphen_type, argparse_many, \
22+
argparse_filestring_type
2223

23-
def get_default_options_parser(add_clean=True, add_options=True):
24+
def get_default_options_parser(add_clean=True, add_options=True,
25+
add_app_config=False):
2426
"""Create a new options parser with the default compiler options added
2527
2628
Keyword arguments:
@@ -78,4 +80,9 @@ def get_default_options_parser(add_clean=True, add_options=True):
7880
'analyze'],
7981
"build option"))
8082

83+
if add_app_config:
84+
parser.add_argument("--app-config", default=None, dest="app_config",
85+
type=argparse_filestring_type,
86+
help="Path of an app configuration file (Default is to look for 'mbed_app.json')")
87+
8188
return parser

tools/test.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
if __name__ == '__main__':
4242
try:
4343
# Parse Options
44-
parser = get_default_options_parser()
44+
parser = get_default_options_parser(add_app_config=True)
4545

4646
parser.add_argument("-D",
4747
action="append",
@@ -94,10 +94,6 @@
9494
default=False,
9595
help="Verbose diagnostic output")
9696

97-
parser.add_argument("--app-config", default=None, dest="app_config",
98-
type=argparse_filestring_type,
99-
help="Path of an app configuration file (Default is to look for 'mbed_app.json')")
100-
10197
options = parser.parse_args()
10298

10399
# Filter tests by path if specified

0 commit comments

Comments
 (0)