Hi Tomasz,

Tomasz Dziendzielski <[email protected]> escreveu no dia
quinta, 11/05/2023 à(s) 09:53:

> Using TOPDIR variable breaks the sstate-cache every time build directory
> changes totally breaking build from sstate-cache among different
> workspaces.
>
> Changing that to TMPDIR that is included in BB_HASHEXCLUDE_COMMON.
>
> Another thing is disabling the UNINATIVE_LOADER, causing sstate-cache
> artifacts not working in different workspaces. On
> populate_sysroot_setscene patchelf-uninative --set-interpreter with
> empty argument is ran which does not change the interpreter path and
> then bazel binary ends up with path to the interpreter that might not
> exist, since the bazel was taken from sstate-cache.
>
> Removing the UNINATIVE_LOADER = "" so that uninative.bbclass can
> correctly replace the interpreter path and make bazel binary usable.
> One could think that it will reintroduce the original issue behind
> disabling uninative, which was some java file corrupted (see commit
> dd7642b), but I think we don't have this problem anymore and also I
> don't think it was the correct solution - since the loader is anyway
> included in the binary, so it wasn't really disabling it, just disabling
> the yocto functionality around uninative. If the error re-occurs I think
> different solution should be found.
>
> Signed-off-by: Tomasz Dziendzielski <[email protected]>
> ---
>  recipes-devtools/bazel/bazel-native_5.3.2.bb | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/recipes-devtools/bazel/bazel-native_5.3.2.bb
> b/recipes-devtools/bazel/bazel-native_5.3.2.bb
> index da13a53..c9e4343 100644
> --- a/recipes-devtools/bazel/bazel-native_5.3.2.bb
> +++ b/recipes-devtools/bazel/bazel-native_5.3.2.bb
> @@ -38,7 +38,7 @@ EXTRA_BAZEL_ARGS = " \
>
>  do_compile[network] = "1"
>  do_compile () {
> -    TMPDIR="${TOPDIR}/bazel" \
> +    TMPDIR="${TMPDIR}/../bazel" \
>

Why not use the something inside the bitbake TMPDIR, if it is temporary the
correct place will be in TMPDIR.

so TMPDIR="${TMPDIR}/bazel" looks better IMO.

Jose


>      VERBOSE=yes \
>      EXTRA_BAZEL_ARGS="${EXTRA_BAZEL_ARGS}" \
>      ./compile.sh
> @@ -49,7 +49,4 @@ do_install () {
>      install -m 0755 ${S}/output/bazel ${D}${bindir}
>  }
>
> -# Explicitly disable uninative
> -UNINATIVE_LOADER = ""
> -
>  inherit native
> --
> 2.40.1
>
>
> 
>
>

-- 
Best regards,

José Quaresma
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#59982): https://lists.yoctoproject.org/g/yocto/message/59982
Mute This Topic: https://lists.yoctoproject.org/mt/98823739/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to