Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vulkan-validationlayers 1.4.309 #208549

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

  • resource blocks have been checked for updates.

@github-actions github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Feb 21, 2025
@cho-m
Copy link
Member

cho-m commented Feb 21, 2025

I wonder if we should align with SDK releases instead.

This would allow us to match with SPIRV releases and avoid the resources (as SDK's components should be compatible)

@chenrui333
Copy link
Member

I wonder if we should align with SDK releases instead.

This would allow us to match with SPIRV releases and avoid the resources (as SDK's components should be compatible)

I dont think they are always in sync?

  # https://github.com/KhronosGroup/Vulkan-ValidationLayers/blob/v#{version}/scripts/known_good.json#L32
  resource "SPIRV-Headers" do
    url "https://github.com/KhronosGroup/SPIRV-Headers.git",
        revision: "2b2e05e088841c63c0b6fd4c9fb380d8688738d3"
  end
  # https://github.com/KhronosGroup/Vulkan-ValidationLayers/blob/v#{version}/scripts/known_good.json#L46
  resource "SPIRV-Tools" do
    url "https://github.com/KhronosGroup/SPIRV-Tools.git",
        revision: "066c3d52c2fca8d9df79ca37055c3f5eddf2ffce"
  end

@cho-m
Copy link
Member

cho-m commented Feb 22, 2025

I dont think they are always in sync?

The SDK is designed to be compatible. SDK is a state of all components that get shipped together in a full package - https://vulkan.lunarg.com/sdk/home#mac

known_good is just a single confirmed test point. There is often no need to be on exact version specified there for same reason we don't vendor every dependency. As long as within an API/ABI compatible version, which SDK provides.

@chenrui333
Copy link
Member

chenrui333 commented Feb 22, 2025

IIRC those were known_good.json was brought in when there was compatible issue before.

(yeah, we can always go back to the specific commit if the formulae do not work)

@cho-m
Copy link
Member

cho-m commented Feb 22, 2025

IIRC those were known_good.json was brought in when there was compatible issue before.

But was that on the SDK release cycle (i.e. x.y.z releases with a vulkan-sdk-x.y.z tags, not those with only x.y.z tags), i.e.

SPIRV only provides releases on SDK cycle:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants