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

Android build error on Mac 10.15.4 #2097

Closed
xing-mei opened this issue Apr 6, 2020 · 5 comments
Closed

Android build error on Mac 10.15.4 #2097

xing-mei opened this issue Apr 6, 2020 · 5 comments

Comments

@xing-mei
Copy link

xing-mei commented Apr 6, 2020

Pulled latest bx, bimg, bgfx. Ran make android-arm or make android-arm64 and failed with the following error message:

../../../src/renderer_vk.cpp:139:1: error: unknown type name 'PFN_vkSetDebugUtilsObjectNameEXT'
...
../../../src/renderer_vk.cpp:139:1: error: unknown type name 'PFN_vkSetDebugUtilsObjectTagEXT'
...

Seeme something wrong with vulkan related macros.

My android NDK is r16b. ANDROID_NDK_ROOT, ANDROID_NDK_ARM, ANDROID_NDK_MIPS. ANDROID_NDK_X86 has been set following https://bkaradzic.github.io/bgfx/build.html. Anyone ran into the same issue?

Thanks.

@bkaradzic
Copy link
Owner

cc @pezcode

@pezcode
Copy link
Contributor

pezcode commented Apr 6, 2020

Sounds like it's not using the Vulkan headers coming with bgfx, those function pointers types are definitely defined in the bundled vulkan_core.h

If you have the Vulkan SDK installed, try renaming vulkan.h temporarily so it picks up the correct headers from bgfx/3rdparty.

@xing-mei
Copy link
Author

xing-mei commented Apr 7, 2020

@pezcode thanks . I do have a vulkan.h in ndk_dir/sysroot/usr/include/vulkan/. I renamed it and problem solved, but I can't leave ndk like that since I have multiple android projects going on. Any way to enforce the search path for bgfx android build?

@pezcode
Copy link
Contributor

pezcode commented Apr 7, 2020

From a quick glance $(ANDROID_NDK_ROOT)/usr/include is added as an include path by bx and then transitively added to bgfx. I'm not familiar enough with the build system but @bkaradzic might have an idea on how to improve include path ordering here.

@bkaradzic
Copy link
Owner

I'll probably change directory name in 3rdparty/khronos. vulkan -> vulkan1 or something like that.

bkaradzic added a commit that referenced this issue Apr 8, 2020
rozgo pushed a commit to VertexStudio/bgfx that referenced this issue Aug 14, 2020
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

No branches or pull requests

3 participants