* [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
@ 2025-08-20 13:29 Ross Burton
2025-08-20 13:29 ` [PATCH 2/3] spirv-tools: improve packaging Ross Burton
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Ross Burton @ 2025-08-20 13:29 UTC (permalink / raw)
To: openembedded-core
- There is no need for python3native, as this recipe doesn't depend on
other python modules.
- Inherit lib_package so the llvm-spirv binary is in a separate package
to the libLLVMSPIRVLib.so library.
- Remove obsolete CMake options that no longer appear to be required.
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
index 64a73ce55aa..3a090b7b5cb 100644
--- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
+++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
@@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+(\.\d+)+)"
DEPENDS = "llvm spirv-tools spirv-headers"
-inherit cmake pkgconfig python3native
+inherit cmake pkgconfig lib_package
-# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
-# for CMAKE_SHARED_LIBS=OFF see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
EXTRA_OECMAKE = "\
-DBUILD_SHARED_LIBS=ON \
- -DCMAKE_BUILD_TYPE=Release \
- -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
- -DCMAKE_SKIP_RPATH=ON \
- -DLLVM_EXTERNAL_LIT=lit \
- -DLLVM_INCLUDE_TESTS=ON \
- -Wno-dev \
-DLLVM_EXTERNAL_SPIRV_HEADERS_SOURCE_DIR=${STAGING_INCDIR}/.. \
"
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 2/3] spirv-tools: improve packaging
2025-08-20 13:29 [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Ross Burton
@ 2025-08-20 13:29 ` Ross Burton
2025-08-20 13:29 ` [PATCH 3/3] mesa: clean up libopencl-mesa dependencies Ross Burton
` (2 subsequent siblings)
3 siblings, 0 replies; 10+ messages in thread
From: Ross Burton @ 2025-08-20 13:29 UTC (permalink / raw)
To: openembedded-core
spirv-tools is a set of libraries and some binaries that link to those
libraries, so inherit lib_package so that the binaries and the libraries
are packaged separately.
The lesspipe script hasn't needed bash since 2018[1], so remove the bash
dependency and package the script with the other binaries.
The build type unless specified is Debug, explicitly set RelWithDebInfo.
[1] spirv-tools f70e9e "tools/lesspipe: Allow generic shell (#2255)"
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
meta/recipes-graphics/spir/spirv-tools_1.4.321.0.bb | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/meta/recipes-graphics/spir/spirv-tools_1.4.321.0.bb b/meta/recipes-graphics/spir/spirv-tools_1.4.321.0.bb
index 319b25b7f43..5242473e6bc 100644
--- a/meta/recipes-graphics/spir/spirv-tools_1.4.321.0.bb
+++ b/meta/recipes-graphics/spir/spirv-tools_1.4.321.0.bb
@@ -18,11 +18,12 @@ PE = "1"
# upstream considers a release.
UPSTREAM_CHECK_GITTAGREGEX = "sdk-(?P<pver>\d+(\.\d+)+)"
-inherit cmake
+inherit cmake lib_package
DEPENDS = "spirv-headers"
EXTRA_OECMAKE += "\
+ -DCMAKE_BUILD_TYPE=RelWithDebInfo \
-DSPIRV-Headers_SOURCE_DIR=${STAGING_EXECPREFIXDIR} \
-DSPIRV_TOOLS_BUILD_STATIC=OFF \
-DBUILD_SHARED_LIBS=ON \
@@ -45,8 +46,4 @@ do_install:append:class-target() {
SOLIBS = ".so"
FILES_SOLIBSDEV = ""
-PACKAGES =+ "${PN}-lesspipe"
-FILES:${PN}-lesspipe = "${base_bindir}/spirv-lesspipe.sh"
-RDEPENDS:${PN}-lesspipe += "${PN} bash"
-
BBCLASSEXTEND = "native nativesdk"
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 3/3] mesa: clean up libopencl-mesa dependencies
2025-08-20 13:29 [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Ross Burton
2025-08-20 13:29 ` [PATCH 2/3] spirv-tools: improve packaging Ross Burton
@ 2025-08-20 13:29 ` Ross Burton
2025-08-20 17:17 ` [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Khem Raj
2025-08-21 8:59 ` Mathieu Dubois-Briand
3 siblings, 0 replies; 10+ messages in thread
From: Ross Burton @ 2025-08-20 13:29 UTC (permalink / raw)
To: openembedded-core
The libraries in spirv-tools and spirv-llvm-translator are linked to
libopencl, so the correct dependencies are generated automatically.
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
meta/recipes-graphics/mesa/mesa.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
index a848a2ad8c7..6b040e9f839 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -244,7 +244,7 @@ DEV_PKG_DEPENDENCY = ""
RPROVIDES:libgles2-mesa += "libgles3-mesa"
RPROVIDES:libgles2-mesa-dev += "libgles3-mesa-dev"
-RDEPENDS:libopencl-mesa += "${@bb.utils.contains('PACKAGECONFIG', 'opencl', 'libclc spirv-tools spirv-llvm-translator', '', d)}"
+RDEPENDS:libopencl-mesa += "${@bb.utils.contains('PACKAGECONFIG', 'opencl', 'libclc', '', d)}"
PACKAGES =+ "libegl-mesa libegl-mesa-dev \
libgallium \
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-08-20 13:29 [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Ross Burton
2025-08-20 13:29 ` [PATCH 2/3] spirv-tools: improve packaging Ross Burton
2025-08-20 13:29 ` [PATCH 3/3] mesa: clean up libopencl-mesa dependencies Ross Burton
@ 2025-08-20 17:17 ` Khem Raj
2025-08-20 19:50 ` Ross Burton
2025-08-21 8:59 ` Mathieu Dubois-Briand
3 siblings, 1 reply; 10+ messages in thread
From: Khem Raj @ 2025-08-20 17:17 UTC (permalink / raw)
To: ross.burton; +Cc: openembedded-core
On Wed, Aug 20, 2025 at 6:29 AM Ross Burton via lists.openembedded.org
<ross.burton=arm.com@lists.openembedded.org> wrote:
>
> - There is no need for python3native, as this recipe doesn't depend on
> other python modules.
>
> - Inherit lib_package so the llvm-spirv binary is in a separate package
> to the libLLVMSPIRVLib.so library.
>
> - Remove obsolete CMake options that no longer appear to be required.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
> .../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> index 64a73ce55aa..3a090b7b5cb 100644
> --- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> +++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> @@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+(\.\d+)+)"
>
> DEPENDS = "llvm spirv-tools spirv-headers"
>
> -inherit cmake pkgconfig python3native
> +inherit cmake pkgconfig lib_package
>
> -# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
> -# for CMAKE_SHARED_LIBS=OFF see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
> EXTRA_OECMAKE = "\
> -DBUILD_SHARED_LIBS=ON \
> - -DCMAKE_BUILD_TYPE=Release \
Is Release already the default ? Usually this option reduces the debug
info size, which may not matter
for this but would be good to know.
Other changes look fine.
> - -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
> - -DCMAKE_SKIP_RPATH=ON \
> - -DLLVM_EXTERNAL_LIT=lit \
> - -DLLVM_INCLUDE_TESTS=ON \
> - -Wno-dev \
> -DLLVM_EXTERNAL_SPIRV_HEADERS_SOURCE_DIR=${STAGING_INCDIR}/.. \
> "
>
> --
> 2.43.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#222179): https://lists.openembedded.org/g/openembedded-core/message/222179
> Mute This Topic: https://lists.openembedded.org/mt/114799103/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-08-20 17:17 ` [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Khem Raj
@ 2025-08-20 19:50 ` Ross Burton
2025-08-20 20:49 ` Khem Raj
0 siblings, 1 reply; 10+ messages in thread
From: Ross Burton @ 2025-08-20 19:50 UTC (permalink / raw)
To: Khem Raj; +Cc: openembedded-core@lists.openembedded.org
On 20 Aug 2025, at 18:17, Khem Raj <raj.khem@gmail.com> wrote:
>
> On Wed, Aug 20, 2025 at 6:29 AM Ross Burton via lists.openembedded.org
> <ross.burton=arm.com@lists.openembedded.org> wrote:
>>
>> - There is no need for python3native, as this recipe doesn't depend on
>> other python modules.
>>
>> - Inherit lib_package so the llvm-spirv binary is in a separate package
>> to the libLLVMSPIRVLib.so library.
>>
>> - Remove obsolete CMake options that no longer appear to be required.
>>
>> Signed-off-by: Ross Burton <ross.burton@arm.com>
>> ---
>> .../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
>> 1 file changed, 1 insertion(+), 9 deletions(-)
>>
>> diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
>> index 64a73ce55aa..3a090b7b5cb 100644
>> --- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
>> +++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
>> @@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+(\.\d+)+)"
>>
>> DEPENDS = "llvm spirv-tools spirv-headers"
>>
>> -inherit cmake pkgconfig python3native
>> +inherit cmake pkgconfig lib_package
>>
>> -# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
>> -# for CMAKE_SHARED_LIBS=OFF see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
>> EXTRA_OECMAKE = "\
>> -DBUILD_SHARED_LIBS=ON \
>> - -DCMAKE_BUILD_TYPE=Release \
>
> Is Release already the default ? Usually this option reduces the debug
> info size, which may not matter
> for this but would be good to know.
The upstream cmakelists don’t set it either way, and changing this doesn’t change the output.
(I also need to finish off the branch which makes our class pass RelWithDebInfo by default)
Ross
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-08-20 19:50 ` Ross Burton
@ 2025-08-20 20:49 ` Khem Raj
0 siblings, 0 replies; 10+ messages in thread
From: Khem Raj @ 2025-08-20 20:49 UTC (permalink / raw)
To: Ross Burton; +Cc: openembedded-core@lists.openembedded.org
[-- Attachment #1: Type: text/plain, Size: 2050 bytes --]
On Wed, Aug 20, 2025 at 12:50 PM Ross Burton <Ross.Burton@arm.com> wrote:
> On 20 Aug 2025, at 18:17, Khem Raj <raj.khem@gmail.com> wrote:
> >
> > On Wed, Aug 20, 2025 at 6:29 AM Ross Burton via lists.openembedded.org
> > <ross.burton=arm.com@lists.openembedded.org> wrote:
> >>
> >> - There is no need for python3native, as this recipe doesn't depend on
> >> other python modules.
> >>
> >> - Inherit lib_package so the llvm-spirv binary is in a separate package
> >> to the libLLVMSPIRVLib.so library.
> >>
> >> - Remove obsolete CMake options that no longer appear to be required.
> >>
> >> Signed-off-by: Ross Burton <ross.burton@arm.com>
> >> ---
> >> .../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
> >> 1 file changed, 1 insertion(+), 9 deletions(-)
> >>
> >> diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> >> index 64a73ce55aa..3a090b7b5cb 100644
> >> --- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> >> +++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> >> @@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX =
> "v(?P<pver>\d+(\.\d+)+)"
> >>
> >> DEPENDS = "llvm spirv-tools spirv-headers"
> >>
> >> -inherit cmake pkgconfig python3native
> >> +inherit cmake pkgconfig lib_package
> >>
> >> -# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
> >> -# for CMAKE_SHARED_LIBS=OFF see
> https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
> >> EXTRA_OECMAKE = "\
> >> -DBUILD_SHARED_LIBS=ON \
> >> - -DCMAKE_BUILD_TYPE=Release \
> >
> > Is Release already the default ? Usually this option reduces the debug
> > info size, which may not matter
> > for this but would be good to know.
>
> The upstream cmakelists don’t set it either way, and changing this doesn’t
> change the output.
>
Ok
> (I also need to finish off the branch which makes our class pass
> RelWithDebInfo by default)
>
> Ross
[-- Attachment #2: Type: text/html, Size: 3670 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-08-20 13:29 [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Ross Burton
` (2 preceding siblings ...)
2025-08-20 17:17 ` [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Khem Raj
@ 2025-08-21 8:59 ` Mathieu Dubois-Briand
2025-08-21 13:34 ` Khem Raj
3 siblings, 1 reply; 10+ messages in thread
From: Mathieu Dubois-Briand @ 2025-08-21 8:59 UTC (permalink / raw)
To: ross.burton, openembedded-core
On Wed Aug 20, 2025 at 3:29 PM CEST, Ross Burton via lists.openembedded.org wrote:
> - There is no need for python3native, as this recipe doesn't depend on
> other python modules.
>
> - Inherit lib_package so the llvm-spirv binary is in a separate package
> to the libLLVMSPIRVLib.so library.
>
> - Remove obsolete CMake options that no longer appear to be required.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
Hi Ross,
It looks like we have a reproducibility issue:
AssertionError: The following deb packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_amd64.deb
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_amd64.deb
The following ipk packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_x86-64-v3.ipk
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_x86-64-v3.ipk
The following rpm packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib-dbg-20.1.4-r0.x86_64_v3.rpm
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib20.1-20.1.4-r0.x86_64_v3.rpm
http://valkyrie.yocto.io/pub/repro-fail/oe-reproducible-20250821-1clwvk91/packages/diff-html/
https://autobuilder.yoctoproject.org/valkyrie/#/builders/37/builds/2325
Best regards,
Mathieu
--
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-08-21 8:59 ` Mathieu Dubois-Briand
@ 2025-08-21 13:34 ` Khem Raj
0 siblings, 0 replies; 10+ messages in thread
From: Khem Raj @ 2025-08-21 13:34 UTC (permalink / raw)
To: mathieu.dubois-briand; +Cc: ross.burton, openembedded-core
[-- Attachment #1: Type: text/plain, Size: 2671 bytes --]
On Thu, Aug 21, 2025 at 1:59 AM Mathieu Dubois-Briand via
lists.openembedded.org <mathieu.dubois-briand=
bootlin.com@lists.openembedded.org> wrote:
> On Wed Aug 20, 2025 at 3:29 PM CEST, Ross Burton via
> lists.openembedded.org wrote:
> > - There is no need for python3native, as this recipe doesn't depend on
> > other python modules.
> >
> > - Inherit lib_package so the llvm-spirv binary is in a separate package
> > to the libLLVMSPIRVLib.so library.
> >
> > - Remove obsolete CMake options that no longer appear to be required.
> >
> > Signed-off-by: Ross Burton <ross.burton@arm.com>
> > ---
>
> Hi Ross,
>
> It looks like we have a reproducibility issue:
>
> AssertionError: The following deb packages are different and not in
> exclusion list:
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_amd64.deb
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_amd64.deb
> The following ipk packages are different and not in exclusion list:
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_x86-64-v3.ipk
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_x86-64-v3.ipk
> The following rpm packages are different and not in exclusion list:
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib-dbg-20.1.4-r0.x86_64_v3.rpm
>
> /srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib20.1-20.1.4-r0.x86_64_v3.rpm
>
>
> http://valkyrie.yocto.io/pub/repro-fail/oe-reproducible-20250821-1clwvk91/packages/diff-html/
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/37/builds/2325
>
I think we still need RPATH_SKIP option
>
> Best regards,
> Mathieu
>
> --
> Mathieu Dubois-Briand, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#222224):
> https://lists.openembedded.org/g/openembedded-core/message/222224
> Mute This Topic: https://lists.openembedded.org/mt/114799103/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
[-- Attachment #2: Type: text/html, Size: 4183 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
@ 2025-09-05 11:08 Ross Burton
2025-09-05 17:29 ` [OE-core] " Khem Raj
2025-09-08 12:28 ` Mathieu Dubois-Briand
0 siblings, 2 replies; 10+ messages in thread
From: Ross Burton @ 2025-09-05 11:08 UTC (permalink / raw)
To: openembedded-core
- There is no need for python3native, as this recipe doesn't depend on
other python modules.
- Inherit lib_package so the llvm-spirv binary is in a separate package
to the libLLVMSPIRVLib.so library.
- Remove obsolete CMake options that no longer appear to be required.
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
.../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
index 64a73ce55aa..3a090b7b5cb 100644
--- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
+++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
@@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+(\.\d+)+)"
DEPENDS = "llvm spirv-tools spirv-headers"
-inherit cmake pkgconfig python3native
+inherit cmake pkgconfig lib_package
-# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
-# for CMAKE_SHARED_LIBS=OFF see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
EXTRA_OECMAKE = "\
-DBUILD_SHARED_LIBS=ON \
- -DCMAKE_BUILD_TYPE=Release \
- -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
- -DCMAKE_SKIP_RPATH=ON \
- -DLLVM_EXTERNAL_LIT=lit \
- -DLLVM_INCLUDE_TESTS=ON \
- -Wno-dev \
-DLLVM_EXTERNAL_SPIRV_HEADERS_SOURCE_DIR=${STAGING_INCDIR}/.. \
"
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-09-05 11:08 Ross Burton
@ 2025-09-05 17:29 ` Khem Raj
2025-09-08 12:28 ` Mathieu Dubois-Briand
1 sibling, 0 replies; 10+ messages in thread
From: Khem Raj @ 2025-09-05 17:29 UTC (permalink / raw)
To: ross.burton; +Cc: openembedded-core
what is the effect of
- -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
- -DCMAKE_SKIP_RPATH=ON \
On Fri, Sep 5, 2025 at 4:08 AM Ross Burton via lists.openembedded.org
<ross.burton=arm.com@lists.openembedded.org> wrote:
>
> - There is no need for python3native, as this recipe doesn't depend on
> other python modules.
>
> - Inherit lib_package so the llvm-spirv binary is in a separate package
> to the libLLVMSPIRVLib.so library.
>
> - Remove obsolete CMake options that no longer appear to be required.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
> .../spir/spirv-llvm-translator_20.1.4.bb | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> index 64a73ce55aa..3a090b7b5cb 100644
> --- a/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> +++ b/meta/recipes-graphics/spir/spirv-llvm-translator_20.1.4.bb
> @@ -15,18 +15,10 @@ UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+(\.\d+)+)"
>
> DEPENDS = "llvm spirv-tools spirv-headers"
>
> -inherit cmake pkgconfig python3native
> +inherit cmake pkgconfig lib_package
>
> -# Specify any options you want to pass to cmake using EXTRA_OECMAKE:
> -# for CMAKE_SHARED_LIBS=OFF see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/issues/1868
> EXTRA_OECMAKE = "\
> -DBUILD_SHARED_LIBS=ON \
> - -DCMAKE_BUILD_TYPE=Release \
> - -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
> - -DCMAKE_SKIP_RPATH=ON \
> - -DLLVM_EXTERNAL_LIT=lit \
> - -DLLVM_INCLUDE_TESTS=ON \
> - -Wno-dev \
> -DLLVM_EXTERNAL_SPIRV_HEADERS_SOURCE_DIR=${STAGING_INCDIR}/.. \
> "
>
> --
> 2.43.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#222992): https://lists.openembedded.org/g/openembedded-core/message/222992
> Mute This Topic: https://lists.openembedded.org/mt/115080036/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup
2025-09-05 11:08 Ross Burton
2025-09-05 17:29 ` [OE-core] " Khem Raj
@ 2025-09-08 12:28 ` Mathieu Dubois-Briand
1 sibling, 0 replies; 10+ messages in thread
From: Mathieu Dubois-Briand @ 2025-09-08 12:28 UTC (permalink / raw)
To: ross.burton, openembedded-core
On Fri Sep 5, 2025 at 1:08 PM CEST, Ross Burton via lists.openembedded.org wrote:
> - There is no need for python3native, as this recipe doesn't depend on
> other python modules.
>
> - Inherit lib_package so the llvm-spirv binary is in a separate package
> to the libLLVMSPIRVLib.so library.
>
> - Remove obsolete CMake options that no longer appear to be required.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
Hi Ross,
It looks like we have some reproducibility issues:
AssertionError: The following deb packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_amd64.deb
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/deb/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_amd64.deb
The following ipk packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib-dbg_20.1.4-r0_x86-64-v3.ipk
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/ipk/./x86-64-v3/libllvmspirvlib20.1_20.1.4-r0_x86-64-v3.ipk
The following rpm packages are different and not in exclusion list:
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib-dbg-20.1.4-r0.x86_64_v3.rpm
/srv/pokybuild/yocto-worker/reproducible/build/build-st/reproducibleB-extended/tmp/deploy/rpm/./x86_64_v3/libllvmspirvlib20.1-20.1.4-r0.x86_64_v3.rpm
https://autobuilder.yoctoproject.org/valkyrie/#/builders/37/builds/2442
https://valkyrie.yocto.io/pub/repro-fail/oe-reproducible-20250908-q5zzw7wr/packages/
Can you have a look?
Thanks,
Mathieu
--
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-09-08 12:28 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-20 13:29 [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Ross Burton
2025-08-20 13:29 ` [PATCH 2/3] spirv-tools: improve packaging Ross Burton
2025-08-20 13:29 ` [PATCH 3/3] mesa: clean up libopencl-mesa dependencies Ross Burton
2025-08-20 17:17 ` [OE-core] [PATCH 1/3] spirv-llvm-translator: small recipe cleanup Khem Raj
2025-08-20 19:50 ` Ross Burton
2025-08-20 20:49 ` Khem Raj
2025-08-21 8:59 ` Mathieu Dubois-Briand
2025-08-21 13:34 ` Khem Raj
-- strict thread matches above, loose matches on Subject: below --
2025-09-05 11:08 Ross Burton
2025-09-05 17:29 ` [OE-core] " Khem Raj
2025-09-08 12:28 ` Mathieu Dubois-Briand
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox