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

[ogre-next] vulkan no osx #35968

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Conversation

autoantwort
Copy link
Contributor

@autoantwort autoantwort commented Jan 1, 2024

It requires xcb/X11 on osx

/Users/leanderSchulten/git_projekte/vcpkg2/buildtrees/ogre-next/src/v2.3.3-d8d38bba95.clean/RenderSystems/Vulkan/src/Windowing/X11/OgreVulkanXcbSupport.cpp:34:10: fatal error: 'xcb/randr.h' file not found

@dg0yt
Copy link
Contributor

dg0yt commented Jan 2, 2024

Why not just disable xcb/x11?

@MonicaLiu0311 MonicaLiu0311 added the category:port-bug The issue is with a library, which is something the port should already support label Jan 2, 2024
@autoantwort
Copy link
Contributor Author

Why not just disable xcb/x11?

Because the supported window systems are win32, xcb and android

@Neumann-A
Copy link
Contributor

(could try building xcb on osx. There is a port for it, it is currently configured to do nothing but there is a triplet variables to change that.)

@dg0yt
Copy link
Contributor

dg0yt commented Jan 2, 2024

Why not just disable xcb/x11?

Because the supported window systems are win32, xcb and android

Does it need a supported window system like xcb to be useful with a supported rendering backend like Metal?

@autoantwort
Copy link
Contributor Author

Does it need a supported window system like xcb to be useful with a supported rendering backend like Metal?

No. xcb is only needed when using vulkan

@MonicaLiu0311 MonicaLiu0311 added the info:reviewed Pull Request changes follow basic guidelines label Jan 4, 2024
@BillyONeal BillyONeal merged commit 45c8b19 into microsoft:master Jan 4, 2024
15 checks passed
@BillyONeal
Copy link
Member

Thanks!

Osyotr pushed a commit to Osyotr/vcpkg that referenced this pull request Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants