diff mbox series

vulkan-samples: update to latest revision

Message ID 20250911211137.3678240-1-skandigraun@gmail.com
State Accepted, archived
Commit 69e3758a820a2283ec19f4fcf217a0065ef2ad96
Headers show
Series vulkan-samples: update to latest revision | expand

Commit Message

Gyorgy Sarvari Sept. 11, 2025, 9:11 p.m. UTC
Dropped 0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch,
because the dependency it was patching (glslang submodule) has been
removed by the project, so it became obsolete.

Shortlog:
Unify vkb::PhysicalDevice and vkb::core::HPPPhysicalDevice into vkb::core::PhysicalDevice<bindingType>
Update Apple min deployment versions and iOS Simulator build config
Add example for dynamic_rendering_local_read
Add Slang and HLSL shaders to first set of samples
Fix use of --width and --height by setting extent earlier
Unify vkb::Instance and vkb::core::HPPInstance into vkb::core::Instance<bindingType>
Remove vkb::filesystem::init() from astc to fix memory leak and batch mode stability
Update VVL version to 1.4.321.0 for Android
Update NDK version to r28c for Android
Fix variable descriptor count info not used in descriptor set creation
Fix layer settings regression on Apple platforms for MoltenVK config
Update shader interfaces to match sample's vertex input description
Fix regression on iOS Simulator by including TargetConditionals.h in gui.h
Add CMAKE_OSX_DEPLOYMENT_TARGET to macOS build docs and CI script
Add possible Linux WSI options
Unify vkb::Gui and vkb::HPPGui into vkb::Gui<bindingType>
Remove submodule CTPL, replace its functionality by calls to std::async
fix: fix the surface resize logic when use wayland wsi
Implement simple cache for decoded ASTC textures
Unify vkb::Device and vkb::core::HPPDevice into vkb::core::Device<bindingType>
Port of latest changes in api sample hello_triangle to hpp_hello_triangle
Fix crash in sample on android
Improve and fix async compute sample
Fix MacOS cmake build instructions
Replace runtime shader compilation with offline compiled SPIR-V shaders
Transcoded version of api sample hello_triangle_1_3 based on Vulkan-Hpp
Unify vkb::RenderFrame and vkb::rendering::HPPRenderFrame into vkb::rendering::RenderFrame<bindingType>
Increase base version to Vulkan 1.1

Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
 ...uilder.h-add-missing-cstdint-include.patch | 28 -------------------
 .../vulkan/vulkan-samples_git.bb              |  6 ++--
 2 files changed, 2 insertions(+), 32 deletions(-)
 delete mode 100644 meta/recipes-graphics/vulkan/vulkan-samples/0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch

Comments

patchtest@automation.yoctoproject.org Sept. 11, 2025, 9:15 p.m. UTC | #1
Thank you for your submission. Patchtest identified one
or more issues with the patch. Please see the log below for
more information:

---
Testing patch /home/patchtest/share/mboxes/vulkan-samples-update-to-latest-revision.patch

FAIL: test src uri left files: Patches not removed from tree. Remove them and amend the submitted mbox (test_metadata.TestMetadata.test_src_uri_left_files)

PASS: pretest src uri left files (test_metadata.TestMetadata.pretest_src_uri_left_files)
PASS: test CVE check ignore (test_metadata.TestMetadata.test_cve_check_ignore)
PASS: test Signed-off-by presence (test_mbox.TestMbox.test_signed_off_by_presence)
PASS: test author valid (test_mbox.TestMbox.test_author_valid)
PASS: test commit message presence (test_mbox.TestMbox.test_commit_message_presence)
PASS: test commit message user tags (test_mbox.TestMbox.test_commit_message_user_tags)
PASS: test lic files chksum modified not mentioned (test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned)
PASS: test max line length (test_metadata.TestMetadata.test_max_line_length)
PASS: test mbox format (test_mbox.TestMbox.test_mbox_format)
PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade)
PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format)
PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length)
PASS: test target mailing list (test_mbox.TestMbox.test_target_mailing_list)

