Skip to content

loader: Include limits.h for PATH_MAX#1760

Merged
charles-lunarg merged 1 commit into
KhronosGroup:mainfrom
mixi:limits_h
Aug 25, 2025
Merged

loader: Include limits.h for PATH_MAX#1760
charles-lunarg merged 1 commit into
KhronosGroup:mainfrom
mixi:limits_h

Conversation

@mixi

@mixi mixi commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

This fixes compiling against musl libc:

 loader/loader.c: In function 'normalize_pat':
 loader/loader.c:405:52: error: 'PATH_MAX' undeclared (first use in this function)
   405 |     char *path = loader_instance_heap_calloc(inst, PATH_MAX, VK_SYSTEM_ALLOCATION_SCOPE_INSTANCE);
       |                                                    ^~~~~~~~

This fixes compiling against musl libc:

 loader/loader.c: In function 'normalize_pat':
 loader/loader.c:405:52: error: 'PATH_MAX' undeclared (first use in this function)
   405 |     char *path = loader_instance_heap_calloc(inst, PATH_MAX, VK_SYSTEM_ALLOCATION_SCOPE_INSTANCE);
       |                                                    ^~~~~~~~
@ci-tester-lunarg

Copy link
Copy Markdown

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

1 similar comment
@ci-tester-lunarg

Copy link
Copy Markdown

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

@CLAassistant

CLAassistant commented Aug 14, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@charles-lunarg charles-lunarg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the fix!

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Loader build queued with queue ID 516453.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Loader build # 3160 running.

@ci-tester-lunarg

Copy link
Copy Markdown

CI Vulkan-Loader build # 3160 passed.

@charles-lunarg charles-lunarg merged commit e753770 into KhronosGroup:main Aug 25, 2025
45 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