[OE-core] [PATCH 2/2] populate_sdk_base: ensure that filenames with empty space character are handled
João Henrique Ferreira de Freitas
joaohf at gmail.com
Sat Jul 12 13:20:47 UTC 2014
Hi Otavio
Em 08-07-2014 22:33, Otavio Salvador escreveu:
> Now talking about the patch itself, Joao, did you try to avoid the
> extra fork calling echo? Using '-print0' option in find might help.
Not until now. But I agree about your concern. So, instead of
executable_files=$($SUDO_EXEC find $native_sysroot -type f -perm /111
-exec echo \'{}\' \;)
Its better do:
executable_files=$($SUDO_EXEC find $native_sysroot -type f -perm /111
-printf "'%h/%f' ")
No extra fork was used with -printf "'%h/%f' ".
The argument -print0 is not an option because the output of find command
will be 'path1 path2 path 3' a concatenation of strings not strings
split by space and protected with ' '.
Both results are valid. But the second is nicer. What do you think?
When I did this patch I was not thinking to solve all problems about
spaces in filename. However a deep dig is needed to track all points
that have this issue.
Thanks.
--
João Henrique Ferreira de Freitas - joaohf_at_gmail.com
Campinas-SP-Brasil
More information about the Openembedded-core
mailing list