[OE-core] [PATCH 1/2] package: Add INHIBIT_PACKAGE_STRIP_FILES to allow files to remain unstripped

[Alejandro Enedino Hernandez Samaniego]

Hey Richard,


Do we expect to backport this to older releases?


Cheers,


Alejandro


On 02/14/2018 07:07 AM, Richard Purdie wrote:
> There are cases where its useful to allow only a select few files
> to be excluded from the package stripping mechanism. Currently
> this isn't possible so add a variable to allow this.
>
> This is to be used sparingly as in general the core code should be
> doing the right thing. This is better than the alternative of leaving the whole
> package unstripped.
>
> Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
> ---
>   meta/classes/package.bbclass | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
> index 6a7f35a..90e5236 100644
> --- a/meta/classes/package.bbclass
> +++ b/meta/classes/package.bbclass
> @@ -946,6 +946,7 @@ python split_and_strip_files () {
>       inodes = {}
>       libdir = os.path.abspath(dvar + os.sep + d.getVar("libdir"))
>       baselibdir = os.path.abspath(dvar + os.sep + d.getVar("base_libdir"))
> +    skipfiles = (d.getVar("INHIBIT_PACKAGE_STRIP_FILES") or "").split()
>       if (d.getVar('INHIBIT_PACKAGE_STRIP') != '1' or \
>               d.getVar('INHIBIT_PACKAGE_DEBUG_SPLIT') != '1'):
>           for root, dirs, files in cpath.walk(dvar):
> @@ -961,6 +962,9 @@ python split_and_strip_files () {
>                   if debugdir and debugdir in os.path.dirname(file[len(dvar):]):
>                       continue
>   
> +                if file in skipfiles:
> +                    continue
> +
>                   try:
>                       ltarget = cpath.realpath(file, dvar, False)
>                       s = cpath.lstat(ltarget)