diff options
author | Jianxun Zhang <jianxun.zhang@intel.com> | 2013-04-17 15:19:19 -0700 |
---|---|---|
committer | Andrew Boie <andrew.p.boie@intel.com> | 2013-08-27 15:31:13 -0700 |
commit | 098494981d03f47e4358496488c2664cfc655965 (patch) | |
tree | c240a5382692f8f9f8037d3d78d78d00a0ed583a /tools | |
parent | d7a9f708870ebd82bc57e478541068bb7ed987c4 (diff) | |
download | build-098494981d03f47e4358496488c2664cfc655965.zip build-098494981d03f47e4358496488c2664cfc655965.tar.gz build-098494981d03f47e4358496488c2664cfc655965.tar.bz2 |
Fix parsing string parameters in BOARD_MKBOOTIMG_ARGS
The existing logic in common.py breaks string arguments incorrectly:
e.g. --para1 val1 --para2 "val2 is a string" will be output as:
'--para', 'val1, '--para2', 'val2' 'is' 'a' 'string'
This will cause mkbootimg command fails due to the invalid arguments
generated from the wrong parsing.
The patch fixes this issue to get:
'--para', 'val1, '--para2', 'val2 is a string'
Change-Id: Ia34ec357550f11ae9d6adc719d86a0c6a9099fbc
Signed-off-by: Jianxun Zhang <jianxun.zhang@intel.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/releasetools/common.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/releasetools/common.py b/tools/releasetools/common.py index 05c2a4a..f179717 100644 --- a/tools/releasetools/common.py +++ b/tools/releasetools/common.py @@ -309,7 +309,7 @@ def BuildBootableImage(sourcedir, fs_config_file, info_dict=None): args = info_dict.get("mkbootimg_args", None) if args and args.strip(): - cmd.extend(args.split()) + cmd.extend(shlex.split(args)) cmd.extend(["--ramdisk", ramdisk_img.name, "--output", img.name]) |