SKIP: pretest pylint: No python related patches, skipping test (test_python_pylint.PyLint.pretest_pylint)
SKIP: test CVE tag format: No new CVE patches introduced (test_patch.TestPatch.test_cve_tag_format)
SKIP: test Signed-off-by presence: No new CVE patches introduced (test_patch.TestPatch.test_signed_off_by_presence)
SKIP: test Upstream-Status presence: No new CVE patches introduced (test_patch.TestPatch.test_upstream_status_presence_format)
SKIP: test bugzilla entry format: No bug ID found (test_mbox.TestMbox.test_bugzilla_entry_format)
SKIP: test lic files chksum presence: No added recipes, skipping test (test_metadata.TestMetadata.test_lic_files_chksum_presence)
SKIP: test license presence: No added recipes, skipping test (test_metadata.TestMetadata.test_license_presence)
SKIP: test pylint: No python related patches, skipping test (test_python_pylint.PyLint.test_pylint)
SKIP: test series merge on head: Merge test is disabled for now (test_mbox.TestMbox.test_series_merge_on_head)
SKIP: test summary presence: No added recipes, skipping test (test_metadata.TestMetadata.test_summary_presence)

---

Please address the issues identified and
submit a new revision of the patch, or alternatively, reply to this
email with an explanation of why the patch should be accepted. If you
believe these results are due to an error in patchtest, please submit a
bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' category
under 'Yocto Project Subprojects'). For more information on specific
failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank
you!
Mathieu Dubois-Briand Sept. 12, 2025, 11:49 a.m. UTC | #2
On Thu Sep 11, 2025 at 11:11 PM CEST, Gyorgy Sarvari via lists.openembedded.org wrote:
> Dropped 0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch,
> because the dependency it was patching (glslang submodule) has been
> removed by the project, so it became obsolete.
>
> Shortlog:
> Unify vkb::PhysicalDevice and vkb::core::HPPPhysicalDevice into vkb::core::PhysicalDevice<bindingType>
> Update Apple min deployment versions and iOS Simulator build config
> Add example for dynamic_rendering_local_read
> Add Slang and HLSL shaders to first set of samples
> Fix use of --width and --height by setting extent earlier
> Unify vkb::Instance and vkb::core::HPPInstance into vkb::core::Instance<bindingType>
> Remove vkb::filesystem::init() from astc to fix memory leak and batch mode stability
> Update VVL version to 1.4.321.0 for Android
> Update NDK version to r28c for Android
> Fix variable descriptor count info not used in descriptor set creation
> Fix layer settings regression on Apple platforms for MoltenVK config
> Update shader interfaces to match sample's vertex input description
> Fix regression on iOS Simulator by including TargetConditionals.h in gui.h
> Add CMAKE_OSX_DEPLOYMENT_TARGET to macOS build docs and CI script
> Add possible Linux WSI options
> Unify vkb::Gui and vkb::HPPGui into vkb::Gui<bindingType>
> Remove submodule CTPL, replace its functionality by calls to std::async
> fix: fix the surface resize logic when use wayland wsi
> Implement simple cache for decoded ASTC textures
> Unify vkb::Device and vkb::core::HPPDevice into vkb::core::Device<bindingType>
> Port of latest changes in api sample hello_triangle to hpp_hello_triangle
> Fix crash in sample on android
> Improve and fix async compute sample
> Fix MacOS cmake build instructions
> Replace runtime shader compilation with offline compiled SPIR-V shaders
> Transcoded version of api sample hello_triangle_1_3 based on Vulkan-Hpp
> Unify vkb::RenderFrame and vkb::rendering::HPPRenderFrame into vkb::rendering::RenderFrame<bindingType>
> Increase base version to Vulkan 1.1
>
> Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
> ---

Hi Gyorgy,

It looks like vulkan-samples is used by the test_devtool_git_submodules
selftest, so it results in some error after this update:

