Skip to content

shader-slang: 2025.14.3 -> 2025.15#437524

Merged
niklaskorz merged 1 commit intoNixOS:masterfrom
samestep:shader-slang-2025.15
Aug 27, 2025
Merged

shader-slang: 2025.14.3 -> 2025.15#437524
niklaskorz merged 1 commit intoNixOS:masterfrom
samestep:shader-slang-2025.15

Conversation

@samestep
Copy link
Contributor

@samestep samestep commented Aug 27, 2025

Changelog: https://github.com/shader-slang/slang/releases/tag/v2025.15

See shader-slang/slang#7987 and shader-slang/slang#8031. Supersedes #436961.

@niklaskorz I did have a question: how should we handle updating Slang and its VS Code extension once #436862 lands? I feel like the ideal would be to update them in lockstep with each other so that the Slang version used by the VS Code extension matches the version on Marketplace, but that doesn't seem to necessarily match with the strategy used in Nixpkgs generally. Thoughts?

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@samestep samestep requested a review from niklaskorz August 27, 2025 13:36
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Aug 27, 2025
@niklaskorz
Copy link
Member

niklaskorz commented Aug 27, 2025

As Slang implements the Language Server Protocol, tight version coupling shouldn't be required I think? If they do tend to be released at the same time, I don't have any issue with updating them in the same PR (but still separate commits) though.

@niklaskorz
Copy link
Member

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 437524
Commit: 8d547ed606f176317e4420fd0cd85179463720ad


aarch64-linux

✅ 4 packages built:
  • shader-slang
  • shader-slang.debug
  • shader-slang.dev
  • shader-slang.doc

aarch64-darwin

✅ 3 packages built:
  • shader-slang
  • shader-slang.dev
  • shader-slang.doc

Copy link
Member

@niklaskorz niklaskorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thx. I'm fine with including the LLVM bump here as-is and discuss how to handle LLVM in general in a follow up.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Aug 27, 2025
@niklaskorz niklaskorz merged commit b433e43 into NixOS:master Aug 27, 2025
39 of 41 checks passed
@samestep samestep mentioned this pull request Aug 27, 2025
13 tasks
@samestep samestep deleted the shader-slang-2025.15 branch August 27, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants