On Tue, Sep 20, 2022 at 6:12 PM Richard Purdie
<richard.pur...@linuxfoundation.org> wrote:
>
> With recent kernel changes in stable, the full path to python was being
> encoded into binaries and python install files. Add some workarounds
> for both issues to fix build reproducibility issues.

Ack'd.

Looks good to me. I was looking into this, but had only barely figured
out what was being encoded .. much less fix.

Bruce

>
> Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
> ---
>  meta/recipes-kernel/perf/perf.bb | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/meta/recipes-kernel/perf/perf.bb 
> b/meta/recipes-kernel/perf/perf.bb
> index 9f7c300a40e..31bc04635d8 100644
> --- a/meta/recipes-kernel/perf/perf.bb
> +++ b/meta/recipes-kernel/perf/perf.bb
> @@ -145,6 +145,9 @@ do_install() {
>         # we are checking for this make target to be compatible with older 
> perf versions
>         if ${@bb.utils.contains('PACKAGECONFIG', 'scripting', 'true', 
> 'false', d)} && grep -q install-python_ext ${S}/tools/perf/Makefile*; then
>             oe_runmake DESTDIR=${D} install-python_ext
> +           if [ -e ${D}${libdir}/python*/site-packages/perf-*/SOURCES.txt ]; 
> then
> +               sed -i -e 's#${WORKDIR}##g' 
> ${D}${libdir}/python*/site-packages/perf-*/SOURCES.txt
> +           fi
>         fi
>  }
>
> @@ -245,6 +248,9 @@ do_configure:prepend () {
>          # change the Makefile line to remove everything before 'tools/perf'
>          sed -i -e "s%srcdir_SQ = \$(subst ','\\\'',\$(srcdir))%srcdir_SQ = 
> \$(patsubst \%tools/perf,tools/perf,\$(subst ','\\\'',\$(srcdir)))%g" \
>              ${S}/tools/perf/Makefile.config
> +        # Avoid hardcoded path to python-native
> +        sed -i -e 's#\(PYTHON_WORD := \)$(call shell-wordify,$(PYTHON))#\1 
> python3#g' \
> +            ${S}/tools/perf/Makefile.config
>      fi
>      if [ -e "${S}/tools/perf/tests/Build" ]; then
>          # OUTPUT is the full path, we have python on the path so we remove 
> it from the
> --
> 2.34.1
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#170914): 
https://lists.openembedded.org/g/openembedded-core/message/170914
Mute This Topic: https://lists.openembedded.org/mt/93814862/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to