2025-09-12 06:53:10,870 - oe-selftest - INFO - devtool.DevtoolUpdateTests.test_devtool_git_submodules (subunit.RemotedTestCase)
2025-09-12 06:53:10,871 - oe-selftest - INFO -  ... FAIL
...
2025-09-12 06:53:10,872 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
  File "/srv/pokybuild/yocto-worker/oe-selftest-armhost/build/meta/lib/oeqa/selftest/cases/devtool.py", line 1729, in test_devtool_git_submodules
    self._check_repo_status(recipedir, expected_status)
  File "/srv/pokybuild/yocto-worker/oe-selftest-armhost/build/meta/lib/oeqa/selftest/cases/devtool.py", line 135, in _check_repo_status
    self.fail('Unexpected modified file in line: %s' % line)
  File "/usr/lib/python3.12/unittest/case.py", line 715, in fail
    raise self.failureException(msg)
AssertionError: Unexpected modified file in line: ?? meta/recipes-graphics/vulkan/vulkan-samples/

Can you have a look at what is going on there?

Thanks,
Mathieu
diff mbox series

Patch

diff --git a/meta/recipes-graphics/vulkan/vulkan-samples/0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch b/meta/recipes-graphics/vulkan/vulkan-samples/0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch
deleted file mode 100644
index f50aa806da..0000000000
--- a/meta/recipes-graphics/vulkan/vulkan-samples/0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch
+++ /dev/null
@@ -1,28 +0,0 @@ 
-From e40c14a3e007fac0e4f2e4164fdf14d1712355bd Mon Sep 17 00:00:00 2001
-From: Sergei Trofimovich <slyich@gmail.com>
-Date: Fri, 2 Aug 2024 22:44:21 +0100
-Subject: [PATCH] SPIRV/SpvBuilder.h: add missing <cstdint> include
-
-Without the change `glslang` build fails on upcoming `gcc-15` as:
-
-    In file included from /build/source/SPIRV/GlslangToSpv.cpp:45:
-    SPIRV/SpvBuilder.h:248:30: error: 'uint32_t' has not been declared
-      248 |     Id makeDebugLexicalBlock(uint32_t line);
-          |                              ^~~~~~~~
-
-Upstream-Status: Backport [https://github.com/KhronosGroup/glslang/commit/e40c14a3e007fac0e4f2e4164fdf14d1712355bd]
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- SPIRV/SpvBuilder.h | 1 +
- 1 file changed, 1 insertion(+)
-
---- a/SPIRV/SpvBuilder.h
-+++ b/SPIRV/SpvBuilder.h
-@@ -56,6 +56,7 @@ namespace spv {
- }
- 
- #include <algorithm>
-+#include <cstdint>
- #include <map>
- #include <memory>
- #include <set>
diff --git a/meta/recipes-graphics/vulkan/vulkan-samples_git.bb b/meta/recipes-graphics/vulkan/vulkan-samples_git.bb
index b7c9ddd2fe..8011c3a81c 100644
--- a/meta/recipes-graphics/vulkan/vulkan-samples_git.bb
+++ b/meta/recipes-graphics/vulkan/vulkan-samples_git.bb
@@ -5,12 +5,10 @@  LICENSE = "Apache-2.0"
 
 LIC_FILES_CHKSUM = "file://LICENSE;md5=48aa35cefb768436223a6e7f18dc2a2a"
 
-SRC_URI = "gitsm://github.com/KhronosGroup/Vulkan-Samples.git;branch=main;protocol=https;lfs=0 \
-           file://0001-SPIRV-SpvBuilder.h-add-missing-cstdint-include.patch;patchdir=third_party/glslang \
-           "
+SRC_URI = "gitsm://github.com/KhronosGroup/Vulkan-Samples.git;branch=main;protocol=https;lfs=0"
 
 UPSTREAM_CHECK_COMMITS = "1"
-SRCREV = "d27205d14d01ea7d33efc8ba2862478612370182"
+SRCREV = "b9961792604af2ede4c9d0868947de2a8eccd549"
 
 UPSTREAM_CHECK_GITTAGREGEX = "These are not the releases you're looking for"