Re: [PATCH] package.bbclass: allow using EXCLUDE_FROM_SHLIBS for subpackages

[Andrii Bordunov <aborduno@cisco.com>]

Ping-2. Guys? Anything?


Thank you,
Andrii

On 19.10.16 16:58, Andrii Bordunov wrote:
> Ping? Any comments?
>
>
>
> Thank you,
> Andrii
>
> On 10.10.16 20:02, Andrii Bordunov wrote:
>> Some packages containing shared libraries might be registered
>> as shlib providers when they shouldn't (for example, the lib is for
>> their private use and must not generate any dependency).
>>
>> EXCLUDE_FROM_SHLIBS is targeted at that, but it could be set
>> for entire recipe only.
>>
>> This patch expands EXCLUDE_FROM_SHLIBS usage, so now it's possible
>> to set it in a style similar with RDEPENDS. For example:
>>  EXCLUDE_FROM_SHLIBS_${PN}-ptest = "1"
>>
>> Signed-off-by: Andrii Bordunov <aborduno@cisco.com>
>> ---
>>  meta/classes/package.bbclass | 12 ++++++++++--
>>  1 file changed, 10 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
>> index a6f0a7a..9bf43dc 100644
>> --- a/meta/classes/package.bbclass
>> +++ b/meta/classes/package.bbclass
>> @@ -1499,6 +1499,14 @@ python package_do_shlibs() {
>>      libdir_re = re.compile(".*/%s$" % d.getVar('baselib', True))
>>
>>      packages = d.getVar('PACKAGES', True)
>> +
>> +    shlib_pkgs = []
>> +    for pkg in packages.split():
>> +        if d.getVar('EXCLUDE_FROM_SHLIBS_' + pkg, 0):
>> +            bb.note("not generating shlibs for %s" % pkg)
>> +        else:
>> +            shlib_pkgs.append(pkg)
>> +
>>      targetos = d.getVar('TARGET_OS', True)
>>
>>      workdir = d.getVar('WORKDIR', True)
>> @@ -1614,7 +1622,7 @@ python package_do_shlibs() {
>>      needed = {}
>>      shlib_provider = oe.package.read_shlib_providers(d)
>>
>> -    for pkg in packages.split():
>> +    for pkg in shlib_pkgs:
>>          private_libs = d.getVar('PRIVATE_LIBS_' + pkg, True) or
>> d.getVar('PRIVATE_LIBS', True) or ""
>>          private_libs = private_libs.split()
>>          needs_ldconfig = False
>> @@ -1684,7 +1692,7 @@ python package_do_shlibs() {
>>
>>      libsearchpath = [d.getVar('libdir', True),
>> d.getVar('base_libdir', True)]
>>
>> -    for pkg in packages.split():
>> +    for pkg in shlib_pkgs:
>>          bb.debug(2, "calculating shlib requirements for %s" % pkg)
>>
>>          deps = list()
>>