FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Improve Vulkan and Slang SDK discovery by Foereaper · Pull Request #61 · novusengine/Engine · GitHub

Improve Vulkan and Slang SDK discovery - #61

Closed
Foereaper wants to merge 2 commits into
masterfrom
foe/premake-vulkan-slang
Closed

Foereaper wants to merge 2 commits into
masterfrom
foe/premake-vulkan-slang

Conversation

Copy link
Copy Markdown
Member

Summary

  • prefer a sourced Vulkan tarball SDK on Linux and validate its header and loader layout
  • retain system header/library discovery when no SDK environment is configured
  • discover Slang through SLANG_SDK or the Vulkan SDK
  • copy the complete versioned Slang runtime library set beside engine binaries

Why

Ubuntu system packages split Vulkan and Slang across system paths, while the former LunarG apt SDK left Slang older than the engine-facing API. The official tarball keeps compatible Vulkan and Slang components together, but places the Vulkan loader in a separate SDK-relative directory.

Validation

  • clean Debug generation and full compile on the Ubuntu self-hosted runner using Vulkan SDK 1.4.357.1 and Slang 2026.13.1
  • compiled the restored SLANG_STAGE_DISPATCH code without a compatibility bridge
  • verified generated makefiles reference the tarball include, Slang library, and Vulkan loader directories
  • verified Engine-Tests resolves the tarball Vulkan loader and bundled versioned Slang runtime library

Foereaper marked this pull request as ready for review August 10, 2026 08:52
Foereaper closed this Aug 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL