Re: [OE-core] [PATCH] llvm: extend llvm-config reproducivility fix to all classes

Richard Purdie
2024-07-16T07:12:37Z

On Mon, 2024-07-15 at 21:36 -0400, Denys Dmytriyenko via
lists.openembedded.org wrote:
> It's not limited to "target", but also happens to e.g. nativesdk:
>
> ERROR: nativesdk-llvm-18.1.8-r0 do_package_qa: QA Issue: File
> /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-
> linux/usr/bin/llvm-config in package nativesdk-llvm contains
> reference to TMPDIR [buildpaths]
>
> Signed-off-by: Denys Dmytriyenko <denis@denix.org>
> ---
>  meta/recipes-devtools/llvm/llvm_18.1.8.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/llvm/llvm_18.1.8.bb
> b/meta/recipes-devtools/llvm/llvm_18.1.8.bb
> index 8dad952923..6afca9c7d7 100644
> --- a/meta/recipes-devtools/llvm/llvm_18.1.8.bb
> +++ b/meta/recipes-devtools/llvm/llvm_18.1.8.bb
> @@ -95,7 +95,7 @@ EXTRA_OECMAKE:append:class-nativesdk = "\
>                   "
>
>  # patch out build host paths for reproducibility
> -do_compile:prepend:class-target() {
> +do_compile:prepend() {
>          sed -i -e "s,${WORKDIR},,g" ${B}/tools/llvm-
> config/BuildVariables.inc
>  }
>

Do we need those buildpaths in llvm-native?

Cheers,

Richard