[yocto] perf.bb on poky master branch

[Bruce Ashfield]

On Wed, Sep 4, 2019 at 4:53 AM vygu <vygu at protonmail.ch> wrote:
>
> Hello,
>
> Just to signal there are several mistakes with the recipe "perf.bb" from poky on master (currently: e2bde009b325b40641407ecf50f54ed8c41726d6)
>
> - If "Python 3 is not yet supported" by "perf" sources, it causes an issue during the do_compile task

In master the decision was that If you still have a kernel that
doesn't support python3, you'll need to create your own python2 based
perf recipe. Or create a patch that makes the version of python
configurable (I'm doing something for 4.19 at the moment). But in
master, we need to support the release for a period that is longer
than the python2 EOL, so the python2 dependencies need to go.

> - The last "install" in the do_configure_prepend task about "tools/include/linux/bits.h" does not work with a linux kernel 4.5 to 4.18 because this file does not exist

Again, those are older kernels that we support in master. Which are
the only ones we can test. So the copying of that file was tested
against the versions I could see/support. We can definitely protect
those copies with conditionals, but someone that is actively using a
kernel of that vintage, would be better to send the patches.

> [- The usage of _prepend or _append in this bb file does not help to create a bbappend]

We don't have a lot of choices for this. The perf recipe is unique in
that it used to work directly in the kernel source and it has to
support some undefined set of kernels (hence it can't patch the
source). We could move some of that work to new tasks and have them
run before configure, etc, there just hasn't been a need until now.

As usual, patches are welcome (and/or bugzilla entries), since I'm not
currently the issues it isn't as easy to spin patches.

Cheers,

Bruce

>
>
> Vygu
>
> --
> _______________________________________________
> yocto mailing list
> yocto at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II