Skip to content

crystal: expose version 1.16#437046

Merged
emilazy merged 1 commit intoNixOS:masterfrom
kachick:crystal-expose-1_16
Sep 3, 2025
Merged

crystal: expose version 1.16#437046
emilazy merged 1 commit intoNixOS:masterfrom
kachick:crystal-expose-1_16

Conversation

@kachick
Copy link
Member

@kachick kachick commented Aug 26, 2025

The crystal package's default version is currently 1.16.

However, this version becomes unusable once 1.17 or higher is introduced.
Additionally, we are unable to pin the specific version of crystal in other packages.

Follow-up: #399318

> nix run nixpkgs#crystal -- --version
Crystal 1.16.3 (2025-05-12)

LLVM: 18.1.8
Default target: x86_64-unknown-linux-gnu

> nix run nixpkgs#crystal_1_15 -- --version
Crystal 1.15.1 (2025-02-04)

LLVM: 18.1.8
Default target: x86_64-unknown-linux-gnu

> nix run nixpkgs#crystal_1_16 -- --version
error: flake 'flake:nixpkgs' does not provide attribute 'apps.x86_64-linux.crystal_1_16', 'packages.x86_64-linux.crystal_1_16', 'legacyPackages.x86_64-linux.crystal_1_16' or 'crystal_1_16'
       Did you mean one of crystal_1_11, crystal_1_12, crystal_1_14, crystal_1_15 or crystal_1_2?

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.

@david50407
@manveru
@peterhoeg
@donovanglover
@straight-shoota


Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Aug 26, 2025
@kachick
Copy link
Member Author

kachick commented Aug 26, 2025

There are no rebuilds

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 437046
Commit: 538fd5e4e4036a3fdeeb7a33716297e58fba98cb

@nixpkgs-ci nixpkgs-ci bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Aug 26, 2025
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Aug 26, 2025
@kachick kachick mentioned this pull request Aug 27, 2025
13 tasks
@emilazy
Copy link
Member

emilazy commented Sep 3, 2025

Could you rebase?

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Sep 3, 2025
@kachick kachick force-pushed the crystal-expose-1_16 branch from 538fd5e to a3eea68 Compare September 3, 2025 16:38
@emilazy emilazy merged commit dae1624 into NixOS:master Sep 3, 2025
28 of 31 checks passed
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 2.status: merge conflict This PR has merge conflicts with the target branch 12.approvals: 1 This PR was reviewed and approved by one person. labels Sep 3, 2025
@kachick
Copy link
Member Author

kachick commented Sep 3, 2025

Thank you!

@kachick kachick deleted the crystal-expose-1_16 branch September 3, 2025 16:46
@kachick kachick mentioned this pull request Oct 18, 2025
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants