[OE-core] [PATCH 2/3] liburcu: Explicitly add pthread options to cflags

[Khem Raj]

On Fri, Mar 9, 2018 at 7:56 AM, Richard Purdie
<richard.purdie at linuxfoundation.org> wrote:
> On Fri, 2018-03-09 at 03:00 -0800, Khem Raj wrote:
>> Some architectures e.g. riscv gcc does not add -D_REENTRANT
>> when enabling pthreads. Help it here by adding these options
>> while gcc gets fixed
>>
>> Signed-off-by: Khem Raj <raj.khem at gmail.com>
>> ---
>>  meta/recipes-support/liburcu/liburcu_0.10.1.bb | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-support/liburcu/liburcu_0.10.1.bb
>> b/meta/recipes-support/liburcu/liburcu_0.10.1.bb
>> index 0c8cd3571a..a73890af90 100644
>> --- a/meta/recipes-support/liburcu/liburcu_0.10.1.bb
>> +++ b/meta/recipes-support/liburcu/liburcu_0.10.1.bb
>> @@ -15,3 +15,5 @@ SRC_URI[sha256sum] =
>> "9c09220be4435dc27fcd22d291707b94b97f159e0c442fbcd60c168f8f
>>
>>  S = "${WORKDIR}/userspace-rcu-${PV}"
>>  inherit autotools
>> +
>> +CPPFLAGS  += "-pthread -D_REENTRANT"
>
> Should we use a riscv override here so that we have some idea why we're
> doing this and show its not needed for other arches?

we could, its a difference in gcc specs for riscv, it seems a bug to me in gcc.
this change actually provides the options that configure tries to
otherwise guess.