diff options
author | Leonardo Augusto <leobsd@gmail.com> | 2019-02-08 08:35:12 -0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-02-14 11:20:55 +0000 |
commit | 89d4a8df074598cfb3a76e41db7c45d845afd961 (patch) | |
tree | b4c9587b54b8a076ee5209f1e8ed3917a8851528 /meta/recipes-graphics/menu-cache | |
parent | 5501f12ef605a874479a050c4a391f9610a1cebf (diff) | |
download | poky-89d4a8df074598cfb3a76e41db7c45d845afd961.tar.gz |
scripts/lib/wic/engine: Fix cp's target path for ext* filesystems
Python subprocess' shell=True defaults to /bin/sh[1][2], which
often refers to a POSIX-compliant shell. As the -e flag is
not defined in the POSIX standard[3], some shells may
interpret "-e" as the first argument to the echo command,
causing the rest of the command line to fail.
In this particular case, "echo -e 'cd {}'" is interpreted
as "-e cd {}", which causes the first line of the command
to fail, and causing cp to always place the source file
in the filesystem's root.
Replacing "echo -e" for a printf command makes this
more portable.
This issue only affects "wic cp" for ext* filesystems.
[1] https://docs.python.org/2/library/subprocess.html
[2] https://docs.python.org/3/library/subprocess.html
[3] http://pubs.opengroup.org/onlinepubs/9699919799
(From OE-Core rev: 9a5b703d4e60c3ccdf80b5289b8e6fc91133fcde)
Signed-off-by: Leonardo Augusto <leobsd@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-graphics/menu-cache')
0 files changed, 0 insertions, 0 deletions