[oe] [meta-oe][PATCH] pam-plugin-ccreds: add recipe

[Richard Leitner]

On 06/04/2018 09:34 PM, Andre McCurdy wrote:
> On Sun, Jun 3, 2018 at 11:24 PM, Richard Leitner
> <richard.leitner at skidata.com> wrote:
>> On 06/01/2018 08:01 PM, Andre McCurdy wrote:
>>> On Fri, Jun 1, 2018 at 10:40 AM, Khem Raj <raj.khem at gmail.com> wrote:
>>>> On 6/1/18 4:41 AM, Richard Leitner wrote:
>>>>> Add version 11 of the pam-plugin-ccreds with the debian patches applied.
>>>>
>>>> I see QA errors like below
>>>>
>>>> ERROR: pam-plugin-ccreds-11-r0 do_package_qa: QA Issue: non
>>>> -dev/-dbg/nativesdk- package contains symlink .so: pam-plugin-ccreds
>>>> path
>>>> '/work/core2-64-bec-linux-musl/pam-plugin-ccreds/11-r0/packages-split/pam-plugin-ccreds/lib/security/pam_ccreds.so'
>>>> [dev-so]
>>
>> Thanks for that information.
>> Do you know if other targets than "linux-musl" are affected?
>>
>>>
>>> According to OE's sanity checks, a .so plug-in should not be a symlink.
>>>
>>> See the do_install_append() in the libcgroup recipe in oe-core for an
>>> example of how to fix the issue.
>>
>> IMHO in this case the better solution is to add the missing targets
>> to the following patch... Or does anybody know how to pass wildcards to
>> string comparisons in AM_CONDITIONAL? Google wasn't very helpful here...
> 
> Untested, but I guess something like this would work:
> 
>   AM_CONDITIONAL(EXTENSION_SO, case "$target_os" in linux*) true;; *)
> false;; esac)

Thank you :-)

> 
> The approach in your patch also looks OK.
> 

Ok. Then I won't send a v3 for that and stay with v2.