Re: [meta-oe][PATCH v2] llvm-common: move cross script install to sysroot preprocess function

[Jonathan Liu <net147@gmail.com>]

On 11/06/2013 5:44 PM, Khem Raj wrote:
> On Sun, Jun 2, 2013 at 1:06 AM, Jonathan Liu <net147@gmail.com> wrote:
>> The llvm-config script doesn't get placed in the target sysroot
>> crossscripts directory otherwise.
>>
>> Signed-off-by: Jonathan Liu <net147@gmail.com>
>> ---
>>   meta-oe/recipes-core/llvm/llvm-common.bb | 6 +++++-
>>   1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta-oe/recipes-core/llvm/llvm-common.bb b/meta-oe/recipes-core/llvm/llvm-common.bb
>> index 612fd34..3a4232f 100644
>> --- a/meta-oe/recipes-core/llvm/llvm-common.bb
>> +++ b/meta-oe/recipes-core/llvm/llvm-common.bb
>> @@ -6,7 +6,11 @@ LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3
>>
>>   SRC_URI = "file://llvm-config"
>>
>> -do_install() {
>> +ALLOW_EMPTY_${PN} = "1"
>> +SYSROOT_PREPROCESS_FUNCS += "llvm_common_sysroot_preprocess"
>> +SYSROOT_PREPROCESS_FUNCS_virtclass-native += ""
>> +
> so you want this to run for target recipe only it seems. above won't
> do it. you could do something like
>
> SYSROOT_PREPROCESS_FUNCS_append_class-target = "
> llvm_common_sysroot_preprocess "
It does run only for the target recipe last I checked. That's why I 
added SYSROOT_PREPROCESS_FUNCS_virtclass-native += "".

Regards,
Jonathan
>> +llvm_common_sysroot_preprocess() {
>>       install -d ${SYSROOT_DESTDIR}${bindir_crossscripts}/
>>       install -m 0755 ${WORKDIR}/llvm-config ${SYSROOT_DESTDIR}${bindir_crossscripts}/
>>   }
>> --
>> 1.8.3
>>