Skip to content

Dedupe identical vulkan_sdk toolchain tags across modules#150

Merged
mxpv merged 1 commit into
mxpv:mainfrom
0xbeefd1ed:dedupe-toolchain-tags
Jul 2, 2026
Merged

Dedupe identical vulkan_sdk toolchain tags across modules#150
mxpv merged 1 commit into
mxpv:mainfrom
0xbeefd1ed:dedupe-toolchain-tags

Conversation

@0xbeefd1ed

Copy link
Copy Markdown
Contributor

A module extension is evaluated once per build and receives the toolchain tags of every module that used it. When two modules request the same SDK version (e.g. a library and the app consuming it both pin 1.4.341.1), both tags resolve to the same repository name and install_sdk fails with a "repository already defined" error, breaking any multi-module graph that shares an SDK.

Dedupe by the resolved repository name so identical requests collapse to a single repo. When the same name is requested with different attributes -- a genuine conflict between two distinct SDKs -- fail with an actionable message telling the user to give one tag a distinct name, rather than silently installing whichever tag was iterated first.

A module extension is evaluated once per build and receives the toolchain
tags of every module that used it. When two modules request the same SDK
version (e.g. a library and the app consuming it both pin 1.4.341.1), both
tags resolve to the same repository name and install_sdk fails with a
"repository already defined" error, breaking any multi-module graph that
shares an SDK.

Dedupe by the resolved repository name so identical requests collapse to a
single repo. When the same name is requested with different attributes --
a genuine conflict between two distinct SDKs -- fail with an actionable
message telling the user to give one tag a distinct `name`, rather than
silently installing whichever tag was iterated first.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@mxpv mxpv merged commit a83fe15 into mxpv:main Jul 2, 2026
16 checks passed
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.

2 participants