diff options
author | 2024-11-13 13:05:49 -0800 | |
---|---|---|
committer | 2024-11-13 23:23:11 +0000 | |
commit | 7cd1d422c72f8ae0f0b9d8b312e113c3110c29a7 (patch) | |
tree | c4200a8c6600543d5d18f99e2c3da9ba3986a109 /apex/builder.go | |
parent | 06540bb808debb9efe410e518ce9f420bac9042c (diff) |
Don't use echo -e
A rule that uses echo -e may execute in a dash shell instead of bash
when running on the build servers. The dash echo builtin is posix
compliant, and does not treat -e as an option[1]. Replace all uses
of echo -e with echo if it does not need escape sequences, or printf
if it does.
[1] https://wiki.ubuntu.com/DashAsBinSh#echo
Bug: 378931009
Test: manual
Change-Id: I2974f59ea10a613bc3bd36e3591fd81f38a1769a
Diffstat (limited to 'apex/builder.go')
-rw-r--r-- | apex/builder.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apex/builder.go b/apex/builder.go index b04a9d729..ca1e66d67 100644 --- a/apex/builder.go +++ b/apex/builder.go @@ -196,7 +196,7 @@ var ( Command: `diff --unchanged-group-format='' \` + `--changed-group-format='%<' \` + `${image_content_file} ${allowed_files_file} || (` + - `echo -e "New unexpected files were added to ${apex_module_name}." ` + + `echo "New unexpected files were added to ${apex_module_name}." ` + ` "To fix the build run following command:" && ` + `echo "system/apex/tools/update_allowed_list.sh ${allowed_files_file} ${image_content_file}" && ` + `exit 1); touch ${out}`, |