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

build: API parameterization changes #1427

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

aqnuep
Copy link
Contributor

@aqnuep aqnuep commented Jan 4, 2024

API variant parameterization changes to the loader analogously to other Vulkan ecosystem components.

@ci-tester-lunarg
Copy link

CI Vulkan-Loader build queued with queue ID 110983.

@ci-tester-lunarg
Copy link

CI Vulkan-Loader build # 2440 running.

@ci-tester-lunarg
Copy link

CI Vulkan-Loader build # 2440 passed.

Copy link
Contributor

@spencer-lunarg spencer-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the scripts/ logic is on par how it has been intergrated into the other repos

the rest I have no in-depth knowledge of the loader so will let @charles-lunarg finish the review

@aqnuep
Copy link
Contributor Author

aqnuep commented Jan 4, 2024

Thanks, @spencer-lunarg, sure, I hope @charles-lunarg and @juan-lunarg will comment on the respective parts of this change.

Copy link
Collaborator

@charles-lunarg charles-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one nit - everything else is solid and in many cases an improvement.

@aqnuep aqnuep force-pushed the api-parameterization branch from 0a28574 to 84e5336 Compare January 5, 2024 17:02
@ci-tester-lunarg
Copy link

CI Vulkan-Loader build queued with queue ID 111557.

@ci-tester-lunarg
Copy link

CI Vulkan-Loader build # 2441 running.

@ci-tester-lunarg
Copy link

CI Vulkan-Loader build # 2441 passed.

@charles-lunarg charles-lunarg merged commit 66a0f14 into KhronosGroup:main Jan 5, 2024
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants