Skip to content

[BUG] vulkanmemallocator is missing snprintf header #3193

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

Closed
kasperk81 opened this issue Mar 11, 2025 · 2 comments · Fixed by mono/skia#152 or #3196
Closed

[BUG] vulkanmemallocator is missing snprintf header #3193

kasperk81 opened this issue Mar 11, 2025 · 2 comments · Fixed by mono/skia#152 or #3196

Comments

@kasperk81
Copy link
Contributor

I am converging debian an alpine toolchains on top of #3192 and ran into an error with clang-19 on debian amd64 cross-compiling for alpine riscv64:

error: 'snprintf' was not declared in this scope
[  424s]  2817 |         snprintf(outStr, strLen, "%u", static_cast<unsigned int>(num));
[  424s]       |         ^~~~~~~~

tracked it to
https://github.com/mono/skia/blob/1c3113af31028b19affaff5d71f47c920acbe4f4/DEPS#L59

which is >2y old because it's missing this patch GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator@29d492b

@kasperk81
Copy link
Contributor Author

@mattleibow can we bump the BAZEL dependency in mono/skia or apply that tiny patch some other way?

@mattleibow
Copy link
Contributor

I think you can bump the SHA in the DEPS file in the skia repo root.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
2 participants