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

CMakeLists.txt: drop OSDependent/OGLCompiler from lists of glslang libraries #1276

Closed
wants to merge 3 commits into from

Conversation

kanavin
Copy link

@kanavin kanavin commented Nov 2, 2022

glslang no longer installs them separately, and all needed functionality has been merged into glslang shared library itself.

This wasn't a problem previously as they were still provided, as static libraries but in latest glslang they no longer are: KhronosGroup/glslang@7cd5195

@google-cla
Copy link

google-cla bot commented Nov 2, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

…braries

glslang no longer installs them separately, and all needed
functionality has been merged into glslang shared library itself.

This wasn't a problem previously as they were still provided
as static libraries, but in latest glslang they no longer are:
KhronosGroup/glslang@7cd5195
@dneto0 dneto0 enabled auto-merge (rebase) July 24, 2023 14:11
@juan-lunarg
Copy link
Contributor

@dneto0 this can now be closed as well.

@dneto0 dneto0 closed this Jan 23, 2024
auto-merge was automatically disabled January 23, 2024 18:04

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants