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.

Sign up for GitHub

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

Jump to bottom

Linux: current build does not complete vk ext. undeclared #55

Closed
mdeguzis opened this issue Aug 12, 2016 · 6 comments
Closed

Linux: current build does not complete vk ext. undeclared #55

mdeguzis opened this issue Aug 12, 2016 · 6 comments

Comments

@mdeguzis
Copy link
Contributor

mdeguzis commented Aug 12, 2016 •
edited
Loading

The debug extension addition seems to have introduced errors when making Linux binaries. The following error is thrown:

gl_vidsdl.c:674:15: error: ‘VK_EXT_DEBUG_MARKER_EXTENSION_NAME’ undeclared (first use in this function)
    if (strcmp(VK_EXT_DEBUG_MARKER_EXTENSION_NAME, device_extensions[i].extensionName) == 0)
               ^

Log: http://sprunge.us/NMAY

Steps to reproduce:

sudo apt-get install git make gcc libsdl2-dev libvulkan-dev libvorbis-dev libmad0-dev
git clone https://github.com/Novum/vkQuake/
cd vkQuake/QUake
make
@SaschaWillems
Copy link
Contributor

Are you using a recent Vulkan header? The extension has been added to the Vulkan headers some time ago.

@mdeguzis
Copy link
Contributor Author

mdeguzis commented Aug 12, 2016 •
edited
Loading

I backported libvulkan-dev from Debian Stretch and have been using that for some time. I can build 0.50 ok. Is this library version too old?

@SaschaWillems
Copy link
Contributor

Afair the debug marker extension was introduced with 1.0.12.0 and it seems that package is only 1.0.8.0.

@mdeguzis
Copy link
Contributor Author

mdeguzis commented Aug 12, 2016 •
edited
Loading

Lovely :) I guess I'll have to look into building a package for that version. Thank you for the heads up. If this is true @Novum, this definitely should be noted in the readme for the master tree.

Update: It's been fun updating the Debian Sid package for libvulkan-dev 1.0.21.0. Had to make a few modifications to the packaging, but I think I almost have it.

@mdeguzis
Copy link
Contributor Author

mdeguzis commented Aug 12, 2016 •
edited
Loading

I updated my libvulkan-dev package in my pool to 1.0.21.0, merged the Debian patches, and after some tinkering got that built. After installing this during a pbuilder chroot build, everything completed ok. @SaschaWillems thank you for the heads up (and for your Vulkan work).

Testing the resulting binary now.

@mdeguzis
Copy link
Contributor Author

Confirmed working.

@mdeguzis mdeguzis mentioned this issue Aug 12, 2016
Merged
Novum pushed a commit that referenced this issue Aug 13, 2016
Note minimum version of libvulkan-dev (#56)
14e802a 
See #55 for why this is required.
@mdeguzis mdeguzis mentioned this issue Dec 9, 2016
Closed
@sezero sezero mentioned this issue Dec 22, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdeguzis @SaschaWillems