Re: [PATCH 1/2] oprofile: Add PTEST regression testing to oprofile

[Chong Lu <Chong.Lu@windriver.com>]

On 06/05/2014 11:21 PM, Otavio Salvador wrote:
> On Thu, Jun 5, 2014 at 5:44 AM, Chong Lu <Chong.Lu@windriver.com> wrote:
>> The commit changes the oe-core oprofile recipe by adding the PTEST
>> interface, implementing do_compile_ptest() and do_install_ptest().
>>
>> The install routine adds several files that only the regression tests
>> require. Also, a patch is necessary to inhibit executing the tests on
>> the build host and to point to the correct target directory for the
>> libutil++ test 'file-manip-tests'.
>>
>> That test requires the recipe to set SRCDIR in the build environment
>> instead of determining the value from build-time host paths as for a
>> self-hosted build.
>>
>> Signed-off-by: Dave Lerner <dave.lerner@windriver.com>
>> Signed-off-by: Chong Lu <Chong.Lu@windriver.com>
>> ---
>>   meta/recipes-kernel/oprofile/oprofile.inc          | 31 ++++++-
>>   .../oprofile/oprofile-cross-compile-tests.patch    | 98 ++++++++++++++++++++++
>>   meta/recipes-kernel/oprofile/oprofile/run-ptest    | 18 ++++
>>   3 files changed, 145 insertions(+), 2 deletions(-)
>>   create mode 100644 meta/recipes-kernel/oprofile/oprofile/oprofile-cross-compile-tests.patch
>>   create mode 100644 meta/recipes-kernel/oprofile/oprofile/run-ptest
>>
>> diff --git a/meta/recipes-kernel/oprofile/oprofile.inc b/meta/recipes-kernel/oprofile/oprofile.inc
>> index ccf9f9f..0a7f7fd 100644
>> --- a/meta/recipes-kernel/oprofile/oprofile.inc
>> +++ b/meta/recipes-kernel/oprofile/oprofile.inc
>> @@ -20,9 +20,11 @@ FILES_${PN}-staticdev += "${libdir}/${BPN}/lib*.a"
>>
>>   SRC_URI = "file://opstart.patch \
>>              file://oprofile-root.patch \
>> -           file://acinclude.m4"
>> +           file://acinclude.m4 \
>> +           file://oprofile-cross-compile-tests.patch \
>> +           file://run-ptest"
>>
>> -inherit autotools-brokensep pkgconfig
>> +inherit autotools-brokensep ptest
> You removed pkgconfig, why?
>
It's my mistake. pkgconfig should not be removed. I will resend V2.

Best Regards
Chong