Skip to content

spirv-tools: build static library as well#281067

Open
crueter wants to merge 1 commit intoHomebrew:mainfrom
crueter:patch-1
Open

spirv-tools: build static library as well#281067
crueter wants to merge 1 commit intoHomebrew:mainfrom
crueter:patch-1

Conversation

@crueter
Copy link
Copy Markdown
Contributor

@crueter crueter commented May 5, 2026

Not sure why this was off. It's particularly useful for certain CI applications where SPIRV-Tools is better off being linked statically


  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>?
  • Is your test running fine brew test <formula>?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes.

Not sure why this was off. It's particularly useful for certain CI applications where SPIRV-Tools is better off being linked statically
@GunniBusch
Copy link
Copy Markdown
Contributor

GunniBusch commented May 5, 2026

I think as per brews policy, it is preferred to have shared libs. https://docs.brew.sh/Acceptable-Formulae#shared-vs-static-libraries. And if there are static libs required formulae might ship both. (ignore this if you actually meant to have shared AND static)

@crueter
Copy link
Copy Markdown
Contributor Author

crueter commented May 5, 2026

Yeah I want to have shared and static. Their handling is kind of goofed up right now, which might cause the build failure; I do have a pending PR there to fix it KhronosGroup/SPIRV-Tools#6481 but it stalled because I'm lazy and also haven't called for reviews yet. Will investigate more when I next get access to my ARM linux machine

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