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

#335 makes it hard to create Vulkan-Headers updates #339

Closed
oddhack opened this issue Jun 10, 2019 · 4 comments
Closed

#335 makes it hard to create Vulkan-Headers updates #339

oddhack opened this issue Jun 10, 2019 · 4 comments

Comments

@oddhack
Copy link

oddhack commented Jun 10, 2019

In #335, it looks like the cmake rules changed to use a Vulkan-Headers submodule instead of Vulkan-Docs. This caused two issues with trying to update Vulkan-Headers for the 1.1.111 spec update. First, when I cloned the Vulkan-Hpp repo locally, I got an empty Vulkan-Headers directory, which is causing the error "VkGenerate: failed to load file /home/tree/git/Vulkan-Hpp/Vulkan-Headers/registry/vk.xml . Error code: 3" from VulkanHppGenerator, and there aren't any build instructions that say how to pull the submodule (in fairness, there weren't when using Vulkan-Docs, either - I just removed the entire Vulkan-Docs directory and symlinked to my local up-to-date repo).

Second, and more problematic, is that I'm trying to update Vulkan-Headers with the new Vulkan-Hpp. I can't do that if Vulkan-Hpp is pulling its content from Vulkan-Headers, at least without a two-stage process in which I first update the XML, scripts and C headers; pull Vulkan-Hpp and update the Vulkan-Headers submodule; and then update vulkan.hpp in Vulkan-Headers in an additional step. This seems like annoying extra effort involving multiple round trips to github, which I'd rather avoid since I'm doing this every time I push a spec update.

Can we restore, or offer as a documented alternative, making Vulkan-Hpp build out of Vulkan-Docs again? I don't think I can move forward with the headers update until this circular dependency situation is addressed, and that's what would be the easiest way for me.

@mtavenrath
Copy link
Contributor

@oddhack For your case there's no need to use the 'default' vk.xml specified in the project file which currently resides in Vulkan-Headers. You can pass whatever vk.xml you want as argument to the generator. In your case pass the path to your Vulkan-Docs vk.xml to the generator and it'll use it.

@oddhack
Copy link
Author

oddhack commented Jun 11, 2019

@mtavenrath do you mean like

(cd build && ./VulkanHppGenerator some-path/vk.xml)

?

@mtavenrath
Copy link
Contributor

mtavenrath commented Jun 11, 2019

@oddhack Yes

@oddhack
Copy link
Author

oddhack commented Jun 11, 2019

Thanks, Vulkan-Headers is updated now.

@oddhack oddhack closed this as completed Jun 11, 2019
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

2 participants