[OE-core] [PATCH] lvm2.inc: EXTRA_OECONF is been overwrited

[Koen Kooi]

Op 6 jun. 2014, om 11:17 heeft Martin Jansa <martin.jansa at gmail.com> het volgende geschreven:

> On Thu, Jun 05, 2014 at 11:26:17PM -0400, Yao Xinpan wrote:
>> EXTRA_OECONF has been assigned before ‘EXTRA_OECONF = "--with-usrlibdir=${libdir}"’.
>> 
>> So, ‘EXTRA_OECONF = "--with-usrlibdir=${libdir}"’ should be changed to
>> ‘EXTRA_OECONF += "--with-usrlibdir=${libdir}"’.
>> 
>> Signed-off-by: Yao Xinpan <yaoxp at cn.fujitsu.com>
>> ---
>> meta-oe/recipes-support/lvm2/lvm2.inc | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/meta-oe/recipes-support/lvm2/lvm2.inc b/meta-oe/recipes-support/lvm2/lvm2.inc
>> index c6a9ec0..71f782a 100644
>> --- a/meta-oe/recipes-support/lvm2/lvm2.inc
>> +++ b/meta-oe/recipes-support/lvm2/lvm2.inc
>> @@ -47,4 +47,4 @@ FILES_${PN} += "${libdir}/device-mapper/*.so ${base_libdir}/udev"
>> FILES_${PN}-dbg += "${libdir}/device-mapper/.debug"
>> 
>> RDEPENDS_${PN} = "bash"
>> -EXTRA_OECONF = "--with-usrlibdir=${libdir}"
>> +EXTRA_OECONF += "--with-usrlibdir=${libdir}"
> 
> Please send it to openembedded-devel at lists.openembedded.org with right
> subject tag (see README file)


And just collaps the 2 lines into a single EXTRA_OECONF. I don't know how that EXTRA_OECONF below RDEPENDS got through review, but it shouldn't have.