diff mbox series

mesa: on x64 machines llvm-config --libdir locate libraries

Message ID 20221111080415.1046023-1-vince@underview.tech
State New
Headers show
Series mesa: on x64 machines llvm-config --libdir locate libraries | expand

Commit Message

Vincent Davis Jr Nov. 11, 2022, 8:04 a.m. UTC
mesa's meson requires the llvm-config --libdir command which in turn
returns ${STAGING_EXECPREFIXDIR}/lib, but on an x86_64 bit machine
libs are located at ${STAGING_EXECPREFIXDIR}/lib64. Symlink directories
so that the when llvm-config --libdir is called proper libraries are located.

Signed-off-by: Vincent Davis Jr <vince@underview.tech>
---
 meta/recipes-graphics/mesa/mesa.inc | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Alexander Kanavin Nov. 11, 2022, 8:06 a.m. UTC | #1
This seems like brittle special-casing for a specific target. Can we
fix the issue in llvm-config itself, so that it returns the correct
path?

Alex


On Fri, 11 Nov 2022 at 09:04, Vincent Davis Jr <vince@underview.tech> wrote:
>
> mesa's meson requires the llvm-config --libdir command which in turn
> returns ${STAGING_EXECPREFIXDIR}/lib, but on an x86_64 bit machine
> libs are located at ${STAGING_EXECPREFIXDIR}/lib64. Symlink directories
> so that the when llvm-config --libdir is called proper libraries are located.
>
> Signed-off-by: Vincent Davis Jr <vince@underview.tech>
> ---
>  meta/recipes-graphics/mesa/mesa.inc | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
> index 77f9a5c81d..76247932ec 100644
> --- a/meta/recipes-graphics/mesa/mesa.inc
> +++ b/meta/recipes-graphics/mesa/mesa.inc
> @@ -72,6 +72,14 @@ do_configure:prepend () {
>         fi
>  }
>
> +do_configure:prepend:x86-64:class-target () {
> +       # mesa's meson requires the llvm-config --libdir command which in turn
> +       # returns ${STAGING_EXECPREFIXDIR}/lib, but on an x86_64 bit machine
> +       # libs are located at ${STAGING_EXECPREFIXDIR}/lib64. Symlink directories
> +       # so that the when llvm-config --libdir is called proper libraries are located.
> +       ln -fs ${STAGING_LIBDIR} ${STAGING_EXECPREFIXDIR}/lib
> +}
> +
>  # set the MESA_BUILD_TYPE to either 'release' (default) or 'debug'
>  # by default the upstream mesa sources build a debug release
>  # here we assume the user will want a release build by default
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#173120): https://lists.openembedded.org/g/openembedded-core/message/173120
> Mute This Topic: https://lists.openembedded.org/mt/94954642/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Vincent Davis Jr Nov. 11, 2022, 8:24 a.m. UTC | #2
Yes, can check if possible later.
diff mbox series

Patch

diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
index 77f9a5c81d..76247932ec 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -72,6 +72,14 @@  do_configure:prepend () {
 	fi
 }
 
+do_configure:prepend:x86-64:class-target () {
+	# mesa's meson requires the llvm-config --libdir command which in turn
+	# returns ${STAGING_EXECPREFIXDIR}/lib, but on an x86_64 bit machine
+	# libs are located at ${STAGING_EXECPREFIXDIR}/lib64. Symlink directories
+	# so that the when llvm-config --libdir is called proper libraries are located.
+	ln -fs ${STAGING_LIBDIR} ${STAGING_EXECPREFIXDIR}/lib
+}
+
 # set the MESA_BUILD_TYPE to either 'release' (default) or 'debug'
 # by default the upstream mesa sources build a debug release
 # here we assume the user will want a release build by default