ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Unify look of argument errors

pull/2714/head
Jimmy Brisson 2016-08-04 14:48:18 -05:00 committed by Anna Bridge
parent 1f155569dc
commit e9032d8003
4 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/get_config.py
View File

@ -37,7 +37,7 @@ except:
if __name__ == '__main__': if __name__ == '__main__':
# Parse Options # Parse Options
parser = get_default_options_parser(add_clean=False, add_options=False) parser = get_default_options_parser(add_clean=False, add_options=False)
parser.add_argument("--source", dest="source_dir", type=argparse_filestring_type, parser.add_argument("--source", dest="source_dir", type=argparse_filestring_type, required=True,
default=[], help="The source (input) directory", action="append") default=[], help="The source (input) directory", action="append")
parser.add_argument("--prefix", dest="prefix", action="append", parser.add_argument("--prefix", dest="prefix", action="append",
default=[], help="Restrict listing to parameters that have this prefix") default=[], help="Restrict listing to parameters that have this prefix")
@ -48,12 +48,12 @@ if __name__ == '__main__':
# Target # Target
if options.mcu is None : if options.mcu is None :
args_error(parser, "[ERROR] You should specify an MCU") args_error(parser, "argument -m/--mcu is required")
target = options.mcu[0] target = options.mcu[0]
# Toolchain # Toolchain
if options.tool is None: if options.tool is None:
args_error(parser, "[ERROR] You should specify a TOOLCHAIN") args_error(parser, "argument -t/--toolchain is required")
toolchain = options.tool[0] toolchain = options.tool[0]
options.prefix = options.prefix or [""] options.prefix = options.prefix or [""]

4
tools/make.py
View File

@ -207,12 +207,12 @@ if __name__ == '__main__':
# Target # Target
if options.mcu is None : if options.mcu is None :
args_error(parser, "[ERROR] You should specify an MCU") args_error(parser, "argument -m/--mcu is required")
mcu = options.mcu[0] mcu = options.mcu[0]
# Toolchain # Toolchain
if options.tool is None: if options.tool is None:
args_error(parser, "[ERROR] You should specify a TOOLCHAIN") args_error(parser, "argument -t/--toolchain is required")
toolchain = options.tool[0] toolchain = options.tool[0]
if options.color: if options.color:

4
tools/test.py
View File

@ -107,12 +107,12 @@ if __name__ == '__main__':
# Target # Target
if options.mcu is None : if options.mcu is None :
args_error(parser, "[ERROR] You should specify an MCU") args_error(parser, "argument -m/--mcu is required")
mcu = options.mcu[0] mcu = options.mcu[0]
# Toolchain # Toolchain
if options.tool is None: if options.tool is None:
args_error(parser, "[ERROR] You should specify a TOOLCHAIN") args_error(parser, "argument -t/--toolchain is required")
toolchain = options.tool[0] toolchain = options.tool[0]
# Find all tests in the relevant paths # Find all tests in the relevant paths

5
tools/utils.py
View File

@ -282,9 +282,8 @@ def args_error(parser, message):
parser - the ArgumentParser object that parsed the command line parser - the ArgumentParser object that parsed the command line
message - what went wrong message - what went wrong
""" """
print "\n\n%s\n\n" % message parser.error(message)
parser.print_help() sys.exit(2)
sys.exit()
def construct_enum(**enums): def construct_enum(**enums):