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

[glslang] fix tools #35191

Merged
merged 1 commit into from
Nov 20, 2023
Merged

[glslang] fix tools #35191

merged 1 commit into from
Nov 20, 2023

Conversation

moritz-h
Copy link
Contributor

  • Changes comply with the maintainer guide
  • SHA512s are updated for each updated download
  • The "supports" clause reflects platforms that may be fixed by this new version
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

Currently the tool glslangValidator is broken, because it is only a symbolic link to not existent file.
With 12.3.0 release glslang renamed the tool binary glslangValidator to glslang, but keeping the old name as copy (Windows) or symbolic link to the new one (other platforms), see https://github.com/KhronosGroup/glslang/releases/tag/12.3.0. It is required to install the new tool. #34700 fixed this for windows only, but the same solution is required on all platforms.

@MonicaLiu0311 MonicaLiu0311 added the category:port-bug The issue is with a library, which is something the port should already support label Nov 20, 2023
@MonicaLiu0311 MonicaLiu0311 added the info:reviewed Pull Request changes follow basic guidelines label Nov 20, 2023
@data-queue data-queue merged commit 32d07fb into microsoft:master Nov 20, 2023
15 checks passed
@moritz-h moritz-h deleted the glslang branch November 20, 2023 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants