Skip to content

vulkaninfo: Make vulkan entry point variables static#1234

Open
hapakj wants to merge 1 commit into
KhronosGroup:mainfrom
RasterGrid:rastergrid/vulkaninfo-static-entry-points
Open

vulkaninfo: Make vulkan entry point variables static#1234
hapakj wants to merge 1 commit into
KhronosGroup:mainfrom
RasterGrid:rastergrid/vulkaninfo-static-entry-points

Conversation

@hapakj

@hapakj hapakj commented Jul 9, 2026

Copy link
Copy Markdown

Make vulkan entry point variables static in vulkaninfo.

Prevent link issues in case vulkan info directly linked to a library which exports the related vulkan entry points. e.g. a vulkan loader or and icd.

@ci-tester-lunarg

Copy link
Copy Markdown

Author hapakj not on autobuild list. Waiting for curator authorization before starting CI build.

1 similar comment
@ci-tester-lunarg

Copy link
Copy Markdown

Author hapakj not on autobuild list. Waiting for curator authorization before starting CI build.

@charles-lunarg charles-lunarg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't understand the purpose of these changes.

Vulkaninfo is an application that links vulkan-1.dll/libvulkan.so and indirectly loads vulkan drivers. Why would the loader or a driver link to vulkaninfo? If you were doing that, there would need to be significantly more changes.

Is this PR the result of a dynamic linker warning about duplicate symbols?

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Tools build queued with queue ID 35238.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Tools build # 1999 running.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Tools build queued with queue ID 35259.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Tools build # 2000 running.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Tools build # 2000 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.

3 participants